zikula / core

src/system/BlocksModule/Helper/InstallerHelper.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
             $blockClassName = preg_match('/.*Block$/', $blockClassName) ? $blockClassName : $blockClassName . 'Block';
         } catch (\Exception $e) {
             $moduleBundle = null;
-            $blockClassName = '\\' . ucwords($moduleName).'\\'.'Block\\'.ucwords($blockEntity->getBkey());
+            $blockClassName = '\\' . ucwords($moduleName) . '\\' . 'Block\\' . ucwords($blockEntity->getBkey());
             $blockClassName = preg_match('/.*Block$/', $blockClassName) ? $blockClassName : $blockClassName . 'Block';
             $blockClassNameOld = '\\' . ucwords($moduleName) . '_' . 'Block_' . ucwords($blockEntity->getBkey());
             $blockClassName = class_exists($blockClassName) ? $blockClassName : $blockClassNameOld;

src/system/RoutesModule/Translation/ZikulaPatternGenerationStrategy.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
                 || (self::STRATEGY_PREFIX_EXCEPT_DEFAULT === $this->strategy && $this->defaultLocale !== $locale)) {
                 $i18nPattern = '/' . $locale . $i18nPattern;
                 if (null !== $route->getOption('i18n_prefix')) {
-                    $i18nPattern = $route->getOption('i18n_prefix').$i18nPattern;
+                    $i18nPattern = $route->getOption('i18n_prefix') . $i18nPattern;
                 }
             }
 

src/system/RoutesModule/Helper/Base/AbstractControllerHelper.php

Spacing +3 added lines, -3 removed lines

@@ -145,7 +145,7 @@ 
             } else {
                 // single identifier
                 if (array_key_exists($idField, $routeParams)) {
-                    $id = (int) !empty($routeParams[$idField]) ? $routeParams[$idField] : $defaultValue;
+                    $id = (int)!empty($routeParams[$idField]) ? $routeParams[$idField] : $defaultValue;
                 } elseif ($request->query->has($idField)) {
                     $id = $request->query->getInt($idField, $defaultValue);
                 } else {
@@ -157,9 +157,9 @@ 
             if (!$id && $idField != 'id' && count($idFields) == 1) {
                 $defaultValue = isset($args['id']) && is_numeric($args['id']) ? $args['id'] : 0;
                 if (array_key_exists('id', $routeParams)) {
-                    $id = (int) !empty($routeParams['id']) ? $routeParams['id'] : $defaultValue;
+                    $id = (int)!empty($routeParams['id']) ? $routeParams['id'] : $defaultValue;
                 } elseif ($request->query->has('id')) {
-                    $id = (int) $request->query->getInt('id', $defaultValue);
+                    $id = (int)$request->query->getInt('id', $defaultValue);
                 } else {
                     $id = $defaultValue;
                 }

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
      * @param string $context Usage context (allowed values: controllerAction, api, helper, actionHandler, block, contentType, util)
      * @param array  $args    Additional arguments
      *
-     * @return array List of allowed object types
+     * @return string[] List of allowed object types
      */
     public function getObjectTypes($context = '', $args = [])
     {

src/system/RoutesModule/Helper/Base/AbstractSelectionHelper.php

Indentation +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
         $slimMode = (bool) $slimMode; 
     
         if (!empty($idList)) {
-           return $repository->selectByIdList($idList, $useJoins, $slimMode);
+            return $repository->selectByIdList($idList, $useJoins, $slimMode);
         }
     
         return $repository->selectWhere($where, $orderBy, $useJoins, $slimMode);

Spacing +6 added lines, -6 removed lines

@@ -106,8 +106,8 @@ 
     
         $repository = $this->getRepository($objectType);
     
-        $useJoins = (bool) $useJoins;
-        $slimMode = (bool) $slimMode; 
+        $useJoins = (bool)$useJoins;
+        $slimMode = (bool)$slimMode; 
     
         $entity = $repository->selectById($id, $useJoins, $slimMode);
     
@@ -133,8 +133,8 @@ 
         }
         $repository = $this->getRepository($objectType);
     
-        $useJoins = (bool) $useJoins;
-        $slimMode = (bool) $slimMode; 
+        $useJoins = (bool)$useJoins;
+        $slimMode = (bool)$slimMode; 
     
         if (!empty($idList)) {
            return $repository->selectByIdList($idList, $useJoins, $slimMode);
@@ -163,8 +163,8 @@ 
         }
         $repository = $this->getRepository($objectType);
     
-        $useJoins = (bool) $useJoins;
-        $slimMode = (bool) $slimMode; 
+        $useJoins = (bool)$useJoins;
+        $slimMode = (bool)$slimMode; 
     
         return $repository->selectWherePaginated($where, $orderBy, $currentPage, $resultsPerPage, $useJoins, $slimMode);
     }

src/system/MenuModule/Block/Form/Type/MenuType.php

Spacing +2 added lines, -2 removed lines

@@ -39,10 +39,10 @@
         ;
         $builder->get('options')
             ->addModelTransformer(new CallbackTransformer(
-                function ($text) {
+                function($text) {
                     return $text;
                 },
-                function ($text) {
+                function($text) {
                     if (empty($text)) {
                         return '{}';
                     }

src/system/MenuModule/Controller/NodeController.php

Spacing +2 added lines, -2 removed lines

@@ -122,9 +122,9 @@
         $entityId = str_replace($this->domTreeNodePrefix, '', $node['id']);
         $menuItemEntity = $repo->find($entityId);
         $oldParent = $request->request->get('old_parent');
-        $oldPosition = (int) $request->request->get('old_position');
+        $oldPosition = (int)$request->request->get('old_position');
         $parent = $request->request->get('parent');
-        $position = (int) $request->request->get('position');
+        $position = (int)$request->request->get('position');
         if ($oldParent == $parent) {
             $diff = $oldPosition - $position; // if $diff is positive, then node moved up
             $methodName = $diff > 0 ? 'moveUp' : 'moveDown';

Braces +2 added lines, -1 removed lines

@@ -56,7 +56,8 @@
                         $parent = $repo->find($request->request->get('parent'));
                         $menuItemEntity->setParent($parent);
                         $menuItemEntity->setRoot($parent->getRoot());
-                    } elseif (empty($parent) && $request->request->has('after')) { // sibling of top-level child
+                    } elseif (empty($parent) && $request->request->has('after')) {
+// sibling of top-level child
                         $sibling = $repo->find($request->request->get('after'));
                         $menuItemEntity->setParent($sibling->getParent());
                         $menuItemEntity->setRoot($sibling->getRoot());

src/system/CategoriesModule/Entity/Repository/CategoryRepository.php

Indentation +2 added lines, -2 removed lines

@@ -36,12 +36,12 @@
 
         if ($parentId > 0) {
             $qb->andWhere('c.parent = :parentid')
-               ->setParameter('parentid', $parentId);
+                ->setParameter('parentid', $parentId);
         }
 
         if ($excludedId > 0) {
             $qb->andWhere('c.id != :id')
-               ->setParameter('id', $excludedId);
+                ->setParameter('id', $excludedId);
         }
 
         $query = $qb->getQuery();

src/system/CategoriesModule/Form/DataTransformer/CategoriesCollectionTransformer.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         $class = $this->entityCategoryClass;
 
         foreach ($value as $regId => $categories) {
-            $regId = (int) substr($regId, strpos($regId, '_') + 1);
+            $regId = (int)substr($regId, strpos($regId, '_') + 1);
             $subCollection = new ArrayCollection();
             if (!is_array($categories) && $categories instanceof CategoryEntity) {
                 $categories = [$categories];

src/system/UsersModule/Block/OnlineBlock.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      */
     public function display(array $properties)
     {
-        if (!$this->hasPermission('Onlineblock::', $properties['bid'].'::', ACCESS_READ)) {
+        if (!$this->hasPermission('Onlineblock::', $properties['bid'] . '::', ACCESS_READ)) {
             return '';
         }