zikula / core

src/system/SecurityCenterModule/SecurityCenterModuleInstaller.php

Spacing +9 added lines, -9 removed lines

@@ -54,7 +54,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);
@@ -67,7 +67,7 @@ 
         $this->setSystemVar('filtercookievars', 1);
 
         // Location of HTML Purifier
-        $this->setSystemVar('htmlpurifierlocation', __DIR__.'/vendor/htmlpurifier/');
+        $this->setSystemVar('htmlpurifierlocation', __DIR__ . '/vendor/htmlpurifier/');
 
         // HTML Purifier cache dir
         $this->container->get('cache_clearer')->clear('purifier');
@@ -80,13 +80,13 @@ 
         $this->setSystemVar('useids', 0);
         $this->setSystemVar('idsmail', 0);
         $this->setSystemVar('idsrulepath', __DIR__ . '/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/AbstractAdminController.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      *
      * @param Request  $request      Current request instance
      *
-     * @return mixed Output
+     * @return RedirectResponse Output
      *
      * @throws AccessDeniedException Thrown if the user doesn't have required permissions
      */

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

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      *
      * @param Request  $request      Current request instance
      *
-     * @return mixed Output
+     * @return RedirectResponse Output
      *
      * @throws AccessDeniedException Thrown if the user doesn't have required permissions
      */

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

Doc Comments +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
     }
 
     /**
-     * @param $moduleName
+     * @param string $moduleName
      * @return bool
      */
     protected function installModule($moduleName)
@@ -119,7 +119,7 @@ 
     /**
      * Set an admin category for a module
      * @param $moduleName
-     * @param $translatedCategoryName
+     * @param string $translatedCategoryName
      */
     protected function setModuleCategory($moduleName, $translatedCategoryName)
     {

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

Indentation +22 added lines, -22 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;
@@ -534,7 +534,7 @@ 
         $offset = ($currentPage-1) * $resultsPerPage;
     
         $query->setFirstResult($offset)
-              ->setMaxResults($resultsPerPage);
+                ->setMaxResults($resultsPerPage);
     
         return $query;
     }
@@ -600,14 +600,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);
+                    }
                 }
             }
         }
@@ -647,7 +647,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);
@@ -787,7 +787,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);
@@ -832,7 +832,7 @@ 
     {
         $qb = $this->getCountQuery('', false);
         $qb->andWhere('tbl.' . $fieldName . ' = :' . $fieldName)
-           ->setParameter($fieldName, $fieldValue);
+            ->setParameter($fieldName, $fieldValue);
     
         $qb = $this->addExclusion($qb, $excludeId);
     
@@ -878,7 +878,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);
@@ -956,7 +956,7 @@ 
             
             $userId = $this->getRequest()->getSession()->get('uid');
             $qb->andWhere('tbl.createdBy = :creator')
-               ->setParameter('creator', $userId);
+                ->setParameter('creator', $userId);
         }
     
         return $qb;
@@ -975,7 +975,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;

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

Spacing +1 added lines, -1 removed lines

@@ -216,7 +216,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);

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

Spacing +17 added lines, -17 removed lines

@@ -100,7 +100,7 @@ 
             'attr' => [
                 'class' => '',
                 'title' => $this->__('Choose the route type')
-            ],'choices' => $choices,
+            ], 'choices' => $choices,
             'choice_attr' => $choiceAttributes,
             'multiple' => false,
             'expanded' => false
@@ -112,7 +112,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') . ':',
@@ -121,7 +121,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the bundle of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('controller', TextType::class, [
             'label' => $this->__('Controller') . ':',
@@ -130,7 +130,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the controller of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('action', TextType::class, [
             'label' => $this->__('Action') . ':',
@@ -139,7 +139,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the action of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('path', TextType::class, [
             'label' => $this->__('Path') . ':',
@@ -148,7 +148,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the path of the route')
-            ],'required' => true,
+            ], 'required' => true,
         ]);
         $builder->add('host', TextType::class, [
             'label' => $this->__('Host') . ':',
@@ -157,7 +157,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the host of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
 
         $listEntries = $this->listHelper->getEntries('route', 'schemes');
@@ -173,7 +173,7 @@ 
             'attr' => [
                 'class' => '',
                 'title' => $this->__('Choose the schemes')
-            ],'choices' => $choices,
+            ], 'choices' => $choices,
             'choice_attr' => $choiceAttributes,
             'multiple' => true,
             'expanded' => false
@@ -192,7 +192,7 @@ 
             'attr' => [
                 'class' => '',
                 'title' => $this->__('Choose the methods')
-            ],'choices' => $choices,
+            ], 'choices' => $choices,
             'choice_attr' => $choiceAttributes,
             'multiple' => true,
             'expanded' => false
@@ -202,14 +202,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') . ':',
@@ -218,7 +218,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') . ':',
@@ -227,7 +227,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the condition of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
         $builder->add('description', TextType::class, [
             'label' => $this->__('Description') . ':',
@@ -236,7 +236,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the description of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
         $builder->add('sort', IntegerType::class, [
             'label' => $this->__('Sort') . ':',
@@ -245,7 +245,7 @@ 
                 'maxlength' => 255,
                 'class' => ' validate-digits',
                 'title' => $this->__('Enter the sort of the route. Only digits are allowed.')
-            ],'required' => false,
+            ], 'required' => false,
             'maxlength' => 11,
             'scale' => 0
         ]);
@@ -256,7 +256,7 @@ 
                 'maxlength' => 255,
                 'class' => '',
                 'title' => $this->__('Enter the group of the route')
-            ],'required' => false,
+            ], 'required' => false,
         ]);
     }
 
@@ -331,7 +331,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' => [

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

Spacing +1 added lines, -1 removed lines

@@ -615,7 +615,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 %s action.', ['%s' => $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()]);
             }