Completed
Push — master ( ef0404...81c807 )
by Craig
23:17 queued 14:56
created
src/system/UsersModule/Controller/FileIOController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
             if ($form->get('download')->isClicked()) {
47 47
                 $data = $form->getData();
48 48
                 $response = new StreamedResponse();
49
-                $response->setCallback(function () use ($data) {
49
+                $response->setCallback(function() use ($data) {
50 50
                     $fields = ['uid', 'uname', 'activated', 'email', 'user_regdate', 'lastlogin', 'groups'];
51 51
                     foreach ($fields as $k => $field) {
52 52
                         if (isset($data[$field]) && !$data[$field]) {
Please login to merge, or discard this patch.
src/system/UsersModule/UsersModuleInstaller.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
                     UsersConstant::MODVAR_LOGIN_DISPLAY_APPROVAL_STATUS
118 118
                 ];
119 119
                 foreach ($modvarsToConvertToBool as $modvarToConvert) {
120
-                    $this->setVar($modvarToConvert, (bool) $this->getVar($modvarToConvert));
120
+                    $this->setVar($modvarToConvert, (bool)$this->getVar($modvarToConvert));
121 121
                 }
122 122
                 $this->schemaTool->update(['Zikula\UsersModule\Entity\UserEntity']);
123 123
                 $this->delVar('login_redirect');
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
             $value = in_array($migratedModVarName, [
291 291
                 ZAuthConstant::MODVAR_EMAIL_VERIFICATION_REQUIRED,
292 292
                 ZAuthConstant::MODVAR_PASSWORD_STRENGTH_METER_ENABLED
293
-            ]) ? (bool) $value : $value;
293
+            ]) ? (bool)$value : $value;
294 294
             $this->container->get('zikula_extensions_module.api.variable')->set('ZikulaZAuthModule', $migratedModVarName, $value);
295 295
         }
296 296
     }
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -295,7 +295,7 @@
 block discarded – undo
295 295
 
296 296
     /**
297 297
      * These modvar names used to have UsersConstant values, but have been moved to ZAuthConstant and maintain their actual values.
298
-     * @return array
298
+     * @return string[]
299 299
      */
300 300
     private function getMigratedModVarNames()
301 301
     {
Please login to merge, or discard this patch.
src/system/MailerModule/Api/MailerApi.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -175,7 +175,7 @@
 block discarded – undo
175 175
 
176 176
         // add body with formatting
177 177
         $bodyFormat = 'text/plain';
178
-        if (!empty($altBody) || ((bool) $html) || $this->dataValues['html']) {
178
+        if (!empty($altBody) || ((bool)$html) || $this->dataValues['html']) {
179 179
             $bodyFormat = 'text/html';
180 180
         }
181 181
         if (isset($body)) {
Please login to merge, or discard this patch.
src/system/SecurityCenterModule/Entity/Repository/IntrusionRepository.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
                 $qb->from('ZikulaUsersModule:UserEntity', 'u');
109 109
                 $qb->andWhere($qb->expr()->eq('tbl.user', 'u.uid'));
110 110
                 $qb->andWhere($qb->expr()->eq('tbl.user', ':uid'))
111
-                   ->setParameter('uid', $uid);
111
+                    ->setParameter('uid', $uid);
112 112
             }
113 113
         }
114 114
 
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
             $i = 1;
118 118
             foreach ($filters as $w_key => $w_value) {
119 119
                 $qb->andWhere($qb->expr()->eq('tbl.' . $w_key, '?' . $i))
120
-                   ->setParameter($i, $w_value);
120
+                    ->setParameter($i, $w_value);
121 121
                 $i++;
122 122
             }
123 123
         }
@@ -144,12 +144,12 @@  discard block
 block discarded – undo
144 144
 
145 145
         if ($fieldName == 'uid') {
146 146
             $qb->select('DISTINCT(u.' . $fieldName . ')')
147
-               ->from('ZikulaUsersModule:UserEntity', 'u')
148
-               ->where($qb->expr()->eq('tbl.user', 'u.uid'))
149
-               ->addOrderBy('u.uname', 'ASC');
147
+                ->from('ZikulaUsersModule:UserEntity', 'u')
148
+                ->where($qb->expr()->eq('tbl.user', 'u.uid'))
149
+                ->addOrderBy('u.uname', 'ASC');
150 150
         } else {
151 151
             $qb->select('DISTINCT(tbl.' . $fieldName . ')')
152
-               ->addOrderBy('tbl.' . $fieldName, 'ASC');
152
+                ->addOrderBy('tbl.' . $fieldName, 'ASC');
153 153
         }
154 154
 
155 155
         $query = $qb->getQuery();
Please login to merge, or discard this patch.
src/system/AdminModule/Controller/AdminInterfaceController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -184,9 +184,9 @@
 block discarded – undo
184 184
                     $ldir = dirname(__FILE__);
185 185
                     $p = strpos($ldir, DIRECTORY_SEPARATOR . 'system'); // we are in system/AdminModule
186 186
                     $b = substr($ldir, 0, $p);
187
-                    $filePath = $b.'/'.$appDir.'/.htaccess';
187
+                    $filePath = $b . '/' . $appDir . '/.htaccess';
188 188
                 } else {
189
-                    $filePath = $appDir.'/.htaccess';
189
+                    $filePath = $appDir . '/.htaccess';
190 190
                 }
191 191
                 $app_htaccess = file_exists($filePath);
192 192
             }
Please login to merge, or discard this patch.
src/lib/Zikula/Bundle/CoreBundle/EventListener/ExceptionListener.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@
 block discarded – undo
85 85
      * Handle an AccessDeniedException
86 86
      *
87 87
      * @param GetResponseForExceptionEvent $event
88
-     * @param $userLoggedIn
88
+     * @param boolean $userLoggedIn
89 89
      * @param string $message a custom error message (default: 'Access Denied') (The default message from Symfony)
90 90
      * @see http://api.symfony.com/2.8/Symfony/Component/Security/Core/Exception/AccessDeniedException.html
91 91
      */
Please login to merge, or discard this patch.
src/lib/Zikula/Core/AbstractExtensionInstaller.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -112,8 +112,8 @@
 block discarded – undo
112 112
 
113 113
     /**
114 114
      * Convenience shortcut to add a session flash message.
115
-     * @param $type
116
-     * @param $message
115
+     * @param string $type
116
+     * @param string $message
117 117
      */
118 118
     public function addFlash($type, $message)
119 119
     {
Please login to merge, or discard this patch.
src/lib/Zikula/Core/LinkContainer/LinkContainerCollector.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -31,6 +31,10 @@
 block discarded – undo
31 31
         $this->linkContainers[$linkContainer->getBundleName()] = $linkContainer;
32 32
     }
33 33
 
34
+    /**
35
+     * @param string $containerName
36
+     * @param string $type
37
+     */
34 38
     public function getLinks($containerName, $type = LinkContainerInterface::TYPE_ADMIN)
35 39
     {
36 40
         $links = [];
Please login to merge, or discard this patch.
src/system/ExtensionsModule/ExtensionsModuleInstaller.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@
 block discarded – undo
55 55
      * This function must consider all the released versions of the module!
56 56
      * If the upgrade fails at some point, it returns the last upgraded version.
57 57
      *
58
-     * @param string $oldversion Version number string to upgrade from
58
+     * @param string $oldVersion Version number string to upgrade from
59 59
      *
60 60
      * @return  boolean|string True on success, last valid version string or false if fails
61 61
      */
Please login to merge, or discard this patch.