Porto-SAP / Hello-API

app/Ship/Parents/Tests/PhpUnit/TestCaseTrait.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
             return;
         }
 
-        $url = ($url) ? : $this->baseUrl;
+        $url = ($url) ?: $this->baseUrl;
 
         $info = parse_url($url);
 

app/Containers/User/UI/API/Tests/Functional/CreateAdminTest.php

Indentation +2 added lines, -2 removed lines

@@ -38,10 +38,10 @@
             'name'  => $data['name'],
         ], $response);
 
-         // assert response contain the token
+            // assert response contain the token
         $this->assertResponseContainKeys(['id', 'token'], $response);
 
-         // assert the data is stored in the database
+            // assert the data is stored in the database
         $this->seeInDatabase('users', ['email' => $data['email']]);
 
         $responseContent = $this->getResponseContent($response);

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),