zikula / core

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

src/system/UsersModule/Controller/AccessController.php

Braces +2 added lines, -1 removed lines

@@ -123,7 +123,8 @@
                 $form = $this->createForm(DefaultLoginType::class, ['uid' => $uid]);
                 $loginFormEvent = new UserFormAwareEvent($form);
                 $eventDispatcher->dispatch($loginFormEvent, AccessEvents::AUTHENTICATION_FORM);
-                if ($form->count() > 3) { // count > 3 means that the AUTHENTICATION_FORM event added some form children
+                if ($form->count() > 3) {
+// count > 3 means that the AUTHENTICATION_FORM event added some form children
                     $form->handleRequest($request);
                     if ($form->isSubmitted() && $form->isValid()) {
                         $uid = $form->get('uid')->getData();

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

Spacing +1 added lines, -1 removed lines

@@ -282,7 +282,7 @@
                 return $this->router->generate($routePrefix . 'view');
             case 'userOwnView':
             case 'adminOwnView':
-                return $this->router->generate($routePrefix . 'view', [ 'own' => 1 ]);
+                return $this->router->generate($routePrefix . 'view', ['own' => 1]);
             case 'userDisplay':
             case 'adminDisplay':
                 if (

src/system/MailerModule/Api/MailerApi.php

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

@@ -256,7 +256,7 @@
                 // access the logging channel
                 $logger = new Logger('mailer');
                 $logger->pushHandler(new StreamHandler($logFile, Logger::INFO));
-                $logger->addError("Could not send message to: ${emailList} :: " . $this->message->toString());
+                $logger->addError("Could not send message to: ${emaillist} :: " . $this->message->toString());
             }
 
             $this->eventDispatcher->dispatch($event, MailerEvents::SEND_MESSAGE_FAILURE);

src/system/ExtensionsModule/Command/ZikulaExtensionUninstallCommand.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
         $requiredDependents = $this->dependencyHelper->getDependentExtensions($extension);
         if (!empty($requiredDependents)) {
             if ($input->isInteractive()) {
-                $names = implode(', ', array_map(function ($dependent) {
+                $names = implode(', ', array_map(function($dependent) {
                     return $dependent->getModname();
                 }, $requiredDependents));
 

src/system/ZAuthModule/Listener/UserDeleteListener.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 
     public function deleteUsers(GenericEvent $event): void
     {
-        $deletedUid = (int) $event->getSubject();
+        $deletedUid = (int)$event->getSubject();
         $this->mappingRepository->removeByZikulaId($deletedUid);
         $this->verificationRepository->removeByZikulaId($deletedUid);
     }