Completed
Push — master ( 21b5e5...d039a4 )
by Mahmoud
03:43
created
app/Containers/User/Actions/DeleteUserAction.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@
 block discarded – undo
43 43
      */
44 44
     public function run($userId = null)
45 45
     {
46
-        if(!$userId){
46
+        if (!$userId) {
47 47
             $userId = $this->getAuthenticatedUserTask->run()->id;
48 48
         }
49 49
 
Please login to merge, or discard this patch.
app/Containers/User/UI/API/Tests/Functional/RegisterUserTest.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,10 +33,10 @@
 block discarded – undo
33 33
             'name'  => $data['name'],
34 34
         ], $response);
35 35
 
36
-         // assert response contain the token
36
+            // assert response contain the token
37 37
         $this->assertResponseContainKeys(['id', 'token'], $response);
38 38
 
39
-         // assert the data is stored in the database
39
+            // assert the data is stored in the database
40 40
         $this->seeInDatabase('users', ['email' => $data['email']]);
41 41
     }
42 42
 
Please login to merge, or discard this patch.
app/Containers/User/UI/API/Tests/Functional/CreateAdminTest.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,10 +36,10 @@
 block discarded – undo
36 36
             'name'  => $data['name'],
37 37
         ], $response);
38 38
 
39
-         // assert response contain the token
39
+            // assert response contain the token
40 40
         $this->assertResponseContainKeys(['id', 'token'], $response);
41 41
 
42
-         // assert the data is stored in the database
42
+            // assert the data is stored in the database
43 43
         $this->seeInDatabase('users', ['email' => $data['email']]);
44 44
 
45 45
         $responseObject = $this->getResponseObject($response);
Please login to merge, or discard this patch.
app/Port/HashId/Traits/HashIdTrait.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
     public function runEndpointsHashedIdsDecoder()
30 30
     {
31 31
         if (Config::get('hello.hash-id')) {
32
-            Route::bind('id', function ($id, $route) {
32
+            Route::bind('id', function($id, $route) {
33 33
                 // skip decoding some endpoints
34 34
                 if (!in_array($route->getUri(), $this->skippedEndpoints)) {
35 35
 
Please login to merge, or discard this patch.
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -61,13 +61,13 @@
 block discarded – undo
61 61
         return $id;
62 62
     }
63 63
 
64
-  /**
65
-   * Will be used by the Eloquent Models (since it's used as trait there).
66
-   *
67
-   * @param null $key
68
-   *
69
-   * @return  mixed
70
-   */
64
+    /**
65
+     * Will be used by the Eloquent Models (since it's used as trait there).
66
+     *
67
+     * @param null $key
68
+     *
69
+     * @return  mixed
70
+     */
71 71
     public function getHashedKey($key = null)
72 72
     {
73 73
         // hash the ID only if hash-id enabled in the config
Please login to merge, or discard this patch.
app/Containers/Debugger/Traits/DebuggerTrait.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
     public function runQueryDebugger($log = true, $terminal = false)
26 26
     {
27 27
         if (Config::get('database.query_debugging')) {
28
-            DB::listen(function ($event) use ($terminal, $log) {
28
+            DB::listen(function($event) use ($terminal, $log) {
29 29
                 $fullQuery = vsprintf(str_replace(['%', '?'], ['%%', '%s'], $event->sql), $event->bindings);
30 30
 
31 31
                 $text = $event->connectionName . ' (' . $event->time . '): ' . $fullQuery;
Please login to merge, or discard this patch.
app/Containers/User/Data/Criterias/AdminsCriteria.php 2 patches
Indentation   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -13,16 +13,16 @@
 block discarded – undo
13 13
 class AdminsCriteria extends Criteria
14 14
 {
15 15
 
16
-  /**
17
-   * @param                                                   $model
18
-   * @param \Prettus\Repository\Contracts\RepositoryInterface $repository
19
-   *
20
-   * @return mixed
21
-   */
22
-  public function apply($model, PrettusRepositoryInterface $repository)
23
-  {
16
+    /**
17
+     * @param                                                   $model
18
+     * @param \Prettus\Repository\Contracts\RepositoryInterface $repository
19
+     *
20
+     * @return mixed
21
+     */
22
+    public function apply($model, PrettusRepositoryInterface $repository)
23
+    {
24 24
     return $model->whereHas('roles', function ($q) {
25
-      $q->where('name', 'admin');
25
+        $q->where('name', 'admin');
26 26
     });
27
-  }
27
+    }
28 28
 }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
    */
22 22
   public function apply($model, PrettusRepositoryInterface $repository)
23 23
   {
24
-    return $model->whereHas('roles', function ($q) {
24
+    return $model->whereHas('roles', function($q) {
25 25
       $q->where('name', 'admin');
26 26
     });
27 27
   }
Please login to merge, or discard this patch.
app/Containers/User/UI/API/Controllers/Controller.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -54,12 +54,12 @@
 block discarded – undo
54 54
         return $this->response->paginator($users, new UserTransformer());
55 55
     }
56 56
 
57
-  /**
58
-   * @param \App\Containers\User\UI\API\Requests\ListAllUsersRequest $request
59
-   * @param \App\Containers\User\Actions\ListAndSearchUsersAction    $action
60
-   *
61
-   * @return  \Dingo\Api\Http\Response
62
-   */
57
+    /**
58
+     * @param \App\Containers\User\UI\API\Requests\ListAllUsersRequest $request
59
+     * @param \App\Containers\User\Actions\ListAndSearchUsersAction    $action
60
+     *
61
+     * @return  \Dingo\Api\Http\Response
62
+     */
63 63
     public function listAllAdmins(ListAllUsersRequest $request, ListAndSearchUsersAction $action)
64 64
     {
65 65
         $users = $action->run(true, true);
Please login to merge, or discard this patch.
Authorization/UI/API/Tests/Functional/AttachPermissionsToRoleTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use App\Containers\Authorization\Models\Permission;
6 6
 use App\Containers\Authorization\Models\Role;
7
-use App\Containers\User\Models\User;
8 7
 use App\Port\Test\PHPUnit\Abstracts\TestCase;
9 8
 
10 9
 /**
Please login to merge, or discard this patch.
app/Port/Test/PHPUnit/Traits/TestingTrait.php 2 patches
Doc Comments   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
     public $loggedInTestingUser;
38 38
 
39 39
     /**
40
-     * @param        $endpoint
40
+     * @param        string $endpoint
41 41
      * @param string $verb
42 42
      * @param array  $data
43 43
      * @param bool   $protected
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
     }
134 134
 
135 135
     /**
136
-     * @return  App\Containers\User\Models\User|mixed
136
+     * @return  User
137 137
      */
138 138
     public function getLoggedInTestingAdmin()
139 139
     {
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
     /**
172 172
      * @param null $userDetails
173 173
      *
174
-     * @return  mixed
174
+     * @return  User
175 175
      */
176 176
     public function registerAndLoginTestingUser($userDetails = null)
177 177
     {
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
     /**
203 203
      * @param null $userDetails
204 204
      *
205
-     * @return  mixed
205
+     * @return  User
206 206
      */
207 207
     public function registerAndLoginTestingAdmin($userDetails = null)
208 208
     {
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
      *
219 219
      * @param null $userDetails
220 220
      *
221
-     * @return  mixed
221
+     * @return  User
222 222
      */
223 223
     public function registerAndLoginTestingDeveloper($userDetails = null)
224 224
     {
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
     }
234 234
 
235 235
     /**
236
-     * @param $keys
236
+     * @param string[] $keys
237 237
      * @param $response
238 238
      */
239 239
     public function assertResponseContainKeys($keys, $response)
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
         );
197 197
 
198 198
         // if permission property is defined than attach permission on the user
199
-        if(isset($this->permissions)){
199
+        if (isset($this->permissions)) {
200 200
             $user->givePermissionTo($this->permissions);
201 201
         }
202 202
 
@@ -392,7 +392,7 @@  discard block
 block discarded – undo
392 392
      */
393 393
     public function overrideSubDomain($subDomain, $url = null)
394 394
     {
395
-        $url = ($url) ? : $this->baseUrl;
395
+        $url = ($url) ?: $this->baseUrl;
396 396
 
397 397
         $info = parse_url($url);
398 398
 
Please login to merge, or discard this patch.