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

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");
 

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());

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/SettingsModule/SettingsModuleInstaller.php

Spacing +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
     private function setGuestTimeZone(): void
     {
         $existingOffset = $this->getSystemVar('timezone_offset');
-        $actualOffset = (float) $existingOffset * 60; // express in minutes
+        $actualOffset = (float)$existingOffset * 60; // express in minutes
         $timezoneAbbreviations = DateTimeZone::listAbbreviations();
         $timeZone = date_default_timezone_get();
         foreach ($timezoneAbbreviations as $abbreviation => $zones) {

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 +1 added lines, -1 removed lines

@@ -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);
         }

build/BuildPackageCommand.php

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

@@ -47,23 +47,23 @@ 
 
         $pwd = getcwd();
         if (is_dir($buildDir)) {
-            system("rm -rf ${buildDir}");
+            system("rm -rf ${builddir}");
         }
 
         // build env
         $filesystem->mkdir($buildDir, 0755);
         $progress->advance();
 
-        $filesystem->mirror($sourceDir, "${buildDir}/${name}");
+        $filesystem->mirror($sourceDir, "${builddir}/${name}");
         $progress->advance();
 
-        PurgeVendorsCommand::cleanVendors("${buildDir}/${name}/vendor", $progress);
-        FixAutoloaderCommand::fix("${buildDir}/${name}/vendor", $progress);
+        PurgeVendorsCommand::cleanVendors("${builddir}/${name}/vendor", $progress);
+        FixAutoloaderCommand::fix("${builddir}/${name}/vendor", $progress);
 
         $writableArray = [
-            "${buildDir}/${name}/var/cache",
-            "${buildDir}/${name}/var/log",
-            "${buildDir}/${name}/web/uploads",
+            "${builddir}/${name}/var/cache",
+            "${builddir}/${name}/var/log",
+            "${builddir}/${name}/web/uploads",
         ];
         $filesystem->chmod($writableArray, 0777);
         $progress->advance();
@@ -86,7 +86,7 @@ 
         $zip = new ZipArchive();
         $fileName = "${name}.zip";
         if (true !== $zip->open($fileName, ZipArchive::CREATE)) {
-            $output->writeln("<error>Error creating ${fileName}</error>");
+            $output->writeln("<error>Error creating ${filename}</error>");
         }
 
         foreach ($allFiles as $file) {
@@ -99,9 +99,9 @@ 
 
         // build tar
         $fileName = "${name}.tar";
-        system("tar cp ${name} > ${fileName}");
+        system("tar cp ${name} > ${filename}");
         $progress->advance();
-        system("gzip ${fileName}");
+        system("gzip ${filename}");
         $progress->advance();
 
         // checksums
@@ -117,16 +117,16 @@ 
 -----------------sha1sums-----------------
 ${zipSha1}  ${name}.zip
 ${tarSha1}  ${name}.tar.gz
-CHECKSUM;
+checksum;
         file_put_contents("${name}-checksum.txt", $checksum);
         $progress->advance();
 
         // cleanup
-        system("rm -rf ${buildDir} ${name}");
+        system("rm -rf ${builddir} ${name}");
         chdir($pwd);
         $progress->advance();
         $progress->finish();
 
-        $output->writeln("<info>Artifacts built in ${buildDir}/ folder</info>");
+        $output->writeln("<info>Artifacts built in ${builddir}/ folder</info>");
     }
 }