Porto-SAP / Hello-API

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
      */

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',
     ],
 ]);

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

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Ship\Parents\Tests\PhpUnit;
 
 use App\Ship\Engine\Traits\HashIdTrait;
-use App\Ship\Features\Tests\PhpUnit\TestingTrait;
 use App\Ship\Features\Tests\PhpUnit\TestsAuthHelperTrait;
 use App\Ship\Features\Tests\PhpUnit\TestsCustomHelperTrait;
 use App\Ship\Features\Tests\PhpUnit\TestsMockHelperTrait;

app/Ship/Engine/Traits/HashIdTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -182,7 +182,7 @@ 
     /**
      * @param $subject
      * @param $findKey
-     * @param $callback
+     * @param \Closure $callback
      *
      * @return  array
      */
@@ -206,7 +206,7 @@ 
     }
 
     /**
-     * @param $search
+     * @param string $search
      * @param $subject
      *
      * @return  mixed
@@ -225,7 +225,7 @@ 
     }
 
     /**
-     * @param $needle
+     * @param string $needle
      * @param $haystack
      *
      * @return  int

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
     {
         // hash the ID only if hash-id enabled in the config
         if (Config::get('hello.hash-id')) {
-            return $this->encoder(($key) ? : $this->getKey());
+            return $this->encoder(($key) ?: $this->getKey());
         }
 
         return $this->getKey();
@@ -115,7 +115,7 @@ 
         // get the last part of the key, which should be the ID that needs decoding
         $idToDecode = substr($key, strrpos($key, '.*.') + 3);
 
-        array_walk_recursive($requestData, function (&$value, $key) use ($idToDecode) {
+        array_walk_recursive($requestData, function(&$value, $key) use ($idToDecode) {
 
             if ($key == $idToDecode) {
                 $value = $this->decode($value, $key);
@@ -132,7 +132,7 @@ 
     {
         $idToDecode = $this->removeLastOccurrenceFromString($key, '.*');
 
-        $this->findKeyAndReturnValue($requestData, $idToDecode, function ($ids) use ($key) {
+        $this->findKeyAndReturnValue($requestData, $idToDecode, function($ids) use ($key) {
 
             if (!is_array($ids)) {
                 throw new IncorrectIdException('Expected ID\'s to be in array. Please wrap your ID\'s in an Array and send them back.');
@@ -274,7 +274,7 @@ 
     public function runHashedIdsDecoder()
     {
         if (Config::get('hello.hash-id')) {
-            Route::bind('id', function ($id, $route) {
+            Route::bind('id', function($id, $route) {
                 // skip decoding some endpoints
                 if (!in_array($route->uri(), $this->skippedEndpoints)) {
 

app/Ship/Features/Validations/ValidationTrait.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
     public function extendValidationRules()
     {
         // Validate String contains no space.
-        Validator::extend('no_spaces', function ($attribute, $value, $parameters, $validator) {
+        Validator::extend('no_spaces', function($attribute, $value, $parameters, $validator) {
             return preg_match('/^\S*$/u', $value);
         }, ['String should not contain space.']);
 
@@ -26,7 +26,7 @@ 
         // Usage: unique_composite:table,this-attribute-column,the-other-attribute-column
         // Example:    'values'               => 'required|unique_composite:item_variant_values,value,item_variant_name_id',
         //             'item_variant_name_id' => 'required',
-        Validator::extend('unique_composite', function ($attribute, $value, $parameters, $validator) {
+        Validator::extend('unique_composite', function($attribute, $value, $parameters, $validator) {
 
             $queryBuilder = \DB::table($parameters[0]);
 

app/Containers/Localization/Middlewares/LocalizationMiddleware.php

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');
     }
 
     /**

Doc Comments +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
     }
 
     /**
-     * @param $lang
+     * @param string $lang
      *
      * @return string|Exception
      */
@@ -57,7 +57,7 @@ 
     }
 
     /**
-     * @param $request
+     * @param Request $request
      *
      * @return  string
      */

app/Ship/Engine/Loaders/MiddlewaresLoaderTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     /**
      * Registering Route Group's
      *
-     * @param array $middlewareGroups
+     * @param array $middlewares
      */
     private function registerMiddleware(array $middlewares = [])
     {