Porto-SAP / Hello-API

app/Containers/Localization/Middlewares/Localization.php

Doc Comments +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
     }
 
     /**
-     * @param $lang
+     * @param string $lang
      *
      * @return string|Exception
      */
@@ -67,7 +67,7 @@ 
     }
 
     /**
-     * @param $request
+     * @param \Illuminate\Http\Request $request
      *
      * @return  string
      */

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     private function findLanguage($request)
     {
         // read the language from the request header, if the header is missed, take the default local language
-        return $request->header('Content-Language') ? : Config::get('app.locale');
+        return $request->header('Content-Language') ?: Config::get('app.locale');
     }
 
     /**

app/Ship/Engine/Loaders/LocalizationLoaderTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     }
 
     /**
-     * @param $directory
+     * @param string $directory
      * @param $containerName
      */
     private function loadLocals($directory, $containerName)

app/Ship/Engine/Loaders/MigrationsLoaderTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     }
 
     /**
-     * @param $directory
+     * @param string $directory
      */
     private function loadMigrations($directory)
     {

app/Ship/Engine/Loaders/RoutesLoaderTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
 
     /**
      * @param $file
-     * @param $controllerNamespace
+     * @param string $controllerNamespace
      */
     private function loadWebRoute($file, $controllerNamespace)
     {
@@ -145,7 +145,7 @@ 
     /**
      * @param \Symfony\Component\Finder\SplFileInfo $file
      *
-     * @return  mixed
+     * @return  string
      */
     private function getRouteFileNameWithoutExtension(SplFileInfo $file)
     {

Spacing +3 added lines, -3 removed lines

@@ -84,7 +84,7 @@ 
         $this->webRouter->group([
             'middleware' => ['web'],
             'namespace'  => $controllerNamespace,
-        ], function (LaravelRouter $router) use ($file) {
+        ], function(LaravelRouter $router) use ($file) {
             require $file->getPathname();
         });
     }
@@ -100,7 +100,7 @@ 
         $apiVersionNumber = $this->getRouteFileVersionNumber($file);
 
         $this->apiRouter->version('v' . $apiVersionNumber,
-            function (DingoApiRouter $router) use ($file, $containerPath, $containersNamespace) {
+            function(DingoApiRouter $router) use ($file, $containerPath, $containersNamespace) {
 
                 $controllerNamespace = $containersNamespace . '\\Containers\\' . basename($containerPath) . '\\UI\API\Controllers';
 
@@ -113,7 +113,7 @@ 
                     'limit'      => Config::get('hello.api.limit'),
                     // The API limit expiry time.
                     'expires'    => Config::get('hello.api.limit_expires'),
-                ], function ($router) use ($file) {
+                ], function($router) use ($file) {
 
                     require $file->getPathname();
 

app/Ship/Engine/Loaders/ViewsLoaderTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     }
 
     /**
-     * @param $directory
+     * @param string $directory
      * @param $containerName
      */
     private function loadViews($directory, $containerName)

app/Containers/Authorization/Traits/AuthorizationTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 trait AuthorizationTrait
 {
     /**
-     * @return  \App\Containers\User\Models\User|null
+     * @return  \Illuminate\Contracts\Auth\Authenticatable|null
      */
     public function getUser()
     {

app/Ship/Parents/Requests/Request.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace App\Ship\Parents\Requests;
 
-use App\Containers\Authorization\Traits\AuthorizationTrait;
 use App\Ship\Engine\Traits\HashIdTrait;
 use Illuminate\Foundation\Http\FormRequest as LaravelFormRequest;
 

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     public function hasAccess(User $user = null)
     {
         // if not in parameters, take from the request object {$this}
-        $user = $user ? : $this->user();
+        $user = $user ?: $this->user();
 
         $hasAccess = array_merge(
             $this->hasAnyPermissionAccess($user),

app/Ship/Parents/Requests/RequestTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
      * To call functions and compare their bool responses to determine
      * if the user can proceed with the request or not.
      *
-     * @param array $functions
+     * @param string[] $functions
      *
      * @return  bool
      */

Unused Use Statements -3 removed lines

@@ -2,11 +2,8 @@
 
 namespace App\Ship\Parents\Requests;
 
-use App\Containers\Authorization\Traits\AuthorizationTrait;
-use App\Ship\Engine\Traits\HashIdTrait;
 use App\Ship\Features\Exceptions\ValidationFailedException;
 use Illuminate\Contracts\Validation\Validator;
-use Illuminate\Foundation\Http\FormRequest as LaravelFormRequest;
 
 /**
  * Class RequestTrait

Spacing +2 added lines, -2 removed lines

@@ -125,7 +125,7 @@ 
 
         $permissions = explode('|', $this->access['permissions']);
 
-        $hasAccess = array_map(function ($permission) use ($user) {
+        $hasAccess = array_map(function($permission) use ($user) {
             // Note: internal return
             return $user->hasPermissionTo($permission);
         }, $permissions);
@@ -146,7 +146,7 @@ 
 
         $roles = explode('|', $this->access['roles']);
 
-        $hasAccess = array_map(function ($role) use ($user) {
+        $hasAccess = array_map(function($role) use ($user) {
             // Note: internal return
             return $user->hasRole($role);
         }, $roles);

app/Containers/User/UI/API/Routes/CreateAdmin.v1.private.php

Indentation +1 added lines, -1 removed lines

@@ -42,6 +42,6 @@
 $router->post('admins', [
     'uses'  => 'Controller@createAdmin',
     'middleware' => [
-      'api.auth',
+        'api.auth',
     ],
 ]);