zikula / core

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

Doc Comments +2 added lines, -2 removed lines

@@ -88,7 +88,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 = '', array $args = [])
     {
@@ -381,7 +381,7 @@ 
      * @param string $context    Usage context (allowed values: controllerAction, api, actionHandler, block, contentType)
      * @param array  $args       Additional arguments
      *
-     * @return array List of template variables to be assigned
+     * @return string List of template variables to be assigned
      */
     public function addTemplateParameters($objectType = '', array $parameters = [], $context = '', array $args = [])
     {

src/system/RoutesModule/Listener/Base/AbstractEntityLifecycleListener.php

Doc Comments +1 added lines, -1 removed lines

@@ -262,7 +262,7 @@
      *
      * @param EntityAccess $entity The given entity
      *
-     * @return Event The created event instance
+     * @return string The created event instance
      */
     protected function createFilterEvent($entity)
     {

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

Indentation +6 added lines, -6 removed lines

@@ -171,14 +171,14 @@ 
                 if ((!is_numeric($v) && $v != '') || (is_numeric($v) && $v > 0)) {
                     if ($k == 'workflowState' && substr($v, 0, 1) == '!') {
                         $qb->andWhere('tbl.' . $k . ' != :' . $k)
-                           ->setParameter($k, substr($v, 1, strlen($v)-1));
+                            ->setParameter($k, substr($v, 1, strlen($v)-1));
                     } elseif (substr($v, 0, 1) == '%') {
                         $qb->andWhere('tbl.' . $k . ' LIKE :' . $k)
-                           ->setParameter($k, '%' . $v . '%');
+                            ->setParameter($k, '%' . $v . '%');
                     } else {
                         $qb->andWhere('tbl.' . $k . ' = :' . $k)
-                           ->setParameter($k, $v);
-                   }
+                            ->setParameter($k, $v);
+                    }
                 }
             }
         }
@@ -282,10 +282,10 @@ 
     
         if (is_array($userId)) {
             $qb->andWhere('tbl.createdBy IN (:userIds)')
-               ->setParameter('userIds', $userId);
+                ->setParameter('userIds', $userId);
         } else {
             $qb->andWhere('tbl.createdBy = :userId')
-               ->setParameter('userId', $userId);
+                ->setParameter('userId', $userId);
         }
     
         return $qb;

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
                 if ((!is_numeric($v) && $v != '') || (is_numeric($v) && $v > 0)) {
                     if ($k == 'workflowState' && substr($v, 0, 1) == '!') {
                         $qb->andWhere('tbl.' . $k . ' != :' . $k)
-                           ->setParameter($k, substr($v, 1, strlen($v)-1));
+                           ->setParameter($k, substr($v, 1, strlen($v) - 1));
                     } elseif (substr($v, 0, 1) == '%') {
                         $qb->andWhere('tbl.' . $k . ' LIKE :' . $k)
                            ->setParameter($k, '%' . $v . '%');

src/system/PermissionsModule/Api/PermissionApi.php

Doc Comments +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
      * @param PermissionRepositoryInterface $permRepository Permission repository
      * @param UserRepositoryInterface $userRepository User repository
      * @param CurrentUserApiInterface $currentUserApi
-     * @param TranslatorInterface $translator Translator service instance
+     * @param \Zikula\Common\Translator\IdentityTranslator $translator Translator service instance
      */
     public function __construct(
         PermissionRepositoryInterface $permRepository,

src/system/PermissionsModule/Tests/Api/PermissionApiTest.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
         $this->userRepo
             ->method('findByUids')
             ->with($this->anything())
-            ->will($this->returnCallback(function (array $uids) {
+            ->will($this->returnCallback(function(array $uids) {
                 $groups = [];
                 // getGroups returns [gid => $group, gid => $group, ...]
                 if (in_array(self::RANDOM_USER_ID, $uids)) {
@@ -125,7 +125,7 @@ 
         $this->currentUserApi
             ->method('get')
             ->with($this->equalTo('uid'))
-            ->will($this->returnCallback(function () use ($uid) {
+            ->will($this->returnCallback(function() use ($uid) {
                 return isset($uid) ? $uid : Constant::USER_ID_ANONYMOUS;
             }));
         $api = new PermissionApi($this->permRepo, $this->userRepo, $this->currentUserApi, $this->translator);

src/system/GroupsModule/GroupsModuleInstaller.php

Spacing +2 added lines, -2 removed lines

@@ -70,8 +70,8 @@
         switch ($oldVersion) {
             case '2.3.2':
             case '2.4.0':
-                $this->setVar('mailwarning', (bool) $this->getVar('mailwarning'));
-                $this->setVar('hideclosed', (bool) $this->getVar('hideclosed'));
+                $this->setVar('mailwarning', (bool)$this->getVar('mailwarning'));
+                $this->setVar('hideclosed', (bool)$this->getVar('hideclosed'));
                 $this->setVar('hidePrivate', false);
             case '2.4.1':
                 /** @var UserEntity $anonymousUser */

src/system/CategoriesModule/Entity/CategoryEntity.php

Doc Comments +1 added lines, -1 removed lines

@@ -383,7 +383,7 @@
 
     /**
      * get the category display name
-     * @param $lang
+     * @param string $lang
      *
      * @return array|string the category display name(s)
      */

src/system/UsersModule/Twig/Extension/OnlineExtension.php

Spacing +1 added lines, -1 removed lines

@@ -69,6 +69,6 @@
             ->andWhere(Criteria::expr()->gt('lastused', $since));
         $online = $this->sessionRepository->matching($c)->count();
 
-        return (bool) $online;
+        return (bool)$online;
     }
 }

src/system/RoutesModule/Form/Handler/Common/Base/AbstractEditHandler.php

Spacing +1 added lines, -1 removed lines

@@ -292,7 +292,7 @@
         $routeParams = $this->request->get('_route_params', []);
         if (empty($this->idValue)) {
             if (array_key_exists($this->idField, $routeParams)) {
-                $this->idValue = (int) !empty($routeParams[$this->idField]) ? $routeParams[$this->idField] : 0;
+                $this->idValue = (int)!empty($routeParams[$this->idField]) ? $routeParams[$this->idField] : 0;
             }
             if (0 === $this->idValue) {
                 $this->idValue = $this->request->query->getInt($this->idField, 0);