zikula / core

src/Zikula/CoreInstallerBundle/Helper/ParameterHelper.php

Spacing +2 added lines, -2 removed lines

@@ -123,7 +123,7 @@ 
             $this->variableApi->set(VariableApi::CONFIG, 'locale', $params['locale']);
             // Set the System Identifier as a unique string.
             if (!$this->variableApi->get(VariableApi::CONFIG, 'system_identifier')) {
-                $this->variableApi->set(VariableApi::CONFIG, 'system_identifier', str_replace('.', '', uniqid((string) (random_int(1000000000, 9999999999)), true)));
+                $this->variableApi->set(VariableApi::CONFIG, 'system_identifier', str_replace('.', '', uniqid((string)(random_int(1000000000, 9999999999)), true)));
             }
             // add admin email as site email
             $this->variableApi->set(VariableApi::CONFIG, 'adminmail', $params['email']);
@@ -162,7 +162,7 @@ 
             }
 
             // on upgrade, if a user doesn't add their custom theme back to the /theme dir, it should be reset to a core theme, if available.
-            $defaultTheme = (string) $this->variableApi->getSystemVar('Default_Theme');
+            $defaultTheme = (string)$this->variableApi->getSystemVar('Default_Theme');
             if (!$this->kernel->isBundle($defaultTheme) && $this->kernel->isBundle('ZikulaBootstrapTheme')) {
                 $this->variableApi->set(VariableApi::CONFIG, 'Default_Theme', 'ZikulaBootstrapTheme');
             }

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}/public/uploads",
+            "${builddir}/${name}/var/cache",
+            "${builddir}/${name}/var/log",
+            "${builddir}/${name}/public/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,17 +117,17 @@ 
 -----------------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>");
 
         return 0;
     }

src/system/RoutesModule/Routing/RouteLoader.php

Braces +36 added lines, -36 removed lines

@@ -36,8 +36,8 @@ 
  *

  * @see https://symfony.com/doc/current/routing/custom_route_loader.html

  */

-class RouteLoader extends Loader

-{

+class RouteLoader extends Loader
+{
     /**

      * @var bool

      */

@@ -85,8 +85,8 @@ 
         ExtractTranslationHelper $extractTranslationHelper,

         PathBuilderHelper $pathBuilderHelper,

         SanitizeHelper $sanitizeHelper,

-        string $locale)

-    {

+        string $locale)
+    {
         $this->kernel = $kernel;

         $this->translator = $translator;

         $this->entityFactory = $entityFactory;

@@ -96,9 +96,9 @@ 
         $this->locale = $locale;

     }

 

-    public function load($resource, string $type = null)

-    {

-        if (true === $this->loaded) {

+    public function load($resource, string $type = null)
+    {
+        if (true === $this->loaded) {
             throw new RuntimeException('Do not add the "zikularoutesmodule" loader twice');

         }

 

@@ -108,9 +108,9 @@ 
 

         $routeCollection->addCollection($topRoutes);

 

-        try {

+        try {
             $customRoutes = $this->entityFactory->getRepository('route')->findBy([], ['sort' => 'ASC']);

-        } catch (Exception $exception) {

+        } catch (Exception $exception) {
             $routeCollection->addCollection($middleRoutes);

             $routeCollection->addCollection($bottomRoutes);

 

@@ -118,7 +118,7 @@ 
             return $routeCollection;

         }

 

-        if (!empty($customRoutes)) {

+        if (!empty($customRoutes)) {
             $this->addCustomRoutes($routeCollection, $customRoutes);

         }

 

@@ -144,7 +144,7 @@ 
         $topRoutes = new RouteCollection();

         $middleRoutes = new RouteCollection();

         $bottomRoutes = new RouteCollection();

-        foreach ($extensions as $extension) {

+        foreach ($extensions as $extension) {
             [$currentMiddleRoutes, $currentTopRoutes, $currentBottomRoutes] = $this->find($extension);

             $middleRoutes->addCollection($currentMiddleRoutes);

             $topRoutes->addCollection($currentTopRoutes);

@@ -161,9 +161,9 @@ 
      */

     private function find(AbstractExtension $extension): array

     {

-        try {

+        try {
             $path = $this->kernel->locateResource($extension->getRoutingConfig());

-        } catch (InvalidArgumentException $exception) {

+        } catch (InvalidArgumentException $exception) {
             // Routing file does not exist (e.g. because the extension could not be located).

             return [new RouteCollection(), new RouteCollection(), new RouteCollection()];

         }

@@ -182,19 +182,19 @@ 
         // Add all resources from the imported route collection to the middleRouteCollection.

         // The actual collection (top, middle, bottom) to add the resources too does not matter,

         // they just must be added to one of them, so that they don't get lost.

-        foreach ($routeCollection->getResources() as $resource) {

+        foreach ($routeCollection->getResources() as $resource) {
             $middleRoutes->addResource($resource);

         }

 

         /** @var Route $route */

-        foreach ($routeCollection as $oldRouteName => $route) {

+        foreach ($routeCollection as $oldRouteName => $route) {
             // set break here with $oldRouteName == 'zikula_routesmodule_route_renew'

             $this->prependExtensionPrefix($route, $extension);

             [$type, $func] = $this->setZikulaDefaults($route, $extension, $name);

             $routeName = $this->getRouteName($oldRouteName, $name, $type, $func);

 

-            if ($route->hasOption('zkPosition')) {

-                switch ($route->getOption('zkPosition')) {

+            if ($route->hasOption('zkPosition')) {
+                switch ($route->getOption('zkPosition')) {
                     case 'top':

                         $topRoutes->add($routeName, $route);

                         break;

@@ -204,7 +204,7 @@ 
                     default:

                         $middleRoutes->add($routeName, $route);

                 }

-            } else {

+            } else {
                 $middleRoutes->add($routeName, $route);

             }

         }

@@ -218,9 +218,9 @@ 
     private function addCustomRoutes(RouteCollection $routeCollection, array $customRoutes = []): void

     {

         /** @var RouteEntity $dbRoute */

-        foreach ($customRoutes as $dbRoute) {

+        foreach ($customRoutes as $dbRoute) {
             $extensionName = $dbRoute->getBundle();

-            if (!$this->kernel->isBundle($extensionName)) {

+            if (!$this->kernel->isBundle($extensionName)) {
                 continue;

             }

             $extension = $this->kernel->getBundle($extensionName);

@@ -237,9 +237,9 @@ 
             // We have to prepend the extension prefix (see detailed description in docblock of prependExtensionPrefix() method).

             $options = $dbRoute->getOptions();

             $prependExtension = empty($this->extractTranslationHelper->getBundleName()) && isset($options['i18n']) && !$options['i18n'];

-            if ($prependExtension) {

+            if ($prependExtension) {
                 $path = $this->pathBuilderHelper->getPathWithBundlePrefix($dbRoute);

-            } else {

+            } else {
                 $path = $dbRoute->getPath();

             }

 

@@ -272,9 +272,9 @@ 
         $defaults = $route->getDefaults();

 

         $defaults['_zkBundle'] = $extensionName;

-        if ($extension instanceof AbstractModule) {

+        if ($extension instanceof AbstractModule) {
             $defaults['_zkModule'] = $extensionName;

-        } else if ($extension instanceof AbstractTheme) {

+        } else if ($extension instanceof AbstractTheme) {
             $defaults['_zkTheme'] = $extensionName;

         }

 

@@ -300,24 +300,24 @@ 
      * If i18n is set to true, Zikula's customized pattern generation strategy will take care of it.

      * See Zikula\RoutesModule\Translation\ZikulaPatternGenerationStrategy

      */

-    private function prependExtensionPrefix(Route $route, AbstractExtension $extension)

-    {

+    private function prependExtensionPrefix(Route $route, AbstractExtension $extension)
+    {
         $prefix = '';

         $options = $route->getOptions();

         $prependExtension = empty($this->extractTranslationHelper->getBundleName()) && isset($options['i18n']) && !$options['i18n'];

-        if (!$prependExtension) {

+        if (!$prependExtension) {
             return;

         }

-        if ((isset($options['zkNoBundlePrefix']) && $options['zkNoBundlePrefix'])) {

+        if ((isset($options['zkNoBundlePrefix']) && $options['zkNoBundlePrefix'])) {
             return;

         }

 

         // get url from extension meta data first. May be empty.

         $untranslatedPrefix = $extension->getMetaData()->getUrl(false);

-        if (!empty($untranslatedPrefix)) {

-            if ($this->translator->getCatalogue($this->locale)->has($untranslatedPrefix, strtolower($extension->getName()))) {

+        if (!empty($untranslatedPrefix)) {
+            if ($this->translator->getCatalogue($this->locale)->has($untranslatedPrefix, strtolower($extension->getName()))) {
                 $prefix = $this->translator->trans(/** @Ignore */$untranslatedPrefix, [], strtolower($extension->getName()), $this->locale);

-            } else {

+            } else {
                 $prefix = $untranslatedPrefix;

             }

         }

@@ -333,7 +333,7 @@ 
      */

     private function sanitizeController(string $extensionName, string $controllerString): string

     {

-        if (0 === preg_match('#^(.*?\\\\Controller\\\\(.+)Controller)::(.+)Action$#', $controllerString, $match)) {

+        if (0 === preg_match('#^(.*?\\\\Controller\\\\(.+)Controller)::(.+)Action$#', $controllerString, $match)) {
             return $controllerString;

         }

 

@@ -348,9 +348,9 @@ 
     {

         $suffix = '';

         $lastHit = strrpos($oldRouteName, '_');

-        if (false !== $lastHit) {

+        if (false !== $lastHit) {
             $lastPart = substr($oldRouteName, $lastHit);

-            if (is_numeric($lastPart)) {

+            if (is_numeric($lastPart)) {
                 // If the last part of the old route name is numeric, also append it to the new route name.

                 // This allows multiple routes for the same action.

                 $suffix = '_' . $lastPart;

@@ -365,8 +365,8 @@ 
      *

      * @return boolean

      */

-    public function supports($resource, string $type = null)

-    {

+    public function supports($resource, string $type = null)
+    {
         return 'zikularoutesmodule' === $type;

     }

 }

src/system/RoutesModule/Translation/ConsoleCommandListener.php

Braces +11 added lines, -11 removed lines

@@ -18,20 +18,20 @@ 
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;

 use Zikula\RoutesModule\Helper\ExtractTranslationHelper;

 

-class ConsoleCommandListener implements EventSubscriberInterface

-{

+class ConsoleCommandListener implements EventSubscriberInterface
+{
     /**

      * @var ExtractTranslationHelper

      */

     private $extractTranslationHelper;

 

-    public function __construct(ExtractTranslationHelper $extractTranslationHelper)

-    {

+    public function __construct(ExtractTranslationHelper $extractTranslationHelper)
+    {
         $this->extractTranslationHelper = $extractTranslationHelper;

     }

 

-    public static function getSubscribedEvents()

-    {

+    public static function getSubscribedEvents()
+    {
         return [

             ConsoleEvents::COMMAND => ['checkBundleForTranslatingRoutes']

         ];

@@ -43,21 +43,21 @@ 
      */

     public function checkBundleForTranslatingRoutes(ConsoleCommandEvent $event): void

     {

-        if ('translation:extract' !== $event->getCommand()->getName()) {

+        if ('translation:extract' !== $event->getCommand()->getName()) {
             return;

         }

 

         $this->extractTranslationHelper->setExtensionName('');

 

         $bundle = null;

-        if ($event->getInput()->hasParameterOption('-b')) {

+        if ($event->getInput()->hasParameterOption('-b')) {
             $bundle = $event->getInput()->getParameterOption('-b');

-        } elseif ($event->getInput()->hasParameterOption('--bundle')) {

+        } elseif ($event->getInput()->hasParameterOption('--bundle')) {
             $bundle = $event->getInput()->getParameterOption('--bundle');

         }

 

-        if (null !== $bundle) {

-            if ('@' === $bundle[0]) {

+        if (null !== $bundle) {
+            if ('@' === $bundle[0]) {
                 $bundle = substr($bundle, 1);

             }

 

src/system/RoutesModule/Helper/PathBuilderHelper.php

Braces +7 added lines, -7 removed lines

@@ -18,15 +18,15 @@ 
 use Zikula\ExtensionsModule\AbstractExtension;

 use Zikula\RoutesModule\Entity\RouteEntity;

 

-class PathBuilderHelper

-{

+class PathBuilderHelper
+{
     /**

      * @var ZikulaHttpKernelInterface

      */

     private $kernel;

 

-    public function __construct(ZikulaHttpKernelInterface $kernel)

-    {

+    public function __construct(ZikulaHttpKernelInterface $kernel)
+    {
         $this->kernel = $kernel;

     }

 

@@ -36,7 +36,7 @@ 
     public function getPathWithBundlePrefix(RouteEntity $route): string

     {

         $options = $route->getOptions();

-        if (isset($options['zkNoBundlePrefix']) && $options['zkNoBundlePrefix']) {

+        if (isset($options['zkNoBundlePrefix']) && $options['zkNoBundlePrefix']) {
             // return path only

             return $route->getPath();

         }

@@ -45,9 +45,9 @@ 
          * @var AbstractExtension $extensionBundle

          */

         $extensionBundle = null;

-        try {

+        try {
             $extensionBundle = $this->kernel->getBundle($route->getBundle());

-        } catch (Exception $exception) {

+        } catch (Exception $exception) {
             return $route->getPath();

         }

 

src/system/RoutesModule/Helper/ExtractTranslationHelper.php

Braces +2 added lines, -2 removed lines

@@ -13,8 +13,8 @@
 

 namespace Zikula\RoutesModule\Helper;

 

-class ExtractTranslationHelper

-{

+class ExtractTranslationHelper
+{
     /**

      * @var string

      */

src/system/AtomTheme/ZikulaAtomTheme.php

Braces +2 added lines, -2 removed lines

@@ -17,8 +17,8 @@
 use Twig\Environment;

 use Zikula\ExtensionsModule\AbstractCoreTheme;

 

-class ZikulaAtomTheme extends AbstractCoreTheme

-{

+class ZikulaAtomTheme extends AbstractCoreTheme
+{
     /**

      * Override parent method in order to add Content-type header to Response.

      */

src/system/BlocksModule/Api/BlockApi.php

Braces +12 added lines, -12 removed lines

@@ -32,8 +32,8 @@ 
  * This class provides an API for interaction with and management of blocks. The class is mainly used internally

  * by Twig-based theme tags in order to 'decorate' a page with the requested blocks.

  */

-class BlockApi implements BlockApiInterface

-{

+class BlockApi implements BlockApiInterface
+{
     public const BLOCK_ACTIVE = 1;

 

     public const BLOCK_INACTIVE = 0;

@@ -63,7 +63,7 @@ 
         BlockFactoryApiInterface $blockFactoryApi,

         ExtensionRepositoryInterface $extensionRepository,

         BlockCollector $blockCollector

-    ) {

+    ) {
         $this->blockPositionRepository = $blockPositionRepository;

         $this->blockFactory = $blockFactoryApi;

         $this->extensionRespository = $extensionRepository;

@@ -72,18 +72,18 @@ 
 

     public function getBlocksByPosition(string $positionName): array

     {

-        if (empty($positionName)) {

+        if (empty($positionName)) {
             throw new InvalidArgumentException('Name must not be empty.');

         }

 

         /** @var BlockPositionEntity $position */

         $position = $this->blockPositionRepository->findByName($positionName);

         $blocks = [];

-        if (null === $position) {

+        if (null === $position) {
             return $blocks;

         }

 

-        foreach ($position->getPlacements() as $placement) {

+        foreach ($position->getPlacements() as $placement) {
             /** @var BlockEntity $block */

             $block = $placement->getBlock();

             $blocks[$block->getBid()] = $block;

@@ -94,7 +94,7 @@ 
 

     public function createInstanceFromBKey(string $bKey): BlockHandlerInterface

     {

-        if (false !== mb_strpos($bKey, ':')) {

+        if (false !== mb_strpos($bKey, ':')) {
             @trigger_error('The BKey should not contain the module name with a colon as of Core-3.0.0');

         }

 

@@ -106,15 +106,15 @@ 
         $modulesByName = [];

         $modules = isset($moduleEntity) ? [$moduleEntity] : $this->extensionRespository->findBy(['state' => Constant::STATE_ACTIVE]);

         /** @var ExtensionEntity $module */

-        foreach ($modules as $module) {

+        foreach ($modules as $module) {
             $modulesByName[$module->getName()] = $module;

         }

 

         $foundBlocks = [];

-        foreach ($this->blockCollector->getBlocks() as $id => $blockInstance) {

+        foreach ($this->blockCollector->getBlocks() as $id => $blockInstance) {
             /** @var AbstractBlockHandler $blockInstance */

             $bundleName = $blockInstance->getExtension()->getName();

-            if (!array_key_exists($bundleName, $modulesByName)) {

+            if (!array_key_exists($bundleName, $modulesByName)) {
                 continue;

             }

 

@@ -129,10 +129,10 @@ 
     {

         $modules = $this->extensionRespository->findBy(['state' => Constant::STATE_ACTIVE]);

         $modulesContainingBlocks = [];

-        foreach ($modules as $module) {

+        foreach ($modules as $module) {
             /** @var ExtensionEntity $module */

             $blocks = $this->getAvailableBlockTypes($module);

-            if (!empty($blocks)) {

+            if (!empty($blocks)) {
                 $modulesContainingBlocks[$module->getId()] = $module->getName();

             }

         }

src/system/BlocksModule/Controller/BlockController.php

Braces +20 added lines, -20 removed lines

@@ -35,8 +35,8 @@ 
  * Class BlockController

  * @Route("/admin/block")

  */

-class BlockController extends AbstractController

-{

+class BlockController extends AbstractController
+{
     /**

      * @Route("/new")

      * @Theme("admin")

@@ -47,7 +47,7 @@ 
     {

         $form = $this->createForm(NewBlockType::class);

         $form->handleRequest($request);

-        if ($form->isSubmitted() && $form->isValid()) {

+        if ($form->isSubmitted() && $form->isValid()) {
             $bkey = json_encode($form->getData()['bkey']);

 

             return $this->redirectToRoute('zikulablocksmodule_block_edit', ['bkey' => $bkey]);

@@ -74,10 +74,10 @@ 
     ): Response {

         $requiredAccessLevel = ACCESS_EDIT;

         $isCreation = false;

-        if (null === $blockEntity) {

+        if (null === $blockEntity) {
             $isCreation = true;

             $bKey = json_decode($request->query->get('bkey'));

-            if (empty($bKey)) {

+            if (empty($bKey)) {
                 return $this->redirectToRoute('zikulablocksmodule_block_new');

             }

             $blockEntity = new BlockEntity(); // sets defaults in constructor

@@ -85,31 +85,31 @@ 
             $requiredAccessLevel = ACCESS_ADD;

         }

 

-        if (!$this->hasPermission('ZikulaBlocksModule::', $blockEntity->getBlocktype() . ':' . $blockEntity->getTitle() . ':' . $blockEntity->getBid(), $requiredAccessLevel)) {

+        if (!$this->hasPermission('ZikulaBlocksModule::', $blockEntity->getBlocktype() . ':' . $blockEntity->getTitle() . ':' . $blockEntity->getBid(), $requiredAccessLevel)) {
             throw new AccessDeniedException();

         }

 

         $blockInstance = $blockApi->createInstanceFromBKey($blockEntity->getBkey());

-        if (true === $isCreation) {

+        if (true === $isCreation) {
             $blockEntity->setProperties($blockInstance->getPropertyDefaults());

         }

         $blockType = $blockEntity->getBlocktype();

-        if (empty($blockType)) {

+        if (empty($blockType)) {
             $blockEntity->setBlocktype($blockInstance->getType());

         }

 

         $form = $this->createForm(BlockType::class, $blockEntity, ['locale' => $request->getLocale()]);

-        if ($blockInstance instanceof BlockHandlerInterface && '' !== $blockInstance->getFormClassName()) {

+        if ($blockInstance instanceof BlockHandlerInterface && '' !== $blockInstance->getFormClassName()) {
             $form->add('properties', $blockInstance->getFormClassName(), $blockInstance->getFormOptions());

         }

         $form->handleRequest($request);

 

         $moduleName = $blockInstance->getExtension()->getName();

-        if ($form->isSubmitted()) {

-            if ($form->isValid() && $form->get('save')->isClicked()) {

+        if ($form->isSubmitted()) {
+            if ($form->isValid() && $form->get('save')->isClicked()) {
                 // remove orphan properties (#3892)

                 $properties = [];

-                foreach ($form->get('properties')->all() as $child) {

+                foreach ($form->get('properties')->all() as $child) {
                     $properties[$child->getName()] = $child->getData();

                 }

                 $blockEntity->setProperties($properties);

@@ -128,7 +128,7 @@ 
                 $em->persist($blockEntity);

                 $em->flush();

                 $this->addFlash('status', 'Done! Block saved.');

-            } elseif ($form->get('cancel')->isClicked()) {

+            } elseif ($form->get('cancel')->isClicked()) {
                 $this->addFlash('status', 'Operation cancelled.');

             }

 

@@ -152,19 +152,19 @@ 
      */

     public function deleteAction(Request $request, BlockEntity $blockEntity): Response

     {

-        if (!$this->hasPermission('ZikulaBlocksModule::', $blockEntity->getBkey() . ':' . $blockEntity->getTitle() . ':' . $blockEntity->getBid(), ACCESS_DELETE)) {

+        if (!$this->hasPermission('ZikulaBlocksModule::', $blockEntity->getBkey() . ':' . $blockEntity->getTitle() . ':' . $blockEntity->getBid(), ACCESS_DELETE)) {
             throw new AccessDeniedException();

         }

 

         $form = $this->createForm(DeletionType::class);

         $form->handleRequest($request);

-        if ($form->isSubmitted() && $form->isValid()) {

-            if ($form->get('delete')->isClicked()) {

+        if ($form->isSubmitted() && $form->isValid()) {
+            if ($form->get('delete')->isClicked()) {
                 $em = $this->getDoctrine()->getManager();

                 $em->remove($blockEntity);

                 $em->flush();

                 $this->addFlash('status', 'Done! Block deleted.');

-            } elseif ($form->get('cancel')->isClicked()) {

+            } elseif ($form->get('cancel')->isClicked()) {
                 $this->addFlash('status', 'Operation cancelled.');

             }

 

@@ -186,16 +186,16 @@ 
      */

     public function toggleblockAction(Request $request): JsonResponse

     {

-        if (!$this->hasPermission('ZikulaBlocksModule::', '::', ACCESS_ADMIN)) {

+        if (!$this->hasPermission('ZikulaBlocksModule::', '::', ACCESS_ADMIN)) {
             return $this->json($this->trans('No permission for this action.'), Response::HTTP_FORBIDDEN);

         }

         $bid = $request->request->getInt('bid', -1);

-        if (-1 === $bid) {

+        if (-1 === $bid) {
             return $this->json($this->trans('No block ID passed.'), Response::HTTP_BAD_REQUEST);

         }

         $em = $this->getDoctrine()->getManager();

         $block = $em->find('ZikulaBlocksModule:BlockEntity', $bid);

-        if (null !== $block) {

+        if (null !== $block) {
             $block->setActive(BlockApi::BLOCK_ACTIVE === $block->getActive() ? BlockApi::BLOCK_INACTIVE : BlockApi::BLOCK_ACTIVE);

             $em->flush();

         }