Skip to content

Commit c3c2399

Browse files
committed
psr-4 fixes
1 parent ffeccce commit c3c2399

File tree

6 files changed

+80
-54
lines changed

6 files changed

+80
-54
lines changed

composer.json

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,11 @@
1919
"laravel/framework": "6.*|7.*|8.*|9.*"
2020
},
2121
"require-dev": {
22-
"orchestra/testbench": "^6.0",
23-
"laravel/tinker": "^2.4",
2422
"illuminate/support": "^8.5",
25-
"laravel/laravel": "^8.0"
23+
"laravel/laravel": "^8.0",
24+
"laravel/tinker": "^2.4",
25+
"orchestra/testbench": "^6.0",
26+
"suin/phpcs-psr4-sniff": "^3.0"
2627
},
2728
"autoload": {
2829
"psr-4": {
@@ -42,7 +43,10 @@
4243
}
4344
},
4445
"config": {
45-
"sort-packages": true
46+
"sort-packages": true,
47+
"allow-plugins": {
48+
"dealerdirect/phpcodesniffer-composer-installer": true
49+
}
4650
},
4751
"minimum-stability": "stable",
4852
"prefer-stable": true

src/App/Http/Controllers/TwoStepController.php

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,13 @@ public function __construct()
2828
{
2929
$this->middleware('auth');
3030

31-
$this->middleware(function ($request, $next) {
32-
$this->setUser2StepData();
31+
$this->middleware(
32+
function ($request, $next) {
33+
$this->setUser2StepData();
3334

34-
return $next($request);
35-
});
35+
return $next($request);
36+
}
37+
);
3638
}
3739

3840
/**
@@ -146,12 +148,14 @@ public function verify(Request $request)
146148
}
147149

148150
if ($request->ajax()) {
149-
$validator = Validator::make($request->all(), [
151+
$validator = Validator::make(
152+
$request->all(), [
150153
'v_input_1' => 'required|min:1|max:1',
151154
'v_input_2' => 'required|min:1|max:1',
152155
'v_input_3' => 'required|min:1|max:1',
153156
'v_input_4' => 'required|min:1|max:1',
154-
]);
157+
]
158+
);
155159

156160
if ($validator->fails()) {
157161
$returnData = $this->invalidCodeReturnData($validator->errors());

src/App/Http/Middleware/Laravel2step.php

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -25,23 +25,23 @@ public function handle($request, Closure $next)
2525
$nextUri = config('app.url').'/'.$uri;
2626

2727
switch ($uri) {
28-
case 'verification/needed':
29-
case 'password/reset':
30-
case 'register':
31-
case 'logout':
32-
case 'login':
33-
case '/':
34-
break;
35-
36-
default:
37-
session(['nextUri' => $nextUri]);
38-
39-
if (config('laravel2step.laravel2stepEnabled')) {
40-
if ($this->twoStepVerification($request) !== true) {
41-
return redirect('verification/needed');
42-
}
28+
case 'verification/needed':
29+
case 'password/reset':
30+
case 'register':
31+
case 'logout':
32+
case 'login':
33+
case '/':
34+
break;
35+
36+
default:
37+
session(['nextUri' => $nextUri]);
38+
39+
if (config('laravel2step.laravel2stepEnabled')) {
40+
if ($this->twoStepVerification($request) !== true) {
41+
return redirect('verification/needed');
4342
}
44-
break;
43+
}
44+
break;
4545
}
4646

4747
return $response;

src/Laravel2stepServiceProvider.php

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -49,28 +49,40 @@ private function publishFiles()
4949
{
5050
$publishTag = 'laravel2step';
5151

52-
$this->publishes([
52+
$this->publishes(
53+
[
5354
__DIR__.'/config/laravel2step.php' => base_path('config/laravel2step.php'),
54-
], $publishTag);
55+
], $publishTag
56+
);
5557

56-
$this->publishes([
58+
$this->publishes(
59+
[
5760
__DIR__.'/database/migrations/' => base_path('/database/migrations'),
58-
], $publishTag);
61+
], $publishTag
62+
);
5963

60-
$this->publishes([
64+
$this->publishes(
65+
[
6166
__DIR__.'/public/css' => public_path('css/laravel2step'),
62-
], $publishTag);
67+
], $publishTag
68+
);
6369

64-
$this->publishes([
70+
$this->publishes(
71+
[
6572
__DIR__.'/resources/assets/scss' => resource_path('assets/scss/laravel2step'),
66-
], $publishTag);
73+
], $publishTag
74+
);
6775

68-
$this->publishes([
76+
$this->publishes(
77+
[
6978
__DIR__.'/resources/views' => resource_path('views/vendor/laravel2step'),
70-
], $publishTag);
79+
], $publishTag
80+
);
7181

72-
$this->publishes([
82+
$this->publishes(
83+
[
7384
__DIR__.'/resources/lang' => base_path('resources/lang/vendor/laravel2step'),
74-
], $publishTag);
85+
], $publishTag
86+
);
7587
}
7688
}

src/database/migrations/2017_12_09_070937_create_two_step_auth_table.php

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,19 @@ public function up()
2020
$tableCheck = Schema::connection($connection)->hasTable($table);
2121

2222
if (!$tableCheck) {
23-
Schema::connection($connection)->create($table, function (Blueprint $table) {
24-
$table->increments('id');
25-
$table->unsignedBigInteger('userId')->unsigned()->index();
26-
$table->foreign('userId')->references('id')->on('users')->onDelete('cascade');
27-
$table->string('authCode')->nullable();
28-
$table->integer('authCount');
29-
$table->boolean('authStatus')->default(false);
30-
$table->dateTime('authDate')->nullable();
31-
$table->dateTime('requestDate')->nullable();
32-
$table->timestamps();
33-
});
23+
Schema::connection($connection)->create(
24+
$table, function (Blueprint $table) {
25+
$table->increments('id');
26+
$table->unsignedBigInteger('userId')->unsigned()->index();
27+
$table->foreign('userId')->references('id')->on('users')->onDelete('cascade');
28+
$table->string('authCode')->nullable();
29+
$table->integer('authCount');
30+
$table->boolean('authStatus')->default(false);
31+
$table->dateTime('authDate')->nullable();
32+
$table->dateTime('requestDate')->nullable();
33+
$table->timestamps();
34+
}
35+
);
3436
}
3537
}
3638

src/routes/web.php

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,14 @@
55
| Laravel 2-Step Verification Web Routes
66
|--------------------------------------------------------------------------
77
|
8+
| Here are the routes for two step verification middleware.
9+
|
810
*/
911

10-
Route::group(['prefix' => 'verification', 'as' => 'laravel2step::', 'namespace' => 'jeremykenedy\laravel2step\App\Http\Controllers', 'middleware' => ['web']], function () {
11-
Route::get('/needed', ['uses' => 'TwoStepController@showVerification'])->name('verificationNeeded');
12-
Route::post('/verify', ['uses' => 'TwoStepController@verify'])->name('verify');
13-
Route::post('/resend', ['uses' => 'TwoStepController@resend'])->name('resend');
14-
});
12+
Route::group(
13+
['prefix' => 'verification', 'as' => 'laravel2step::', 'namespace' => 'jeremykenedy\laravel2step\App\Http\Controllers', 'middleware' => ['web']], function () {
14+
Route::get('/needed', ['uses' => 'TwoStepController@showVerification'])->name('verificationNeeded');
15+
Route::post('/verify', ['uses' => 'TwoStepController@verify'])->name('verify');
16+
Route::post('/resend', ['uses' => 'TwoStepController@resend'])->name('resend');
17+
}
18+
);

0 commit comments

Comments
 (0)