superdesk / web-publisher

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

Indentation +26 added lines, -26 removed lines

@@ -28,32 +28,32 @@ 
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class SlideshowItemController extends Controller {
-  private ArticleRepositoryInterface $articleRepository;
-  private SlideshowRepositoryInterface $slideshowRepository;
-  private SlideshowItemRepositoryInterface $slideshowItemRepository;
-  private EventDispatcherInterface $eventDispatcher;
+    private ArticleRepositoryInterface $articleRepository;
+    private SlideshowRepositoryInterface $slideshowRepository;
+    private SlideshowItemRepositoryInterface $slideshowItemRepository;
+    private EventDispatcherInterface $eventDispatcher;
 
-  /**
-   * @param ArticleRepositoryInterface $articleRepository
-   * @param SlideshowRepositoryInterface $slideshowRepository
-   * @param SlideshowItemRepositoryInterface $slideshowItemRepository
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(ArticleRepositoryInterface       $articleRepository,
-                              SlideshowRepositoryInterface     $slideshowRepository,
-                              SlideshowItemRepositoryInterface $slideshowItemRepository,
-                              EventDispatcherInterface         $eventDispatcher) {
+    /**
+     * @param ArticleRepositoryInterface $articleRepository
+     * @param SlideshowRepositoryInterface $slideshowRepository
+     * @param SlideshowItemRepositoryInterface $slideshowItemRepository
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(ArticleRepositoryInterface       $articleRepository,
+                                SlideshowRepositoryInterface     $slideshowRepository,
+                                SlideshowItemRepositoryInterface $slideshowItemRepository,
+                                EventDispatcherInterface         $eventDispatcher) {
     $this->articleRepository = $articleRepository;
     $this->slideshowRepository = $slideshowRepository;
     $this->slideshowItemRepository = $slideshowItemRepository;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/content/slideshows/{articleId}/{id}/items/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_slideshow_items", requirements={"id"="\d+"})
-   */
-  public function listAction(Request $request, string $articleId, string $id) {
+    /**
+     * @Route("/api/{version}/content/slideshows/{articleId}/{id}/items/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_slideshow_items", requirements={"id"="\d+"})
+     */
+    public function listAction(Request $request, string $articleId, string $id) {
     $article = $this->findArticleOr404($articleId);
     $repository = $this->slideshowItemRepository;
 
@@ -68,21 +68,21 @@ 
     );
 
     return new ResourcesListResponse($items);
-  }
+    }
 
-  private function findOr404($id): ?SlideshowInterface {
+    private function findOr404($id): ?SlideshowInterface {
     if (null === $slideshow = $this->slideshowRepository->findOneById($id)) {
-      throw new NotFoundHttpException(sprintf('Slideshow with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('Slideshow with id "%s" was not found.', $id));
     }
 
     return $slideshow;
-  }
+    }
 
-  private function findArticleOr404($id) {
+    private function findArticleOr404($id) {
     if (null === $article = $this->articleRepository->findOneById($id)) {
-      throw new NotFoundHttpException(sprintf('Article with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('Article with id "%s" was not found.', $id));
     }
 
     return $article;
-  }
+    }
 }

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/Controller/SeoMetadataController.php

Indentation +28 added lines, -28 removed lines

@@ -19,34 +19,34 @@ 
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class SeoMetadataController extends AbstractController {
-  private FormFactoryInterface $formFactory;
-  private FactoryInterface $seoMetadataFactory;
-  private RepositoryInterface $seoMetadataRepository;
-  private EventDispatcherInterface $eventDispatcher;
+    private FormFactoryInterface $formFactory;
+    private FactoryInterface $seoMetadataFactory;
+    private RepositoryInterface $seoMetadataRepository;
+    private EventDispatcherInterface $eventDispatcher;
 
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param FactoryInterface $seoMetadataFactory
-   * @param RepositoryInterface $seoMetadataRepository
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(FormFactoryInterface $formFactory, FactoryInterface $seoMetadataFactory,
-                              RepositoryInterface  $seoMetadataRepository, EventDispatcherInterface $eventDispatcher) {
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param FactoryInterface $seoMetadataFactory
+     * @param RepositoryInterface $seoMetadataRepository
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(FormFactoryInterface $formFactory, FactoryInterface $seoMetadataFactory,
+                                RepositoryInterface  $seoMetadataRepository, EventDispatcherInterface $eventDispatcher) {
     $this->formFactory = $formFactory;
     $this->seoMetadataFactory = $seoMetadataFactory;
     $this->seoMetadataRepository = $seoMetadataRepository;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/seo/{packageGuid}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PUT"}, name="swp_api_core_seo_metadata_put")
-   */
-  public function put(Request $request, string $packageGuid): SingleResourceResponse {
+    /**
+     * @Route("/api/{version}/packages/seo/{packageGuid}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PUT"}, name="swp_api_core_seo_metadata_put")
+     */
+    public function put(Request $request, string $packageGuid): SingleResourceResponse {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
 
     $seoMetadata = $this->seoMetadataRepository->findOneByPackageGuid($packageGuid);
     if (null === $seoMetadata) {
-      $seoMetadata = $this->seoMetadataFactory->create();
+        $seoMetadata = $this->seoMetadataFactory->create();
     }
 
     $form = $this->formFactory->createNamed('', SeoMetadataType::class, $seoMetadata, ['method' => $request->getMethod()]);
@@ -54,26 +54,26 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $seoMetadata->setPackageGuid($packageGuid);
-      $this->seoMetadataRepository->add($seoMetadata);
+        $seoMetadata->setPackageGuid($packageGuid);
+        $this->seoMetadataRepository->add($seoMetadata);
 
-      return new SingleResourceResponse($seoMetadata, new ResponseContext(200));
+        return new SingleResourceResponse($seoMetadata, new ResponseContext(200));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/seo/{packageGuid}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_seo_metadata_get")
-   */
-  public function getAction(string $packageGuid): SingleResourceResponse {
+    /**
+     * @Route("/api/{version}/packages/seo/{packageGuid}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_seo_metadata_get")
+     */
+    public function getAction(string $packageGuid): SingleResourceResponse {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
 
     $existingSeoMetadata = $this->seoMetadataRepository->findOneByPackageGuid($packageGuid);
     if (null === $existingSeoMetadata) {
-      throw new NotFoundHttpException('SEO metadata not found!');
+        throw new NotFoundHttpException('SEO metadata not found!');
     }
 
     return new SingleResourceResponse($existingSeoMetadata, new ResponseContext(200));
-  }
+    }
 }

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

Indentation +79 added lines, -79 removed lines

@@ -44,35 +44,35 @@ 
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class ContentListItemController extends AbstractController {
-  private ContentListItemRepositoryInterface $contentListItemRepository;
-  private EntityManagerInterface $entityManager;
-  private ContentListServiceInterface $contentListService;
-  private EventDispatcherInterface $eventDispatcher;
-
-  /**
-   * @param ContentListItemRepositoryInterface $contentListItemRepository
-   * @param EntityManagerInterface $entityManager
-   * @param ContentListServiceInterface $contentListService
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(ContentListItemRepositoryInterface                          $contentListItemRepository,
-                              EntityManagerInterface                                      $entityManager,
-                              ContentListServiceInterface                                 $contentListService,
-                              EventDispatcherInterface $eventDispatcher) {
+    private ContentListItemRepositoryInterface $contentListItemRepository;
+    private EntityManagerInterface $entityManager;
+    private ContentListServiceInterface $contentListService;
+    private EventDispatcherInterface $eventDispatcher;
+
+    /**
+     * @param ContentListItemRepositoryInterface $contentListItemRepository
+     * @param EntityManagerInterface $entityManager
+     * @param ContentListServiceInterface $contentListService
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(ContentListItemRepositoryInterface                          $contentListItemRepository,
+                                EntityManagerInterface                                      $entityManager,
+                                ContentListServiceInterface                                 $contentListService,
+                                EventDispatcherInterface $eventDispatcher) {
     $this->contentListItemRepository = $contentListItemRepository;
     $this->entityManager = $entityManager;
     $this->contentListService = $contentListService;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}/items/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_items", requirements={"id"="\d+"})
-   */
-  public function listAction(Request $request, int $id): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}/items/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_items", requirements={"id"="\d+"})
+     */
+    public function listAction(Request $request, int $id): ResourcesListResponseInterface {
     $sort = $request->query->all('sorting');
     if (empty($sort)) {
-      $sort = ['sticky' => 'desc'];
+        $sort = ['sticky' => 'desc'];
     }
 
     $items = $this->contentListItemRepository->getPaginatedByCriteria(
@@ -105,20 +105,20 @@ 
     );
 
     return new ResourcesListResponse($items, $responseContext);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{listId}/items/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_show_lists_item", requirements={"id"="\d+"})
-   */
-  public function getAction($listId, $id) {
+    /**
+     * @Route("/api/{version}/content/lists/{listId}/items/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_show_lists_item", requirements={"id"="\d+"})
+     */
+    public function getAction($listId, $id) {
     return new SingleResourceResponse($this->findOr404($listId, $id));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{listId}/items/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_lists_item", requirements={"id"="\d+", "listId"="\d+"})
-   */
-  public function updateAction(Request $request, FormFactoryInterface $formFactory, $listId,
-                                       $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{listId}/items/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_lists_item", requirements={"id"="\d+", "listId"="\d+"})
+     */
+    public function updateAction(Request $request, FormFactoryInterface $formFactory, $listId,
+                                        $id): SingleResourceResponseInterface {
     $contentListItem = $this->findOr404($listId, $id);
     $form = $formFactory->createNamed(
         '',
@@ -130,59 +130,59 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $contentListItem->getContentList()->setUpdatedAt(new DateTime());
+        $contentListItem->getContentList()->setUpdatedAt(new DateTime());
 
-      if (null !== $contentListItem->getStickyPosition()) {
+        if (null !== $contentListItem->getStickyPosition()) {
         $contentListItem->setPosition($contentListItem->getStickyPosition());
-      }
+        }
 
-      $this->entityManager->flush();
+        $this->entityManager->flush();
 
-      return new SingleResourceResponse($contentListItem);
+        return new SingleResourceResponse($contentListItem);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
-
-  /**
-   * @Route("/api/{version}/content/lists/{listId}/items/", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_batch_update_lists_item", requirements={"listId"="\d+"})
-   */
-  public function batchUpdateAction(
-      Request                        $request,
-      FormFactoryInterface           $formFactory,
-      ContentListRepositoryInterface $contentListRepository,
-      ArticleRepositoryInterface     $articleRepository,
-      EventDispatcherInterface       $eventDispatcher,
-      int                            $listId
-  ): SingleResourceResponseInterface {
+    }
+
+    /**
+     * @Route("/api/{version}/content/lists/{listId}/items/", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_batch_update_lists_item", requirements={"listId"="\d+"})
+     */
+    public function batchUpdateAction(
+        Request                        $request,
+        FormFactoryInterface           $formFactory,
+        ContentListRepositoryInterface $contentListRepository,
+        ArticleRepositoryInterface     $articleRepository,
+        EventDispatcherInterface       $eventDispatcher,
+        int                            $listId
+    ): SingleResourceResponseInterface {
     /** @var ContentListInterface $list */
     $list = $contentListRepository->findOneBy(['id' => $listId]);
     if (null === $list) {
-      throw new NotFoundHttpException(sprintf('Content list with id "%s" was not found.', $list));
+        throw new NotFoundHttpException(sprintf('Content list with id "%s" was not found.', $list));
     }
 
     $form = $formFactory->createNamed('', ContentListItemsType::class, [], ['method' => $request->getMethod()]);
 
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $data = $form->getData();
-      $updatedAt = DateTime::createFromFormat(DateTime::RFC3339, $data['updatedAt'], new DateTimeZone('UTC'));
-      $updatedAt->setTimezone(new DateTimeZone('UTC'));
-      $listUpdatedAt = $list->getUpdatedAt();
-      $listUpdatedAt->setTimezone(new DateTimeZone('UTC'));
-      if ($updatedAt < $listUpdatedAt) {
+        $data = $form->getData();
+        $updatedAt = DateTime::createFromFormat(DateTime::RFC3339, $data['updatedAt'], new DateTimeZone('UTC'));
+        $updatedAt->setTimezone(new DateTimeZone('UTC'));
+        $listUpdatedAt = $list->getUpdatedAt();
+        $listUpdatedAt->setTimezone(new DateTimeZone('UTC'));
+        if ($updatedAt < $listUpdatedAt) {
         throw new ConflictHttpException('List was already updated');
-      }
+        }
 
-      $updatedArticles = [];
-      /** @var ContentListAction $item */
-      foreach ($data['items'] as $item) {
+        $updatedArticles = [];
+        /** @var ContentListAction $item */
+        foreach ($data['items'] as $item) {
         $position = $item->getPosition();
         $isSticky = $item->isSticky();
         $contentId = $item->getContentId();
 
         switch ($item->getAction()) {
-          case ContentListAction::ACTION_MOVE:
+            case ContentListAction::ACTION_MOVE:
             $contentListItem = $this->findByContentOr404($list, $contentId);
 
             $this->ensureThereIsNoItemOnPositionOrThrow409($listId, $position, $isSticky);
@@ -195,7 +195,7 @@ 
             $updatedArticles[$contentId] = $contentListItem->getContent();
 
             break;
-          case ContentListAction::ACTION_ADD:
+            case ContentListAction::ACTION_ADD:
             $this->ensureThereIsNoItemOnPositionOrThrow409($listId, $position, $isSticky);
 
             $object = $articleRepository->findOneById($contentId);
@@ -204,7 +204,7 @@ 
             $updatedArticles[$contentId] = $contentListItem->getContent();
 
             break;
-          case ContentListAction::ACTION_DELETE:
+            case ContentListAction::ACTION_DELETE:
             $contentListItem = $this->findByContentOr404($list, $contentId);
             $this->entityManager->remove($contentListItem);
             $list->setUpdatedAt(new DateTime('now'));
@@ -213,25 +213,25 @@ 
 
             break;
         }
-      }
+        }
 
-      $this->contentListService->repositionStickyItems($list);
+        $this->contentListService->repositionStickyItems($list);
 
-      foreach ($updatedArticles as $updatedArticle) {
+        foreach ($updatedArticles as $updatedArticle) {
         $eventDispatcher->dispatch(new ArticleEvent(
             $updatedArticle,
             $updatedArticle->getPackage(),
             ArticleEvents::POST_UPDATE
         ), ArticleEvents::POST_UPDATE);
-      }
+        }
 
-      return new SingleResourceResponse($list, new ResponseContext(201));
+        return new SingleResourceResponse($list, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  private function findByContentOr404($listId, $contentId): ContentListItemInterface {
+    private function findByContentOr404($listId, $contentId): ContentListItemInterface {
     /** @var ContentListItemInterface $listItem */
     $listItem = $this->contentListItemRepository->findOneBy([
         'contentList' => $listId,
@@ -239,13 +239,13 @@ 
     ]);
 
     if (null === $listItem) {
-      throw new NotFoundHttpException(sprintf('Content list item with content_id "%s" was not found on that list. If You want to add new item - use action type "add".', $contentId));
+        throw new NotFoundHttpException(sprintf('Content list item with content_id "%s" was not found on that list. If You want to add new item - use action type "add".', $contentId));
     }
 
     return $listItem;
-  }
+    }
 
-  private function findOr404($listId, $id): ContentListItemInterface {
+    private function findOr404($listId, $id): ContentListItemInterface {
     /** @var ContentListItemInterface $listItem */
     $listItem = $this->contentListItemRepository->findOneBy([
         'contentList' => $listId,
@@ -253,17 +253,17 @@ 
     ]);
 
     if (null === $listItem) {
-      throw new NotFoundHttpException(sprintf('Content list item with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('Content list item with id "%s" was not found.', $id));
     }
 
     return $listItem;
-  }
+    }
 
-  private function ensureThereIsNoItemOnPositionOrThrow409(int $listId, int $position, bool $isSticky): void {
+    private function ensureThereIsNoItemOnPositionOrThrow409(int $listId, int $position, bool $isSticky): void {
     $existingContentListItem = $this->contentListService->isAnyItemPinnedOnPosition($listId, $position);
 
     if (null !== $existingContentListItem && $isSticky && $existingContentListItem->isSticky()) {
-      throw new ConflictHttpException('There is already an item pinned on that position. Unpin it first.');
+        throw new ConflictHttpException('There is already an item pinned on that position. Unpin it first.');
+    }
     }
-  }
 }

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

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
   }
 
   public function viewAction(AmpInterface $object): Response {
-    return $this->cacheService->get($this->getCacheKey($object), function () use ($object) {
+    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,

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/ContentBundle/Provider/ORM/RouteProvider.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
         $redirectRoute = $this->redirectRouteRepository->findOneBy(['staticPrefix' => $candidates[0]]);
 
         if (null !== $redirectRoute) {
-            $collection->add($redirectRoute->getRouteName() ??  '', $redirectRoute);
+            $collection->add($redirectRoute->getRouteName() ?? '', $redirectRoute);
 
             return $collection;
         }
@@ -249,7 +249,7 @@ 
      */
     private function getArrayOfIdsFromRoutesArray(array $routes): array
     {
-        return array_map(function ($route) {
+        return array_map(function($route) {
             return $route->getId();
         }, $routes);
     }

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