zikula / core

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()

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

Spacing +5 added lines, -5 removed lines

@@ -225,8 +225,8 @@ 
     {
         $string = preg_replace_callback(
             '/(.)@(.)/s',
-            function ($m) {
-                return "&#" . sprintf("%03d", ord($m[1])) . ";@&#" .sprintf("%03d", ord($m[2])) . ";";
+            function($m) {
+                return "&#" . sprintf("%03d", ord($m[1])) . ";@&#" . sprintf("%03d", ord($m[2])) . ";";
             },
             $string
         );
@@ -273,9 +273,9 @@ 
         }
 
         // ensure proper variable types
-        $value = (string) $value;
-        $type = (string) $type;
-        $weight = (int) $weight;
+        $value = (string)$value;
+        $type = (string)$type;
+        $weight = (int)$weight;
 
         if ('stylesheet' == $type) {
             $this->container->get('zikula_core.common.theme.assets_css')->add([$value => $weight]);

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

@@ -221,7 +221,7 @@ 
         try {
             // execute the workflow action
             $success = $this->workflowHelper->executeAction($entity, $action);
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
             $flashBag->add('error', $this->__f('Sorry, but an error occured during the %action% action. Please apply the changes again!', ['%action%' => $action]) . ' ' . $e->getMessage());
             $logArgs = ['app' => 'ZikulaRoutesModule', 'user' => $this->currentUserApi->get('uname'), 'entity' => 'route', 'id' => $entity->createCompositeIdentifier(), 'errorMessage' => $e->getMessage()];
             $this->logger->error('{app}: User {user} tried to edit the {entity} with id {id}, but failed. Error details: {errorMessage}.', $logArgs);
@@ -275,7 +275,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')) {

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();