superdesk / web-publisher

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

Indentation +26 added lines, -26 removed lines

@@ -29,28 +29,28 @@ 
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class SlideshowController extends Controller {
-  private ArticleRepositoryInterface $articleRepository;
-  private SlideshowRepositoryInterface $slideshowRepository;
-  private EventDispatcherInterface $eventDispatcher;
+    private ArticleRepositoryInterface $articleRepository;
+    private SlideshowRepositoryInterface $slideshowRepository;
+    private EventDispatcherInterface $eventDispatcher;
 
-  /**
-   * @param ArticleRepositoryInterface $articleRepository
-   * @param SlideshowRepositoryInterface $slideshowRepository
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(ArticleRepositoryInterface   $articleRepository,
-                              SlideshowRepositoryInterface $slideshowRepository,
-                              EventDispatcherInterface     $eventDispatcher) {
+    /**
+     * @param ArticleRepositoryInterface $articleRepository
+     * @param SlideshowRepositoryInterface $slideshowRepository
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(ArticleRepositoryInterface   $articleRepository,
+                                SlideshowRepositoryInterface $slideshowRepository,
+                                EventDispatcherInterface     $eventDispatcher) {
     $this->articleRepository = $articleRepository;
     $this->slideshowRepository = $slideshowRepository;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/content/slideshows/{articleId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_slideshows_list")
-   */
-  public function listAction(Request $request, string $articleId): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/content/slideshows/{articleId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_slideshows_list")
+     */
+    public function listAction(Request $request, string $articleId): ResourcesListResponseInterface {
     $repository = $this->slideshowRepository;
 
     $article = $this->findArticleOr404($articleId);
@@ -60,29 +60,29 @@ 
     ]), $request->query->all('sorting'), new PaginationData($request));
 
     return new ResourcesListResponse($slideshows);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/slideshows/{articleId}/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_get_slideshow", requirements={"id"="\d+"})
-   */
-  public function getAction($id, string $articleId): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/slideshows/{articleId}/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_get_slideshow", requirements={"id"="\d+"})
+     */
+    public function getAction($id, string $articleId): SingleResourceResponseInterface {
     $article = $this->findArticleOr404($articleId);
 
     if (null === $list = $this->slideshowRepository->findOneBy([
             'id' => $id,
             'article' => $article,
         ])) {
-      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 new SingleResourceResponse($list);
-  }
+    }
 
-  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/PackageSeoMediaUploadController.php

Indentation +22 added lines, -22 removed lines

@@ -28,50 +28,50 @@
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class PackageSeoMediaUploadController extends AbstractController {
-  private FactoryInterface $seoMetadataFactory;
-  private RepositoryInterface $seoMetadataRepository;
-  private FormFactoryInterface $formFactory;
+    private FactoryInterface $seoMetadataFactory;
+    private RepositoryInterface $seoMetadataRepository;
+    private FormFactoryInterface $formFactory;
 
-  /**
-   * @param FactoryInterface $seoMetadataFactory
-   * @param RepositoryInterface $seoMetadataRepository
-   * @param FormFactoryInterface $formFactory
-   */
-  public function __construct(FactoryInterface     $seoMetadataFactory, RepositoryInterface $seoMetadataRepository,
-                              FormFactoryInterface $formFactory) {
+    /**
+     * @param FactoryInterface $seoMetadataFactory
+     * @param RepositoryInterface $seoMetadataRepository
+     * @param FormFactoryInterface $formFactory
+     */
+    public function __construct(FactoryInterface     $seoMetadataFactory, RepositoryInterface $seoMetadataRepository,
+                                FormFactoryInterface $formFactory) {
     $this->seoMetadataFactory = $seoMetadataFactory;
     $this->seoMetadataRepository = $seoMetadataRepository;
     $this->formFactory = $formFactory;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/packages/seo/upload/{packageGuid}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_package_seo_image")
-   */
-  public function uploadAction(Request                   $request, string $packageGuid,
-                               SeoImageUploaderInterface $seoImageUploader): SingleResourceResponse {
+    /**
+     * @Route("/api/{version}/packages/seo/upload/{packageGuid}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_package_seo_image")
+     */
+    public function uploadAction(Request                   $request, string $packageGuid,
+                                SeoImageUploaderInterface $seoImageUploader): SingleResourceResponse {
     $seoMetadata = $this->seoMetadataRepository->findOneByPackageGuid($packageGuid);
 
     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 {
         $seoMetadata->setPackageGuid($packageGuid);
         $seoImageUploader->handleUpload($seoMetadata);
 
         $this->seoMetadataRepository->add($seoMetadata);
-      } 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));
-  }
+    }
 }

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