superdesk / web-publisher

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

Indentation +48 added lines, -48 removed lines

@@ -35,102 +35,102 @@
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class RelatedArticleOrganizationController extends Controller {
-  private EventDispatcherInterface $eventDispatcher;
-  private DataTransformerInterface $dataTransformer;
-  private CachedTenantContextInterface $cachedTenantContext;
-  private PackageRepositoryInterface $packageRepository;
-  private ArticleRepositoryInterface $articleRepository;
-  private TenantRepositoryInterface $tenantRepository;
-
-  /**
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param DataTransformerInterface $dataTransformer
-   * @param CachedTenantContextInterface $cachedTenantContext
-   * @param PackageRepositoryInterface $packageRepository
-   * @param ArticleRepositoryInterface $articleRepository
-   * @param TenantRepositoryInterface $tenantRepository
-   */
-  public function __construct(EventDispatcherInterface     $eventDispatcher,
-                              DataTransformerInterface     $dataTransformer,
-                              CachedTenantContextInterface $cachedTenantContext,
-                              PackageRepositoryInterface   $packageRepository,
-                              ArticleRepositoryInterface   $articleRepository,
-                              TenantRepositoryInterface    $tenantRepository) {
+    private EventDispatcherInterface $eventDispatcher;
+    private DataTransformerInterface $dataTransformer;
+    private CachedTenantContextInterface $cachedTenantContext;
+    private PackageRepositoryInterface $packageRepository;
+    private ArticleRepositoryInterface $articleRepository;
+    private TenantRepositoryInterface $tenantRepository;
+
+    /**
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param DataTransformerInterface $dataTransformer
+     * @param CachedTenantContextInterface $cachedTenantContext
+     * @param PackageRepositoryInterface $packageRepository
+     * @param ArticleRepositoryInterface $articleRepository
+     * @param TenantRepositoryInterface $tenantRepository
+     */
+    public function __construct(EventDispatcherInterface     $eventDispatcher,
+                                DataTransformerInterface     $dataTransformer,
+                                CachedTenantContextInterface $cachedTenantContext,
+                                PackageRepositoryInterface   $packageRepository,
+                                ArticleRepositoryInterface   $articleRepository,
+                                TenantRepositoryInterface    $tenantRepository) {
     $this->eventDispatcher = $eventDispatcher;
     $this->dataTransformer = $dataTransformer;
     $this->cachedTenantContext = $cachedTenantContext;
     $this->packageRepository = $packageRepository;
     $this->articleRepository = $articleRepository;
     $this->tenantRepository = $tenantRepository;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/organization/articles/related/", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_organization_related_articles")
-   */
-  public function postAction(Request $request) {
+    /**
+     * @Route("/api/{version}/organization/articles/related/", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_organization_related_articles")
+     */
+    public function postAction(Request $request) {
     $content = $request->getContent();
     $package = $this->dataTransformer->transform($content);
     $relatedArticlesList = $this->getRelated($package);
 
     return new SingleResourceResponse($relatedArticlesList);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/{id}/related/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_packages_related_articles", requirements={"id"="\d+"})
-   */
-  public function getRelatedAction(string $id) {
+    /**
+     * @Route("/api/{version}/packages/{id}/related/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_packages_related_articles", requirements={"id"="\d+"})
+     */
+    public function getRelatedAction(string $id) {
     $package = $this->findOr404((int)$id);
 
     $relatedArticlesList = $this->getRelated($package);
 
     return new SingleResourceResponse($relatedArticlesList);
-  }
+    }
 
-  private function getRelated(PackageInterface $package): RelatedArticleList {
+    private function getRelated(PackageInterface $package): RelatedArticleList {
     $relatedItemsGroups = $package->getItems()->filter(static function ($group) {
-      return ItemInterface::TYPE_TEXT === $group->getType();
+        return ItemInterface::TYPE_TEXT === $group->getType();
     });
 
     $relatedArticlesList = new RelatedArticleList();
 
     if (null === $package || (null !== $package && 0 === \count($relatedItemsGroups))) {
-      return $relatedArticlesList;
+        return $relatedArticlesList;
     }
 
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $articleRepository = $this->articleRepository;
 
     foreach ($relatedItemsGroups as $item) {
-      if (null === ($existingArticles = $articleRepository->findBy(['code' => $item->getGuid()]))) {
+        if (null === ($existingArticles = $articleRepository->findBy(['code' => $item->getGuid()]))) {
         continue;
-      }
+        }
 
-      $tenants = [];
-      foreach ($existingArticles as $existingArticle) {
+        $tenants = [];
+        foreach ($existingArticles as $existingArticle) {
         $tenantCode = $existingArticle->getTenantCode();
         $tenant = $this->tenantRepository->findOneByCode($tenantCode);
 
         $tenants[] = $tenant;
-      }
+        }
 
-      $relatedArticleListItem = new RelatedArticleListItem();
-      $relatedArticleListItem->setTenants($tenants);
-      $relatedArticleListItem->setTitle($item->getHeadline());
+        $relatedArticleListItem = new RelatedArticleListItem();
+        $relatedArticleListItem->setTenants($tenants);
+        $relatedArticleListItem->setTitle($item->getHeadline());
 
-      $relatedArticlesList->addRelatedArticleItem($relatedArticleListItem);
+        $relatedArticlesList->addRelatedArticleItem($relatedArticleListItem);
     }
 
     return $relatedArticlesList;
-  }
+    }
 
-  private function findOr404(int $id): PackageInterface {
+    private function findOr404(int $id): PackageInterface {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $tenantContext = $this->cachedTenantContext;
     if (null === $package = $this->packageRepository->findOneBy(['id' => $id, 'organization' => $tenantContext->getTenant()->getOrganization()])) {
-      throw new NotFoundHttpException('Package was not found.');
+        throw new NotFoundHttpException('Package was not found.');
     }
 
     return $package;
-  }
+    }
 }

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

Indentation +36 added lines, -36 removed lines

@@ -37,75 +37,75 @@
 use FOS\RestBundle\Controller\Annotations\Route as FOSRoute;
 
 class SeoMediaController extends AbstractMediaController {
-  private FactoryInterface $seoMetadataFactory;
-  private ArticleProviderInterface $articleProvider;
-  private EntityManagerInterface $seoObjectManager;
-  private FormFactoryInterface $formFactory;
-
-  public function __construct(
-      FactoryInterface              $seoMetadataFactory,
-      ArticleProviderInterface      $articleProvider,
-      FormFactoryInterface          $formFactory,
-      EntityManagerInterface        $seoObjectManager,
-      MediaManagerInterface         $seoMediaManager,
-      CacheInterface                $cacheProvider,
-      FileProviderInterface         $fileProvider,
-      FileExtensionCheckerInterface $fileExtensionChecker
-  ) {
+    private FactoryInterface $seoMetadataFactory;
+    private ArticleProviderInterface $articleProvider;
+    private EntityManagerInterface $seoObjectManager;
+    private FormFactoryInterface $formFactory;
+
+    public function __construct(
+        FactoryInterface              $seoMetadataFactory,
+        ArticleProviderInterface      $articleProvider,
+        FormFactoryInterface          $formFactory,
+        EntityManagerInterface        $seoObjectManager,
+        MediaManagerInterface         $seoMediaManager,
+        CacheInterface                $cacheProvider,
+        FileProviderInterface         $fileProvider,
+        FileExtensionCheckerInterface $fileExtensionChecker
+    ) {
     $this->seoMetadataFactory = $seoMetadataFactory;
     $this->formFactory = $formFactory;
     $this->articleProvider = $articleProvider;
     $this->seoObjectManager = $seoObjectManager;
 
     parent::__construct($seoMediaManager, $cacheProvider, $fileProvider, $fileExtensionChecker);
-  }
+    }
 
-  /**
-   * @Route("/seo/media/{mediaId}.{extension}", methods={"GET"}, options={"expose"=true}, requirements={"mediaId"=".+"}, name="swp_seo_media_get")
-   */
-  public function getAction(string $mediaId, string $extension): Response {
+    /**
+     * @Route("/seo/media/{mediaId}.{extension}", methods={"GET"}, options={"expose"=true}, requirements={"mediaId"=".+"}, name="swp_seo_media_get")
+     */
+    public function getAction(string $mediaId, string $extension): Response {
     return $this->getMedia($mediaId, $extension);
-  }
+    }
 
-  /**
-   * @FOSRoute("/api/{version}/upload/seo_image/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_seo_image")
-   */
-  public function uploadSeoImageAction(Request                   $request, string $id,
-                                       SeoImageUploaderInterface $seoImageUploader): SingleResourceResponse {
+    /**
+     * @FOSRoute("/api/{version}/upload/seo_image/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_seo_image")
+     */
+    public function uploadSeoImageAction(Request                   $request, string $id,
+                                        SeoImageUploaderInterface $seoImageUploader): SingleResourceResponse {
     $article = $this->findOr404($id);
     $seoMetadata = $article->getSeoMetadata();
 
     if (null === $seoMetadata) {
-      $seoMetadata = $this->seoMetadataFactory->create();
+        $seoMetadata = $this->seoMetadataFactory->create();
     }
 
     $form = $this->formFactory->createNamed('', SeoImageType::class, $seoMetadata);
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      try {
+        try {
         $seoImageUploader->handleUpload($seoMetadata);
 
         if (null === $article->getSeoMetadata()) {
-          $article->setSeoMetadata($seoMetadata);
+            $article->setSeoMetadata($seoMetadata);
         }
 
         $this->seoObjectManager->flush();
-      } catch (\Exception $e) {
+        } catch (\Exception $e) {
         return new SingleResourceResponse(['message' => 'Could not upload an image.'], new ResponseContext(400));
-      }
+        }
 
-      return new SingleResourceResponse($seoMetadata, new ResponseContext(201));
+        return new SingleResourceResponse($seoMetadata, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  private function findOr404(string $id): ArticleInterface {
+    private function findOr404(string $id): ArticleInterface {
     if (null === $article = $this->articleProvider->getOneById($id)) {
-      throw new NotFoundHttpException('Article was not found.');
+        throw new NotFoundHttpException('Article was not found.');
     }
 
     return $article;
-  }
+    }
 }

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

Indentation +5 added lines, -5 removed lines

@@ -23,12 +23,12 @@
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class FailedQueueController extends AbstractController {
-  /**
-   * @Route("/api/{version}/failed_queue/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_failed_queue")
-   */
-  public function listAction(Request $request, FailedEntriesProvider $failedEntriesProvider) {
+    /**
+     * @Route("/api/{version}/failed_queue/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_failed_queue")
+     */
+    public function listAction(Request $request, FailedEntriesProvider $failedEntriesProvider) {
     $max = $request->query->getInt('limit', 50);
 
     return new SingleResourceResponse($failedEntriesProvider->getFailedEntries($max));
-  }
+    }
 }

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

Indentation +51 added lines, -51 removed lines

@@ -28,35 +28,35 @@ 
 
 class BodyListener
 {
-  private $decoderProvider;
-  private $throwExceptionOnUnsupportedContentType;
-  private $defaultFormat;
-  private $arrayNormalizer;
-  private $normalizeForms;
-
-  public function __construct(
-      DecoderProviderInterface $decoderProvider,
-      bool $throwExceptionOnUnsupportedContentType = false,
-      ArrayNormalizerInterface $arrayNormalizer = null,
-      bool $normalizeForms = false
-  ) {
+    private $decoderProvider;
+    private $throwExceptionOnUnsupportedContentType;
+    private $defaultFormat;
+    private $arrayNormalizer;
+    private $normalizeForms;
+
+    public function __construct(
+        DecoderProviderInterface $decoderProvider,
+        bool $throwExceptionOnUnsupportedContentType = false,
+        ArrayNormalizerInterface $arrayNormalizer = null,
+        bool $normalizeForms = false
+    ) {
     $this->decoderProvider = $decoderProvider;
     $this->throwExceptionOnUnsupportedContentType = $throwExceptionOnUnsupportedContentType;
     $this->arrayNormalizer = $arrayNormalizer;
     $this->normalizeForms = true;
-  }
+    }
 
-  public function setDefaultFormat(?string $defaultFormat): void
-  {
+    public function setDefaultFormat(?string $defaultFormat): void
+    {
     $this->defaultFormat = $defaultFormat;
-  }
+    }
 
-  public function onKernelRequest(RequestEvent $event): void
-  {
+    public function onKernelRequest(RequestEvent $event): void
+    {
     $request = $event->getRequest();
 
     if (!$request->attributes->get(FOSRestBundle::ZONE_ATTRIBUTE, true)) {
-      return;
+        return;
     }
 
     $method = $request->getMethod();
@@ -64,83 +64,83 @@ 
     $normalizeRequest = $this->normalizeForms && $this->isFormRequest($request);
 
     if ($this->isDecodeable($request)) {
-      $format = null === $contentType
+        $format = null === $contentType
           ? $request->getRequestFormat()
           : $request->getFormat($contentType);
 
-      $format = $format ?: $this->defaultFormat;
+        $format = $format ?: $this->defaultFormat;
 
-      $content = $request->getContent();
+        $content = $request->getContent();
 
-      if (null === $format || !$this->decoderProvider->supports($format)) {
+        if (null === $format || !$this->decoderProvider->supports($format)) {
         if ($this->throwExceptionOnUnsupportedContentType
             && $this->isNotAnEmptyDeleteRequestWithNoSetContentType($method, $content, $contentType)
         ) {
-          throw new UnsupportedMediaTypeHttpException("Request body format '$format' not supported");
+            throw new UnsupportedMediaTypeHttpException("Request body format '$format' not supported");
         }
 
         return;
-      }
+        }
 
-      if (!empty($content)) {
+        if (!empty($content)) {
         $decoder = $this->decoderProvider->getDecoder($format);
         $data = $decoder->decode($content);
         if (is_array($data)) {
-          $request->request = new ParameterBag($data);
-          $normalizeRequest = true;
+            $request->request = new ParameterBag($data);
+            $normalizeRequest = true;
         } else {
-          throw new BadRequestHttpException('Invalid '.$format.' message received');
+            throw new BadRequestHttpException('Invalid '.$format.' message received');
+        }
         }
-      }
     }
 
     if (null !== $this->arrayNormalizer && $normalizeRequest) {
-      $data = $request->request->all();
+        $data = $request->request->all();
 
-      try {
+        try {
         $data = $this->arrayNormalizer->normalize($data);
-      } catch (NormalizationException $e) {
+        } catch (NormalizationException $e) {
         throw new BadRequestHttpException($e->getMessage());
-      }
+        }
 
-      $request->request = new ParameterBag($data);
+        $request->request = new ParameterBag($data);
 
-      if(!empty($request->files->all())) {
+        if(!empty($request->files->all())) {
         $data = $request->files->all();
 
         try {
-          $data = $this->arrayNormalizer->normalize($data);
+            $data = $this->arrayNormalizer->normalize($data);
         } catch (NormalizationException $e) {
-          throw new BadRequestHttpException($e->getMessage());
+            throw new BadRequestHttpException($e->getMessage());
         }
 
         $request->files = new ParameterBag($data);
-      }
+        }
+    }
     }
-  }
 
-  private function isNotAnEmptyDeleteRequestWithNoSetContentType(string $method, $content, ?string $contentType): bool
-  {
+    private function isNotAnEmptyDeleteRequestWithNoSetContentType(string $method, $content, ?string $contentType): bool
+    {
     return false === ('DELETE' === $method && empty($content) && empty($contentType));
-  }
+    }
 
-  private function isDecodeable(Request $request): bool
-  {
+    private function isDecodeable(Request $request): bool
+    {
     if (!in_array($request->getMethod(), ['POST', 'PUT', 'PATCH', 'DELETE'])) {
-      return false;
+        return false;
     }
 
     return !$this->isFormRequest($request);
-  }
+    }
 
-  private function isFormRequest(Request $request): bool
-  {
+    private function isFormRequest(Request $request): bool
+    {
     $contentTypeParts = explode(';', $request->headers->get('Content-Type', ''));
 
     if (isset($contentTypeParts[0])) {
-      return in_array($contentTypeParts[0], ['multipart/form-data', 'application/x-www-form-urlencoded']);
+        return in_array($contentTypeParts[0], ['multipart/form-data', 'application/x-www-form-urlencoded']);
     }
 
     return false;
-  }
+    }
 }

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

Indentation +24 added lines, -24 removed lines

@@ -26,46 +26,46 @@
 
 final class AmpController extends AbstractController {
 
-  private Environment $twig;
-  private AmpConverterInterface $converter;
-  private ThemeLoaderInterface $themeLoader;
-  private CacheInterface $cacheService;
+    private Environment $twig;
+    private AmpConverterInterface $converter;
+    private ThemeLoaderInterface $themeLoader;
+    private CacheInterface $cacheService;
 
-  public function __construct(
-      Environment           $twig,
-      AmpConverterInterface $ampConverter,
-      ThemeLoaderInterface  $ampThemeLoader,
-      CacheInterface        $cacheService
-  ) {
+    public function __construct(
+        Environment           $twig,
+        AmpConverterInterface $ampConverter,
+        ThemeLoaderInterface  $ampThemeLoader,
+        CacheInterface        $cacheService
+    ) {
     $this->twig = $twig;
     $this->converter = $ampConverter;
     $this->themeLoader = $ampThemeLoader;
     $this->cacheService = $cacheService;
-  }
+    }
 
-  public function viewAction(AmpInterface $object): Response {
+    public function viewAction(AmpInterface $object): Response {
     return $this->cacheService->get($this->getCacheKey($object), function () use ($object) {
-      $this->themeLoader->load();
-      $content = $this->twig->render(sprintf('@%s/index.html.twig', ThemeLoaderInterface::THEME_NAMESPACE), [
-          'object' => $object,
-      ]);
+        $this->themeLoader->load();
+        $content = $this->twig->render(sprintf('@%s/index.html.twig', ThemeLoaderInterface::THEME_NAMESPACE), [
+            'object' => $object,
+        ]);
 
-      $response = new Response();
-      $response->setContent($this->converter->convertToAmp($content));
-      return $response;
+        $response = new Response();
+        $response->setContent($this->converter->convertToAmp($content));
+        return $response;
     });
-  }
+    }
 
-  private function getCacheKey(AmpInterface $object): string {
+    private function getCacheKey(AmpInterface $object): string {
     $elements = ['amp_article'];
     if ($object instanceof PersistableInterface) {
-      $elements[] = $object->getId();
+        $elements[] = $object->getId();
     }
 
     if ($object instanceof TimestampableInterface && null !== $object->getUpdatedAt()) {
-      $elements[] = $object->getUpdatedAt()->getTimestamp();
+        $elements[] = $object->getUpdatedAt()->getTimestamp();
     }
 
     return md5(implode('__', $elements));
-  }
+    }
 }

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