@@ -22,7 +22,7 @@ |
||
| 22 | 22 | { |
| 23 | 23 | $newFactoriesPath = '/app/Port/Loader/Loaders/Factories'; |
| 24 | 24 | |
| 25 | - App::singleton(Factory::class, function ($app) use ($newFactoriesPath) { |
|
| 25 | + App::singleton(Factory::class, function($app) use ($newFactoriesPath) { |
|
| 26 | 26 | $faker = $app->make(Generator::class); |
| 27 | 27 | |
| 28 | 28 | return Factory::construct($faker, base_path() . $newFactoriesPath); |
@@ -15,7 +15,6 @@ |
||
| 15 | 15 | |
| 16 | 16 | |
| 17 | 17 | /** |
| 18 | - * @param array $aliases |
|
| 19 | 18 | */ |
| 20 | 19 | public function loadPortInternalAliases() |
| 21 | 20 | { |
@@ -39,6 +39,7 @@ |
||
| 39 | 39 | |
| 40 | 40 | /** |
| 41 | 41 | * @param $consoleClass |
| 42 | + * @param string $directory |
|
| 42 | 43 | */ |
| 43 | 44 | private function loadTheConsoles($directory) |
| 44 | 45 | { |
@@ -57,7 +57,7 @@ |
||
| 57 | 57 | $this->baseUrl = env('API_FULL_URL'); // this reads the value from `phpunit.xml` during testing |
| 58 | 58 | |
| 59 | 59 | // override the default subDomain of the base URL when subDomain property is declared inside a test |
| 60 | - if(property_exists($this, 'subDomain')){ |
|
| 60 | + if (property_exists($this, 'subDomain')) { |
|
| 61 | 61 | $this->overrideSubDomain($this->subDomain); |
| 62 | 62 | } |
| 63 | 63 | |
@@ -70,8 +70,8 @@ discard block |
||
| 70 | 70 | } |
| 71 | 71 | |
| 72 | 72 | /** |
| 73 | - * @param array $directories |
|
| 74 | - * @param $seedersClasses |
|
| 73 | + * @param string[] $directories |
|
| 74 | + * @param Collection $seedersClasses |
|
| 75 | 75 | * |
| 76 | 76 | * @return mixed |
| 77 | 77 | */ |
@@ -103,7 +103,7 @@ discard block |
||
| 103 | 103 | /** |
| 104 | 104 | * @param $seedersClasses |
| 105 | 105 | * |
| 106 | - * @return \Illuminate\Support\Collection|static |
|
| 106 | + * @return Collection|null |
|
| 107 | 107 | */ |
| 108 | 108 | private function sortSeeders($seedersClasses) |
| 109 | 109 | { |
@@ -140,7 +140,7 @@ discard block |
||
| 140 | 140 | } |
| 141 | 141 | |
| 142 | 142 | /** |
| 143 | - * @param $seedersClasses |
|
| 143 | + * @param Collection|null $seedersClasses |
|
| 144 | 144 | */ |
| 145 | 145 | private function loadSeeders($seedersClasses) |
| 146 | 146 | { |
@@ -122,7 +122,7 @@ |
||
| 122 | 122 | } |
| 123 | 123 | |
| 124 | 124 | // sort the classes that needed to be ordered |
| 125 | - $orderedSeederClasses = $orderedSeederClasses->sortBy(function ($seederFullClassName) { |
|
| 125 | + $orderedSeederClasses = $orderedSeederClasses->sortBy(function($seederFullClassName) { |
|
| 126 | 126 | // get the order number form the end of each class name |
| 127 | 127 | $orderNumber = substr($seederFullClassName, strpos($seederFullClassName, "_") + 1); |
| 128 | 128 | |
@@ -44,6 +44,6 @@ |
||
| 44 | 44 | $router->post('admins/create', [ |
| 45 | 45 | 'uses' => 'Controller@createAdmin', |
| 46 | 46 | 'middleware' => [ |
| 47 | - 'api.auth', |
|
| 47 | + 'api.auth', |
|
| 48 | 48 | ], |
| 49 | 49 | ]); |
@@ -47,7 +47,7 @@ discard block |
||
| 47 | 47 | public function hasAccess(User $user = null) |
| 48 | 48 | { |
| 49 | 49 | // if not in parameters, take from the request object {$this} |
| 50 | - $user = $user ? : $this->user(); |
|
| 50 | + $user = $user ?: $this->user(); |
|
| 51 | 51 | |
| 52 | 52 | $hasAccess = array_merge( |
| 53 | 53 | $this->hasAnyPermissionAccess($user), |
@@ -71,7 +71,7 @@ discard block |
||
| 71 | 71 | |
| 72 | 72 | $permissions = explode('|', $this->access['permissions']); |
| 73 | 73 | |
| 74 | - $hasAccess = array_map(function ($permission) use ($user) { |
|
| 74 | + $hasAccess = array_map(function($permission) use ($user) { |
|
| 75 | 75 | // Note: internal return |
| 76 | 76 | return $user->hasPermissionTo($permission); |
| 77 | 77 | }, $permissions); |
@@ -92,7 +92,7 @@ discard block |
||
| 92 | 92 | |
| 93 | 93 | $roles = explode('|', $this->access['roles']); |
| 94 | 94 | |
| 95 | - $hasAccess = array_map(function ($role) use ($user) { |
|
| 95 | + $hasAccess = array_map(function($role) use ($user) { |
|
| 96 | 96 | // Note: internal return |
| 97 | 97 | return $user->hasRole($role); |
| 98 | 98 | }, $roles); |
@@ -13,7 +13,7 @@ |
||
| 13 | 13 | { |
| 14 | 14 | |
| 15 | 15 | /** |
| 16 | - * @return \App\Containers\User\Models\User|null |
|
| 16 | + * @return \Illuminate\Contracts\Auth\Authenticatable|null |
|
| 17 | 17 | */ |
| 18 | 18 | public function getUser() |
| 19 | 19 | { |
@@ -37,7 +37,7 @@ discard block |
||
| 37 | 37 | public $loggedInTestingAdmin; |
| 38 | 38 | |
| 39 | 39 | /** |
| 40 | - * @param $endpoint |
|
| 40 | + * @param string $endpoint |
|
| 41 | 41 | * @param string $verb |
| 42 | 42 | * @param array $data |
| 43 | 43 | * @param bool $protected |
@@ -224,7 +224,7 @@ discard block |
||
| 224 | 224 | } |
| 225 | 225 | |
| 226 | 226 | /** |
| 227 | - * @param $keys |
|
| 227 | + * @param string[] $keys |
|
| 228 | 228 | * @param $response |
| 229 | 229 | */ |
| 230 | 230 | public function assertResponseContainKeys($keys, $response) |
@@ -168,7 +168,7 @@ discard block |
||
| 168 | 168 | { |
| 169 | 169 | |
| 170 | 170 | // if no user detail provided, use the default details. |
| 171 | - $userDetails = $userDetails ? : [ |
|
| 171 | + $userDetails = $userDetails ?: [ |
|
| 172 | 172 | 'name' => 'Testing User', |
| 173 | 173 | 'email' => $this->faker->email, |
| 174 | 174 | 'password' => 'testing-pass', |
@@ -182,7 +182,7 @@ discard block |
||
| 182 | 182 | ]); |
| 183 | 183 | |
| 184 | 184 | // assign roles and permissions |
| 185 | - $user = $this->setupTestingUserAccess($user, $access ? : (isset($this->access) ? $this->access : null)); |
|
| 185 | + $user = $this->setupTestingUserAccess($user, $access ?: (isset($this->access) ? $this->access : null)); |
|
| 186 | 186 | |
| 187 | 187 | // log the user in |
| 188 | 188 | $user = App::make(ApiLoginThisUserObjectTask::class)->run($user); |
@@ -379,7 +379,7 @@ discard block |
||
| 379 | 379 | */ |
| 380 | 380 | public function overrideSubDomain($subDomain, $url = null) |
| 381 | 381 | { |
| 382 | - $url = ($url) ? : $this->baseUrl; |
|
| 382 | + $url = ($url) ?: $this->baseUrl; |
|
| 383 | 383 | |
| 384 | 384 | $info = parse_url($url); |
| 385 | 385 | |
@@ -24,7 +24,7 @@ |
||
| 24 | 24 | public function runQueryDebugger($log = true, $terminal = false) |
| 25 | 25 | { |
| 26 | 26 | if (Config::get('hello.query_debugger')) { |
| 27 | - DB::listen(function ($event) use ($terminal, $log) { |
|
| 27 | + DB::listen(function($event) use ($terminal, $log) { |
|
| 28 | 28 | $fullQuery = vsprintf(str_replace(['%', '?'], ['%%', '%s'], $event->sql), $event->bindings); |
| 29 | 29 | |
| 30 | 30 | $text = $event->connectionName . ' (' . $event->time . '): ' . $fullQuery; |