zikula / core

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')) {

Unused Use Statements +2 added lines, -4 removed lines

@@ -12,13 +12,11 @@
 
 namespace Zikula\RoutesModule\Form\Handler\Route\Base;
 
+use RuntimeException;
+use Symfony\Component\HttpFoundation\RedirectResponse;
 use Zikula\RoutesModule\Form\Handler\Common\EditHandler;
 use Zikula\RoutesModule\Form\Type\RouteType;
 
-use Symfony\Component\HttpFoundation\RedirectResponse;
-use Symfony\Component\Security\Core\Exception\AccessDeniedException;
-use RuntimeException;
-
 /**
  * This handler class handles the page events of editing forms.
  * It aims on the route object type.

src/system/SettingsModule/SettingsModuleInstaller.php

Doc Comments +3 added lines

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

Spacing +3 added lines, -3 removed lines

@@ -142,9 +142,9 @@
                 $newStargArgs = str_replace(',', '&', $this->getSystemVar('startargs')); // replace comma with `&`
                 $this->setSystemVar('startargs', $newStargArgs);
             case '2.9.11':
-                $this->setSystemVar('shorturls', (bool) $this->getSystemVar('shorturls'));
-                $this->setSystemVar('shorturlsstripentrypoint', (bool) $this->getSystemVar('shorturlsstripentrypoint'));
-                $this->setSystemVar('useCompression', (bool) $this->getSystemVar('useCompression'));
+                $this->setSystemVar('shorturls', (bool)$this->getSystemVar('shorturls'));
+                $this->setSystemVar('shorturlsstripentrypoint', (bool)$this->getSystemVar('shorturlsstripentrypoint'));
+                $this->setSystemVar('useCompression', (bool)$this->getSystemVar('useCompression'));
             case '2.9.12': // ship with Core-1.4.4
                 // reconfigure TZ settings
                 $this->setGuestTimeZone();

src/system/ThemeModule/Engine/AssetFilter.php

Spacing +2 added lines, -2 removed lines

@@ -85,14 +85,14 @@
         $header .= implode("\n", $this->headers->all()) . "\n";
         $header .= ($this->scriptPosition == 'head') ? $this->jsResolver->compile() : '';
         if (strripos($source, '</head>')) {
-            $source = str_replace('</head>', $header."\n</head>", $source);
+            $source = str_replace('</head>', $header . "\n</head>", $source);
         }
 
         // compile and replace foot
         $footer = ($this->scriptPosition == 'foot') ? $this->jsResolver->compile() : '';
         $footer .= trim(implode("\n", $this->footers->all()) . "\n");
         if (false === empty($footer)) {
-            $source = str_replace('</body>', $footer."\n</body>", $source);
+            $source = str_replace('</body>', $footer . "\n</body>", $source);
         }
 
         return $source;

src/system/MenuModule/DependencyInjection/ZikulaMenuExtension.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      */
     public function load(array $configs, ContainerBuilder $container)
     {
-        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__.'/../Resources/config')));
+        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__ . '/../Resources/config')));
 
         $loader->load('services.yml');
     }

src/system/SearchModule/Entity/Repository/SearchResultRepository.php

Indentation +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
 
         if ($sessionId != '') {
             $qb->where('tbl.sesid = :sid')
-               ->setParameter('sid', $sessionId);
+                ->setParameter('sid', $sessionId);
         }
 
         $query = $qb->getQuery();
@@ -64,7 +64,7 @@ 
             $i = 1;
             foreach ($filters as $w_key => $w_value) {
                 $qb->andWhere($qb->expr()->eq('tbl.' . $w_key, '?' . $i))
-                   ->setParameter($i, $w_value);
+                    ->setParameter($i, $w_value);
                 $i++;
             }
         }
@@ -104,7 +104,7 @@ 
 
         if ($sessionId != '') {
             $qb->orWhere('tbl.sesid = :sid')
-               ->setParameter('sid', $sessionId);
+                ->setParameter('sid', $sessionId);
         }
 
         $query = $qb->getQuery();

src/system/PermissionsModule/DependencyInjection/ZikulaPermissionsExtension.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      */
     public function load(array $configs, ContainerBuilder $container)
     {
-        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__.'/../Resources/config')));
+        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__ . '/../Resources/config')));
 
         $loader->load('services.yml');
     }

src/system/GroupsModule/DependencyInjection/ZikulaGroupsExtension.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      */
     public function load(array $configs, ContainerBuilder $container)
     {
-        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__.'/../Resources/config')));
+        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__ . '/../Resources/config')));
 
         $loader->load('services.yml');
     }

src/system/GroupsModule/Entity/Repository/GroupRepository.php

Indentation +5 added lines, -5 removed lines

@@ -46,12 +46,12 @@ 
 
         if (null !== $groupType) {
             $qb->where('g.gtype = :gtype')
-               ->setParameter('gtype', $groupType);
+                ->setParameter('gtype', $groupType);
         }
 
         if (null !== $excludedState) {
             $qb->andWhere('g.state != :state')
-               ->setParameter('state', $excludedState);
+                ->setParameter('state', $excludedState);
         }
 
         $query = $qb->getQuery();
@@ -79,7 +79,7 @@ 
             $i = 1;
             foreach ($filters as $w_key => $w_value) {
                 $qb->andWhere($qb->expr()->eq('g.' . $w_key, '?' . $i))
-                   ->setParameter($i, $w_value);
+                    ->setParameter($i, $w_value);
                 $i++;
             }
         }
@@ -87,7 +87,7 @@ 
             $i = 1;
             foreach ($exclusions as $w_key => $w_value) {
                 $qb->andWhere($qb->expr()->neq('g.' . $w_key, '?' . $i))
-                   ->setParameter($i, $w_value);
+                    ->setParameter($i, $w_value);
                 $i++;
             }
         }
@@ -163,7 +163,7 @@ 
         // already another group by that name.
         if (is_numeric($excludedGroupId) && $excludedGroupId > 0) {
             $qb->andWhere($qb->expr()->neq('g.gid', ':ggid'))
-               ->setParameter('ggid', $excludedGroupId);
+                ->setParameter('ggid', $excludedGroupId);
         }
 
         $query = $qb->getQuery();