superdesk / web-publisher

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/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/CoreBundle/Routing/MetaRouter.php

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/Routing/ArticleAuthorMediaRouter.php

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/Bundle/ContentBundle/Routing/SeoMediaRouter.php

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/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;
-  }
+    }
 }

src/SWP/Bundle/CoreBundle/Controller/PackagePreviewController.php

Indentation +81 added lines, -81 removed lines

@@ -47,36 +47,36 @@ 
 
 class PackagePreviewController extends Controller {
 
-  private EventDispatcherInterface $eventDispatcher;
-  private DataTransformerInterface $dataTransformer;
-  private PackageRepositoryInterface $packageRepository;
-  private RouteRepositoryInterface $routeRepository;
-  private RepositoryInterface $packagePreviewTokenRepository;
-  private PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory;
-  private TemplateNameResolverInterface $templateNameResolver;
-  private ArticlePreviewerInterface $articlePreviewer;
-  private ArticlePreviewContextInterface $articlePreviewContext;
-
-  /**
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param DataTransformerInterface $dataTransformer
-   * @param PackageRepositoryInterface $packageRepository
-   * @param RouteRepositoryInterface $routeRepository
-   * @param RepositoryInterface $packagePreviewTokenRepository
-   * @param PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory
-   * @param TemplateNameResolverInterface $templateNameResolver
-   * @param ArticlePreviewerInterface $articlePreviewer
-   * @param ArticlePreviewContextInterface $articlePreviewContext
-   */
-  public function __construct(EventDispatcherInterface            $eventDispatcher,
-                              DataTransformerInterface            $dataTransformer,
-                              PackageRepositoryInterface          $packageRepository,
-                              RouteRepositoryInterface            $routeRepository,
-                              RepositoryInterface                 $packagePreviewTokenRepository,
-                              PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory,
-                              TemplateNameResolverInterface       $templateNameResolver,
-                              ArticlePreviewerInterface           $articlePreviewer,
-                              ArticlePreviewContextInterface      $articlePreviewContext) {
+    private EventDispatcherInterface $eventDispatcher;
+    private DataTransformerInterface $dataTransformer;
+    private PackageRepositoryInterface $packageRepository;
+    private RouteRepositoryInterface $routeRepository;
+    private RepositoryInterface $packagePreviewTokenRepository;
+    private PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory;
+    private TemplateNameResolverInterface $templateNameResolver;
+    private ArticlePreviewerInterface $articlePreviewer;
+    private ArticlePreviewContextInterface $articlePreviewContext;
+
+    /**
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param DataTransformerInterface $dataTransformer
+     * @param PackageRepositoryInterface $packageRepository
+     * @param RouteRepositoryInterface $routeRepository
+     * @param RepositoryInterface $packagePreviewTokenRepository
+     * @param PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory
+     * @param TemplateNameResolverInterface $templateNameResolver
+     * @param ArticlePreviewerInterface $articlePreviewer
+     * @param ArticlePreviewContextInterface $articlePreviewContext
+     */
+    public function __construct(EventDispatcherInterface            $eventDispatcher,
+                                DataTransformerInterface            $dataTransformer,
+                                PackageRepositoryInterface          $packageRepository,
+                                RouteRepositoryInterface            $routeRepository,
+                                RepositoryInterface                 $packagePreviewTokenRepository,
+                                PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory,
+                                TemplateNameResolverInterface       $templateNameResolver,
+                                ArticlePreviewerInterface           $articlePreviewer,
+                                ArticlePreviewContextInterface      $articlePreviewContext) {
     $this->eventDispatcher = $eventDispatcher;
     $this->dataTransformer = $dataTransformer;
     $this->packageRepository = $packageRepository;
@@ -86,13 +86,13 @@ 
     $this->templateNameResolver = $templateNameResolver;
     $this->articlePreviewer = $articlePreviewer;
     $this->articlePreviewContext = $articlePreviewContext;
-  }
+    }
 
 
-  /**
-   * @Route("/preview/package/{routeId}/{id}", options={"expose"=true}, requirements={"id"="\d+", "routeId"="\d+", "token"=".+"}, methods={"GET"}, name="swp_package_preview")
-   */
-  public function previewAction(int $routeId, $id) {
+    /**
+     * @Route("/preview/package/{routeId}/{id}", options={"expose"=true}, requirements={"id"="\d+", "routeId"="\d+", "token"=".+"}, methods={"GET"}, name="swp_package_preview")
+     */
+    public function previewAction(int $routeId, $id) {
     /** @var RouteInterface $route */
     $route = $this->findRouteOr404($routeId);
     /** @var PackageInterface $package */
@@ -106,22 +106,22 @@ 
     $this->eventDispatcher->dispatch(new GenericEvent($articlePreview), ArticleEvents::PREVIEW);
 
     if (null !== ($url = $articlePreview->getPreviewUrl())) {
-      return new RedirectResponse($url);
+        return new RedirectResponse($url);
     }
 
     $route = $this->ensureRouteTemplateExists($route, $article);
 
     try {
-      return $this->render($route->getArticlesTemplateName());
+        return $this->render($route->getArticlesTemplateName());
     } catch (\Exception $e) {
-      throw $this->createNotFoundException(sprintf('Template for route with id "%d" (%s) not found!', $route->getId(), $route->getName()));
+        throw $this->createNotFoundException(sprintf('Template for route with id "%d" (%s) not found!', $route->getId(), $route->getName()));
+    }
     }
-  }
 
-  /**
-   * @FOSRoute("/api/{version}/preview/package/generate_token/{routeId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_preview_package_token", requirements={"routeId"="\d+"})
-   */
-  public function generateTokenAction(Request $request, int $routeId): SingleResourceResponseInterface {
+    /**
+     * @FOSRoute("/api/{version}/preview/package/generate_token/{routeId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_preview_package_token", requirements={"routeId"="\d+"})
+     */
+    public function generateTokenAction(Request $request, int $routeId): SingleResourceResponseInterface {
     $route = $this->findRouteOr404($routeId);
 
     /** @var string $content */
@@ -134,27 +134,27 @@ 
     $existingPreviewToken = $tokenRepository->findOneBy(['route' => $route]);
 
     if (null === $existingPreviewToken) {
-      $packagePreviewToken = $this->packagePreviewTokenFactory->createTokenizedWith($route, $content);
+        $packagePreviewToken = $this->packagePreviewTokenFactory->createTokenizedWith($route, $content);
 
-      $tokenRepository->persist($packagePreviewToken);
-      $tokenRepository->flush();
+        $tokenRepository->persist($packagePreviewToken);
+        $tokenRepository->flush();
 
-      return $this->returnResponseWithPreviewUrl($packagePreviewToken);
+        return $this->returnResponseWithPreviewUrl($packagePreviewToken);
     }
 
     $this->updatePackagePreviewTokenBody($content, $existingPreviewToken);
 
     return $this->returnResponseWithPreviewUrl($existingPreviewToken);
-  }
+    }
 
-  /**
-   * @Route("/preview/publish/package/{token}", options={"expose"=true}, requirements={"token"=".+"}, methods={"GET"}, name="swp_package_preview_publish")
-   */
-  public function publishPreviewAction(string $token) {
+    /**
+     * @Route("/preview/publish/package/{token}", options={"expose"=true}, requirements={"token"=".+"}, methods={"GET"}, name="swp_package_preview_publish")
+     */
+    public function publishPreviewAction(string $token) {
     $existingPreviewToken = $this->packagePreviewTokenRepository->findOneBy(['token' => $token]);
 
     if (null === $existingPreviewToken) {
-      throw $this->createNotFoundException(sprintf('Token %s is not valid.', $token));
+        throw $this->createNotFoundException(sprintf('Token %s is not valid.', $token));
     }
 
     $article = $this->getArticleForPreview($existingPreviewToken);
@@ -162,17 +162,17 @@ 
     $route = $this->ensureRouteTemplateExists($route, $article);
 
     return $this->renderTemplateOr404($route);
-  }
+    }
 
-  private function updatePackagePreviewTokenBody(string $content, PackagePreviewTokenInterface $packagePreviewToken) {
+    private function updatePackagePreviewTokenBody(string $content, PackagePreviewTokenInterface $packagePreviewToken) {
     if (md5($content) !== md5($packagePreviewToken->getBody())) {
-      $packagePreviewToken->setBody($content);
+        $packagePreviewToken->setBody($content);
 
-      $this->packagePreviewTokenRepository->flush();
+        $this->packagePreviewTokenRepository->flush();
+    }
     }
-  }
 
-  private function returnResponseWithPreviewUrl(PackagePreviewTokenInterface $packagePreviewToken): SingleResourceResponseInterface {
+    private function returnResponseWithPreviewUrl(PackagePreviewTokenInterface $packagePreviewToken): SingleResourceResponseInterface {
     $article = $this->getArticleForPreview($packagePreviewToken);
     $articlePreview = new ArticlePreview();
     $articlePreview->setArticle($article);
@@ -182,19 +182,19 @@ 
     $url = $articlePreview->getPreviewUrl();
 
     if (null === $url) {
-      $url = $this->generateUrl(
-          'swp_package_preview_publish',
-          ['token' => $packagePreviewToken->getToken()],
-          UrlGeneratorInterface::ABSOLUTE_URL
-      );
+        $url = $this->generateUrl(
+            'swp_package_preview_publish',
+            ['token' => $packagePreviewToken->getToken()],
+            UrlGeneratorInterface::ABSOLUTE_URL
+        );
     }
 
     return new SingleResourceResponse([
         'preview_url' => $url,
     ]);
-  }
+    }
 
-  private function getArticleForPreview(PackagePreviewTokenInterface $packagePreviewToken): ArticleInterface {
+    private function getArticleForPreview(PackagePreviewTokenInterface $packagePreviewToken): ArticleInterface {
     $dispatcher = $this->eventDispatcher;
     $package = $this->dataTransformer->transform($packagePreviewToken->getBody());
     $dispatcher->dispatch(new GenericEvent($package), Events::SWP_VALIDATION);
@@ -205,40 +205,40 @@ 
     $articlePreviewContext->setIsPreview(true);
 
     return $articlePreviewer->preview($package, $packagePreviewToken->getRoute());
-  }
+    }
 
-  private function renderTemplateOr404(RouteInterface $route): Response {
+    private function renderTemplateOr404(RouteInterface $route): Response {
     try {
-      return $this->render($templateName = $route->getArticlesTemplateName());
+        return $this->render($templateName = $route->getArticlesTemplateName());
     } catch (\InvalidArgumentException | LoaderError $e) {
-      throw $this->createNotFoundException(sprintf('Template %s for route with id "%d" (%s) not found!', $templateName, $route->getId(), $route->getName()));
+        throw $this->createNotFoundException(sprintf('Template %s for route with id "%d" (%s) not found!', $templateName, $route->getId(), $route->getName()));
+    }
     }
-  }
 
-  private function ensureRouteTemplateExists(RouteInterface $route, ArticleInterface $article): RouteInterface {
+    private function ensureRouteTemplateExists(RouteInterface $route, ArticleInterface $article): RouteInterface {
     if (null === $route->getArticlesTemplateName()) {
-      $templateNameResolver = $this->templateNameResolver;
-      $route->setArticlesTemplateName($templateNameResolver->resolve($article));
+        $templateNameResolver = $this->templateNameResolver;
+        $route->setArticlesTemplateName($templateNameResolver->resolve($article));
     }
 
     return $route;
-  }
+    }
 
-  private function findRouteOr404(int $id): RouteInterface {
+    private function findRouteOr404(int $id): RouteInterface {
     /** @var RouteInterface $route */
     if (null === ($route = $this->routeRepository->findOneBy(['id' => $id]))) {
-      throw $this->createNotFoundException(sprintf('Route with id: "%s" not found!', $id));
+        throw $this->createNotFoundException(sprintf('Route with id: "%s" not found!', $id));
     }
 
     return $route;
-  }
+    }
 
-  private function findPackageOr404(string $id): PackageInterface {
+    private function findPackageOr404(string $id): PackageInterface {
     /** @var PackageInterface $package */
     if (null === ($package = $this->packageRepository->findOneBy(['id' => $id]))) {
-      throw $this->createNotFoundException(sprintf('Package with id: "%s" not found!', $id));
+        throw $this->createNotFoundException(sprintf('Package with id: "%s" not found!', $id));
     }
 
     return $package;
-  }
+    }
 }

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]);
-  }
+    }
 }

src/SWP/Bundle/UserBundle/Controller/ResetPasswordController.php

Indentation +70 added lines, -70 removed lines

@@ -36,88 +36,88 @@ 
  * @Route("/reset-password")
  */
 class ResetPasswordController extends AbstractController {
-  use ResetPasswordControllerTrait;
+    use ResetPasswordControllerTrait;
 
-  private ResetPasswordHelperInterface $resetPasswordHelper;
-  private EntityManagerInterface  $entityManager;
+    private ResetPasswordHelperInterface $resetPasswordHelper;
+    private EntityManagerInterface  $entityManager;
 
-  /**
-   * @param ResetPasswordHelperInterface $resetPasswordHelper
-   * @param EntityManagerInterface $entityManager
-   */
-  public function __construct(ResetPasswordHelperInterface $resetPasswordHelper,
-                              EntityManagerInterface       $entityManager) {
+    /**
+     * @param ResetPasswordHelperInterface $resetPasswordHelper
+     * @param EntityManagerInterface $entityManager
+     */
+    public function __construct(ResetPasswordHelperInterface $resetPasswordHelper,
+                                EntityManagerInterface       $entityManager) {
     $this->resetPasswordHelper = $resetPasswordHelper;
     $this->entityManager = $entityManager;
-  }
+    }
 
 
-  /**
-   * Display & process form to request a password reset.
-   *
-   * @Route("", name="swp_user_forgot_password_request")
-   */
-  public function request(Request $request, MailerInterface $mailer): Response {
+    /**
+     * Display & process form to request a password reset.
+     *
+     * @Route("", name="swp_user_forgot_password_request")
+     */
+    public function request(Request $request, MailerInterface $mailer): Response {
     $form = $this->createForm(ResetPasswordRequestFormType::class);
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      return $this->processSendingPasswordResetEmail(
-          $form->get('email')->getData(),
-          $mailer
-      );
+        return $this->processSendingPasswordResetEmail(
+            $form->get('email')->getData(),
+            $mailer
+        );
     }
 
     return $this->render('@SWPUser/reset_password/request.html.twig', [
         'requestForm' => $form->createView(),
     ]);
-  }
-
-  /**
-   * Confirmation page after a user has requested a password reset.
-   *
-   * @Route("/check-email", name="swp_user_check_email")
-   */
-  public function checkEmail(): Response {
+    }
+
+    /**
+     * Confirmation page after a user has requested a password reset.
+     *
+     * @Route("/check-email", name="swp_user_check_email")
+     */
+    public function checkEmail(): Response {
     // We prevent users from directly accessing this page
     if (!$this->canCheckEmail()) {
-      return $this->redirectToRoute('swp_user_forgot_password_request');
+        return $this->redirectToRoute('swp_user_forgot_password_request');
     }
 
     return $this->render('@SWPUser/reset_password/check_email.html.twig', [
         'tokenLifetime' => $this->resetPasswordHelper->getTokenLifetime(),
     ]);
-  }
-
-  /**
-   * Validates and process the reset URL that the user clicked in their email.
-   *
-   * @Route("/reset/{token}", name="swp_user_reset_password")
-   */
-  public function reset(Request $request, UserPasswordEncoderInterface $passwordEncoder,
+    }
+
+    /**
+     * Validates and process the reset URL that the user clicked in their email.
+     *
+     * @Route("/reset/{token}", name="swp_user_reset_password")
+     */
+    public function reset(Request $request, UserPasswordEncoderInterface $passwordEncoder,
                         string  $token = null): Response {
     if ($token) {
-      // We store the token in session and remove it from the URL, to avoid the URL being
-      // loaded in a browser and potentially leaking the token to 3rd party JavaScript.
-      $this->storeTokenInSession($token);
+        // We store the token in session and remove it from the URL, to avoid the URL being
+        // loaded in a browser and potentially leaking the token to 3rd party JavaScript.
+        $this->storeTokenInSession($token);
 
-      return $this->redirectToRoute('swp_user_reset_password');
+        return $this->redirectToRoute('swp_user_reset_password');
     }
 
     $token = $this->getTokenFromSession();
     if (null === $token) {
-      throw $this->createNotFoundException('No reset password token found in the URL or in the session.');
+        throw $this->createNotFoundException('No reset password token found in the URL or in the session.');
     }
 
     try {
-      $user = $this->resetPasswordHelper->validateTokenAndFetchUser($token);
+        $user = $this->resetPasswordHelper->validateTokenAndFetchUser($token);
     } catch (ResetPasswordExceptionInterface $e) {
-      $this->addFlash('reset_password_error', sprintf(
-          'There was a problem validating your reset request - %s',
-          $e->getReason()
-      ));
+        $this->addFlash('reset_password_error', sprintf(
+            'There was a problem validating your reset request - %s',
+            $e->getReason()
+        ));
 
-      return $this->redirectToRoute('swp_user_forgot_password_request');
+        return $this->redirectToRoute('swp_user_forgot_password_request');
     }
 
     // The token is valid; allow the user to change their password.
@@ -125,30 +125,30 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      // A password reset token should be used only once, remove it.
-      $this->resetPasswordHelper->removeResetRequest($token);
+        // A password reset token should be used only once, remove it.
+        $this->resetPasswordHelper->removeResetRequest($token);
 
-      // Encode the plain password, and set it.
-      $encodedPassword = $passwordEncoder->encodePassword(
-          $user,
-          $form->get('plainPassword')->getData()
-      );
+        // Encode the plain password, and set it.
+        $encodedPassword = $passwordEncoder->encodePassword(
+            $user,
+            $form->get('plainPassword')->getData()
+        );
 
-      $user->setPassword($encodedPassword);
-      $this->entityManager->flush();
+        $user->setPassword($encodedPassword);
+        $this->entityManager->flush();
 
-      // The session is cleaned up after the password has been changed.
-      $this->cleanSessionAfterReset();
+        // The session is cleaned up after the password has been changed.
+        $this->cleanSessionAfterReset();
 
-      return $this->redirectToRoute('homepage');
+        return $this->redirectToRoute('homepage');
     }
 
     return $this->render('@SWPUser/reset_password/reset.html.twig', [
         'resetForm' => $form->createView(),
     ]);
-  }
+    }
 
-  private function processSendingPasswordResetEmail(string $emailFormData, MailerInterface $mailer): RedirectResponse {
+    private function processSendingPasswordResetEmail(string $emailFormData, MailerInterface $mailer): RedirectResponse {
     $user = $this->entityManager->getRepository(User::class)->findOneBy([
         'email' => $emailFormData,
     ]);
@@ -158,22 +158,22 @@ 
 
     // Do not reveal whether a user account was found or not.
     if (!$user) {
-      return $this->redirectToRoute('swp_user_check_email');
+        return $this->redirectToRoute('swp_user_check_email');
     }
 
     try {
-      $resetToken = $this->resetPasswordHelper->generateResetToken($user);
+        $resetToken = $this->resetPasswordHelper->generateResetToken($user);
     } catch (ResetPasswordExceptionInterface $e) {
-      $this->addFlash('reset_password_error', sprintf(
-          'There was a problem handling your password reset request - %s',
-          $e->getReason()
-      ));
+        $this->addFlash('reset_password_error', sprintf(
+            'There was a problem handling your password reset request - %s',
+            $e->getReason()
+        ));
 
-      return $this->redirectToRoute('swp_user_check_email');
+        return $this->redirectToRoute('swp_user_check_email');
     }
 
     $mailer->sendResetPasswordEmail($user, $resetToken);
 
     return $this->redirectToRoute('swp_user_check_email');
-  }
+    }
 }