Completed
Push — master ( a5512e...739835 )
by Mahmoud
06:34 queued 02:31
created
app/Port/Tests/PHPUnit/Traits/TestingTrait.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -30,10 +30,10 @@  discard block
 block discarded – undo
30 30
     public $loggedInTestingUser;
31 31
 
32 32
     /**
33
-     * @param           $endpoint
33
+     * @param           string $endpoint
34 34
      * @param string    $verb
35 35
      * @param array     $data
36
-     * @param bool|true $protected
36
+     * @param boolean $protected
37 37
      * @param array     $header
38 38
      *
39 39
      * @throws \Symfony\Component\Debug\Exception\UndefinedMethodException
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
     }
192 192
 
193 193
     /**
194
-     * @param $keys
194
+     * @param string[] $keys
195 195
      * @param $response
196 196
      */
197 197
     public function assertResponseContainKeys($keys, $response)
Please login to merge, or discard this patch.
app/Containers/SocialAuthentication/UI/API/Routes/v.1.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -156,19 +156,19 @@
 block discarded – undo
156 156
 // FOR LIVE TESTING ONLY
157 157
 
158 158
 $router->any('auth/google/test', [
159
-    'uses'       => function () {
159
+    'uses'       => function() {
160 160
         return Laravel\Socialite\Facades\Socialite::driver('google')->redirect();
161 161
     },
162 162
 ]);
163 163
 
164 164
 $router->any('auth/twitter/test', [
165
-    'uses'       => function () {
165
+    'uses'       => function() {
166 166
         return Laravel\Socialite\Facades\Socialite::driver('twitter')->redirect();
167 167
     },
168 168
 ]);
169 169
 
170 170
 $router->any('auth/facebook/test', [
171
-    'uses'       => function () {
171
+    'uses'       => function() {
172 172
         return Laravel\Socialite\Facades\Socialite::driver('facebook')->redirect();
173 173
     },
174 174
 ]);
Please login to merge, or discard this patch.
app/Containers/User/Tasks/UpdateUserTask.php 1 patch
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -37,10 +37,10 @@
 block discarded – undo
37 37
      * @param null $email
38 38
      * @param null $gender
39 39
      * @param null $birth
40
-     * @param null $token
41
-     * @param null $expiresIn
42
-     * @param null $refreshToken
43
-     * @param null $tokenSecret
40
+     * @param boolean|null $token
41
+     * @param boolean|null $expiresIn
42
+     * @param boolean|null $refreshToken
43
+     * @param boolean|null $tokenSecret
44 44
      *
45 45
      * @return  mixed
46 46
      */
Please login to merge, or discard this patch.
app/Containers/SocialAuthentication/Actions/SocialLoginAction.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -78,10 +78,10 @@
 block discarded – undo
78 78
         // find if that user exist in our database
79 79
         $user = $this->findSocialUserTask->run($provider, $socialUserProfile->id);
80 80
 
81
-        $tokenSecret = isset($socialUserProfile->tokenSecret) ? : null;
82
-        $expiresIn = isset($socialUserProfile->expiresIn) ? : null;
83
-        $refreshToken = isset($socialUserProfile->refreshToken) ? : null;
84
-        $avatar_original = isset($socialUserProfile->avatar_original) ? : null;
81
+        $tokenSecret = isset($socialUserProfile->tokenSecret) ?: null;
82
+        $expiresIn = isset($socialUserProfile->expiresIn) ?: null;
83
+        $refreshToken = isset($socialUserProfile->refreshToken) ?: null;
84
+        $avatar_original = isset($socialUserProfile->avatar_original) ?: null;
85 85
 
86 86
         // if user not found then create new one
87 87
         if (!$user) {
Please login to merge, or discard this patch.
app/Containers/Email/Actions/SetUserEmailWithConfirmationAction.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
      */
66 66
     public function run($email, $userId = null)
67 67
     {
68
-        if(!$userId){
68
+        if (!$userId) {
69 69
             $userId = $this->getAuthenticatedUserTask->run()->id;
70 70
         }
71 71
 
Please login to merge, or discard this patch.
app/Containers/User/Actions/DeleteUserAction.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
      */
66 66
     public function run($email, $userId = null)
67 67
     {
68
-        if(!$userId){
68
+        if (!$userId) {
69 69
             $userId = $this->getAuthenticatedUserTask->run()->id;
70 70
         }
71 71
 
Please login to merge, or discard this patch.
app/Port/Routes/Traits/RoutesServiceProviderTrait.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
                 $apiVersionNumber = $this->getRouteFileVersionNumber($fileNameWithoutExtension);
55 55
 
56 56
                 $this->apiRouter->version('v' . $apiVersionNumber,
57
-                    function (DingoApiRouter $router) use ($file, $containerPath, $containersNamespace) {
57
+                    function(DingoApiRouter $router) use ($file, $containerPath, $containersNamespace) {
58 58
 
59 59
                         $controllerNamespace = $containersNamespace . '\\Containers\\' . basename($containerPath) . '\\UI\API\Controllers';
60 60
 
@@ -62,12 +62,12 @@  discard block
 block discarded – undo
62 62
                             // Routes Namespace
63 63
                             'namespace'  => $controllerNamespace,
64 64
                             // Enable: API Rate Limiting
65
-                            'middleware' => ['api.throttle','mobile.requests.monitor'],
65
+                            'middleware' => ['api.throttle', 'mobile.requests.monitor'],
66 66
                             // The API limit time.
67 67
                             'limit'      => Config::get('api.limit'),
68 68
                             // The API limit expiry time.
69 69
                             'expires'    => Config::get('api.limit_expires'),
70
-                        ], function ($router) use ($file) {
70
+                        ], function($router) use ($file) {
71 71
 
72 72
                             require $file->getPathname();
73 73
 
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
             foreach ($files as $file) {
101 101
                 $this->webRouter->group([
102 102
                     'namespace' => $controllerNamespace,
103
-                ], function (LaravelRouter $router) use ($file) {
103
+                ], function(LaravelRouter $router) use ($file) {
104 104
                     require $file->getPathname();
105 105
                 });
106 106
             }
Please login to merge, or discard this patch.