zikula / core

src/lib/i18n/ZLanguage.php

Doc Comments +3 added lines, -3 removed lines

@@ -296,7 +296,7 @@ 
     /**
      * Get locale.
      *
-     * @return ZLocale
+     * @return string
      */
     public static function getLocale()
     {
@@ -442,7 +442,7 @@ 
      *
      * @param string $modName Module name
      *
-     * @return boolean
+     * @return string|boolean
      */
     public static function bindModuleDomain($modName)
     {
@@ -475,7 +475,7 @@ 
      * @param string $moduleName Module name
      * @param string $pluginName Plugin name
      *
-     * @return boolean
+     * @return string|boolean
      */
     public static function bindModulePluginDomain($moduleName, $pluginName)
     {

Spacing +6 added lines, -6 removed lines

@@ -429,7 +429,7 @@ 
         $domain = self::getThemeDomain($themeName);
         $theme = ThemeUtil::getTheme($themeName);
         if (null !== $theme) {
-            $path = $_this->searchOverrides($domain, $theme->getPath().'/Resources/locale');
+            $path = $_this->searchOverrides($domain, $theme->getPath() . '/Resources/locale');
         } else {
             $path = $_this->searchOverrides($domain, "themes/$themeName/locale");
         }
@@ -452,7 +452,7 @@ 
             return 'zikula';
         }
 
-        $_this  = self::getInstance();
+        $_this = self::getInstance();
         if (!$_this->locale) {
             $request = ServiceUtil::get('request');
             $_this->setup($request);
@@ -461,9 +461,9 @@ 
         $domain = self::getModuleDomain($modName);
         $module = ModUtil::getModule($modName);
         if (null !== $module) {
-            $path = $_this->searchOverrides($domain, $module->getPath().'/Resources/locale');
+            $path = $_this->searchOverrides($domain, $module->getPath() . '/Resources/locale');
         } else {
-            $path = $_this->searchOverrides($domain, 'modules/'.$modName.'/locale');
+            $path = $_this->searchOverrides($domain, 'modules/' . $modName . '/locale');
         }
 
         return self::bindDomain($domain, $path);
@@ -539,7 +539,7 @@ 
     private function searchOverrides($domain, $path)
     {
         $lang = self::transformFS($this->languageCode);
-        $override = realpath('config/locale/'.$lang.'/LC_MESSAGES/'.$domain.'.mo');
+        $override = realpath('config/locale/' . $lang . '/LC_MESSAGES/' . $domain . '.mo');
 
         return $override ? realpath('config/locale') : realpath($path);
     }
@@ -555,7 +555,7 @@ 
     {
         $module = ModUtil::getModule($name);
 
-        return (null === $module) ? strtolower('module_'.$name) : $module->getTranslationDomain();
+        return (null === $module) ? strtolower('module_' . $name) : $module->getTranslationDomain();
     }
 
     /**

src/lib/Zikula/Bundle/CoreInstallerBundle/Controller/AbstractController.php

Doc Comments +2 added lines, -2 removed lines

@@ -74,7 +74,7 @@ 
     }
 
     /**
-     * @param $moduleName
+     * @param string $moduleName
      * @return bool
      */
     protected function installModule($moduleName)
@@ -118,7 +118,7 @@ 
     /**
      * Set an admin category for a module or set to default
      * @param $moduleName
-     * @param $translatedCategoryName
+     * @param string $translatedCategoryName
      */
     protected function setModuleCategory($moduleName, $translatedCategoryName)
     {

src/system/RoutesModule/Controller/AjaxController.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
      *
      * @param Request $request Current request instance
      *
-     * @return JsonResponse
+     * @return \Symfony\Component\HttpFoundation\JsonResponse
      */ 
     public function getCommonUsersListAction(Request $request)
     {

src/system/RoutesModule/Entity/Historical/v110/Base/AbstractRouteEntity.php

Doc Comments +1 added lines, -1 removed lines

@@ -1142,7 +1142,7 @@
      * ToString interceptor implementation.
      * This method is useful for debugging purposes.
      *
-     * @return string The output string for this entity
+     * @return integer The output string for this entity
      */
     public function __toString()
     {

src/system/SecurityCenterModule/SecurityCenterModuleInstaller.php

Spacing +8 added lines, -8 removed lines

@@ -55,7 +55,7 @@ 
         $this->setSystemVar('sessionstoretofile', 0);
         $this->setSystemVar('sessionsavepath', '');
         $this->setSystemVar('gc_probability', 100);
-        $this->setSystemVar('sessioncsrftokenonetime', 1);  // 1 means use same token for entire session
+        $this->setSystemVar('sessioncsrftokenonetime', 1); // 1 means use same token for entire session
         $this->setSystemVar('anonymoussessions', 1); // @deprecated
         $this->setSystemVar('sessionrandregenerate', 1);
         $this->setSystemVar('sessionregenerate', 1);
@@ -78,13 +78,13 @@ 
         $this->setSystemVar('useids', 0);
         $this->setSystemVar('idsmail', 0);
         $this->setSystemVar('idsrulepath', 'system/SecurityCenterModule/Resources/config/phpids_zikula_default.xml');
-        $this->setSystemVar('idssoftblock', 1);                // do not block requests, but warn for debugging
-        $this->setSystemVar('idsfilter', 'xml');               // filter type
-        $this->setSystemVar('idsimpactthresholdone', 1);       // db logging
-        $this->setSystemVar('idsimpactthresholdtwo', 10);      // mail admin
-        $this->setSystemVar('idsimpactthresholdthree', 25);    // block request
-        $this->setSystemVar('idsimpactthresholdfour', 75);     // kick user, destroy session
-        $this->setSystemVar('idsimpactmode', 1);               // per request per default
+        $this->setSystemVar('idssoftblock', 1); // do not block requests, but warn for debugging
+        $this->setSystemVar('idsfilter', 'xml'); // filter type
+        $this->setSystemVar('idsimpactthresholdone', 1); // db logging
+        $this->setSystemVar('idsimpactthresholdtwo', 10); // mail admin
+        $this->setSystemVar('idsimpactthresholdthree', 25); // block request
+        $this->setSystemVar('idsimpactthresholdfour', 75); // kick user, destroy session
+        $this->setSystemVar('idsimpactmode', 1); // per request per default
         $this->setSystemVar('idshtmlfields', ['POST.__wysiwyg']);
         $this->setSystemVar('idsjsonfields', ['POST.__jsondata']);
         $this->setSystemVar('idsexceptions', [

src/system/RoutesModule/Controller/Base/AbstractRouteController.php

Spacing +1 added lines, -1 removed lines

@@ -611,7 +611,7 @@
                 }
                 // execute the workflow action
                 $success = $workflowHelper->executeAction($entity, $action);
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 $this->addFlash('error', $this->__f('Sorry, but an error occured during the %action% action.', ['%action%' => $action]) . '  ' . $e->getMessage());
                 $logger->error('{app}: User {user} tried to execute the {action} workflow action for the {entity} with id {id}, but failed. Error details: {errorMessage}.', ['app' => 'ZikulaRoutesModule', 'user' => $userName, 'action' => $action, 'entity' => 'route', 'id' => $itemid, 'errorMessage' => $e->getMessage()]);
             }

src/system/RoutesModule/Entity/Repository/Base/AbstractRouteRepository.php

Indentation +26 added lines, -26 removed lines

@@ -280,9 +280,9 @@ 
     
         $qb = $this->getEntityManager()->createQueryBuilder();
         $qb->update('Zikula\RoutesModule\Entity\RouteEntity', 'tbl')
-           ->set('tbl.createdBy', $newUserId)
-           ->where('tbl.createdBy= :creator')
-           ->setParameter('creator', $userId);
+            ->set('tbl.createdBy', $newUserId)
+            ->where('tbl.createdBy= :creator')
+            ->setParameter('creator', $userId);
         $query = $qb->getQuery();
         $query->execute();
     
@@ -313,9 +313,9 @@ 
     
         $qb = $this->getEntityManager()->createQueryBuilder();
         $qb->update('Zikula\RoutesModule\Entity\RouteEntity', 'tbl')
-           ->set('tbl.updatedBy', $newUserId)
-           ->where('tbl.updatedBy = :editor')
-           ->setParameter('editor', $userId);
+            ->set('tbl.updatedBy', $newUserId)
+            ->where('tbl.updatedBy = :editor')
+            ->setParameter('editor', $userId);
         $query = $qb->getQuery();
         $query->execute();
     
@@ -344,8 +344,8 @@ 
     
         $qb = $this->getEntityManager()->createQueryBuilder();
         $qb->delete('Zikula\RoutesModule\Entity\RouteEntity', 'tbl')
-           ->where('tbl.createdBy = :creator')
-           ->setParameter('creator', $userId);
+            ->where('tbl.createdBy = :creator')
+            ->setParameter('creator', $userId);
         $query = $qb->getQuery();
     
         $query->execute();
@@ -375,8 +375,8 @@ 
     
         $qb = $this->getEntityManager()->createQueryBuilder();
         $qb->delete('Zikula\RoutesModule\Entity\RouteEntity', 'tbl')
-           ->where('tbl.updatedBy = :editor')
-           ->setParameter('editor', $userId);
+            ->where('tbl.updatedBy = :editor')
+            ->setParameter('editor', $userId);
         $query = $qb->getQuery();
     
         $query->execute();
@@ -472,7 +472,7 @@ 
     {
         if ($excludeId > 0) {
             $qb->andWhere('tbl.id != :excludeId')
-               ->setParameter('excludeId', $excludeId);
+                ->setParameter('excludeId', $excludeId);
         }
     
         return $qb;
@@ -495,10 +495,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;
@@ -560,7 +560,7 @@ 
         $offset = ($currentPage-1) * $resultsPerPage;
     
         $query->setFirstResult($offset)
-              ->setMaxResults($resultsPerPage);
+                ->setMaxResults($resultsPerPage);
     
         return $query;
     }
@@ -626,14 +626,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);
+                    }
                 }
             }
         }
@@ -673,7 +673,7 @@ 
     {
         $qb = $this->genericBaseQuery('', $orderBy, $useJoins);
         if (count($exclude) > 0) {
-        	$qb = $this->addExclusion($qb, $exclude);
+            $qb = $this->addExclusion($qb, $exclude);
         }
     
         $qb = $this->addSearchFilter($qb, $fragment);
@@ -813,7 +813,7 @@ 
     
         $qb = $this->getEntityManager()->createQueryBuilder();
         $qb->select($selection)
-           ->from('Zikula\RoutesModule\Entity\RouteEntity', 'tbl');
+            ->from('Zikula\RoutesModule\Entity\RouteEntity', 'tbl');
     
         if (true === $useJoins) {
             $this->addJoinsToFrom($qb);
@@ -858,7 +858,7 @@ 
     {
         $qb = $this->getCountQuery('', false);
         $qb->andWhere('tbl.' . $fieldName . ' = :' . $fieldName)
-           ->setParameter($fieldName, $fieldValue);
+            ->setParameter($fieldName, $fieldValue);
     
         $qb = $this->addExclusion($qb, $excludeId);
     
@@ -904,7 +904,7 @@ 
     
         $qb = $this->getEntityManager()->createQueryBuilder();
         $qb->select($selection)
-           ->from('Zikula\RoutesModule\Entity\RouteEntity', 'tbl');
+            ->from('Zikula\RoutesModule\Entity\RouteEntity', 'tbl');
     
         if (true === $useJoins) {
             $this->addJoinsToFrom($qb);
@@ -982,7 +982,7 @@ 
             
             $userId = $this->getRequest()->getSession()->get('uid');
             $qb->andWhere('tbl.createdBy = :creator')
-               ->setParameter('creator', $userId);
+                ->setParameter('creator', $userId);
         }
     
         return $qb;
@@ -1001,7 +1001,7 @@ 
         if ($orderBy == 'RAND()') {
             // random selection
             $qb->addSelect('MOD(tbl.id, ' . mt_rand(2, 15) . ') AS HIDDEN randomIdentifiers')
-               ->add('orderBy', 'randomIdentifiers');
+                ->add('orderBy', 'randomIdentifiers');
             $orderBy = '';
         } elseif (empty($orderBy)) {
             $orderBy = $this->defaultSortingField;
@@ -1014,12 +1014,12 @@ 
             }
             if (false !== strpos($orderBy, 'tbl.createdBy')) {
                 $qb->addSelect('tblCreator')
-                   ->leftJoin('tbl.createdBy', 'tblCreator');
+                    ->leftJoin('tbl.createdBy', 'tblCreator');
                 $orderBy = str_replace('tbl.createdBy', 'tblCreator.uname', $orderBy);
             }
             if (false !== strpos($orderBy, 'tbl.updatedBy')) {
                 $qb->addSelect('tblUpdater')
-                   ->leftJoin('tbl.updatedBy', 'tblUpdater');
+                    ->leftJoin('tbl.updatedBy', 'tblUpdater');
                 $orderBy = str_replace('tbl.updatedBy', 'tblUpdater.uname', $orderBy);
             }
             $qb->add('orderBy', $orderBy);

src/system/RoutesModule/Form/Base/AbstractAppSettingsType.php

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
                 'attr' => [
                     'maxlength' => 255,
                     'title' => $this->__('Enter the route entries per page.') . ' ' . $this->__('Only digits are allowed.')
-                ],'scale' => 0
+                ], 'scale' => 0
             ])
         ;
     }

src/system/RoutesModule/Form/Type/Base/AbstractRouteType.php

Spacing +17 added lines, -17 removed lines

@@ -101,7 +101,7 @@ 
             'attr' => [
                 'class' => '',
                 'title' => $this->__('Choose the route type')
-            ],'choices' => $choices,
+            ], 'choices' => $choices,
             'choice_attr' => $choiceAttributes,
             'multiple' => false,
             'expanded' => false
@@ -113,7 +113,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the replaced route name of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
         $builder->add('bundle', TextType::class, [
             'label' => $this->__('Bundle') . ':',
@@ -122,7 +122,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the bundle of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('controller', TextType::class, [
             'label' => $this->__('Controller') . ':',
@@ -131,7 +131,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the controller of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('action', TextType::class, [
             'label' => $this->__('Action') . ':',
@@ -140,7 +140,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the action of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('path', TextType::class, [
             'label' => $this->__('Path') . ':',
@@ -149,7 +149,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the path of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('host', TextType::class, [
             'label' => $this->__('Host') . ':',
@@ -158,7 +158,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the host of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
         
         $listEntries = $this->listHelper->getEntries('route', 'schemes');
@@ -174,7 +174,7 @@ 
             'attr' => [
                 'class' => '',
                 'title' => $this->__('Choose the schemes')
-            ],'choices' => $choices,
+            ], 'choices' => $choices,
             'choice_attr' => $choiceAttributes,
             'multiple' => true,
             'expanded' => false
@@ -193,7 +193,7 @@ 
             'attr' => [
                 'class' => '',
                 'title' => $this->__('Choose the methods')
-            ],'choices' => $choices,
+            ], 'choices' => $choices,
             'choice_attr' => $choiceAttributes,
             'multiple' => true,
             'expanded' => false
@@ -203,14 +203,14 @@ 
             'attr' => [
                 'class' => '',
                 'title' => $this->__('prepend bundle prefix ?')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('translatable', CheckboxType::class, [
             'label' => $this->__('Translatable') . ':',
             'attr' => [
                 'class' => '',
                 'title' => $this->__('translatable ?')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('translationPrefix', TextType::class, [
             'label' => $this->__('Translation prefix') . ':',
@@ -219,7 +219,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the translation prefix of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
         $builder->add('condition', TextType::class, [
             'label' => $this->__('Condition') . ':',
@@ -228,7 +228,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the condition of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
         $builder->add('description', TextType::class, [
             'label' => $this->__('Description') . ':',
@@ -237,7 +237,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the description of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
         $builder->add('sort', IntegerType::class, [
             'label' => $this->__('Sort') . ':',
@@ -246,7 +246,7 @@ 
                 'maxlength' => 11,
                 'class' => ' validate-digits',
                 'title' => $this->__('Enter the sort of the route.') . ' ' . $this->__('Only digits are allowed.')
-            ],'required' => false,
+            ], 'required' => false,
             'scale' => 0
         ]);
         $builder->add('group', TextType::class, [
@@ -256,7 +256,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the group of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
     }
 
@@ -369,7 +369,7 @@ 
             ->setDefaults([
                 // define class for underlying data (required for embedding forms)
                 'data_class' => 'Zikula\RoutesModule\Entity\RouteEntity',
-                'empty_data' => function (FormInterface $form) {
+                'empty_data' => function(FormInterface $form) {
                     return $this->entityFactory->createRoute();
                 },
                 'error_mapping' => [