zikula / core

src/system/SecurityCenterModule/SecurityCenterModuleInstaller.php

Doc Comments +3 added lines

@@ -251,6 +251,9 @@
         return false;
     }
 
+    /**
+     * @param string $name
+     */
     private function setSystemVar($name, $value = '')
     {
         return $this->container->get('zikula_extensions_module.api.variable')->set(VariableApi::CONFIG, $name, $value);

Spacing +8 added lines, -8 removed lines

@@ -55,7 +55,7 @@ 
         $this->setSystemVar('sessionstoretofile', Constant::SESSION_STORAGE_FILE);
         $this->setSystemVar('sessionsavepath', '');
         $this->setSystemVar('gc_probability', 100);
-        $this->setSystemVar('sessioncsrftokenonetime', 1);  // 1 means use same token for entire session
+        $this->setSystemVar('sessioncsrftokenonetime', 1); // 1 means use same token for entire session
         $this->setSystemVar('sessionrandregenerate', 1);
         $this->setSystemVar('sessionregenerate', 1);
         $this->setSystemVar('sessionregeneratefreq', 10);
@@ -77,13 +77,13 @@ 
         $this->setSystemVar('useids', 0);
         $this->setSystemVar('idsmail', 0);
         $this->setSystemVar('idsrulepath', 'system/SecurityCenterModule/Resources/config/phpids_zikula_default.xml');
-        $this->setSystemVar('idssoftblock', 1);                // do not block requests, but warn for debugging
-        $this->setSystemVar('idsfilter', 'xml');               // filter type
-        $this->setSystemVar('idsimpactthresholdone', 1);       // db logging
-        $this->setSystemVar('idsimpactthresholdtwo', 10);      // mail admin
-        $this->setSystemVar('idsimpactthresholdthree', 25);    // block request
-        $this->setSystemVar('idsimpactthresholdfour', 75);     // kick user, destroy session
-        $this->setSystemVar('idsimpactmode', 1);               // per request per default
+        $this->setSystemVar('idssoftblock', 1); // do not block requests, but warn for debugging
+        $this->setSystemVar('idsfilter', 'xml'); // filter type
+        $this->setSystemVar('idsimpactthresholdone', 1); // db logging
+        $this->setSystemVar('idsimpactthresholdtwo', 10); // mail admin
+        $this->setSystemVar('idsimpactthresholdthree', 25); // block request
+        $this->setSystemVar('idsimpactthresholdfour', 75); // kick user, destroy session
+        $this->setSystemVar('idsimpactmode', 1); // per request per default
         $this->setSystemVar('idshtmlfields', ['POST.__wysiwyg']);
         $this->setSystemVar('idsjsonfields', ['POST.__jsondata']);
         $this->setSystemVar('idsexceptions', [

src/system/ThemeModule/Engine/Asset/Merger.php

Doc Comments +1 added lines, -1 removed lines

@@ -284,7 +284,7 @@
 
     /**
      * Remove comments, whitespace and spaces from css files
-     * @param $contents
+     * @param string $contents
      * @return string
      */
     private function minify($contents)

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
                 }
             }
         }
-        $cacheName = in_array($type, ['js', 'css']) ? "{$type}Cache" : null;
+        $cacheName = in_array($type, ['js', 'css']) ? "{$type}cache" : null;
         /** @var CacheProvider $cacheService */
         $cacheService = $this->$cacheName;
         $key = md5(serialize($assets)) . (int)$this->minify . (int)$this->compress . $this->lifetime . '.' . $type;

src/system/UsersModule/Collector/AuthenticationMethodCollector.php

Doc Comments +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
     }
 
     /**
-     * @return array of service aliases
+     * @return integer[] of service aliases
      */
     public function getKeys()
     {
@@ -99,7 +99,7 @@ 
     }
 
     /**
-     * @return array of active service aliases
+     * @return integer[] of active service aliases
      */
     public function getActiveKeys()
     {

src/system/UsersModule/Helper/MailHelper.php

Doc Comments +3 added lines

@@ -262,6 +262,9 @@
         return $this->mailerApi->sendMessage($message, null, null, $textBody, $html);
     }
 
+    /**
+     * @param string $notificationType
+     */
     private function generateEmailSubject($notificationType, array $templateArgs = [])
     {
         $siteName = $this->variableApi->getSystemVar('sitename');

src/system/ZAuthModule/Helper/FileIOHelper.php

Doc Comments +5 added lines, -2 removed lines

@@ -110,6 +110,9 @@ 
         $this->passwordApi = $passwordApi;
     }
 
+    /**
+     * @param TranslatorInterface $translator
+     */
     public function setTranslator($translator)
     {
         $this->translator = $translator;
@@ -273,8 +276,8 @@ 
     /**
      * Convert errors to string and add current line.
      * @param $errors
-     * @param $type
-     * @param $line
+     * @param string $type
+     * @param integer $line
      * @return string
      */
     private function locateErrors($errors, $type, $line)

src/system/ZAuthModule/Helper/MailHelper.php

Doc Comments +3 added lines

@@ -96,6 +96,9 @@
         return $this->mailerApi->sendMessage($message, $subject, $body, $altBody, $html);
     }
 
+    /**
+     * @param string $notificationType
+     */
     private function generateEmailSubject($notificationType, array $templateArgs = [])
     {
         $siteName = $this->variableApi->getSystemVar('sitename');

src/lib/Zikula/Core/AbstractBundle.php

Spacing +6 added lines, -6 removed lines

@@ -54,7 +54,7 @@ 
     public function getInstallerClass()
     {
         $ns = $this->getNamespace();
-        $class = $ns.'\\'.substr($ns, strrpos($ns, '\\') + 1, strlen($ns)).'Installer';
+        $class = $ns . '\\' . substr($ns, strrpos($ns, '\\') + 1, strlen($ns)) . 'Installer';
 
         return $class;
     }
@@ -76,7 +76,7 @@ 
      */
     public function getLocalePath()
     {
-        return $this->getPath().'/Resources/locale';
+        return $this->getPath() . '/Resources/locale';
     }
 
     /**
@@ -86,7 +86,7 @@ 
      */
     public function getViewsPath()
     {
-        return $this->getPath().'/Resources/views';
+        return $this->getPath() . '/Resources/views';
     }
 
     /**
@@ -96,7 +96,7 @@ 
      */
     public function getConfigPath()
     {
-        return $this->getPath().'/Resources/config';
+        return $this->getPath() . '/Resources/config';
     }
 
     /**
@@ -144,9 +144,9 @@ 
         $type = $this->getNameType();
         $typeLower = strtolower($type);
         if (null === $this->extension) {
-            $basename = preg_replace('/'.$type.'/', '', $this->getName());
+            $basename = preg_replace('/' . $type . '/', '', $this->getName());
 
-            $class = $this->getNamespace().'\\DependencyInjection\\'.$basename.'Extension';
+            $class = $this->getNamespace() . '\\DependencyInjection\\' . $basename . 'Extension';
             if (class_exists($class)) {
                 $extension = new $class();
 

src/lib/Zikula/Bundle/CoreBundle/Bundle/Scanner.php

Doc Comments +7 added lines

@@ -92,6 +92,9 @@ 
         return false;
     }
 
+    /**
+     * @param string $file
+     */
     public function getFileContents($file)
     {
         return file_get_contents($file);
@@ -142,6 +145,10 @@ 
         return true;
     }
 
+    /**
+     * @param string $type
+     * @param boolean $indexByShortName
+     */
     private function getMetaData($type, $indexByShortName)
     {
         $array = [];

src/lib/Zikula/Component/FilterUtil/FilterUtil.php

Doc Comments +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
     /**
      * Set filterstring.
      *
-     * @param mixed $filter Filter string or array
+     * @param string $filter Filter string or array
      *
      * @return void
      */

Spacing +7 added lines, -7 removed lines

@@ -125,7 +125,7 @@ 
 
         // Get filter1 ... filterN
         while (true) {
-            $filterURLName = $this->filterKey."$i";
+            $filterURLName = $this->filterKey . "$i";
             $filterStr = $this->request->query->filter(
                 $filterURLName,
                 '',
@@ -153,7 +153,7 @@ 
         if (!isset($this->filter) || empty($this->filter)) {
             $filter = $this->getFiltersFromInput();
             if (is_array($filter) && count($filter) > 0) {
-                $this->filter = "(".implode(')*(', $filter).")";
+                $this->filter = "(" . implode(')*(', $filter) . ")";
             }
         }
 
@@ -174,7 +174,7 @@ 
     public function setFilter($filter)
     {
         if (is_array($filter)) {
-            $this->filter = "(".implode(')*(', $filter).")";
+            $this->filter = "(" . implode(')*(', $filter) . ")";
         } else {
             $this->filter = $filter;
         }
@@ -221,9 +221,9 @@ 
         } elseif (substr($filter, 0, 1) == ',') {
             $this->filter .= $filter;
         } elseif (substr($filter, 0, 1) == '*') {
-            $this->filter .= ','.(substr($filter, 1));
+            $this->filter .= ',' . (substr($filter, 1));
         } else {
-            $this->filter .= ','.($filter);
+            $this->filter .= ',' . ($filter);
         }
     }
 
@@ -243,9 +243,9 @@ 
         } elseif (substr($filter, 0, 1) == '*') {
             $this->filter .= $filter;
         } elseif (substr($filter, 0, 1) == ',') {
-            $this->filter .= '*'.(substr($filter, 1));
+            $this->filter .= '*' . (substr($filter, 1));
         } else {
-            $this->filter .= '*'.($filter);
+            $this->filter .= '*' . ($filter);
         }
     }
 

Braces +4 added lines, -2 removed lines

@@ -349,9 +349,11 @@
                             break;
                         }
                     }
-                    if ($or === null) { // make new or Object
+                    if ($or === null) {
+// make new or Object
                         $or = new Orx();
-                        if ($and !== null) { // add existing and
+                        if ($and !== null) {
+// add existing and
                             $this->addBtoA($or, $and);
                         }
                     }