fabiang / assetic-module

src/Listener.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
         $config = $sm->get('AsseticConfiguration');
         if ($e->getName() === MvcEvent::EVENT_DISPATCH_ERROR) {
             $error = $e->getError();
-            if ($error && ! in_array($error, $config->getAcceptableErrors())) {
+            if ($error && !in_array($error, $config->getAcceptableErrors())) {
                 // break if not an acceptable error
                 return;
             }
@@ -52,7 +52,7 @@ 
 
         /** @var ResponseInterface|null $response */
         $response = $e->getResponse();
-        if (! $response) {
+        if (!$response) {
             $response = new Response();
             $e->setResponse($response);
         }

src/FilterManager.php

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
         }
 
         $service = $this->container;
-        if (! $service->has($alias)) {
+        if (!$service->has($alias)) {
             throw new InvalidArgumentException(
                 sprintf(
                     'There is no "%s" filter in Laminas service manager.',
@@ -52,7 +52,7 @@ 
         }
 
         $filter = $service->get($alias);
-        if (! $filter instanceof FilterInterface) {
+        if (!$filter instanceof FilterInterface) {
             $givenType = is_object($filter) ? FilterInterface::class : gettype($filter);
             $message   = 'Retrieved filter "%s" is not instanceof '
                 . '"Assetic\Filter\FilterInterface", but type was given %s';

src/Service.php

Spacing +16 added lines, -16 removed lines

@@ -177,7 +177,7 @@ 
             $option = null;
             $name   = null;
             if (is_array($options)) {
-                if (! isset($options['name'])) {
+                if (!isset($options['name'])) {
                     throw new InvalidArgumentException(
                         'Filter "' . $alias . '" required option "name"'
                     );
@@ -190,7 +190,7 @@ 
                 unset($options);
             }
 
-            if (! is_string($name)) {
+            if (!is_string($name)) {
                 throw new InvalidArgumentException(
                     'Name of filter could not be found. '
                     . 'Did you provide the `name` option to the filter config?'
@@ -204,8 +204,8 @@ 
             // Filter Id should have optional filter indicator "?"
             $filterId = ltrim($alias, '?');
 
-            if (! $fm->has($filterId)) {
-                if (is_array($option) && ! empty($option)) {
+            if (!$fm->has($filterId)) {
+                if (is_array($option) && !empty($option)) {
                     $r = new ReflectionClass($name);
                     /** @var FilterInterface $filter */
                     $filter = $r->newInstanceArgs($option);
@@ -310,7 +310,7 @@ 
 
     public function setupRendererFromOptions(Renderer $renderer, array $options): void
     {
-        if (! $this->hasStrategyForRenderer($renderer)) {
+        if (!$this->hasStrategyForRenderer($renderer)) {
             throw new InvalidArgumentException(sprintf(
                 'no strategy defined for renderer "%s"',
                 $this->getRendererName($renderer)
@@ -345,12 +345,12 @@ 
      */
     public function getStrategyForRenderer(Renderer $renderer): ?StrategyInterface
     {
-        if (! $this->hasStrategyForRenderer($renderer)) {
+        if (!$this->hasStrategyForRenderer($renderer)) {
             return null;
         }
 
         $rendererName = $this->getRendererName($renderer);
-        if (! isset($this->strategy[$rendererName])) {
+        if (!isset($this->strategy[$rendererName])) {
             $strategyClass = $this->configuration->getStrategyNameForRenderer($rendererName);
 
             if (null === $strategyClass) {
@@ -362,7 +362,7 @@ 
                 );
             }
 
-            if (! class_exists($strategyClass, true)) {
+            if (!class_exists($strategyClass, true)) {
                 throw new InvalidArgumentException(
                     sprintf(
                         'strategy class "%s" dosen\'t exists',
@@ -373,7 +373,7 @@ 
 
             $instance = new $strategyClass();
 
-            if (! $instance instanceof StrategyInterface) {
+            if (!$instance instanceof StrategyInterface) {
                 throw new Exception\DomainException(
                     sprintf(
                         'strategy class "%s" is not instanceof "Fabiang\AsseticBundle\View\StrategyInterface"',
@@ -440,9 +440,9 @@ 
             $sourcePath = $value->getSourcePath() ?? '';
 
             if ($disableSourcePath) {
-                $value->setTargetPath(( $targetPath ?? '' ) . basename($sourcePath));
+                $value->setTargetPath(($targetPath ?? '') . basename($sourcePath));
             } else {
-                $value->setTargetPath(( $targetPath ?? '' ) . $sourcePath);
+                $value->setTargetPath(($targetPath ?? '') . $sourcePath);
             }
 
             $value = $this->cacheAsset($value);
@@ -457,7 +457,7 @@ 
         $options           = $options['options'] ?? [];
         $options['output'] = $options['output'] ?? $name;
         $moveRaw           = isset($options['move_raw']) && $options['move_raw'];
-        $targetPath        = ! empty($options['targetPath']) ? $options['targetPath'] : '';
+        $targetPath        = !empty($options['targetPath']) ? $options['targetPath'] : '';
         if (substr($targetPath, -1) !== DIRECTORY_SEPARATOR) {
             $targetPath .= DIRECTORY_SEPARATOR;
         }
@@ -490,12 +490,12 @@ 
     public function writeAsset(AssetInterface $asset, AssetFactory $factory): void
     {
         // We're not interested in saving assets on request
-        if (! $this->configuration->getBuildOnRequest()) {
+        if (!$this->configuration->getBuildOnRequest()) {
             return;
         }
 
         // Write asset on disk on every request
-        if (! $this->configuration->getWriteIfChanged()) {
+        if (!$this->configuration->getWriteIfChanged()) {
             $this->write($asset, $factory);
 
             return;
@@ -515,7 +515,7 @@ 
         }
 
         // And long requested optimization
-        if (! $created || $isChanged) {
+        if (!$created || $isChanged) {
             $this->write($asset, $factory);
         }
     }
@@ -532,7 +532,7 @@ 
         }
 
         if (
-            $this->configuration->isDebug() && ! $this->configuration->isCombine() && $asset instanceof AssetCollection
+            $this->configuration->isDebug() && !$this->configuration->isCombine() && $asset instanceof AssetCollection
         ) {
             foreach ($asset as $item) {
                 $this->writeAsset($item, $factory);

src/Configuration.php

Spacing +3 added lines, -3 removed lines

@@ -193,11 +193,11 @@ 
 
     public function setDefault(array $default): void
     {
-        if (! isset($default['assets'])) {
+        if (!isset($default['assets'])) {
             $default['assets'] = [];
         }
 
-        if (! isset($default['options'])) {
+        if (!isset($default['options'])) {
             $default['options'] = [];
         }
 
@@ -321,7 +321,7 @@ 
         $parts  = explode('_', $key);
         $parts  = array_map('ucfirst', $parts);
         $setter = 'set' . implode('', $parts);
-        if (! method_exists($this, $setter)) {
+        if (!method_exists($this, $setter)) {
             throw new Exception\BadMethodCallException(
                 'The configuration key "' . $key . '" does not '
                     . 'have a matching ' . $setter . ' setter method '

src/View/ViewHelperStrategy.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         $baseUrl  = $this->str($this->getBaseUrl());
         $basePath = $this->str($this->getBasePath());
 
-        if ($this->isDebug() && ! $this->isCombine() && $asset instanceof AssetCollection) {
+        if ($this->isDebug() && !$this->isCombine() && $asset instanceof AssetCollection) {
             // Move assets as single instance not as a collection
             /** @var AssetCollection $value */
             foreach ($asset as $value) {

src/View/Helper/Asset.php

Spacing +6 added lines, -6 removed lines

@@ -46,7 +46,7 @@ 
         /**
          * @psalm-suppress UndefinedDocblockClass Upstram bug in `@return`
          */
-        if (! $this->service->getAssetManager()->has($collectionName)) {
+        if (!$this->service->getAssetManager()->has($collectionName)) {
             throw new Exception\InvalidArgumentException(
                 'Collection "' . $collectionName . '" does not exist.'
             );
@@ -63,7 +63,7 @@ 
 
         if (
             $this->service->getConfiguration()->isDebug()
-            && ! $this->service->getConfiguration()->isCombine()
+            && !$this->service->getConfiguration()->isCombine()
             && $asset instanceof AssetCollection
         ) {
             // Move assets as single instance not as a collection
@@ -107,16 +107,16 @@ 
 
     protected function getScriptTag(string $path, array $options = []): string
     {
-        $type = isset($options['type']) && ! empty($options['type']) ? $options['type'] : 'text/javascript';
+        $type = isset($options['type']) && !empty($options['type']) ? $options['type'] : 'text/javascript';
 
         return '<script type="' . $this->escape($type) . '" src="' . $this->escape($path) . '"></script>';
     }
 
     protected function getStylesheetTag(string $path, array $options = []): string
     {
-        $media = isset($options['media']) && ! empty($options['media']) ? $options['media'] : 'screen';
-        $type  = isset($options['type']) && ! empty($options['type']) ? $options['type'] : 'text/css';
-        $rel   = isset($options['rel']) && ! empty($options['rel']) ? $options['rel'] : 'stylesheet';
+        $media = isset($options['media']) && !empty($options['media']) ? $options['media'] : 'screen';
+        $type  = isset($options['type']) && !empty($options['type']) ? $options['type'] : 'text/css';
+        $rel   = isset($options['rel']) && !empty($options['rel']) ? $options['rel'] : 'stylesheet';
 
         return '<link href="' . $this->escape($path)
             . '" media="' . $this->escape($media)

src/Cli/SetupCommand.php

Spacing +7 added lines, -7 removed lines

@@ -35,12 +35,12 @@ 
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $config = $this->assetic->getConfiguration();
-        $mode   = null !== ($mode   = $config->getUmask()) ? $mode : 0775;
+        $mode   = null !== ($mode = $config->getUmask()) ? $mode : 0775;
 
-        if (! $this->createPath($output, 'Cache', $config->getCachePath() ?? '', $mode)) {
+        if (!$this->createPath($output, 'Cache', $config->getCachePath() ?? '', $mode)) {
             return 1;
         }
-        if (! $this->createPath($output, 'Web', $config->getWebPath() ?? '', $mode)) {
+        if (!$this->createPath($output, 'Web', $config->getWebPath() ?? '', $mode)) {
             return 1;
         }
 
@@ -54,12 +54,12 @@ 
     {
         $displayMode = decoct($mode);
         $pathExists  = is_dir($path);
-        if (! $path) {
+        if (!$path) {
             $output->writeln('Creation of ' . $which . ' path skipped - no path provided');
 
             return true;
         }
-        if (! $pathExists) {
+        if (!$pathExists) {
             if (mkdir($path, $mode, true)) {
                 $output->writeln($which . ' path created "' . $path . '" with mode "' . $displayMode . '"');
 
@@ -79,13 +79,13 @@ 
             );
 
             return true;
-        } elseif (! $readable && ! $writable) {
+        } elseif (!$readable && !$writable) {
             $output->writeln(
                 '<error>Creation of ' . $which . ' path "'
                     . $path
                     . '" failed - path exists but is neither readable nor writable</error>'
             );
-        } elseif (! $readable) {
+        } elseif (!$readable) {
             $output->writeln(
                 '<error>Creation of ' . $which . ' path "'
                     . $path . '" failed - path exists but is not readable</error>'