superdesk / web-publisher

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

Spacing +3 added lines, -3 removed lines

@@ -94,15 +94,15 @@
 
             if ($seoMetadata->getMetaMedia()) {
                 $metaImage = $seoMetadata->getMetaMedia()->getImage();
-                $response['_links']['meta_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+                $response['_links']['meta_media_url'] = ['href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
             }
             if ($seoMetadata->getOgMedia()) {
                 $metaImage = $seoMetadata->getOgMedia()->getImage();
-                $response['_links']['og_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+                $response['_links']['og_media_url'] = ['href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
             }
             if ($seoMetadata->getTwitterMedia()) {
                 $metaImage = $seoMetadata->getTwitterMedia()->getImage();
-                $response['_links']['twitter_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+                $response['_links']['twitter_media_url'] = ['href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
             }
 
             return new SingleResourceResponse($response, new ResponseContext(201));

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

Indentation +50 added lines, -50 removed lines

@@ -20,10 +20,10 @@ 
 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;
     protected MediaManagerInterface $mediaManager;
 
     /**
@@ -33,29 +33,29 @@ 
      * @param EventDispatcherInterface $eventDispatcher
      * @param MediaManagerInterface $mediaManager
      */
-  public function __construct(
-      FormFactoryInterface $formFactory,
-      FactoryInterface $seoMetadataFactory,
-      RepositoryInterface  $seoMetadataRepository,
-      EventDispatcherInterface $eventDispatcher,
-      MediaManagerInterface $mediaManager
-  ) {
+    public function __construct(
+        FormFactoryInterface $formFactory,
+        FactoryInterface $seoMetadataFactory,
+        RepositoryInterface  $seoMetadataRepository,
+        EventDispatcherInterface $eventDispatcher,
+        MediaManagerInterface $mediaManager
+    ) {
     $this->formFactory = $formFactory;
     $this->seoMetadataFactory = $seoMetadataFactory;
     $this->seoMetadataRepository = $seoMetadataRepository;
     $this->eventDispatcher = $eventDispatcher;
-      $this->mediaManager = $mediaManager;
-  }
+        $this->mediaManager = $mediaManager;
+    }
 
-  /**
-   * @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()]);
@@ -63,48 +63,48 @@ 
     $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);
 
-      $seoMetadata = $this->seoMetadataRepository->findOneByPackageGuid($packageGuid);
+        $seoMetadata = $this->seoMetadataRepository->findOneByPackageGuid($packageGuid);
     if (null === $seoMetadata) {
-      throw new NotFoundHttpException('SEO metadata not found!');
+        throw new NotFoundHttpException('SEO metadata not found!');
     }
 
-      $response = [
-          "meta_title" => $seoMetadata->getMetaTitle(),
-          "meta_description" => $seoMetadata->getMetaDescription(),
-          "og_title" => $seoMetadata->getOgTitle(),
-          "og_description" => $seoMetadata->getOgDescription(),
-          "twitter_title" => $seoMetadata->getTwitterTitle(),
-          "twitter_description" => $seoMetadata->getTwitterDescription(),
-      ];
-
-      if ($seoMetadata->getMetaMedia()) {
-          $metaImage = $seoMetadata->getMetaMedia()->getImage();
-          $response['_links']['meta_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
-      }
-      if ($seoMetadata->getOgMedia()) {
-          $metaImage = $seoMetadata->getOgMedia()->getImage();
-          $response['_links']['og_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
-      }
-      if ($seoMetadata->getTwitterMedia()) {
-          $metaImage = $seoMetadata->getTwitterMedia()->getImage();
-          $response['_links']['twitter_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
-      }
+        $response = [
+            "meta_title" => $seoMetadata->getMetaTitle(),
+            "meta_description" => $seoMetadata->getMetaDescription(),
+            "og_title" => $seoMetadata->getOgTitle(),
+            "og_description" => $seoMetadata->getOgDescription(),
+            "twitter_title" => $seoMetadata->getTwitterTitle(),
+            "twitter_description" => $seoMetadata->getTwitterDescription(),
+        ];
+
+        if ($seoMetadata->getMetaMedia()) {
+            $metaImage = $seoMetadata->getMetaMedia()->getImage();
+            $response['_links']['meta_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+        }
+        if ($seoMetadata->getOgMedia()) {
+            $metaImage = $seoMetadata->getOgMedia()->getImage();
+            $response['_links']['og_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+        }
+        if ($seoMetadata->getTwitterMedia()) {
+            $metaImage = $seoMetadata->getTwitterMedia()->getImage();
+            $response['_links']['twitter_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+        }
 
     return new SingleResourceResponse($response, new ResponseContext(200));
-  }
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -94,15 +94,15 @@
 
       if ($seoMetadata->getMetaMedia()) {
           $metaImage = $seoMetadata->getMetaMedia()->getImage();
-          $response['_links']['meta_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+          $response['_links']['meta_media_url'] = ['href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
       }
       if ($seoMetadata->getOgMedia()) {
           $metaImage = $seoMetadata->getOgMedia()->getImage();
-          $response['_links']['og_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+          $response['_links']['og_media_url'] = ['href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
       }
       if ($seoMetadata->getTwitterMedia()) {
           $metaImage = $seoMetadata->getTwitterMedia()->getImage();
-          $response['_links']['twitter_media_url'] = [ 'href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
+          $response['_links']['twitter_media_url'] = ['href' => $this->mediaManager->getMediaPublicUrl($metaImage)];
       }
 
     return new SingleResourceResponse($response, new ResponseContext(200));

src/SWP/Bundle/CoreBundle/Util/SwpLogger.php

Spacing +3 added lines, -3 removed lines

@@ -5,13 +5,13 @@
 class SwpLogger
 {
 
-    public static function log(string $message,bool $echo = false, string $fileName = 'swp.log')
+    public static function log(string $message, bool $echo = false, string $fileName = 'swp.log')
     {
         $trace = debug_backtrace()[1];
-        $message = '[' . date('Y-m-d H:i:s') . '][' . $trace['class'] . '::' . $trace['function'] . '] ' . $message . PHP_EOL;
+        $message = '['.date('Y-m-d H:i:s').']['.$trace['class'].'::'.$trace['function'].'] '.$message.PHP_EOL;
 
         if (!empty($fileName)) {
-            file_put_contents('/tmp/' . $fileName, $message, FILE_APPEND);
+            file_put_contents('/tmp/'.$fileName, $message, FILE_APPEND);
         }
         if ($echo) {
             echo $message;

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

Spacing +3 added lines, -3 removed lines

@@ -97,7 +97,7 @@ 
               'secret' => $token,
           ];
 
-          $response = $client->request('POST', $url,  [
+          $response = $client->request('POST', $url, [
               'headers' => $headers,
               'json' => $data,
               'query' => $queryParams
@@ -116,9 +116,9 @@ 
               ]
           ];
 
-          file_put_contents('/tmp/cache_invalidation.json', json_encode($result) . PHP_EOL, FILE_APPEND);
+          file_put_contents('/tmp/cache_invalidation.json', json_encode($result).PHP_EOL, FILE_APPEND);
       } catch (\Throwable $e) {
-          file_put_contents('/tmp/cache_invalidation_errors.json', $e->getMessage() . PHP_EOL, FILE_APPEND);
+          file_put_contents('/tmp/cache_invalidation_errors.json', $e->getMessage().PHP_EOL, FILE_APPEND);
       }
   }
 

Indentation +154 added lines, -154 removed lines

@@ -44,36 +44,36 @@ 
 
 class ContentListController extends AbstractController {
 
-  private ContentListRepositoryInterface $contentListRepository;
-  private ContentListItemRepositoryInterface $contentListItemRepository;
-  private ContentListServiceInterface $contentListService;
-  private FormFactoryInterface $formFactory;
-  private EntityManagerInterface $entityManager;
-  private EventDispatcherInterface $eventDispatcher;
-  private FactoryInterface $factory;
-  private string $invalidationCacheUrl;
-  private string $invalidationToken;
-
-  /**
-   * @param ContentListRepositoryInterface $contentListRepository
-   * @param ContentListItemRepositoryInterface $contentListItemRepository
-   * @param ContentListServiceInterface $contentListService
-   * @param FormFactoryInterface $formFactory
-   * @param EntityManagerInterface $entityManager
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param FactoryInterface $factory
-   */
-  public function __construct(
-      ContentListRepositoryInterface     $contentListRepository,
-      ContentListItemRepositoryInterface $contentListItemRepository,
-      ContentListServiceInterface        $contentListService,
-      FormFactoryInterface               $formFactory,
-      EntityManagerInterface             $entityManager,
-      EventDispatcherInterface           $eventDispatcher,
-      FactoryInterface                   $factory,
-      string $invalidationCacheUrl,
-      string $invalidationToken
-  ) {
+    private ContentListRepositoryInterface $contentListRepository;
+    private ContentListItemRepositoryInterface $contentListItemRepository;
+    private ContentListServiceInterface $contentListService;
+    private FormFactoryInterface $formFactory;
+    private EntityManagerInterface $entityManager;
+    private EventDispatcherInterface $eventDispatcher;
+    private FactoryInterface $factory;
+    private string $invalidationCacheUrl;
+    private string $invalidationToken;
+
+    /**
+     * @param ContentListRepositoryInterface $contentListRepository
+     * @param ContentListItemRepositoryInterface $contentListItemRepository
+     * @param ContentListServiceInterface $contentListService
+     * @param FormFactoryInterface $formFactory
+     * @param EntityManagerInterface $entityManager
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param FactoryInterface $factory
+     */
+    public function __construct(
+        ContentListRepositoryInterface     $contentListRepository,
+        ContentListItemRepositoryInterface $contentListItemRepository,
+        ContentListServiceInterface        $contentListService,
+        FormFactoryInterface               $formFactory,
+        EntityManagerInterface             $entityManager,
+        EventDispatcherInterface           $eventDispatcher,
+        FactoryInterface                   $factory,
+        string $invalidationCacheUrl,
+        string $invalidationToken
+    ) {
     $this->contentListRepository = $contentListRepository;
     $this->contentListItemRepository = $contentListItemRepository;
     $this->contentListService = $contentListService;
@@ -83,65 +83,65 @@ 
     $this->factory = $factory;
     $this->invalidationCacheUrl = $invalidationCacheUrl;
     $this->invalidationToken = $invalidationToken;
-  }
-
-  public static function invalidateCache(string $url, string $token, array $data = [])
-  {
-      try {
-          $client = new Client();
-
-          $headers = [
-              'Content-Type' => 'application/json',
-          ];
-          $queryParams = [
-              'secret' => $token,
-          ];
-
-          $response = $client->request('POST', $url,  [
-              'headers' => $headers,
-              'json' => $data,
-              'query' => $queryParams
-          ]);
-          $responseBody = $response->getBody()->getContents();
-          $result = [
-              'request' => [
-                  'headers' => $headers,
-                  'json' => $data,
-                  'query' => $queryParams
-              ],
-              'response' => [
-                  'status' => $response->getStatusCode(),
-                  'body' => $responseBody,
-                  'ReasonPhrase' => $response->getReasonPhrase()
-              ]
-          ];
-
-          file_put_contents('/tmp/cache_invalidation.json', json_encode($result) . PHP_EOL, FILE_APPEND);
-      } catch (\Throwable $e) {
-          file_put_contents('/tmp/cache_invalidation_errors.json', $e->getMessage() . PHP_EOL, FILE_APPEND);
-      }
-  }
-
-  /**
-   * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_list_lists")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    }
+
+    public static function invalidateCache(string $url, string $token, array $data = [])
+    {
+        try {
+            $client = new Client();
+
+            $headers = [
+                'Content-Type' => 'application/json',
+            ];
+            $queryParams = [
+                'secret' => $token,
+            ];
+
+            $response = $client->request('POST', $url,  [
+                'headers' => $headers,
+                'json' => $data,
+                'query' => $queryParams
+            ]);
+            $responseBody = $response->getBody()->getContents();
+            $result = [
+                'request' => [
+                    'headers' => $headers,
+                    'json' => $data,
+                    'query' => $queryParams
+                ],
+                'response' => [
+                    'status' => $response->getStatusCode(),
+                    'body' => $responseBody,
+                    'ReasonPhrase' => $response->getReasonPhrase()
+                ]
+            ];
+
+            file_put_contents('/tmp/cache_invalidation.json', json_encode($result) . PHP_EOL, FILE_APPEND);
+        } catch (\Throwable $e) {
+            file_put_contents('/tmp/cache_invalidation_errors.json', $e->getMessage() . PHP_EOL, FILE_APPEND);
+        }
+    }
+
+    /**
+     * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_list_lists")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $lists = $this->contentListRepository->getPaginatedByCriteria($this->eventDispatcher, new Criteria(), $request->query->all('sorting'), new PaginationData($request));
 
     return new ResourcesListResponse($lists);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_show_lists", requirements={"id"="\d+"})
-   */
-  public function getAction($id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_show_lists", requirements={"id"="\d+"})
+     */
+    public function getAction($id): SingleResourceResponseInterface {
     return new SingleResourceResponse($this->findOr404($id));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_content_create_lists")
-   */
-  public function createAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_content_create_lists")
+     */
+    public function createAction(Request $request): SingleResourceResponseInterface {
     /* @var ContentListInterface $contentList */
     $contentList = $this->factory->create();
     $form = $this->formFactory->createNamed('', ContentListType::class, $contentList, ['method' => $request->getMethod()]);
@@ -150,28 +150,28 @@ 
     $this->ensureContentListExists($contentList->getName());
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->contentListRepository->add($contentList);
-      self::invalidateCache(
-          $this->invalidationCacheUrl,
-          $this->invalidationToken,
-          [
-              'id' => $contentList->getId(),
-              'name' => $contentList->getName(),
-              'type' => $contentList->getType(),
-              'action' => 'CREATE'
-          ]
+        $this->contentListRepository->add($contentList);
+        self::invalidateCache(
+            $this->invalidationCacheUrl,
+            $this->invalidationToken,
+            [
+                'id' => $contentList->getId(),
+                'name' => $contentList->getName(),
+                'type' => $contentList->getType(),
+                'action' => 'CREATE'
+            ]
         );
 
-      return new SingleResourceResponse($contentList, new ResponseContext(201));
+        return new SingleResourceResponse($contentList, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_content_update_lists", requirements={"id"="\d+"})
-   */
-  public function updateAction(Request $request, int $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_content_update_lists", requirements={"id"="\d+"})
+     */
+    public function updateAction(Request $request, int $id): SingleResourceResponseInterface {
     $objectManager = $this->entityManager;
     /** @var ContentListInterface $contentList */
     $contentList = $this->findOr404($id);
@@ -182,12 +182,12 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->eventDispatcher->dispatch(
-          new GenericEvent($contentList, ['filters' => $filters, 'previousLimit' => $listLimit]),
-          ContentListEvents::LIST_CRITERIA_CHANGE
-      );
+        $this->eventDispatcher->dispatch(
+            new GenericEvent($contentList, ['filters' => $filters, 'previousLimit' => $listLimit]),
+            ContentListEvents::LIST_CRITERIA_CHANGE
+        );
 
-      $objectManager->flush();
+        $objectManager->flush();
         self::invalidateCache(
             $this->invalidationCacheUrl,
             $this->invalidationToken,
@@ -199,54 +199,54 @@ 
             ]
         );
 
-      return new SingleResourceResponse($contentList);
+        return new SingleResourceResponse($contentList);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_content_delete_lists", requirements={"id"="\d+"})
-   */
-  public function deleteAction($id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_content_delete_lists", requirements={"id"="\d+"})
+     */
+    public function deleteAction($id): SingleResourceResponseInterface {
     $repository = $this->contentListRepository;
     $contentList = $this->findOr404($id);
 
     $repository->remove($contentList);
-      self::invalidateCache(
-          $this->invalidationCacheUrl,
-          $this->invalidationToken,
-          [
-              'id' => $contentList->getId(),
-              'name' => $contentList->getName(),
-              'type' => $contentList->getType(),
-              'action' => 'DELETE'
-          ]
-      );
+        self::invalidateCache(
+            $this->invalidationCacheUrl,
+            $this->invalidationToken,
+            [
+                'id' => $contentList->getId(),
+                'name' => $contentList->getName(),
+                'type' => $contentList->getType(),
+                'action' => 'DELETE'
+            ]
+        );
 
     return new SingleResourceResponse(null, new ResponseContext(204));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", requirements={"id"="\w+"}, defaults={"version"="v2"}, methods={"LINK","UNLINK"}, name="swp_api_content_list_link_unlink")
-   */
-  public function linkUnlinkToContentListAction(Request $request, string $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", requirements={"id"="\w+"}, defaults={"version"="v2"}, methods={"LINK","UNLINK"}, name="swp_api_content_list_link_unlink")
+     */
+    public function linkUnlinkToContentListAction(Request $request, string $id): SingleResourceResponseInterface {
     $objectManager = $this->entityManager;
     /** @var ContentListInterface $contentList */
     $contentList = $this->findOr404($id);
 
     $matched = false;
     foreach ($request->attributes->get('links', []) as $key => $objectArray) {
-      if (!is_array($objectArray)) {
+        if (!is_array($objectArray)) {
         continue;
-      }
+        }
 
-      $object = $objectArray['object'];
-      if ($object instanceof Exception) {
+        $object = $objectArray['object'];
+        if ($object instanceof Exception) {
         throw $object;
-      }
+        }
 
-      if ($object instanceof ArticleInterface) {
+        if ($object instanceof ArticleInterface) {
         $contentListItem = $this->contentListItemRepository
             ->findOneBy([
                 'contentList' => $contentList,
@@ -254,59 +254,59 @@ 
             ]);
 
         if ('LINK' === $request->getMethod()) {
-          $position = 0;
-          if (count($notConvertedLinks = RequestParser::getNotConvertedLinks($request->attributes->get('links'))) > 0) {
+            $position = 0;
+            if (count($notConvertedLinks = RequestParser::getNotConvertedLinks($request->attributes->get('links'))) > 0) {
             foreach ($notConvertedLinks as $link) {
-              if (isset($link['resourceType']) && 'position' === $link['resourceType']) {
+                if (isset($link['resourceType']) && 'position' === $link['resourceType']) {
                 $position = $link['resource'];
-              }
+                }
+            }
             }
-          }
 
-          if (false === $position && $contentListItem) {
+            if (false === $position && $contentListItem) {
             throw new ConflictHttpException('This content is already linked to Content List');
-          }
+            }
 
-          if (!$contentListItem) {
+            if (!$contentListItem) {
             $contentListItem = $this->contentListService->addArticleToContentList($contentList, $object, $position);
             $objectManager->persist($contentListItem);
-          } else {
+            } else {
             $contentListItem->setPosition($position);
-          }
+            }
 
-          $objectManager->flush();
+            $objectManager->flush();
         } elseif ('UNLINK' === $request->getMethod()) {
-          if ($contentListItem->getContentList() !== $contentList) {
+            if ($contentListItem->getContentList() !== $contentList) {
             throw new ConflictHttpException('Content is not linked to content list');
-          }
-          $objectManager->remove($contentListItem);
+            }
+            $objectManager->remove($contentListItem);
         }
 
         $matched = true;
 
         break;
-      }
+        }
     }
     if (false === $matched) {
-      throw new NotFoundHttpException('Any supported link object was not found');
+        throw new NotFoundHttpException('Any supported link object was not found');
     }
 
     $objectManager->flush();
 
     return new SingleResourceResponse($contentList, new ResponseContext(201));
-  }
+    }
 
-  private function findOr404($id) {
+    private function findOr404($id) {
     if (null === $list = $this->contentListRepository->find($id)) {
-      throw new NotFoundHttpException(sprintf('Content list with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('Content list with id "%s" was not found.', $id));
     }
 
     return $list;
-  }
+    }
 
-  private function ensureContentListExists($name) {
+    private function ensureContentListExists($name) {
     if (null !== $this->contentListRepository->findOneBy(['name' => $name])) {
-      throw new ConflictHttpException(sprintf('Content list named "%s" already exists!', $name));
+        throw new ConflictHttpException(sprintf('Content list named "%s" already exists!', $name));
+    }
     }
-  }
 }

src/SWP/Bundle/ContentBundle/Controller/ArticleController.php

Indentation +63 added lines, -63 removed lines

@@ -36,27 +36,27 @@ 
 
 class ArticleController extends AbstractController {
 
-  private FormFactoryInterface $formFactory;
-  private RouteProviderInterface $routeProvider; // swp.provider.route
-  private ArticleRepositoryInterface $articleRepository; //swp.repository.article
-  private ArticleProviderInterface $articleProvider; //swp.provider.article
-  private EventDispatcherInterface  $eventDispatcher;
-  private EntityManagerInterface $entityManager; // swp.object_manager.article
-  private ArticleServiceInterface $articleService; // swp.service.article
-
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param RouteProviderInterface $routeProvider
-   * @param ArticleRepositoryInterface $articleRepository
-   * @param ArticleProviderInterface $articleProvider
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param EntityManagerInterface $entityManager
-   * @param ArticleServiceInterface $articleService
-   */
-  public function __construct(FormFactoryInterface       $formFactory, RouteProviderInterface $routeProvider,
-                              ArticleRepositoryInterface $articleRepository, ArticleProviderInterface $articleProvider,
-                              EventDispatcherInterface   $eventDispatcher, EntityManagerInterface $entityManager,
-                              ArticleServiceInterface    $articleService) {
+    private FormFactoryInterface $formFactory;
+    private RouteProviderInterface $routeProvider; // swp.provider.route
+    private ArticleRepositoryInterface $articleRepository; //swp.repository.article
+    private ArticleProviderInterface $articleProvider; //swp.provider.article
+    private EventDispatcherInterface  $eventDispatcher;
+    private EntityManagerInterface $entityManager; // swp.object_manager.article
+    private ArticleServiceInterface $articleService; // swp.service.article
+
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param RouteProviderInterface $routeProvider
+     * @param ArticleRepositoryInterface $articleRepository
+     * @param ArticleProviderInterface $articleProvider
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param EntityManagerInterface $entityManager
+     * @param ArticleServiceInterface $articleService
+     */
+    public function __construct(FormFactoryInterface       $formFactory, RouteProviderInterface $routeProvider,
+                                ArticleRepositoryInterface $articleRepository, ArticleProviderInterface $articleProvider,
+                                EventDispatcherInterface   $eventDispatcher, EntityManagerInterface $entityManager,
+                                ArticleServiceInterface    $articleService) {
     $this->formFactory = $formFactory;
     $this->routeProvider = $routeProvider;
     $this->articleRepository = $articleRepository;
@@ -64,30 +64,30 @@ 
     $this->eventDispatcher = $eventDispatcher;
     $this->entityManager = $entityManager;
     $this->articleService = $articleService;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/content/articles/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_list_articles")
-   *
-   * @throws \Exception
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/content/articles/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_list_articles")
+     *
+     * @throws \Exception
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $authors = '';
     if (null !== $request->query->get('author', null)) {
-      $authors = explode(', ', $request->query->get('author'));
+        $authors = explode(', ', $request->query->get('author'));
     }
 
     if ($request->query->get('route', false) && $request->query->get('includeSubRoutes', false)) {
-      $routeObject = $this->routeProvider->getOneById($request->query->get('route'));
+        $routeObject = $this->routeProvider->getOneById($request->query->get('route'));
 
-      if (null !== $routeObject) {
+        if (null !== $routeObject) {
         $ids = [$routeObject->getId()];
         foreach ($routeObject->getChildren() as $child) {
-          $ids[] = $child->getId();
+            $ids[] = $child->getId();
         }
         $request->query->set('route', $ids);
-      }
+        }
     }
 
     $articles = $this->articleRepository
@@ -102,25 +102,25 @@ 
         ]), $request->query->all('sorting'), new PaginationData($request));
 
     return new ResourcesListResponse($articles);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/articles/{id}", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_show_articles", requirements={"id"=".+"})
-   */
-  public function getAction($id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/articles/{id}", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_show_articles", requirements={"id"=".+"})
+     */
+    public function getAction($id): SingleResourceResponseInterface {
     $article = $this->articleProvider->getOneById($id);
 
     if (null === $article) {
-      throw new NotFoundHttpException('Article was not found');
+        throw new NotFoundHttpException('Article was not found');
     }
 
     return new SingleResourceResponse($article);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/article/search-code", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_show_articles_by_code", requirements={"code"=".+"})
-   */
-  public function getByCodeAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/article/search-code", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_show_articles_by_code", requirements={"code"=".+"})
+     */
+    public function getByCodeAction(Request $request): SingleResourceResponseInterface {
     // Extract parameters from the request
     $code = $request->query->get('code', '');
     //$tenantCode = $request->query->get('tenant_code', '');
@@ -134,16 +134,16 @@ 
         ->getOneOrNullResult();
 
     if (null === $article) {
-      throw new NotFoundHttpException('Article was not found');
+        throw new NotFoundHttpException('Article was not found');
     }
 
     return new SingleResourceResponse($article);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/articles/{id}", methods={"PATCH"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_update_articles", requirements={"id"=".+"})
-   */
-  public function updateAction(Request $request, $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/articles/{id}", methods={"PATCH"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_update_articles", requirements={"id"=".+"})
+     */
+    public function updateAction(Request $request, $id): SingleResourceResponseInterface {
     $objectManager = $this->entityManager;
     $article = $this->findOr404($id);
     $originalArticleStatus = $article->getStatus();
@@ -152,32 +152,32 @@ 
 
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->articleService->reactOnStatusChange($originalArticleStatus, $article);
-      $objectManager->flush();
-      $objectManager->refresh($article);
+        $this->articleService->reactOnStatusChange($originalArticleStatus, $article);
+        $objectManager->flush();
+        $objectManager->refresh($article);
 
-      return new SingleResourceResponse($article);
+        return new SingleResourceResponse($article);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(500));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/articles/{id}", methods={"DELETE"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_delete_articles", requirements={"id"=".+"})
-   */
-  public function deleteAction($id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/articles/{id}", methods={"DELETE"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_delete_articles", requirements={"id"=".+"})
+     */
+    public function deleteAction($id): SingleResourceResponseInterface {
     $objectManager = $this->entityManager;
     $objectManager->remove($this->findOr404($id));
     $objectManager->flush();
 
     return new SingleResourceResponse(null, new ResponseContext(204));
-  }
+    }
 
-  private function findOr404($id) {
+    private function findOr404($id) {
     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/EventListener/UpdateContentListItemsListener.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
             $subject = $metadata['subject'] ?? [];
             foreach ($subject as $key => $item) {
                 if (isset($item['scheme']) && $item['scheme'] == 'urgency') {
-                    $filters['metadata']['subject'][$key]['code'] = (string)$item['code'];
+                    $filters['metadata']['subject'][$key]['code'] = (string) $item['code'];
                 }
             }
             $criteria = new Criteria($filters);

src/SWP/Bundle/CoreBundle/MessageHandler/AbstractContentPushHandler.php

Spacing +6 added lines, -6 removed lines

@@ -144,10 +144,10 @@ 
                     ),
                     Events::PACKAGE_PRE_OPTIONS);
             }
-            $this->eventDispatcher->dispatch( new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PRE_UPDATE]), Events::PACKAGE_PRE_UPDATE);
+            $this->eventDispatcher->dispatch(new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PRE_UPDATE]), Events::PACKAGE_PRE_UPDATE);
             $this->packageObjectManager->flush();
-            $this->eventDispatcher->dispatch( new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_POST_UPDATE]), Events::PACKAGE_POST_UPDATE);
-            $this->eventDispatcher->dispatch( new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
+            $this->eventDispatcher->dispatch(new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_POST_UPDATE]), Events::PACKAGE_POST_UPDATE);
+            $this->eventDispatcher->dispatch(new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
             $this->packageObjectManager->flush();
 
             $this->logger->info(sprintf('Package %s was updated', $existingPackage->getGuid()));
@@ -176,10 +176,10 @@ 
             );
         }
 
-        $this->eventDispatcher->dispatch( new GenericEvent($package, ['eventName' => Events::PACKAGE_PRE_CREATE]), Events::PACKAGE_PRE_CREATE);
+        $this->eventDispatcher->dispatch(new GenericEvent($package, ['eventName' => Events::PACKAGE_PRE_CREATE]), Events::PACKAGE_PRE_CREATE);
         $this->packageRepository->add($package);
-        $this->eventDispatcher->dispatch( new GenericEvent($package, ['eventName' => Events::PACKAGE_POST_CREATE]), Events::PACKAGE_POST_CREATE);
-        $this->eventDispatcher->dispatch( new GenericEvent($package, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
+        $this->eventDispatcher->dispatch(new GenericEvent($package, ['eventName' => Events::PACKAGE_POST_CREATE]), Events::PACKAGE_POST_CREATE);
+        $this->eventDispatcher->dispatch(new GenericEvent($package, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
 
         if (!empty($options)) {
             $this->eventDispatcher->dispatch(

src/SWP/Bundle/ContentBundle/Controller/ContentPushController.php

Spacing +4 added lines, -4 removed lines

@@ -44,12 +44,12 @@ 
 class ContentPushController extends AbstractController {
   private EventDispatcherInterface $eventDispatcher;
   private FormFactoryInterface $formFactory;
-  private MessageBusInterface $messageBus;//swp_multi_tenancy.tenant_context
+  private MessageBusInterface $messageBus; //swp_multi_tenancy.tenant_context
   private DataTransformerInterface $dataTransformer; // swp_bridge.transformer.json_to_package
   private MediaManagerInterface $mediaManager; // swp_content_bundle.manager.media
   private EntityManagerInterface $entityManager; // swp.object_manager.media
-  private PackageRepository $packageRepository;//swp.repository.package
-  private  ArticleRepositoryInterface $articleRepository;//swp.repository.article
+  private PackageRepository $packageRepository; //swp.repository.package
+  private  ArticleRepositoryInterface $articleRepository; //swp.repository.article
   private FileProviderInterface $fileProvider;
 
     /**
@@ -172,7 +172,7 @@ 
         );
       }
 
-      throw new \Exception('Uploaded file is not valid:' . $uploadedFile->getErrorMessage());
+      throw new \Exception('Uploaded file is not valid:'.$uploadedFile->getErrorMessage());
     }
 
     return new SingleResourceResponse($form);

Indentation +47 added lines, -47 removed lines

@@ -42,15 +42,15 @@ 
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class ContentPushController extends AbstractController {
-  private EventDispatcherInterface $eventDispatcher;
-  private FormFactoryInterface $formFactory;
-  private MessageBusInterface $messageBus;//swp_multi_tenancy.tenant_context
-  private DataTransformerInterface $dataTransformer; // swp_bridge.transformer.json_to_package
-  private MediaManagerInterface $mediaManager; // swp_content_bundle.manager.media
-  private EntityManagerInterface $entityManager; // swp.object_manager.media
-  private PackageRepository $packageRepository;//swp.repository.package
-  private  ArticleRepositoryInterface $articleRepository;//swp.repository.article
-  private FileProviderInterface $fileProvider;
+    private EventDispatcherInterface $eventDispatcher;
+    private FormFactoryInterface $formFactory;
+    private MessageBusInterface $messageBus;//swp_multi_tenancy.tenant_context
+    private DataTransformerInterface $dataTransformer; // swp_bridge.transformer.json_to_package
+    private MediaManagerInterface $mediaManager; // swp_content_bundle.manager.media
+    private EntityManagerInterface $entityManager; // swp.object_manager.media
+    private PackageRepository $packageRepository;//swp.repository.package
+    private  ArticleRepositoryInterface $articleRepository;//swp.repository.article
+    private FileProviderInterface $fileProvider;
 
     /**
      * @param EventDispatcherInterface $eventDispatcher
@@ -63,17 +63,17 @@ 
      * @param ArticleRepositoryInterface $articleRepository
      * @param FileProviderInterface $fileProvider
      */
-  public function __construct(
-      EventDispatcherInterface $eventDispatcher,
-      FormFactoryInterface $formFactory,
-      MessageBusInterface $messageBus,
-      DataTransformerInterface $dataTransformer,
-      MediaManagerInterface $mediaManager,
-      EntityManagerInterface $entityManager,
-      PackageRepository $packageRepository,
-      ArticleRepositoryInterface $articleRepository,
-      FileProviderInterface    $fileProvider
-  ) {
+    public function __construct(
+        EventDispatcherInterface $eventDispatcher,
+        FormFactoryInterface $formFactory,
+        MessageBusInterface $messageBus,
+        DataTransformerInterface $dataTransformer,
+        MediaManagerInterface $mediaManager,
+        EntityManagerInterface $entityManager,
+        PackageRepository $packageRepository,
+        ArticleRepositoryInterface $articleRepository,
+        FileProviderInterface    $fileProvider
+    ) {
     $this->eventDispatcher = $eventDispatcher;
     $this->formFactory = $formFactory;
     $this->messageBus = $messageBus;
@@ -83,13 +83,13 @@ 
     $this->packageRepository = $packageRepository;
     $this->articleRepository = $articleRepository;
     $this->fileProvider = $fileProvider;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/content/push", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_push")
-   */
-  public function pushContentAction(Request $request, TenantContextInterface $tenantContext): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/push", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_push")
+     */
+    public function pushContentAction(Request $request, TenantContextInterface $tenantContext): SingleResourceResponseInterface {
     $package = $this->dataTransformer->transform($request->getContent());
     $this->eventDispatcher->dispatch(new GenericEvent($package), Events::SWP_VALIDATION);
 
@@ -100,7 +100,7 @@ 
     );
 
     return new SingleResourceResponse(['status' => 'OK'], new ResponseContext(201));
-  }
+    }
 
 
     /**
@@ -128,25 +128,25 @@ 
         return new SingleResourceResponse(['status' => 'OK'], new ResponseContext(201));
     }
 
-  /**
-   * @Route("/api/{version}/assets/push", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_assets_push")
-   */
-  public function pushAssetsAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/assets/push", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_assets_push")
+     */
+    public function pushAssetsAction(Request $request): SingleResourceResponseInterface {
     $form = $this->formFactory->createNamed('', MediaFileType::class);
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $mediaManager = $this->mediaManager;
-      $uploadedFile = $form->getData()['media'];
-      $mediaId = $request->request->get('mediaId');
+        $mediaManager = $this->mediaManager;
+        $uploadedFile = $form->getData()['media'];
+        $mediaId = $request->request->get('mediaId');
 
-      if ($uploadedFile->isValid()) {
+        if ($uploadedFile->isValid()) {
         $fileProvider = $this->fileProvider;
         $file = $fileProvider->getFile(ArticleMedia::handleMediaId($mediaId), $uploadedFile->guessExtension());
 
         if (null === $file) {
-          $file = $mediaManager->handleUploadedFile($uploadedFile, $mediaId);
-          $this->entityManager->flush();
+            $file = $mediaManager->handleUploadedFile($uploadedFile, $mediaId);
+            $this->entityManager->flush();
         }
 
         return new SingleResourceResponse(
@@ -159,23 +159,23 @@ 
             ],
             new ResponseContext(201)
         );
-      }
+        }
 
-      throw new \Exception('Uploaded file is not valid:' . $uploadedFile->getErrorMessage());
+        throw new \Exception('Uploaded file is not valid:' . $uploadedFile->getErrorMessage());
     }
 
     return new SingleResourceResponse($form);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/assets/{action}/{mediaId}.{extension}", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, requirements={"mediaId"=".+", "action"="get|push"}, name="swp_api_assets_get")
-   */
-  public function getAssetsAction(string $mediaId, string $extension): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/assets/{action}/{mediaId}.{extension}", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, requirements={"mediaId"=".+", "action"="get|push"}, name="swp_api_assets_get")
+     */
+    public function getAssetsAction(string $mediaId, string $extension): SingleResourceResponseInterface {
     $fileProvider = $this->fileProvider;
     $file = $fileProvider->getFile(ArticleMedia::handleMediaId($mediaId), $extension);
 
     if (null === $file) {
-      throw new NotFoundHttpException('Media don\'t exist in storage');
+        throw new NotFoundHttpException('Media don\'t exist in storage');
     }
 
     $mediaManager = $this->mediaManager;
@@ -187,9 +187,9 @@ 
         'mime_type' => MimeTypeHelper::getByExtension($file->getFileExtension()),
         'filemeta' => [],
     ]);
-  }
+    }
 
-  protected function getPackageRepository() {
+    protected function getPackageRepository() {
     return $this->packageRepository;
-  }
+    }
 }

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

Indentation +104 added lines, -104 removed lines

@@ -45,41 +45,41 @@ 
 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,
-      string $invalidationCacheUrl,
-      string $invalidationToken
-  ) {
-      $this->contentListItemRepository = $contentListItemRepository;
-      $this->entityManager = $entityManager;
-      $this->contentListService = $contentListService;
-      $this->eventDispatcher = $eventDispatcher;
-      $this->invalidationCacheUrl = $invalidationCacheUrl;
-      $this->invalidationToken = $invalidationToken;
-  }
-
-
-  /**
-   * @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 {
+    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,
+        string $invalidationCacheUrl,
+        string $invalidationToken
+    ) {
+        $this->contentListItemRepository = $contentListItemRepository;
+        $this->entityManager = $entityManager;
+        $this->contentListService = $contentListService;
+        $this->eventDispatcher = $eventDispatcher;
+        $this->invalidationCacheUrl = $invalidationCacheUrl;
+        $this->invalidationToken = $invalidationToken;
+    }
+
+
+    /**
+     * @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(
@@ -112,20 +112,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(
         '',
@@ -137,13 +137,13 @@ 
     $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();
         ContentListController::invalidateCache(
             $this->invalidationCacheUrl,
             $this->invalidationToken,
@@ -155,46 +155,46 @@ 
             ]
         );
 
-      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 = [];
-      $updatedItemsInvalidateCache = [];
-      /** @var ContentListAction $item */
-      foreach ($data['items'] as $item) {
+        $updatedArticles = [];
+        $updatedItemsInvalidateCache = [];
+        /** @var ContentListAction $item */
+        foreach ($data['items'] as $item) {
         $position = $item->getPosition();
         $isSticky = $item->isSticky();
         $contentId = $item->getContentId();
@@ -208,7 +208,7 @@ 
 
 
         switch ($item->getAction()) {
-          case ContentListAction::ACTION_MOVE:
+            case ContentListAction::ACTION_MOVE:
             $updated = false;
             $contentListItem = $this->findByContentOr404($list, $contentId);
 
@@ -236,7 +236,7 @@ 
             $updatedArticles[$contentId] = $contentListItem->getContent();
 
             break;
-          case ContentListAction::ACTION_ADD:
+            case ContentListAction::ACTION_ADD:
             $this->ensureThereIsNoItemOnPositionOrThrow409(
                 $listId,
                 $position,
@@ -250,7 +250,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'));
@@ -259,17 +259,17 @@ 
 
             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);
-      }
+        }
         ContentListController::invalidateCache(
             $this->invalidationCacheUrl,
             $this->invalidationToken,
@@ -282,13 +282,13 @@ 
             ]
         );
 
-      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,
@@ -296,13 +296,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,
@@ -310,29 +310,29 @@ 
     ]);
 
     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,
-      string $action): void {
-      $existingContentListItem = $this->contentListService->isAnyItemPinnedOnPosition($listId, $position);
+    private function ensureThereIsNoItemOnPositionOrThrow409(
+        int $listId,
+        int $position,
+        bool $isSticky,
+        string $action): void {
+        $existingContentListItem = $this->contentListService->isAnyItemPinnedOnPosition($listId, $position);
 
-      if (!$existingContentListItem && !$isSticky) {
-          return;
-      }
+        if (!$existingContentListItem && !$isSticky) {
+            return;
+        }
 
-      if ($existingContentListItem && $existingContentListItem->isSticky()) {
+        if ($existingContentListItem && $existingContentListItem->isSticky()) {
         throw new ConflictHttpException('There is already an item pinned on that position. Unpin it first.');
-      }
+        }
 
-      if ($action === ContentListAction::ACTION_MOVE && $isSticky) {
-          throw new ConflictHttpException('Cannot move pinned item. Unpin it first.');
-      }
-  }
+        if ($action === ContentListAction::ACTION_MOVE && $isSticky) {
+            throw new ConflictHttpException('Cannot move pinned item. Unpin it first.');
+        }
+    }
 }