superdesk / web-publisher

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;