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/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/lib/Zikula/Bundle/CoreInstallerBundle/Util/ZikulaRequirements.php

Spacing +4 added lines, -4 removed lines

@@ -24,7 +24,7 @@ 
     public function runSymfonyChecks($parameters = [])
     {
         try {
-            $path = realpath(__DIR__.'/../../../../../var/SymfonyRequirements.php');
+            $path = realpath(__DIR__ . '/../../../../../var/SymfonyRequirements.php');
             require $path;
             $symfonyRequirements = new \SymfonyRequirements();
             $this->addZikulaPathRequirements($symfonyRequirements, $parameters);
@@ -44,8 +44,8 @@ 
         if ($requirement->isFulfilled()) {
             return;
         }
-        $errorMessage = wordwrap($requirement->getTestMessage(), $lineSize - 3, PHP_EOL.'   ').PHP_EOL;
-        $errorMessage .= '   > '.wordwrap($requirement->getHelpText(), $lineSize - 5, PHP_EOL.'   > ').PHP_EOL;
+        $errorMessage = wordwrap($requirement->getTestMessage(), $lineSize - 3, PHP_EOL . '   ') . PHP_EOL;
+        $errorMessage .= '   > ' . wordwrap($requirement->getHelpText(), $lineSize - 5, PHP_EOL . '   > ') . PHP_EOL;
 
         return $errorMessage;
     }
@@ -66,7 +66,7 @@ 
         $symfonyRequirements->addRequirement(
             is_writable($src . '/' . $parameters['datadir']),
             $parameters['datadir'] . '/ directory must be writable',
-            'Change the permissions of "<strong>' . $parameters['datadir']. '</strong>" directory so that the web server can write into it.'
+            'Change the permissions of "<strong>' . $parameters['datadir'] . '</strong>" directory so that the web server can write into it.'
         );
     }
 }

src/system/BlocksModule/Twig/Extension/BlocksExtension.php

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/ExtensionsModule/Entity/Repository/ExtensionRepository.php

Doc Comments +6 added lines

@@ -53,6 +53,9 @@ 
         return $paginator;
     }
 
+    /**
+     * @param string $indexBy
+     */
     public function getIndexedArrayCollection($indexBy)
     {
         $qb = $this->createQueryBuilder('e')->indexBy('e', 'e.' . $indexBy);
@@ -78,6 +81,9 @@ 
         $this->_em->flush($entity);
     }
 
+    /**
+     * @param ExtensionEntity $entity
+     */
     public function removeAndFlush($entity)
     {
         $this->_em->remove($entity);

src/system/RoutesModule/Form/Handler/Route/Base/AbstractEditHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -378,7 +378,7 @@
     /**
      * Get list of allowed redirect codes.
      *
-     * @return array list of possible redirect codes
+     * @return string[] list of possible redirect codes
      */
     protected function getRedirectCodes()
     {

Spacing +2 added lines, -2 removed lines

@@ -223,7 +223,7 @@ 
         try {
             // execute the workflow action
             $success = $this->workflowHelper->executeAction($entity, $action);
-        } catch(\Exception $exception) {
+        } catch (\Exception $exception) {
             $flashBag->add('error', $this->__f('Sorry, but an error occured during the %action% action. Please apply the changes again!', ['%action%' => $action]) . ' ' . $exception->getMessage());
             $logArgs = ['app' => 'ZikulaRoutesModule', 'user' => $this->currentUserApi->get('uname'), 'entity' => 'route', 'id' => $entity->getKey(), 'errorMessage' => $exception->getMessage()];
             $this->logger->error('{app}: User {user} tried to edit the {entity} with id {id}, but failed. Error details: {errorMessage}.', $logArgs);
@@ -277,7 +277,7 @@ 
                 return $this->router->generate($routePrefix . 'view');
             case 'userOwnView':
             case 'adminOwnView':
-                return $this->router->generate($routePrefix . 'view', [ 'own' => 1 ]);
+                return $this->router->generate($routePrefix . 'view', ['own' => 1]);
             case 'userDisplay':
             case 'adminDisplay':
                 if ($args['commandName'] != 'delete' && !($this->templateParameters['mode'] == 'create' && $args['commandName'] == 'cancel')) {