Porto-SAP / Hello-API

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());
         }
 

app/Containers/User/Actions/DeleteUserAction.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
      */
     public function run($userId = null)
     {
-        $userId = $userId ? : $this->call(GetAuthenticatedUserTask::class)->id;
+        $userId = $userId ?: $this->call(GetAuthenticatedUserTask::class)->id;
 
         $user = $this->call(FindUserByIdTask::class, [$userId]);
 

app/Containers/Documentation/Actions/GenerateDocumentationAction.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 {
 
     /**
-     * @param $console
+     * @param \App\Containers\Documentation\UI\CLI\Commands\GenerateApiDocsCommand $console
      */
     public function run($console)
     {