superdesk / web-publisher

src/SWP/Bundle/ContentBundle/Routing/SeoMediaRouter.php

Spacing +1 added lines, -1 removed lines

@@ -50,6 +50,6 @@
       unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
 
-    return 'Route for media ' . $name->getValues()->getId() . ' not found';
+    return 'Route for media '.$name->getValues()->getId().' not found';
   }
 }

Indentation +14 added lines, -14 removed lines

@@ -26,15 +26,15 @@ 
 
 class SeoMediaRouter extends Router implements VersatileGeneratorInterface {
 
-  const OBJECT_BASED_ROUTE_NAME = "__seo_media_router_route_name__";
+    const OBJECT_BASED_ROUTE_NAME = "__seo_media_router_route_name__";
 
-  /**
-   * {@inheritdoc}
-   */
-  public function generate($name, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH) {
+    /**
+     * {@inheritdoc}
+     */
+    public function generate($name, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH) {
     if (self::OBJECT_BASED_ROUTE_NAME === $name && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)) {
-      $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
-      unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
+        $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
+        unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
 
     $item = $name->getValues()->getImage();
@@ -43,18 +43,18 @@ 
     $parameters['extension'] = $item->getFileExtension();
 
     return parent::generate('swp_seo_media_get', $parameters, $referenceType);
-  }
+    }
 
-  public function supports($name): bool {
+    public function supports($name): bool {
     return (is_string($name) && $name == self::OBJECT_BASED_ROUTE_NAME) || ($name instanceof Meta && $name->getValues() instanceof ArticleSeoMediaInterface);
-  }
+    }
 
-  public function getRouteDebugMessage($name, array $parameters = array()): string {
+    public function getRouteDebugMessage($name, array $parameters = array()): string {
     if (self::OBJECT_BASED_ROUTE_NAME === $name && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)) {
-      $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
-      unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
+        $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
+        unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
 
     return 'Route for media ' . $name->getValues()->getId() . ' not found';
-  }
+    }
 }

src/SWP/Bundle/CoreBundle/Twig/DecoratingRoutingExtension.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
 
     try {
       return $this->routingExtension->getPath($name, $parameters, $relative);
-    } catch (RouteNotFoundException|MissingMandatoryParametersException|InvalidParameterException $e) {
+    } catch (RouteNotFoundException | MissingMandatoryParametersException | InvalidParameterException $e) {
       // allow empty path
     }
 
@@ -64,7 +64,7 @@ 
 
     try {
       return $this->routingExtension->getUrl($name, $parameters, $schemeRelative);
-    } catch (RouteNotFoundException|MissingMandatoryParametersException|InvalidParameterException $e) {
+    } catch (RouteNotFoundException | MissingMandatoryParametersException | InvalidParameterException $e) {
       // allow empty url
     }
 

Indentation +33 added lines, -33 removed lines

@@ -36,88 +36,88 @@
 use Twig\TwigFunction;
 
 final class DecoratingRoutingExtension extends AbstractExtension {
-  private RoutingExtension $routingExtension;
+    private RoutingExtension $routingExtension;
 
-  public function __construct(RoutingExtension $routingExtension) {
+    public function __construct(RoutingExtension $routingExtension) {
     $this->routingExtension = $routingExtension;
-  }
+    }
 
-  public function getPath($name, $parameters = [], $relative = false): ?string {
+    public function getPath($name, $parameters = [], $relative = false): ?string {
     if ($name == null) {
-      return null;
+        return null;
     }
 
     if (is_object($name)) {
-      $object = $name;
-      $name = null;
-      self::setupParams($object, $name, $parameters);
+        $object = $name;
+        $name = null;
+        self::setupParams($object, $name, $parameters);
     }
 
     try {
-      return $this->routingExtension->getPath($name, $parameters, $relative);
+        return $this->routingExtension->getPath($name, $parameters, $relative);
     } catch (RouteNotFoundException|MissingMandatoryParametersException|InvalidParameterException $e) {
-      // allow empty path
+        // allow empty path
     }
 
     return null;
-  }
+    }
 
-  public function getUrl($name, $parameters = [], $schemeRelative = false): ?string {
+    public function getUrl($name, $parameters = [], $schemeRelative = false): ?string {
     if ($name == null) {
-      return null;
+        return null;
     }
 
     if (is_object($name)) {
-      $object = $name;
-      $name = null;
-      self::setupParams($object, $name, $parameters);
+        $object = $name;
+        $name = null;
+        self::setupParams($object, $name, $parameters);
     }
 
     try {
-      return $this->routingExtension->getUrl($name, $parameters, $schemeRelative);
+        return $this->routingExtension->getUrl($name, $parameters, $schemeRelative);
     } catch (RouteNotFoundException|MissingMandatoryParametersException|InvalidParameterException $e) {
-      // allow empty url
+        // allow empty url
     }
 
     return null;
-  }
+    }
 
-  public static function setupParams(object $object, &$name, &$parameters) {
+    public static function setupParams(object $object, &$name, &$parameters) {
     $name = RouteObjectInterface::OBJECT_BASED_ROUTE_NAME;
     $parameters[RouteObjectInterface::ROUTE_OBJECT] = $object;
 
     if ($object instanceof Meta) {
-      $values = $object->getValues();
-      if (($values instanceof ArticleMediaInterface || $values instanceof ImageRenditionInterface)) {
+        $values = $object->getValues();
+        if (($values instanceof ArticleMediaInterface || $values instanceof ImageRenditionInterface)) {
         $name = MediaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
 
-      if ($values instanceof ArticleSeoMediaInterface) {
+        if ($values instanceof ArticleSeoMediaInterface) {
         $name = SeoMediaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
 
-      if ($values instanceof AuthorMediaInterface) {
+        if ($values instanceof AuthorMediaInterface) {
         $name = ArticleAuthorMediaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
 
-      if ($values instanceof ArticleInterface || $values instanceof RouteInterface) {
+        if ($values instanceof ArticleInterface || $values instanceof RouteInterface) {
         $name = MetaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
     }
 
     if ($object instanceof RouteInterface || $object instanceof ArticleInterface) {
-      $name = MetaRouter::OBJECT_BASED_ROUTE_NAME;
+        $name = MetaRouter::OBJECT_BASED_ROUTE_NAME;
+    }
     }
-  }
 
-  public function getFunctions(): array {
+    public function getFunctions(): array {
     return [
         new TwigFunction('url', [$this, 'getUrl'], ['is_safe_callback' => [$this->routingExtension, 'isUrlGenerationSafe']]),
         new TwigFunction('path', [$this, 'getPath'], ['is_safe_callback' => [$this->routingExtension, 'isUrlGenerationSafe']]),
     ];
-  }
+    }
 }

src/SWP/Bundle/CoreBundle/Routing/MetaRouter.php

Spacing +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
       $name = $route->getId();
     }
 
-    return md5($name . serialize($parameters) . $type);
+    return md5($name.serialize($parameters).$type);
   }
 
   /**

Indentation +28 added lines, -28 removed lines

@@ -24,43 +24,43 @@ 
 use Symfony\Component\Routing\Route as SymfonyRoute;
 
 class MetaRouter extends DynamicRouter {
-  const OBJECT_BASED_ROUTE_NAME = "__meta_router_route_name__";
+    const OBJECT_BASED_ROUTE_NAME = "__meta_router_route_name__";
 
-  protected $internalRoutesCache = [];
+    protected $internalRoutesCache = [];
 
-  public function generate($name, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH) {
+    public function generate($name, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH) {
     if (self::OBJECT_BASED_ROUTE_NAME === $name
         && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)
     ) {
-      $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
-      unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
+        $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
+        unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
 
     $cacheKey = $this->getCacheKey($name, $parameters, $referenceType);
     if (array_key_exists($cacheKey, $this->internalRoutesCache)) {
-      return $this->internalRoutesCache[$cacheKey];
+        return $this->internalRoutesCache[$cacheKey];
     }
 
     $route = $name;
     if ($name instanceof Meta) {
-      $object = $name->getValues();
-      if ($object instanceof ArticleInterface) {
+        $object = $name->getValues();
+        if ($object instanceof ArticleInterface) {
         $parameters['slug'] = $object->getSlug();
         $route = $object->getRoute();
         if (null === $route && $name->getContext()->getCurrentPage()) {
-          $parameters['slug'] = null;
-          $route = $name->getContext()->getCurrentPage()->getValues();
+            $parameters['slug'] = null;
+            $route = $name->getContext()->getCurrentPage()->getValues();
         }
-      } elseif ($name->getValues() instanceof RouteInterface) {
+        } elseif ($name->getValues() instanceof RouteInterface) {
         $route = $name->getValues();
-      }
+        }
     } elseif ($name instanceof ArticleInterface) {
-      $route = $name->getRoute();
-      $parameters['slug'] = $name->getSlug();
+        $route = $name->getRoute();
+        $parameters['slug'] = $name->getSlug();
     }
 
     if (null === $route || is_array($route)) {
-      throw new RouteNotFoundException(sprintf('Unable to generate a URL for the named route "%s" as such route does not exist.', $name));
+        throw new RouteNotFoundException(sprintf('Unable to generate a URL for the named route "%s" as such route does not exist.', $name));
     }
 
     $result = parent::generate($route, $parameters, $referenceType);
@@ -68,29 +68,29 @@ 
     unset($route);
 
     return $result;
-  }
+    }
 
-  private function getCacheKey($route, $parameters, $type) {
+    private function getCacheKey($route, $parameters, $type) {
     $name = $route;
     if ($route instanceof Meta) {
-      if ($route->getValues() instanceof ArticleInterface) {
+        if ($route->getValues() instanceof ArticleInterface) {
         $name = $route->getValues()->getId();
-      } elseif ($route->getValues() instanceof RouteInterface) {
+        } elseif ($route->getValues() instanceof RouteInterface) {
         $name = $route->getValues()->getName();
-      }
+        }
     } elseif ($route instanceof RouteInterface) {
-      $name = $route->getName();
+        $name = $route->getName();
     } elseif ($route instanceof ArticleInterface) {
-      $name = $route->getId();
+        $name = $route->getId();
     }
 
     return md5($name . serialize($parameters) . $type);
-  }
+    }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function supports($name) {
+    /**
+     * {@inheritdoc}
+     */
+    public function supports($name) {
     return
         ($name instanceof Meta && (
                 $name->getValues() instanceof ArticleInterface ||
@@ -106,5 +106,5 @@ 
             'swp_media_get' !== $name &&
             false === strpos($name, 'swp_api_')
         );
-  }
+    }
 }

src/SWP/Bundle/CoreBundle/Command/ProcessPackagesCommand.php

Spacing +2 added lines, -2 removed lines

@@ -123,8 +123,8 @@
         /** @var SlidingPagination $pagination */
         $pagination = $this->paginator->paginate(
             $queryBuilder,
-            (int)  $input->getOption('page'),
-            (int)  $input->getOption('limit')
+            (int) $input->getOption('page'),
+            (int) $input->getOption('limit')
         );
 
         $output->writeln(sprintf('<bg=green;options=bold>Packages found: %s</>', $pagination->getTotalItemCount()));

src/SWP/Bundle/MultiTenancyBundle/Routing/TenantAwareRouter.php

Indentation +3 added lines, -3 removed lines

@@ -31,12 +31,12 @@
      */
     public function generate($name, $parameters = [], $referenceType = false)
     {
-      if (RouteObjectInterface::OBJECT_BASED_ROUTE_NAME === $name
+        if (RouteObjectInterface::OBJECT_BASED_ROUTE_NAME === $name
           && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)
-      ) {
+        ) {
         $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
         unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
-      }
+        }
 
         if (null === $name && isset($parameters['content_id'])) {
             $contentId = $this->checkAndRemoveFirstSlash($parameters['content_id']);

src/SWP/Bundle/CoreBundle/Routing/ArticleAuthorMediaRouter.php

Spacing +1 added lines, -1 removed lines

@@ -71,6 +71,6 @@
       $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
       unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
-    return 'Route for article author media ' . $name->getValues()->getId() . ' not found';
+    return 'Route for article author media '.$name->getValues()->getId().' not found';
   }
 }

Indentation +26 added lines, -26 removed lines

@@ -27,53 +27,53 @@
 
 class ArticleAuthorMediaRouter extends Router implements VersatileGeneratorInterface {
 
-  const OBJECT_BASED_ROUTE_NAME = "__article_author_media_router_route_name__";
+    const OBJECT_BASED_ROUTE_NAME = "__article_author_media_router_route_name__";
 
-  protected $authorMediaManager;
+    protected $authorMediaManager;
 
-  public function __construct(
-      ContainerInterface $container,
-                         $resource,
-      array              $options = [],
-      RequestContext     $context = null,
-      ContainerInterface $parameters = null,
-      LoggerInterface    $logger = null,
-      string             $defaultLocale = null
-  ) {
+    public function __construct(
+        ContainerInterface $container,
+                            $resource,
+        array              $options = [],
+        RequestContext     $context = null,
+        ContainerInterface $parameters = null,
+        LoggerInterface    $logger = null,
+        string             $defaultLocale = null
+    ) {
     $this->authorMediaManager = $container->get('swp_core_bundle.manager.author_media');
 
     parent::__construct($container, $resource, $options, $context, $parameters, $logger, $defaultLocale);
-  }
+    }
 
-  public function generate($meta, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH) {
+    public function generate($meta, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH) {
     if (self::OBJECT_BASED_ROUTE_NAME === $meta
         && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)
     ) {
-      $meta = $parameters[RouteObjectInterface::ROUTE_OBJECT];
-      unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
+        $meta = $parameters[RouteObjectInterface::ROUTE_OBJECT];
+        unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
 
     if ($meta instanceof Meta && ($meta->getValues() instanceof AuthorMediaInterface)) {
-      return $this->authorMediaManager->getMediaPublicUrl($meta->getValues()->getImage());
+        return $this->authorMediaManager->getMediaPublicUrl($meta->getValues()->getImage());
     }
 
     return '';
-  }
+    }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function supports($name) {
+    /**
+     * {@inheritdoc}
+     */
+    public function supports($name) {
     return (is_string($name) && $name == self::OBJECT_BASED_ROUTE_NAME) || ($name instanceof Meta && ($name->getValues() instanceof AuthorMediaInterface));
-  }
+    }
 
-  public function getRouteDebugMessage($name, array $parameters = []) {
+    public function getRouteDebugMessage($name, array $parameters = []) {
     if (self::OBJECT_BASED_ROUTE_NAME === $name
         && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)
     ) {
-      $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
-      unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
+        $name = $parameters[RouteObjectInterface::ROUTE_OBJECT];
+        unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
     return 'Route for article author media ' . $name->getValues()->getId() . ' not found';
-  }
+    }
 }

src/SWP/Component/MultiTenancy/Exception/TenantNotFoundException.php

Indentation +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
 class TenantNotFoundException extends NotFoundHttpException {
-  public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = []) {
+    public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = []) {
     parent::__construct(sprintf('Tenant for host "%s" could not be found! Please check tenants configuration.', $message), $previous, $code, $headers);
-  }
+    }
 }

src/SWP/Bundle/ContentBundle/Routing/MediaRouter.php

Indentation +36 added lines, -36 removed lines

@@ -31,82 +31,82 @@ 
 use Symfony\Component\Routing\RequestContext;
 
 class MediaRouter extends Router implements VersatileGeneratorInterface {
-  private $mediaManager;
-
-  const OBJECT_BASED_ROUTE_NAME = "__media_router_route_name__";
-
-  public function __construct(
-      ContainerInterface $container,
-                         $resource,
-      array              $options = [],
-      RequestContext     $context = null,
-      ContainerInterface $parameters = null,
-      LoggerInterface    $logger = null,
-      string             $defaultLocale = null
-  ) {
+    private $mediaManager;
+
+    const OBJECT_BASED_ROUTE_NAME = "__media_router_route_name__";
+
+    public function __construct(
+        ContainerInterface $container,
+                            $resource,
+        array              $options = [],
+        RequestContext     $context = null,
+        ContainerInterface $parameters = null,
+        LoggerInterface    $logger = null,
+        string             $defaultLocale = null
+    ) {
     $this->mediaManager = $container->get('swp_content_bundle.manager.media');
 
     parent::__construct($container, $resource, $options, $context, $parameters, $logger, $defaultLocale);
-  }
+    }
 
-  public function getRouteDebugMessage($meta, array $parameters = array()): string {
+    public function getRouteDebugMessage($meta, array $parameters = array()): string {
     if (self::OBJECT_BASED_ROUTE_NAME === $meta && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)) {
-      $meta = $parameters[RouteObjectInterface::ROUTE_OBJECT];
-      unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
+        $meta = $parameters[RouteObjectInterface::ROUTE_OBJECT];
+        unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
     return 'Route for media ' . $meta->getValues()->getId() . ' not found';
-  }
+    }
 
-  public function supports($name): bool {
+    public function supports($name): bool {
     return (is_string($name) && $name == self::OBJECT_BASED_ROUTE_NAME) ||
         ($name instanceof Meta && (
                 $name->getValues() instanceof ArticleMediaInterface ||
                 $name->getValues() instanceof ImageRenditionInterface));
-  }
+    }
 
-  public function generate($meta, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH): string {
+    public function generate($meta, $parameters = [], $referenceType = UrlGeneratorInterface::ABSOLUTE_PATH): string {
     if (self::OBJECT_BASED_ROUTE_NAME === $meta && array_key_exists(RouteObjectInterface::ROUTE_OBJECT, $parameters)) {
-      $meta = $parameters[RouteObjectInterface::ROUTE_OBJECT];
-      unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
+        $meta = $parameters[RouteObjectInterface::ROUTE_OBJECT];
+        unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
 
     if (!$meta instanceof Meta) {
-      return '';
+        return '';
     }
 
     $item = $this->getItem($meta);
     if (null === $item) {
-      return '';
+        return '';
     }
 
     if ($meta->getValues() instanceof ImageRenditionInterface && null !== ($previewUrl = $meta->getValues()->getPreviewUrl())) {
-      return $previewUrl;
+        return $previewUrl;
     }
 
     if ($item instanceof PreviewUrlAwareInterface && null !== ($previewUrl = $item->getPreviewUrl())) {
-      return $previewUrl;
+        return $previewUrl;
     }
 
     return $this->getUrlWithCorrectExtension($item, $parameters);
-  }
+    }
 
-  private function getItem(Meta $meta): ?FileInterface {
+    private function getItem(Meta $meta): ?FileInterface {
     if (($rendition = $meta->getValues()) instanceof ImageRendition) {
-      return $rendition->getImage();
+        return $rendition->getImage();
     }
 
     if (($image = $meta->getValues()->getImage()) instanceof ImageInterface) {
-      return $image;
+        return $image;
     }
 
     if (($file = $meta->getValues()->getFile()) instanceof FileInterface) {
-      return $file;
+        return $file;
     }
 
     return null;
-  }
+    }
 
-  private function getUrlWithCorrectExtension(FileInterface $item, array $parameters): string {
+    private function getUrlWithCorrectExtension(FileInterface $item, array $parameters): string {
     $url = $this->mediaManager->getMediaPublicUrl($item);
 
     if (
@@ -115,9 +115,9 @@ 
         true === $parameters['webp'] &&
         $item->hasVariant(ImageInterface::VARIANT_WEBP)
     ) {
-      return str_replace('.' . $item->getFileExtension(), '.webp', $url);
+        return str_replace('.' . $item->getFileExtension(), '.webp', $url);
     }
 
     return $url;
-  }
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
       $meta = $parameters[RouteObjectInterface::ROUTE_OBJECT];
       unset($parameters[RouteObjectInterface::ROUTE_OBJECT]);
     }
-    return 'Route for media ' . $meta->getValues()->getId() . ' not found';
+    return 'Route for media '.$meta->getValues()->getId().' not found';
   }
 
   public function supports($name): bool {
@@ -115,7 +115,7 @@ 
         true === $parameters['webp'] &&
         $item->hasVariant(ImageInterface::VARIANT_WEBP)
     ) {
-      return str_replace('.' . $item->getFileExtension(), '.webp', $url);
+      return str_replace('.'.$item->getFileExtension(), '.webp', $url);
     }
 
     return $url;

src/SWP/Bundle/CoreBundle/EventListener/DownloadArticleAuthorAvatarListener.php

Indentation +66 added lines, -66 removed lines

@@ -34,87 +34,87 @@ 
 use SWP\Component\Common\Exception\UnexpectedTypeException;
 
 final class DownloadArticleAuthorAvatarListener {
-  /**
-   * @var MediaManagerInterface
-   */
-  private $authorMediaManager;
-
-  /**
-   * @var EntityManagerInterface
-   */
-  private $entityManager;
-
-  /**
-   * @var string
-   */
-  private $cacheDirectory;
-
-  public function __construct(EntityManagerInterface $entityManager, $cacheDirectory) {
+    /**
+     * @var MediaManagerInterface
+     */
+    private $authorMediaManager;
+
+    /**
+     * @var EntityManagerInterface
+     */
+    private $entityManager;
+
+    /**
+     * @var string
+     */
+    private $cacheDirectory;
+
+    public function __construct(EntityManagerInterface $entityManager, $cacheDirectory) {
     $this->entityManager = $entityManager;
     $this->cacheDirectory = $cacheDirectory;
-  }
+    }
 
-  /**
-   * @param MediaManagerInterface $authorMediaManager
-   */
-  public function setAuthorMediaManager(MediaManagerInterface $authorMediaManager) {
+    /**
+     * @param MediaManagerInterface $authorMediaManager
+     */
+    public function setAuthorMediaManager(MediaManagerInterface $authorMediaManager) {
     $this->authorMediaManager = $authorMediaManager;
-  }
+    }
 
-  /**
-   * @param GenericEvent $event
-   */
-  public function processAuthors(GenericEvent $event): void {
+    /**
+     * @param GenericEvent $event
+     */
+    public function processAuthors(GenericEvent $event): void {
     $package = $event->getSubject();
 
     if (!$package instanceof PackageInterface) {
-      throw new UnexpectedTypeException($package, PackageInterface::class);
+        throw new UnexpectedTypeException($package, PackageInterface::class);
     }
 
     $authors = [];
     /** @var ArticleAuthorInterface $packageAuthor */
     foreach ($package->getAuthors() as $packageAuthor) {
-      $authors[] = $this->handle($packageAuthor);
+        $authors[] = $this->handle($packageAuthor);
     }
     $package->setAuthors(new ArrayCollection($authors));
-  }
-
-  /**
-   * @param ArticleAuthorInterface $object
-   *
-   * @return ArticleAuthorInterface
-   */
-  private function handle(ArticleAuthorInterface $object): ArticleAuthorInterface {
+    }
+
+    /**
+     * @param ArticleAuthorInterface $object
+     *
+     * @return ArticleAuthorInterface
+     */
+    private function handle(ArticleAuthorInterface $object): ArticleAuthorInterface {
     if (null !== $object->getAvatarUrl()) {
-      $filesystem = new Filesystem();
-      $pathParts = \pathinfo($object->getAvatarUrl());
-      $assetId = \sha1($pathParts['filename']);
-      if (null !== $object->getSlug()) {
+        $filesystem = new Filesystem();
+        $pathParts = \pathinfo($object->getAvatarUrl());
+        $assetId = \sha1($pathParts['filename']);
+        if (null !== $object->getSlug()) {
         $assetId = $object->getSlug() . '_' . $assetId;
-      }
-      $existingAvatar = $this->entityManager->getRepository(Image::class)->findBy(['assetId' => $assetId]);
-      if (\count($existingAvatar) > 0) {
+        }
+        $existingAvatar = $this->entityManager->getRepository(Image::class)->findBy(['assetId' => $assetId]);
+        if (\count($existingAvatar) > 0) {
         $object->setAvatarUrl((string)\reset($existingAvatar));
 
         return $object;
-      }
+        }
 
-      try {
+        try {
         $tempDirectory = $this->cacheDirectory . \DIRECTORY_SEPARATOR . 'downloaded_avatars';
         $tempLocation = $tempDirectory . \DIRECTORY_SEPARATOR . \sha1($assetId . date('his'));
         if (!$filesystem->exists($tempDirectory)) {
-          $filesystem->mkdir($tempDirectory);
+            $filesystem->mkdir($tempDirectory);
         }
 
         $avatarUrl = $object->getAvatarUrl();
         if (strpos($avatarUrl, "http://") === 0 || strpos($avatarUrl, "https://") === 0) {
-          self::downloadFile($avatarUrl, $tempLocation);
+            self::downloadFile($avatarUrl, $tempLocation);
         } else {
-          $file = \file_get_contents($avatarUrl);
-          if (false === $file) {
+            $file = \file_get_contents($avatarUrl);
+            if (false === $file) {
             throw new \Exception('File can\'t be downloaded');
-          }
-          $filesystem->dumpFile($tempLocation, $file);
+            }
+            $filesystem->dumpFile($tempLocation, $file);
         }
 
         $uploadedFile = new UploadedFile($tempLocation,
@@ -123,29 +123,29 @@ 
             \filesize($tempLocation),
             true
         );
-      } catch (\Exception $e) {
+        } catch (\Exception $e) {
         return $object;
-      }
-      /** @var Image $image */
-      $image = $this->authorMediaManager->handleUploadedFile($uploadedFile, $assetId);
-      $avatar = $this->createAuthorMedia($object, $image);
-      $this->entityManager->persist($avatar);
-      $this->entityManager->persist($image);
-
-      $object->setAvatar($avatar);
-      $object->setAvatarUrl((string)$image);
+        }
+        /** @var Image $image */
+        $image = $this->authorMediaManager->handleUploadedFile($uploadedFile, $assetId);
+        $avatar = $this->createAuthorMedia($object, $image);
+        $this->entityManager->persist($avatar);
+        $this->entityManager->persist($image);
+
+        $object->setAvatar($avatar);
+        $object->setAvatarUrl((string)$image);
     }
 
     return $object;
-  }
+    }
 
-  private function createAuthorMedia(ArticleAuthorInterface $articleAuthor, Image $image): AuthorMediaInterface {
+    private function createAuthorMedia(ArticleAuthorInterface $articleAuthor, Image $image): AuthorMediaInterface {
     return new AuthorMedia('avatar', $articleAuthor, $image);
-  }
+    }
 
-  private static function downloadFile($url, $location) {
+    private static function downloadFile($url, $location) {
     $handlerStack = HandlerStack::create(new CurlHandler());
     $client = new Client(['handler' => $handlerStack]);
     $client->request('GET', $url, ['sink' => $location]);
-  }
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -90,18 +90,18 @@ 
       $pathParts = \pathinfo($object->getAvatarUrl());
       $assetId = \sha1($pathParts['filename']);
       if (null !== $object->getSlug()) {
-        $assetId = $object->getSlug() . '_' . $assetId;
+        $assetId = $object->getSlug().'_'.$assetId;
       }
       $existingAvatar = $this->entityManager->getRepository(Image::class)->findBy(['assetId' => $assetId]);
       if (\count($existingAvatar) > 0) {
-        $object->setAvatarUrl((string)\reset($existingAvatar));
+        $object->setAvatarUrl((string) \reset($existingAvatar));
 
         return $object;
       }
 
       try {
-        $tempDirectory = $this->cacheDirectory . \DIRECTORY_SEPARATOR . 'downloaded_avatars';
-        $tempLocation = $tempDirectory . \DIRECTORY_SEPARATOR . \sha1($assetId . date('his'));
+        $tempDirectory = $this->cacheDirectory.\DIRECTORY_SEPARATOR.'downloaded_avatars';
+        $tempLocation = $tempDirectory.\DIRECTORY_SEPARATOR.\sha1($assetId.date('his'));
         if (!$filesystem->exists($tempDirectory)) {
           $filesystem->mkdir($tempDirectory);
         }
@@ -133,7 +133,7 @@ 
       $this->entityManager->persist($image);
 
       $object->setAvatar($avatar);
-      $object->setAvatarUrl((string)$image);
+      $object->setAvatarUrl((string) $image);
     }
 
     return $object;