Passed
Push — master ( 793370...b09d10 )
by Carsten
12:59
created
module/Auth/src/Auth/Factory/Adapter/ExternalApplicationAdapterFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
 class ExternalApplicationAdapterFactory implements FactoryInterface
24 24
 {
25 25
 	
26
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
26
+	public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
27 27
 	{
28 28
 		$repository = $container->get('repositories')->get('Auth/User');
29 29
 		$adapter = new ExternalApplication($repository);
Please login to merge, or discard this patch.
module/Auth/src/Auth/Factory/Controller/IndexControllerFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
         $hybridAuthAdapter = $container->get('HybridAuthAdapter');
62 62
 		$externalAdapter = $container->get('ExternalApplicationAdapter');
63 63
 		$repositories = $container->get('repositories');
64
-        $controller = new IndexController($auth, $logger, $userLoginAdapter,$locale,$viewHelperManager,$forms, $options,$hybridAuthAdapter,$externalAdapter,$repositories);
64
+        $controller = new IndexController($auth, $logger, $userLoginAdapter, $locale, $viewHelperManager, $forms, $options, $hybridAuthAdapter, $externalAdapter, $repositories);
65 65
         return $controller;
66 66
     }
67 67
     /**
Please login to merge, or discard this patch.
module/Auth/src/Auth/Factory/Controller/UsersControllerFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,6 +32,6 @@
 block discarded – undo
32 32
         $users = $container->get('repositories')->get('Auth/User');
33 33
 		$formManager = $container->get('forms');
34 34
 		$viewHelper = $container->get('ViewHelperManager');
35
-        return new UsersController($users,$formManager,$viewHelper);
35
+        return new UsersController($users, $formManager, $viewHelper);
36 36
     }
37 37
 }
Please login to merge, or discard this patch.
module/Auth/src/Auth/Controller/ManageGroupsController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
          */
76 76
         $events->attach(
77 77
             MvcEvent::EVENT_DISPATCH,
78
-            function ($event) {
78
+            function($event) {
79 79
                 $routeMatch = $event->getRouteMatch();
80 80
                 $action     = $routeMatch->getParam('action');
81 81
             
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
          */
96 96
         $events->attach(
97 97
             MvcEvent::EVENT_DISPATCH,
98
-            function ($event) {
98
+            function($event) {
99 99
                 $model = $event->getResult();
100 100
                 if (!$model instanceof ViewModel || $model->terminate()) {
101 101
                     return;
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
             $users = $repository->findByQuery($query);
224 224
         
225 225
             $userFilter = $this->filterManager->get('Auth/Entity/UserToSearchResult');
226
-            $filterFunc = function ($user) use ($userFilter) {
226
+            $filterFunc = function($user) use ($userFilter) {
227 227
                 return $userFilter->filter($user);
228 228
             };
229 229
             $result     = array_values(array_map($filterFunc, $users->toArray()));
Please login to merge, or discard this patch.
module/Auth/src/Auth/Controller/Plugin/SocialProfiles.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 */
31 31
     protected $request;
32 32
     
33
-    public function __construct($hybridAuth,RequestInterface $request,array $adapters = array())
33
+    public function __construct($hybridAuth, RequestInterface $request, array $adapters = array())
34 34
     {
35 35
         $this->hybridAuth = $hybridAuth;
36 36
         $this->request = $request;
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
             throw new \InvalidArgumentException(
71 71
                 sprintf(
72 72
                     'Adapter must be either a string or an instance of \Auth\Controller\Plugin\SocialProfiles\AbstractAdapter, but received %s',
73
-                    is_object($adapter) ? get_class($adapter) : '(' . gettype($adapter) . ')'
73
+                    is_object($adapter) ? get_class($adapter) : '('.gettype($adapter).')'
74 74
                 )
75 75
             );
76 76
         }
Please login to merge, or discard this patch.
module/Auth/src/Auth/Controller/Plugin/Service/SocialProfilesFactory.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,11 +16,11 @@
 block discarded – undo
16 16
 
17 17
 class SocialProfilesFactory implements FactoryInterface
18 18
 {
19
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
19
+	public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
20 20
 	{
21 21
 		$request = $container->get('request');
22 22
 		$hybridAuth = $container->get('HybridAuth');
23
-		$plugin     = new SocialProfiles($hybridAuth,$request);
23
+		$plugin     = new SocialProfiles($hybridAuth, $request);
24 24
 		
25 25
 		return $plugin;
26 26
 	}
Please login to merge, or discard this patch.
module/Auth/src/Auth/Controller/IndexController.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -35,8 +35,8 @@  discard block
 block discarded – undo
35 35
 class IndexController extends AbstractActionController
36 36
 {
37 37
 
38
-    const LOGIN='login';
39
-    const REGISTER='register';
38
+    const LOGIN = 'login';
39
+    const REGISTER = 'register';
40 40
 
41 41
     /**
42 42
      * @var AuthenticationService
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
         $registerForm = $this->forms[self::REGISTER];
125 125
 
126 126
         /* @var $request \Zend\Http\Request */
127
-        $request   = $this->getRequest();
127
+        $request = $this->getRequest();
128 128
 
129 129
         if ($request->isPost()) {
130 130
             $data                          = $this->params()->fromPost();
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
             if (array_key_exists('credentials', $data) &&
143 143
                 array_key_exists('login', $data['credentials']) &&
144 144
                 array_key_exists('credential', $data['credentials'])) {
145
-                $adapter->setIdentity($data['credentials']['login'] . $loginSuffix)
145
+                $adapter->setIdentity($data['credentials']['login'].$loginSuffix)
146 146
                     ->setCredential($data['credentials']['credential']);
147 147
             }
148 148
             
@@ -153,14 +153,14 @@  discard block
 block discarded – undo
153 153
             if ($result->isValid()) {
154 154
                 $user = $auth->getUser();
155 155
                 $language = $this->locale->detectLanguage($request, $user);
156
-                $this->logger->info('User ' . $user->getLogin() . ' logged in');
156
+                $this->logger->info('User '.$user->getLogin().' logged in');
157 157
                 
158 158
                 $ref = $this->params()->fromQuery('ref', false);
159 159
 
160 160
                 if ($ref) {
161 161
                     $ref = urldecode($ref);
162
-                    $url = preg_replace('~/[a-z]{2}(/|$)~', '/' . $language . '$1', $ref);
163
-                    $url = $request->getBasePath() . $url;
162
+                    $url = preg_replace('~/[a-z]{2}(/|$)~', '/'.$language.'$1', $ref);
163
+                    $url = $request->getBasePath().$url;
164 164
                 } else {
165 165
                     $urlHelper = $this->viewHelperManager->get('url');
166 166
                     $url = $urlHelper('lang', array('lang' => $language));
@@ -170,9 +170,9 @@  discard block
 block discarded – undo
170 170
             } else {
171 171
                 $loginName = $data['credentials']['login'];
172 172
                 if (!empty($loginSuffix)) {
173
-                    $loginName = $loginName . ' (' . $loginName . $loginSuffix . ')';
173
+                    $loginName = $loginName.' ('.$loginName.$loginSuffix.')';
174 174
                 }
175
-                $this->logger->info('Failed to authenticate User ' . $loginName);
175
+                $this->logger->info('Failed to authenticate User '.$loginName);
176 176
                 $this->notification()->danger(/*@translate*/ 'Authentication failed.');
177 177
             }
178 178
         }
@@ -232,9 +232,9 @@  discard block
 block discarded – undo
232 232
             try {
233 233
                 $user          = $auth->getUser();
234 234
                 $password      = substr(md5(uniqid()), 0, 6);
235
-                $login         = uniqid() . ($this->options->auth_suffix != "" ? '@' . $this->options->auth_suffix : '');
235
+                $login         = uniqid().($this->options->auth_suffix != "" ? '@'.$this->options->auth_suffix : '');
236 236
                 $externalLogin = $user->getLogin() ?: '-- not communicated --';
237
-                $this->logger->debug('first login via ' . $provider . ' as: ' . $externalLogin);
237
+                $this->logger->debug('first login via '.$provider.' as: '.$externalLogin);
238 238
 
239 239
                 $user->setLogin($login);
240 240
                 $user->setPassword($password);
@@ -253,11 +253,11 @@  discard block
 block discarded – undo
253 253
                 );
254 254
                 $mail->addTo($user->getInfo()->getEmail());
255 255
 
256
-                $loggerId = $login . ' (' . $provider . ': ' . $externalLogin . ')';
256
+                $loggerId = $login.' ('.$provider.': '.$externalLogin.')';
257 257
                 if (isset($mail) && $this->mailer($mail)) {
258
-                    $this->logger->info('Mail first-login for ' . $loggerId . ' sent to ' . $user->getInfo()->getEmail());
258
+                    $this->logger->info('Mail first-login for '.$loggerId.' sent to '.$user->getInfo()->getEmail());
259 259
                 } else {
260
-                    $this->logger->warn('No Mail was sent for ' . $loggerId);
260
+                    $this->logger->warn('No Mail was sent for '.$loggerId);
261 261
                 }
262 262
             } catch (\Exception $e) {
263 263
                 $this->logger->crit($e);
@@ -268,11 +268,11 @@  discard block
 block discarded – undo
268 268
         }
269 269
         
270 270
         $user = $auth->getUser();
271
-        $this->logger->info('User ' . $auth->getUser()->getInfo()->getDisplayName() . ' logged in via ' . $provider);
271
+        $this->logger->info('User '.$auth->getUser()->getInfo()->getDisplayName().' logged in via '.$provider);
272 272
         $settings = $user->getSettings('Core');
273 273
         if (null !== $settings->localization->language) {
274 274
             $basePath = $this->getRequest()->getBasePath();
275
-            $ref = preg_replace('~^'.$basePath . '/[a-z]{2}(?=/|$)~', $basePath . '/' . $settings->localization->language, $ref);
275
+            $ref = preg_replace('~^'.$basePath.'/[a-z]{2}(?=/|$)~', $basePath.'/'.$settings->localization->language, $ref);
276 276
         }
277 277
         return $this->redirect()->toUrl($ref);
278 278
     }
@@ -303,8 +303,8 @@  discard block
 block discarded – undo
303 303
         
304 304
         if ($result->isValid()) {
305 305
             $this->logger->info(
306
-                'User ' . $this->params()->fromPost('user') .
307
-                ' logged via ' . $appKey
306
+                'User '.$this->params()->fromPost('user').
307
+                ' logged via '.$appKey
308 308
             );
309 309
             
310 310
             // the external login may include some parameters for an update
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
             if (array_key_exists('firstLogin', $resultMessage) && $resultMessage['firstLogin'] === true) {
336 336
                 // first external Login
337 337
                 $userName = $this->params()->fromPost('user');
338
-                $this->logger->debug('first login for User: ' .  $userName);
338
+                $this->logger->debug('first login for User: '.$userName);
339 339
                 //
340 340
                 if (preg_match('/^(.*)@\w+$/', $userName, $realUserName)) {
341 341
                     $userName = $realUserName[1];
@@ -359,7 +359,7 @@  discard block
 block discarded – undo
359 359
 
360 360
                 try {
361 361
                     $this->mailer($mail);
362
-                    $this->logger->info('Mail first-login sent to ' . $userName);
362
+                    $this->logger->info('Mail first-login sent to '.$userName);
363 363
                 } catch (\Zend\Mail\Transport\Exception\ExceptionInterface $e) {
364 364
                     $this->logger->warn('No Mail was sent');
365 365
                     $this->logger->debug($e);
@@ -374,8 +374,8 @@  discard block
 block discarded – undo
374 374
             );
375 375
         } else {
376 376
             $this->logger->info(
377
-                'Failed to authenticate User ' . $this->params()->fromPost('user') .
378
-                ' via ' . $this->params()->fromPost('appKey')
377
+                'Failed to authenticate User '.$this->params()->fromPost('user').
378
+                ' via '.$this->params()->fromPost('appKey')
379 379
             );
380 380
             
381 381
             $this->getResponse()->setStatusCode(Response::STATUS_CODE_401);
@@ -400,8 +400,8 @@  discard block
 block discarded – undo
400 400
                 array(
401 401
                 'format' => 'json',
402 402
                     'group' => array(
403
-                        0 => 'testuser4711', 1 => 'flatscreen', 2 => 'flatscreen1', 3 => 'flatscreen2', 4 => 'flatscreen3',  5 => 'flatscreen4',
404
-                        6 => 'flatscreen5', 7 => 'flatscreen6', 8 => 'flatscreen7',  9 => 'flatscreen8', 10 => 'flatscreen9'
403
+                        0 => 'testuser4711', 1 => 'flatscreen', 2 => 'flatscreen1', 3 => 'flatscreen2', 4 => 'flatscreen3', 5 => 'flatscreen4',
404
+                        6 => 'flatscreen5', 7 => 'flatscreen6', 8 => 'flatscreen7', 9 => 'flatscreen8', 10 => 'flatscreen9'
405 405
                     ),
406 406
                     'name' => '(die) Rauscher – Unternehmensberatung & Consulting',
407 407
                 )
@@ -410,7 +410,7 @@  discard block
 block discarded – undo
410 410
         }
411 411
         $auth = $this->auth;
412 412
         $userGrpAdmin = $auth->getUser();
413
-        $this->logger->info('User ' . $auth->getUser()->getInfo()->getDisplayName());
413
+        $this->logger->info('User '.$auth->getUser()->getInfo()->getDisplayName());
414 414
         $grp = $this->params()->fromQuery('group');
415 415
         
416 416
         // if the request is made by an external host, add his identification-key to the name
@@ -431,16 +431,16 @@  discard block
 block discarded – undo
431 431
             foreach ($params->group as $grp_member) {
432 432
                 try
433 433
                 {
434
-                    $user = $users->findByLogin($grp_member . $loginSuffix);
434
+                    $user = $users->findByLogin($grp_member.$loginSuffix);
435 435
                     if (!empty($user)) {
436 436
                         $groupUserId[] = $user->id;
437 437
                     } else {
438
-                        $notFoundUsers[] = $grp_member . $loginSuffix;
438
+                        $notFoundUsers[] = $grp_member.$loginSuffix;
439 439
                     }
440 440
                 }
441 441
                 catch (\Auth\Exception\UserDeactivatedException $e)
442 442
                 {
443
-                    $notFoundUsers[] = $grp_member . $loginSuffix;
443
+                    $notFoundUsers[] = $grp_member.$loginSuffix;
444 444
                 }
445 445
             }
446 446
         }
@@ -450,9 +450,9 @@  discard block
 block discarded – undo
450 450
             $group->setUsers($groupUserId);
451 451
         }
452 452
         $this->logger->info(
453
-            'Update Group Name: ' . $name . PHP_EOL . str_repeat(' ', 36) . 'Group Owner: ' . $userGrpAdmin->getLogin() . PHP_EOL .
454
-            str_repeat(' ', 36) . 'Group Members Param: ' . implode(',', $params->group) . PHP_EOL .
455
-            str_repeat(' ', 36) . 'Group Members: ' . count($groupUserId) . PHP_EOL . str_repeat(' ', 36) . 'Group Members not found: ' . implode(',', $notFoundUsers)
453
+            'Update Group Name: '.$name.PHP_EOL.str_repeat(' ', 36).'Group Owner: '.$userGrpAdmin->getLogin().PHP_EOL.
454
+            str_repeat(' ', 36).'Group Members Param: '.implode(',', $params->group).PHP_EOL.
455
+            str_repeat(' ', 36).'Group Members: '.count($groupUserId).PHP_EOL.str_repeat(' ', 36).'Group Members not found: '.implode(',', $notFoundUsers)
456 456
         );
457 457
         
458 458
         return new JsonModel(
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
     public function logoutAction()
470 470
     {
471 471
         $auth = $this->auth;
472
-        $this->logger->info('User ' . ($auth->getUser()->getLogin()==''?$auth->getUser()->getInfo()->getDisplayName():$auth->getUser()->getLogin()) . ' logged out');
472
+        $this->logger->info('User '.($auth->getUser()->getLogin() == '' ? $auth->getUser()->getInfo()->getDisplayName() : $auth->getUser()->getLogin()).' logged out');
473 473
         $auth->clearIdentity();
474 474
         unset($_SESSION['HA::STORE']);
475 475
 
Please login to merge, or discard this patch.
module/Auth/src/Auth/Controller/UsersController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
     /**
41 41
      * @param UserRepository $userRepository
42 42
      */
43
-    public function __construct(UserRepository $userRepository,$formManager,$viewHelper)
43
+    public function __construct(UserRepository $userRepository, $formManager, $viewHelper)
44 44
     {
45 45
         $this->userRepository = $userRepository;
46 46
         $this->formManager = $formManager;
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
         }
187 187
 
188 188
         $switcher = $this->plugin('Auth/User/Switcher');
189
-        $success  = $switcher($this->params()->fromQuery('id'), [ 'ref' => urldecode($this->params()->fromQuery('ref')) ]);
189
+        $success  = $switcher($this->params()->fromQuery('id'), ['ref' => urldecode($this->params()->fromQuery('ref'))]);
190 190
 
191 191
         return new JsonModel(['success' => true]);
192 192
     }
Please login to merge, or discard this patch.
module/Auth/src/Auth/Controller/ManageController.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -95,11 +95,11 @@  discard block
 block discarded – undo
95 95
         /* @var \Auth\Form\UserProfileContainer $userProfileContainer */
96 96
         $userProfileContainer = $this->userProfileContainer;
97 97
         $user = $this->authService->getUser(); /* @var $user \Auth\Entity\User */
98
-        $postProfiles = (array)$this->params()->fromPost('social_profiles');
98
+        $postProfiles = (array) $this->params()->fromPost('social_profiles');
99 99
         $userProfiles = $user->getProfile();
100 100
         $formSocialProfiles = $this->socialProfileForm
101 101
             ->setUseDefaultValidation(true)
102
-            ->setData(['social_profiles' => array_map(function ($array)
102
+            ->setData(['social_profiles' => array_map(function($array)
103 103
             {
104 104
                 return $array['data'];
105 105
             }, $userProfiles)]);
@@ -118,11 +118,11 @@  discard block
 block discarded – undo
118 118
             $form      = $userProfileContainer->getForm($formName);
119 119
             
120 120
             if ($form) {
121
-                $postData  = $form->getOption('use_post_array') ? $_POST : array();
121
+                $postData = $form->getOption('use_post_array') ? $_POST : array();
122 122
                 //@TODO: [ZF3] option use_files_array is false by default
123 123
                 //$filesData = $form->getOption('use_files_array') ? $_FILES : array();
124 124
 	            $filesData = $_FILES;
125
-                $data      = array_merge($postData, $filesData);
125
+                $data = array_merge($postData, $filesData);
126 126
                 $form->setData($data);
127 127
                 
128 128
                 if (!$form->isValid()) {
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
                                 ));
184 184
                             } else {
185 185
                                 $profile = [
186
-                                    'auth' => (array)$authProfile,
186
+                                    'auth' => (array) $authProfile,
187 187
                                     'data' => \Zend\Json\Json::decode($dataProfiles[$network])
188 188
                                 ];
189 189
                                 $user->addProfile($network, $profile);
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
                 }
194 194
                 
195 195
                 // keep data in sync & properly decoded
196
-                $formSocialProfiles->setData(['social_profiles' => array_map(function ($array)
196
+                $formSocialProfiles->setData(['social_profiles' => array_map(function($array)
197 197
                 {
198 198
                     return \Zend\Json\Json::decode($array) ?: '';
199 199
                 }, $dataProfiles)]);
Please login to merge, or discard this patch.