zikula / core

src/system/UsersModule/Entity/Repository/UserRepository.php

Indentation +2 added lines, -2 removed lines

@@ -51,8 +51,8 @@
         // the following process should be unnecessary because cascade = all but MySQL 5.7 not working with that (#3726)
         $qb = $this->_em->createQueryBuilder();
         $qb->delete(UserAttributeEntity::class, 'a')
-           ->where('a.user = :userId')
-           ->setParameter('userId', $user->getUid());
+            ->where('a.user = :userId')
+            ->setParameter('userId', $user->getUid());
         $query = $qb->getQuery();
         $query->execute();
         // end of theoretically unrequired process

Spacing +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
         }
         $query = $qb->getQuery();
 
-        return (int)$query->getSingleScalarResult();
+        return (int) $query->getSingleScalarResult();
     }
 
     /**

src/system/UsersModule/Entity/Repository/UserSessionRepository.php

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

@@ -91,7 +91,7 @@
     public function gc(string $level, int $inactiveMinutes, int $days): void
     {
         $inactive = new DateTime();
-        $inactive->modify("-${inactiveMinutes} minutes");
+        $inactive->modify("-${inactiveminutes} minutes");
         $daysOld = new DateTime();
         $daysOld->modify("-${days} days");
 

Spacing +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
             ->setParameter('guestUser', Constant::USER_ID_ANONYMOUS)
             ->getQuery();
 
-        return (int)$query->getSingleScalarResult();
+        return (int) $query->getSingleScalarResult();
     }
 
     public function countGuestsSince(DateTime $dateTime): int
@@ -69,7 +69,7 @@ 
             ->setParameter('guestUser', Constant::USER_ID_ANONYMOUS)
             ->getQuery();
 
-        return (int)$query->getSingleScalarResult();
+        return (int) $query->getSingleScalarResult();
     }
 
     public function clearUnsavedData(): void

src/system/CategoriesModule/Controller/NodeController.php

Braces +2 added lines, -1 removed lines

@@ -80,7 +80,8 @@
                         $parent = $categoryRepository->find($parentId);
                         $category->setParent($parent);
                         $category->setRoot($parent->getRoot());
-                    } elseif (empty($parentId) && $request->request->has('after')) { // sibling of top-level child
+                    } elseif (empty($parentId) && $request->request->has('after')) {
+// sibling of top-level child
                         /** @var CategoryEntity $sibling */
                         $sibling = $categoryRepository->find($request->request->get('after'));
                         $category->setParent($sibling->getParent());

Spacing +2 added lines, -2 removed lines

@@ -206,9 +206,9 @@
         }
 
         $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';

src/system/CategoriesModule/Api/CategoryPermissionApi.php

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

@@ -48,7 +48,7 @@
             }
             $regId = $categoryAssignment->getCategoryRegistryId();
             $catId = $categoryAssignment->getCategory()->getId();
-            $hasAccess = $this->permissionApi->hasPermission('ZikulaCategoriesModule:PropertyId:CategoryId', "${regId}::${catId}", $permLevel);
+            $hasAccess = $this->permissionApi->hasPermission('ZikulaCategoriesModule:PropertyId:CategoryId', "${regid}::${catid}", $permLevel);
             if ($requireAccessForAll && !$hasAccess) {
                 return false;
             }

src/system/CategoriesModule/Form/Type/CategoryType.php

Spacing +2 added lines, -2 removed lines

@@ -105,7 +105,7 @@ 
                 'mapped' => false,
                 'required' => false
             ])
-            ->addEventListener(FormEvents::PRE_SET_DATA, static function (FormEvent $event) use ($translator, $options) {
+            ->addEventListener(FormEvents::PRE_SET_DATA, static function(FormEvent $event) use ($translator, $options) {
                 // ensure all display name and description exist for all locales
                 /** @var CategoryEntity $category */
                 $category = $event->getData();
@@ -129,7 +129,7 @@ 
 
                 $event->setData($category);
             })
-            ->addEventListener(FormEvents::SUBMIT, static function (FormEvent $event) use ($translator, $options) {
+            ->addEventListener(FormEvents::SUBMIT, static function(FormEvent $event) use ($translator, $options) {
                 // ensure all locales have a display name
                 /** @var CategoryEntity $category */
                 $category = $event->getData();

src/system/CategoriesModule/Tests/Api/CategoryPermissionApiTest.php

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
     {
         $api = $this->getMockBuilder(PermissionApiInterface::class)
             ->getMock();
-        $api->method('hasPermission')->willReturnCallback(static function ($component = null, $instance = null, $level = ACCESS_NONE, $user = null) {
+        $api->method('hasPermission')->willReturnCallback(static function($component = null, $instance = null, $level = ACCESS_NONE, $user = null) {
             list(/*$regId*/, $catId) = explode('::', $instance);
 
             return 0 === $catId % 2;

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

Indentation +6 added lines, -6 removed lines

@@ -163,17 +163,17 @@ 
             if ((!is_numeric($v) && '' !== $v) || (is_numeric($v) && 0 < $v)) {
                 if ('workflowState' === $k && 0 === strpos($v, '!')) {
                     $qb->andWhere('tbl.' . $k . ' != :' . $k)
-                       ->setParameter($k, substr($v, 1));
+                        ->setParameter($k, substr($v, 1));
                 } elseif (0 === strpos($v, '%')) {
                     $qb->andWhere('tbl.' . $k . ' LIKE :' . $k)
-                       ->setParameter($k, '%' . substr($v, 1) . '%');
+                        ->setParameter($k, '%' . substr($v, 1) . '%');
                 } elseif (in_array($k, ['schemes', 'methods'], true)) {
                     // multi list filter
                     $qb->andWhere('tbl.' . $k . ' LIKE :' . $k)
-                       ->setParameter($k, '%' . $v . '%');
+                        ->setParameter($k, '%' . $v . '%');
                 } else {
                     $qb->andWhere('tbl.' . $k . ' = :' . $k)
-                       ->setParameter($k, $v);
+                        ->setParameter($k, $v);
                 }
             }
         }
@@ -206,7 +206,7 @@ 
             // per default we show approved routes only
             $onlineStates = ['approved'];
             $qb->andWhere('tbl.workflowState IN (:onlineStates)')
-               ->setParameter('onlineStates', $onlineStates);
+                ->setParameter('onlineStates', $onlineStates);
         }
     
         return $qb;
@@ -270,7 +270,7 @@ 
         }
     
         $qb->andWhere('tbl.createdBy = :userId')
-           ->setParameter('userId', $userId);
+            ->setParameter('userId', $userId);
     
         return $qb;
     }

Spacing +3 added lines, -3 removed lines

@@ -55,7 +55,7 @@ 
         $this->requestStack = $requestStack;
         $this->permissionHelper = $permissionHelper;
         $this->currentUserApi = $currentUserApi;
-        $this->showOnlyOwnEntries = (bool)$variableApi->get('ZikulaRoutesModule', 'showOnlyOwnEntries');
+        $this->showOnlyOwnEntries = (bool) $variableApi->get('ZikulaRoutesModule', 'showOnlyOwnEntries');
     }
     
     /**
@@ -191,7 +191,7 @@ 
             return $qb;
         }
     
-        $showOnlyOwnEntries = (bool)$request->query->getInt('own', (int) $this->showOnlyOwnEntries);
+        $showOnlyOwnEntries = (bool) $request->query->getInt('own', (int) $this->showOnlyOwnEntries);
         if ($showOnlyOwnEntries) {
             $qb = $this->addCreatorFilter($qb);
         }
@@ -267,7 +267,7 @@ 
     {
         if (null === $userId) {
             $userId = $this->currentUserApi->isLoggedIn()
-                ? (int)$this->currentUserApi->get('uid')
+                ? (int) $this->currentUserApi->get('uid')
                 : UsersConstant::USER_ID_ANONYMOUS
             ;
         }

src/system/UsersModule/Controller/AccessController.php

Braces +2 added lines, -1 removed lines

@@ -123,7 +123,8 @@
                 $form = $this->createForm(DefaultLoginType::class, ['uid' => $uid]);
                 $loginFormEvent = new UserFormAwareEvent($form);
                 $eventDispatcher->dispatch($loginFormEvent, AccessEvents::AUTHENTICATION_FORM);
-                if ($form->count() > 3) { // count > 3 means that the AUTHENTICATION_FORM event added some form children
+                if ($form->count() > 3) {
+// count > 3 means that the AUTHENTICATION_FORM event added some form children
                     $form->handleRequest($request);
                     if ($form->isSubmitted() && $form->isValid()) {
                         $uid = $form->get('uid')->getData();

src/system/RoutesModule/Twig/TwigExtension.php

Spacing +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
                 ->getPathWithBundlePrefix($route)
         );
 
-        $path = preg_replace_callback('#%(.*?)%#', static function ($matches) use ($container) {
+        $path = preg_replace_callback('#%(.*?)%#', static function($matches) use ($container) {
             return '<abbr title="' . htmlspecialchars($matches[0]) . '">'
                 . htmlspecialchars($container->getParameter($matches[1]))
                 . '</abbr>'
@@ -96,7 +96,7 @@ 
 
         $defaults = $route->getDefaults();
         $requirements = $route->getRequirements();
-        $path = preg_replace_callback('#{(.*?)}#', function ($matches) use ($defaults, $requirements) {
+        $path = preg_replace_callback('#{(.*?)}#', function($matches) use ($defaults, $requirements) {
             $title = '';
             if (isset($defaults[$matches[1]])) {
                 $title .= $this->trans('Default: %value%', ['%value%' => htmlspecialchars($defaults[$matches[1]])]);