zikula / core

src/system/RoutesModule/Base/AbstractRoutesModuleInstaller.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
     /**
      * Build array with all entity classes for ZikulaRoutesModule.
      *
-     * @return array list of class names
+     * @return string[] list of class names
      */
     protected function listEntityClasses()
     {

src/system/PermissionsModule/Twig/Extension/PermissionsExtension.php

Spacing +1 added lines, -1 removed lines

@@ -62,6 +62,6 @@
 
         $result = $this->container->get('zikula_permissions_module.api.permission')->hasPermission($component, $instance, constant($level));
 
-        return (bool) $result;
+        return (bool)$result;
     }
 }

src/system/GroupsModule/Controller/MembershipController.php

Spacing +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
         if ($gid < 1 || $uid < 1) {
             throw new \InvalidArgumentException($this->__('Invalid Group ID or User ID.'));
         }
-        if (!$this->hasPermission('ZikulaGroupsModule::', $gid.'::', ACCESS_EDIT)) {
+        if (!$this->hasPermission('ZikulaGroupsModule::', $gid . '::', ACCESS_EDIT)) {
             throw new AccessDeniedException();
         }
         $group = $this->get('zikula_groups_module.group_repository')->find($gid);

src/system/GroupsModule/GroupsModuleInstaller.php

Spacing +2 added lines, -2 removed lines

@@ -66,8 +66,8 @@
         switch ($oldVersion) {
             case '2.3.2':
             case '2.4.0':
-                $this->setVar('mailwarning', (bool) $this->getVar('mailwarning'));
-                $this->setVar('hideclosed', (bool) $this->getVar('hideclosed'));
+                $this->setVar('mailwarning', (bool)$this->getVar('mailwarning'));
+                $this->setVar('hideclosed', (bool)$this->getVar('hideclosed'));
                 $this->setVar('hidePrivate', false);
             case '2.4.1':
             // future upgrade routines

src/lib/Zikula/Bundle/CoreBundle/Twig/Extension/PagerExtension.php

Spacing +13 added lines, -13 removed lines

@@ -139,7 +139,7 @@ 
                     case 'lang':
                         $addcurrentlang2url = $systemVars['languageurl'];
                         if ($addcurrentlang2url == 0) {
-                            $pager['args'][$k] =  $v;
+                            $pager['args'][$k] = $v;
                         }
                         break;
                     default:
@@ -161,19 +161,19 @@ 
                                     }
                                 } elseif (strlen($vv)) {
                                     $tkey = $k . '[' . $kk . ']';
-                                    $pager['args'][$tkey] =  $vv;
+                                    $pager['args'][$tkey] = $vv;
                                 }
                             }
                         } else {
                             if (strlen($v)) {
-                                $pager['args'][$k] =  $v;
+                                $pager['args'][$k] = $v;
                             }
                         }
                 }
             }
         }
 
-        $pagerUrl = function ($pager) use ($routeName, $systemVars) {
+        $pagerUrl = function($pager) use ($routeName, $systemVars) {
             if ($routeName) {
                 return $this->container->get('router')->generate($routeName, $pager['args']);
             }
@@ -373,7 +373,7 @@ 
             // predefined abc
             if (strtolower($params['skin']) == 'hu') {
                 // Hungarian
-                $pager['names']  = $pager['values'] = ['A', '?', 'B', 'C', 'D', 'E', '?', 'F', 'G', 'H', 'I', '?', 'J', 'K', 'L', 'M', 'N', 'O', '?', '?', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', '?', '?', 'U', 'V', 'W', 'X', 'Y', 'Z'];
+                $pager['names'] = $pager['values'] = ['A', '?', 'B', 'C', 'D', 'E', '?', 'F', 'G', 'H', 'I', '?', 'J', 'K', 'L', 'M', 'N', 'O', '?', '?', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', '?', '?', 'U', 'V', 'W', 'X', 'Y', 'Z'];
                 //$params['names']  = array('A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U'    ,'V','W','X','Y','Z');
                 //$params['values'] = array('A,?','B','C','D','E,?','F','G','H','I,?','J','K','L','M','N','O,?,?,O','P','Q','R','S','T','U,?,?,U','V','W','X','Y','Z');
             } else {
@@ -387,7 +387,7 @@ 
         unset($params['names']);
         unset($params['values']);
         unset($params['route']);
-        $pagerUrl = function ($pager) {
+        $pagerUrl = function($pager) {
             return $this->container->get('router')->generate($pager['route'], $pager['args']);
         };
         $allVars = array_merge($request->request->all(), $request->query->all(), $request->attributes->get('_route_params', []));
@@ -418,38 +418,38 @@ 
         }
         unset($pager['args'][$pager['posvar']]);
         // begin to fill the output
-        $output = '<ul class="'.$params['class'].'">'."\n";
+        $output = '<ul class="' . $params['class'] . '">' . "\n";
         $style = '';
         if ($params['printempty']) {
             $active = '';
             if (!empty($params['class_numon'])) {
                 if (!isset($allVars[$pager['posvar']])) {
-                    $style = ' class="'.$params['class_numon'].'"';
+                    $style = ' class="' . $params['class_numon'] . '"';
                     $active = 'class="active"';
                 } elseif (!empty($params['class_num'])) {
-                    $style = ' class="'.$params['class_num'].'"';
+                    $style = ' class="' . $params['class_num'] . '"';
                 } else {
                     $style = '';
                 }
             }
             $vars[$pager['posvar']] = '';
-            $output .= '<li '.$active.'><a '.$style.' href="'.$pagerUrl($pager).'"> -'."\n</a></li>";
+            $output .= '<li ' . $active . '><a ' . $style . ' href="' . $pagerUrl($pager) . '"> -' . "\n</a></li>";
         }
         $style = '';
         foreach (array_keys($pager['names']) as $i) {
             $active = '';
             if (!empty($params['class_numon'])) {
                 if (isset($allVars[$pager['posvar']]) && $allVars[$pager['posvar']] == $pager['values'][$i]) {
-                    $style = ' class="'.$params['class_numon'].'"';
+                    $style = ' class="' . $params['class_numon'] . '"';
                     $active = 'class="active"';
                 } elseif (!empty($params['class_num'])) {
-                    $style = ' class="'.$params['class_num'].'"';
+                    $style = ' class="' . $params['class_num'] . '"';
                 } else {
                     $style = '';
                 }
             }
             $pager['args'][$pager['posvar']] = $pager['values'][$i];
-            $output .= '<li '.$active.'><a '.$style.' href="'.$pagerUrl($pager).'">'.$pager['names'][$i]."</a></li>\n";
+            $output .= '<li ' . $active . '><a ' . $style . ' href="' . $pagerUrl($pager) . '">' . $pager['names'][$i] . "</a></li>\n";
         }
         $output .= "</ul>\n";
 

Doc Comments +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     /**
      * Returns a list of functions to add to the existing list.
      *
-     * @return array An array of functions
+     * @return \Twig_SimpleFunction[] An array of functions
      */
     public function getFunctions()
     {

src/system/SettingsModule/Api/LocaleApi.php

Spacing +2 added lines, -2 removed lines

@@ -59,11 +59,11 @@
         $availableLanguages = [];
         foreach ($matches as $match) {
             list($languageCode, $unusedVar) = explode('-', $match[1]) + ['', ''];
-            $priority = isset($match[2]) ? (float) $match[2] : 1.0;
+            $priority = isset($match[2]) ? (float)$match[2] : 1.0;
             $availableLanguages[][$languageCode] = $priority;
         }
 
-        $defaultPriority = (float) 0;
+        $defaultPriority = (float)0;
         $matchedLanguage = '';
         foreach ($availableLanguages as $key => $value) {
             $languageCode = key($value);

src/system/AdminModule/Controller/AdminInterfaceController.php

Spacing +3 added lines, -3 removed lines

@@ -182,9 +182,9 @@ 
                     $ldir = dirname(__FILE__);
                     $p = strpos($ldir, DIRECTORY_SEPARATOR . 'system'); // we are in system/AdminModule
                     $b = substr($ldir, 0, $p);
-                    $filePath = $b.'/'.$appDir.'/.htaccess';
+                    $filePath = $b . '/' . $appDir . '/.htaccess';
                 } else {
-                    $filePath = $appDir.'/.htaccess';
+                    $filePath = $appDir . '/.htaccess';
                 }
                 $app_htaccess = file_exists($filePath);
             }
@@ -199,7 +199,7 @@ 
         return $this->render('@ZikulaAdminModule/AdminInterface/securityAnalyzer.html.twig', [
             'security' => [
                 'config_php' => is_writable('config/config.php'),
-                'magic_quotes_gpc' => (bool) get_magic_quotes_gpc(), // should be off anyway...
+                'magic_quotes_gpc' => (bool)get_magic_quotes_gpc(), // should be off anyway...
                 'app_htaccess' => $app_htaccess,
                 'updatecheck' => $variableApi->getSystemVar('updatecheck'),
                 'scactive' => $hasSecurityCenter,

src/system/UsersModule/Twig/Extension/ProfileExtension.php

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
             // truncate the user name to $maxLength chars
             $length = strlen($userDisplayName);
             $truncEnd = ($maxLength > $length) ? $length : $maxLength;
-            $show  = htmlspecialchars(substr($userDisplayName, 0, $truncEnd), ENT_QUOTES);
+            $show = htmlspecialchars(substr($userDisplayName, 0, $truncEnd), ENT_QUOTES);
         } else {
             $show = htmlspecialchars($userDisplayName, ENT_QUOTES);
         }

src/lib/Zikula/Bundle/CoreBundle/Console/Application.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         $this->kernel = $kernel;
 
         $this->setName('Zikula');
-        $this->setVersion(ZikulaKernel::VERSION.' - '.$kernel->getName().'/'.$kernel->getEnvironment().($kernel->isDebug() ? '/debug' : ''));
+        $this->setVersion(ZikulaKernel::VERSION . ' - ' . $kernel->getName() . '/' . $kernel->getEnvironment() . ($kernel->isDebug() ? '/debug' : ''));
     }
 
     protected function registerCommands()