zikula / core

src/lib/Zikula/Core/LinkContainer/LinkContainerCollector.php

Doc Comments +3 added lines

@@ -67,6 +67,9 @@
         return $links;
     }
 
+    /**
+     * @param string $containerName
+     */
     public function hasContainer($containerName)
     {
         return isset($this->linkContainers[$containerName]);

src/system/CategoriesModule/Entity/CategoryEntity.php

Doc Comments +3 added lines, -3 removed lines

@@ -383,7 +383,7 @@ 
 
     /**
      * get the category display name
-     * @param $lang
+     * @param string $lang
      *
      * @return array|string the category display name(s)
      */
@@ -463,7 +463,7 @@ 
     /**
      * set the creation date of the category
      *
-     * @param mixed $cr_date the creation date
+     * @param \DateTime $cr_date the creation date
      */
     public function setCr_date($cr_date)
     {
@@ -503,7 +503,7 @@ 
     /**
      * set the last updated timestamp of the category
      *
-     * @param mixed $lu_date the last updated timestamp
+     * @param \DateTime $lu_date the last updated timestamp
      */
     public function setLu_date($lu_date)
     {

src/system/RoutesModule/Form/Handler/Route/EditHandler.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 use Symfony\Component\Routing\RouteCollection;
 use Zikula\Bundle\CoreBundle\CacheClearer;
-use Zikula\RoutesModule\Entity\RouteEntity;
 use Zikula\RoutesModule\Form\Handler\Route\Base\AbstractEditHandler;
 use Zikula\RoutesModule\Helper\PathBuilderHelper;
 use Zikula\RoutesModule\Helper\RouteDumperHelper;

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

Unused Use Statements +1 added lines, -2 removed lines

@@ -14,7 +14,6 @@ 
 
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
-use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
 use Symfony\Component\Form\Extension\Core\Type\DateTimeType;
 use Symfony\Component\Form\Extension\Core\Type\IntegerType;
 use Symfony\Component\Form\Extension\Core\Type\ResetType;
@@ -28,8 +27,8 @@ 
 use Zikula\RoutesModule\Entity\Factory\EntityFactory;
 use Zikula\RoutesModule\Form\Type\Field\ArrayType;
 use Zikula\RoutesModule\Form\Type\Field\MultiListType;
-use Zikula\UsersModule\Form\Type\UserLiveSearchType;
 use Zikula\RoutesModule\Helper\ListEntriesHelper;
+use Zikula\UsersModule\Form\Type\UserLiveSearchType;
 
 /**
  * Route editing form type base class.

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

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Symfony\Component\HttpFoundation\RequestStack;
 use Zikula\UsersModule\Api\ApiInterface\CurrentUserApiInterface;
 use Zikula\UsersModule\Constant as UsersConstant;
-use Zikula\RoutesModule\Entity\RouteEntity;
 
 /**
  * Entity collection filter helper base class.

Spacing +1 added lines, -1 removed lines

@@ -177,7 +177,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, '%' . substr($v, 1) . '%');

Indentation +6 added lines, -6 removed lines

@@ -177,17 +177,17 @@ 
             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, '%' . substr($v, 1) . '%');
+                        ->setParameter($k, '%' . substr($v, 1) . '%');
                 } elseif (in_array($k, ['schemes', 'methods'])) {
                     // 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);
                 }
             }
         }
@@ -293,10 +293,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;

src/system/RoutesModule/Entity/Repository/RouteRepository.php

Indentation +2 added lines, -2 removed lines

@@ -41,8 +41,8 @@
 
         $qb = $this->getEntityManager()->createQueryBuilder();
         $qb->delete($this->mainEntityClass, 'tbl')
-           ->where('tbl.bundle = :bundle')
-           ->setParameter('bundle', $bundleName);
+            ->where('tbl.bundle = :bundle')
+            ->setParameter('bundle', $bundleName);
         $query = $qb->getQuery();
         $query->execute();
     }

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

Doc Comments +1 added lines, -1 removed lines

@@ -294,7 +294,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 = [])
     {

build/BuildPackageCommand.php

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

@@ -319,7 +319,7 @@
 -----------------sha1sums-----------------
 $zipSha1  $name.zip
 $tarSha1  $name.tar.gz
-CHECKSUM;
+checksum;
         file_put_contents("$name-checksum.txt", $checksum);
         $progress->advance();
 

src/system/CategoriesModule/Controller/NodeController.php

Braces +2 added lines, -1 removed lines

@@ -70,7 +70,8 @@
                         $parent = $repo->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
                         $sibling = $repo->find($request->request->get('after'));
                         $category->setParent($sibling->getParent());
                         $category->setRoot($sibling->getRoot());