Skip to content

Commit 1531dcd

Browse files
committed
style: clear old code by pint
1 parent 7f1018a commit 1531dcd

12 files changed

+27
-40
lines changed

app/Http/Controllers/API/AuthController.php

+4-4
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
namespace App\Http\Controllers\API;
44

55
use Illuminate\Http\Request;
6-
use App\Http\Controllers\Controller;
6+
use Illuminate\Routing\Controller;
77
use Validator;
88

99
class AuthController extends Controller
@@ -40,12 +40,12 @@ public function index(Request $request)
4040
'ip' => 'ip',
4141
'mac' => 'string',
4242
]);
43-
$user_status = -1;
43+
$userStatus = -1;
4444
$status = 400;
4545
if (!$validator->fails()) {
46-
$user_status = 1;
46+
$userStatus = 1;
4747
$status = 200;
4848
}
49-
return response()->txt('Auth: ' . $user_status, $status);
49+
return response()->txt('Auth: ' . $userStatus, $status);
5050
}
5151
}

app/Http/Controllers/API/GatewayController.php

+15-15
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
namespace App\Http\Controllers\API;
44

55
use Illuminate\Http\Request;
6-
use App\Http\Controllers\Controller;
6+
use Illuminate\Routing\Controller;
77
use App\Models\Gateway;
88
use Validator;
99

@@ -26,27 +26,27 @@ public function ping(Request $request)
2626
'sys_load' => 'required|numeric',
2727
'wifidog_uptime' => 'required|integer',
2828
]);
29-
$r = 'Error: params wrong';
29+
$responseTxt = 'Error: params wrong';
3030
if ($validator->fails()) {
31-
return response()->txt($r, 400);
31+
return response()->txt($responseTxt, 400);
3232
}
3333
$input = $validator->getData();
3434
$input['id'] = $input['gw_id'];
3535
unset($input['gw_id']);
36-
$status_code = 200;
36+
$statusCode = 200;
3737
if (config('wifidog.allow_unknown_gateway')) {
3838
Gateway::updateOrCreate(['id' => $input['id']], $input);
39-
$r = 'Pong';
40-
} else {
41-
$gw = Gateway::find($input['id']);
42-
if (!empty($gw)) {
43-
$gw->update($input);
44-
$r = 'Pong';
45-
} else {
46-
$r = 'Error: not allow unknown gateway';
47-
$status_code = 400;
48-
}
39+
$responseTxt = 'Pong';
40+
return response()->txt($responseTxt, $statusCode);
4941
}
50-
return response()->txt($r, $status_code);
42+
$gateway = Gateway::find($input['id']);
43+
if (!empty($gateway)) {
44+
$gateway->update($input);
45+
$responseTxt = 'Pong';
46+
return response()->txt($responseTxt, $statusCode);
47+
}
48+
$responseTxt = 'Error: not allow unknown gateway';
49+
$statusCode = 400;
50+
return response()->txt($responseTxt, $statusCode);
5151
}
5252
}

app/Http/Controllers/DashboardController.php

-3
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22

33
namespace App\Http\Controllers;
44

5-
use Illuminate\Support\Facades\Auth;
6-
75
class DashboardController extends Controller
86
{
97
/**
@@ -13,7 +11,6 @@ class DashboardController extends Controller
1311
*/
1412
public function index()
1513
{
16-
$user = auth()->user();
1714
$data = [];
1815
if (session('gw_address') && session('gw_port') && session('token')) {
1916
$data['wifidog_uri'] = "http://" . session('gw_address')

app/Http/Controllers/Welcome.php

+1-4
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,14 @@
22

33
namespace App\Http\Controllers;
44

5-
use Illuminate\Http\Request;
6-
75
class Welcome extends Controller
86
{
97
/**
108
* Handle the incoming request.
119
*
12-
* @param \Illuminate\Http\Request $request
1310
* @return \Illuminate\Http\Response
1411
*/
15-
public function __invoke(Request $request)
12+
public function __invoke()
1613
{
1714
return view('welcome');
1815
}

database/factories/GatewayFactory.php

-2
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@
33
namespace Database\Factories;
44

55
use Illuminate\Database\Eloquent\Factories\Factory;
6-
use Illuminate\Support\Facades\Hash;
7-
use Illuminate\Support\Str;
86

97
/**
108
* @extends \Illuminate\Database\Eloquent\Factories\Factory<\App\Models\User>

database/factories/UserFactory.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public function definition(): array
3737
*/
3838
public function unverified(): static
3939
{
40-
return $this->state(fn (array $attributes) => [
40+
return $this->state(fn () => [
4141
'email_verified_at' => null,
4242
]);
4343
}

database/migrations/0001_01_01_000000_create_users_table.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@
44
use Illuminate\Database\Schema\Blueprint;
55
use Illuminate\Support\Facades\Schema;
66

7-
return new class extends Migration
8-
{
7+
return new class () extends Migration {
98
/**
109
* Run the migrations.
1110
*/

database/migrations/0001_01_01_000001_create_cache_table.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@
44
use Illuminate\Database\Schema\Blueprint;
55
use Illuminate\Support\Facades\Schema;
66

7-
return new class extends Migration
8-
{
7+
return new class () extends Migration {
98
/**
109
* Run the migrations.
1110
*/

database/migrations/0001_01_01_000002_create_jobs_table.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@
44
use Illuminate\Database\Schema\Blueprint;
55
use Illuminate\Support\Facades\Schema;
66

7-
return new class extends Migration
8-
{
7+
return new class () extends Migration {
98
/**
109
* Run the migrations.
1110
*/

database/migrations/2024_11_15_162815_create_personal_access_tokens_table.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@
44
use Illuminate\Database\Schema\Blueprint;
55
use Illuminate\Support\Facades\Schema;
66

7-
return new class extends Migration
8-
{
7+
return new class () extends Migration {
98
/**
109
* Run the migrations.
1110
*/

tests/Feature/API/AuthControllerTest.php

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
use Tests\TestCase;
66
use Illuminate\Foundation\Testing\RefreshDatabase;
77
use App\Models\User;
8-
use Illuminate\Support\Str;
98

109
class AuthControllerTest extends TestCase
1110
{

tests/Feature/DashboardTest.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ public function testIndex()
2121
'gw_port' => '2060',
2222
'token' => $token,
2323
])->get('/dashboard');
24-
$wifidog_uri = "http://192.168.199.1:2060/wifidog/auth?token=" . $token;
25-
$response->assertStatus(200)->assertViewHas('wifidog_uri', $wifidog_uri);
24+
$wifidogUri = "http://192.168.199.1:2060/wifidog/auth?token=" . $token;
25+
$response->assertStatus(200)->assertViewHas('wifidog_uri', $wifidogUri);
2626
$response->assertSeeText('start internet');
2727
}
2828

0 commit comments

Comments
 (0)