Porto-SAP / Hello-API

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 = [])
     {

app/Containers/Debugger/Objects/Output.php

Doc Comments +2 added lines, -1 removed lines

@@ -56,6 +56,7 @@ 
     }
 
     /**
+     * @param string $text
      * @return  string
      */
     protected function set($text)
@@ -82,7 +83,7 @@ 
     /**
      * Add header
      *
-     * @param $name
+     * @param string $name
      */
     public function header($name)
     {

app/Containers/Debugger/Objects/RequestsLogger.php

Unused Use Statements -2 removed lines

@@ -4,8 +4,6 @@
 
 use App;
 use Config;
-use DB;
-use Log;
 use Monolog\Formatter\LineFormatter;
 use Monolog\Handler\StreamHandler;
 use Monolog\Logger;

app/Containers/Authentication/UI/WEB/Controllers/Controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
     /**
      * @param \App\Containers\Authentication\Actions\WebLogoutAction $action
      *
-     * @return  \Illuminate\Contracts\View\Factory|\Illuminate\View\View
+     * @return  \Illuminate\Http\RedirectResponse
      */
     public function logoutAdmin(WebLogoutAction $action)
     {

app/Containers/Authorization/Actions/RevokeUserFromRoleAction.php

Doc Comments +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 class RevokeUserFromRoleAction extends Action
 {
     /**
-     * @param User|integer  $userId
+     * @param User|integer  $user
      * @param integer|array $rolesIds
      *
      * @return  \App\Containers\User\Models\User

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Containers\Authorization\Actions;
 
 use App\Containers\Authorization\Tasks\GetRoleTask;
-use App\Containers\Authorization\Tasks\RevokeUserFromRoleTask;
 use App\Containers\User\Models\User;
 use App\Containers\User\Tasks\FindUserByIdTask;
 use App\Ship\Parents\Actions\Action;

app/Ship/Parents/Exceptions/Exception.php

Doc Comments +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
     /**
      * Exception constructor.
      *
-     * @param null            $message
+     * @param string            $message
      * @param null            $errors
      * @param null            $statusCode
      * @param int             $code
@@ -112,9 +112,9 @@ 
 
 
     /**
-     * @param $statusCode
+     * @param integer $statusCode
      * @param $message
-     * @param $code
+     * @param integer $code
      */
     private function logTheError($statusCode, $message, $code)
     {

app/Containers/Authorization/UI/API/Tests/Functional/AssignUserToRoleTest.php

Spacing +2 added lines, -2 removed lines

@@ -60,13 +60,13 @@
         $response = $this->makeCall($data);
 
         // assert response status is correct. Note: this will return 200 if `HASH_ID=false` in the .env
-        if(\Config::get('hello.hash-id')){
+        if (\Config::get('hello.hash-id')) {
             $this->assertEquals('400', $response->getStatusCode());
 
             $this->assertResponseContainKeyValue([
                 'message' => 'Only Hashed ID\'s allowed (user_id).',
             ], $response);
-        }else{
+        } else {
             $this->assertEquals('200', $response->getStatusCode());
         }
 

Braces +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             $this->assertResponseContainKeyValue([
                 'message' => 'Only Hashed ID\'s allowed (user_id).',
             ], $response);
-        }else{
+        } else{
             $this->assertEquals('200', $response->getStatusCode());
         }
 

app/Containers/Authorization/UI/API/Tests/Functional/RevokeUserFromRoleTest.php

Spacing +2 added lines, -2 removed lines

@@ -66,13 +66,13 @@
 
 
         // assert response status is correct. Note: this will return 200 if `HASH_ID=false` in the .env
-        if(\Config::get('hello.hash-id')){
+        if (\Config::get('hello.hash-id')) {
             $this->assertEquals('400', $response->getStatusCode());
 
             $this->assertResponseContainKeyValue([
                 'message' => 'Only Hashed ID\'s allowed (roles_ids.*).',
             ], $response);
-        }else{
+        } else {
             $this->assertEquals('200', $response->getStatusCode());
         }
 

Braces +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
             $this->assertResponseContainKeyValue([
                 'message' => 'Only Hashed ID\'s allowed (roles_ids.*).',
             ], $response);
-        }else{
+        } else{
             $this->assertEquals('200', $response->getStatusCode());
         }