zikula / core

src/lib/Zikula/Bundle/CoreBundle/DependencyInjection/Configuration.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function __construct($debug)
     {
-        $this->debug = (bool) $debug;
+        $this->debug = (bool)$debug;
     }
 
     /**
@@ -56,7 +56,7 @@ 
         ->fixXmlConfig('fallback')
         ->children()
         ->arrayNode('fallbacks')
-        ->beforeNormalization()->ifString()->then(function ($v) {
+        ->beforeNormalization()->ifString()->then(function($v) {
             return [$v];
         })->end()
         ->prototype('scalar')->end()

src/lib/Zikula/Bundle/CoreBundle/Tests/Twig/Node/SwitchNodeTest.php

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

@@ -63,7 +63,7 @@ 
     case 0:
         echo "case 0";
 }
-EOF
+eof
         ];
 
         // #2 switch with two cases, second with break
@@ -89,7 +89,7 @@ 
         echo "case 1";
         break;
 }
-EOF
+eof
         ];
 
         // #3 switch with two cases (second with break) and default
@@ -117,7 +117,7 @@ 
     default:
         echo "default case";
 }
-EOF
+eof
         ];
 
         // #4 switch with two cases (first without body, second with break) and default
@@ -144,7 +144,7 @@ 
     default:
         echo "default case";
 }
-EOF
+eof
         ];
 
         return $tests;

src/lib/Zikula/Bundle/CoreBundle/Tests/Functional/AppKernel.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
         $fs = new Filesystem();
         if (!$fs->isAbsolutePath($config)) {
-            $config = __DIR__.'/config/'.$config;
+            $config = __DIR__ . '/config/' . $config;
         }
 
         if (!file_exists($config)) {

src/lib/Zikula/Bundle/CoreBundle/Tests/Bundle/MetaDataTest.php

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

@@ -118,7 +118,7 @@
         }
     }
 }
-EOF
+eof
         ;
 
         return json_decode($json, true);

src/lib/Zikula/Bundle/CoreBundle/Command/BootstrapBundlesCommand.php

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

@@ -24,7 +24,7 @@
             ->setDescription('Loads bundles into persistences')
             ->setHelp(<<<'EOT'
 The <info>scan:bundles</info> command loads bundle table.
-EOT
+eot
             )
             ->setDefinition([
                 new InputArgument('create', InputArgument::OPTIONAL, 'Create schema'),

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/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;