superdesk / web-publisher

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

Indentation +100 added lines, -100 removed lines

@@ -38,117 +38,117 @@ 
 
 class AuthController extends AbstractController
 {
-  protected FormFactoryInterface $formFactory;
-  protected ApiKeyRepositoryInterface $apiKeyRepository;
-  protected ApiKeyFactory $apiKeyFactory;
-  protected LockFactory $lockFactory;
-
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param ApiKeyRepositoryInterface $apiKeyRepository
-   * @param ApiKeyFactory $apiKeyFactory
-   * @param LockFactory $lockFactory
-   */
-  public function __construct(
-      FormFactoryInterface $formFactory,
-      ApiKeyRepositoryInterface $apiKeyRepository,
-      ApiKeyFactory$apiKeyFactory,
-      LockFactory $lockFactory
-  ) {
+    protected FormFactoryInterface $formFactory;
+    protected ApiKeyRepositoryInterface $apiKeyRepository;
+    protected ApiKeyFactory $apiKeyFactory;
+    protected LockFactory $lockFactory;
+
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param ApiKeyRepositoryInterface $apiKeyRepository
+     * @param ApiKeyFactory $apiKeyFactory
+     * @param LockFactory $lockFactory
+     */
+    public function __construct(
+        FormFactoryInterface $formFactory,
+        ApiKeyRepositoryInterface $apiKeyRepository,
+        ApiKeyFactory$apiKeyFactory,
+        LockFactory $lockFactory
+    ) {
     $this->formFactory = $formFactory;
     $this->apiKeyRepository = $apiKeyRepository;
     $this->apiKeyFactory = $apiKeyFactory;
     $this->lockFactory = $lockFactory;
-  }
-
-
-  /**
-   * @Route(
-   *     "/api/{version}/auth/",
-   *     options={"expose"=true},
-   *     defaults={"version"="v2"},
-   *     methods={"POST"},
-   *     name="swp_api_auth"
-   * )
-   */
-  public function authenticateAction(
-      Request $request,
-      UserProviderInterface $userProvider,
-      UserPasswordHasherInterface $userPasswordEncoder
-  ) {
+    }
+
+
+    /**
+     * @Route(
+     *     "/api/{version}/auth/",
+     *     options={"expose"=true},
+     *     defaults={"version"="v2"},
+     *     methods={"POST"},
+     *     name="swp_api_auth"
+     * )
+     */
+    public function authenticateAction(
+        Request $request,
+        UserProviderInterface $userProvider,
+        UserPasswordHasherInterface $userPasswordEncoder
+    ) {
     $form = $this->formFactory->createNamed('', UserAuthenticationType::class, []);
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $formData = $form->getData();
+        $formData = $form->getData();
 
-      try {
+        try {
         $user = $userProvider->loadUserByIdentifier($formData['username']);
-      } catch (AuthenticationException $e) {
+        } catch (AuthenticationException $e) {
         $user = null;
-      }
+        }
 
-      if ((null !== $user) && $userPasswordEncoder->isPasswordValid($user, $formData['password'])) {
+        if ((null !== $user) && $userPasswordEncoder->isPasswordValid($user, $formData['password'])) {
         return $this->returnApiTokenResponse($user);
-      }
+        }
     }
 
     return new SingleResourceResponse([
         'status' => 401,
         'message' => 'Unauthorized',
     ], new ResponseContext(401));
-  }
-
-  /**
-   * @Route(
-   *     "/api/{version}/auth/superdesk/",
-   *     options={"expose"=true},
-   *     methods={"POST"},
-   *     defaults={"version"="v2"},
-   *     name="swp_api_auth_superdesk"
-   * )
-   */
-  public function authenticateWithSuperdeskAction(
-      Request               $request,
-      LoggerInterface       $logger,
-      array                 $superdeskServers,
-      UserProviderInterface $userProvider,
-      UserManagerInterface  $userManager
-  ) {
+    }
+
+    /**
+     * @Route(
+     *     "/api/{version}/auth/superdesk/",
+     *     options={"expose"=true},
+     *     methods={"POST"},
+     *     defaults={"version"="v2"},
+     *     name="swp_api_auth_superdesk"
+     * )
+     */
+    public function authenticateWithSuperdeskAction(
+        Request               $request,
+        LoggerInterface       $logger,
+        array                 $superdeskServers,
+        UserProviderInterface $userProvider,
+        UserManagerInterface  $userManager
+    ) {
     $form = $this->formFactory->createNamed('', SuperdeskCredentialAuthenticationType::class, []);
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $formData = $form->getData();
-      $authorizedSuperdeskHosts = $superdeskServers;
-      $superdeskUser = null;
-      $client = new GuzzleHttp\Client();
+        $formData = $form->getData();
+        $authorizedSuperdeskHosts = $superdeskServers;
+        $superdeskUser = null;
+        $client = new GuzzleHttp\Client();
 
-      foreach ($authorizedSuperdeskHosts as $baseUrl) {
+        foreach ($authorizedSuperdeskHosts as $baseUrl) {
         try {
-          $apiRequest = new GuzzleHttp\Psr7\Request('GET', sprintf('%s/api/sessions/%s', $baseUrl, $formData['sessionId']), [
-              'Authorization' => $formData['token'],
-          ]);
+            $apiRequest = new GuzzleHttp\Psr7\Request('GET', sprintf('%s/api/sessions/%s', $baseUrl, $formData['sessionId']), [
+                'Authorization' => $formData['token'],
+            ]);
 
-          $apiResponse = $client->send($apiRequest);
-          if (200 !== $apiResponse->getStatusCode()) {
+            $apiResponse = $client->send($apiRequest);
+            if (200 !== $apiResponse->getStatusCode()) {
             $logger->warning(sprintf('[%s] Unsuccessful response from Superdesk Server: %s', $apiResponse->getStatusCode(), $apiResponse->getBody()->getContents()));
 
             continue;
-          }
+            }
 
-          $content = json_decode($apiResponse->getBody()->getContents(), true);
-          if (is_array($content) && array_key_exists('user', $content)) {
+            $content = json_decode($apiResponse->getBody()->getContents(), true);
+            if (is_array($content) && array_key_exists('user', $content)) {
             $superdeskUser = $content['user'];
 
             break;
-          }
+            }
         } catch (GuzzleHttp\Exception\ClientException $e) {
-          $logger->warning(sprintf('Error when logging in Superdesk: %s', $e->getMessage()));
+            $logger->warning(sprintf('Error when logging in Superdesk: %s', $e->getMessage()));
 
-          continue;
+            continue;
+        }
         }
-      }
 
-      if (null === $superdeskUser) {
+        if (null === $superdeskUser) {
         return new SingleResourceResponse([
             'status' => 401,
             'message' => <<<'MESSAGE'
@@ -156,18 +156,18 @@ 
 Make sure that Publisher can talk to Superdesk instance. Set it's address in "SUPERDESK_SERVERS" environment variable.
 MESSAGE,
         ], new ResponseContext(401));
-      }
+        }
 
-      $publisherUser = $userProvider->findOneByEmail($superdeskUser['email']);
-      if (null === $publisherUser) {
+        $publisherUser = $userProvider->findOneByEmail($superdeskUser['email']);
+        if (null === $publisherUser) {
         try {
-          $publisherUser = $userProvider->loadUserByUsername($superdeskUser['username']);
+            $publisherUser = $userProvider->loadUserByUsername($superdeskUser['username']);
         } catch (AuthenticationException $e) {
-          $publisherUser = null;
+            $publisherUser = null;
+        }
         }
-      }
 
-      if (null === $publisherUser) {
+        if (null === $publisherUser) {
         /** @var UserInterface $publisherUser */
         $publisherUser = $userManager->createUser();
         $publisherUser->setUsername($superdeskUser['username']);
@@ -177,20 +177,20 @@ 
         $publisherUser->setLastName(\array_key_exists('last_name', $superdeskUser) ? $superdeskUser['last_name'] : '');
         $publisherUser->setPassword(password_hash(random_bytes(36), PASSWORD_BCRYPT));
         $userManager->updateUser($publisherUser);
-      }
+        }
 
-      if (null !== $publisherUser) {
+        if (null !== $publisherUser) {
         return $this->returnApiTokenResponse($publisherUser, str_replace('Basic ', '', $formData['token']));
-      }
+        }
     }
 
     return new SingleResourceResponse([
         'status' => 401,
         'message' => 'Unauthorized',
     ], new ResponseContext(401));
-  }
+    }
 
-  private function returnApiTokenResponse(UserInterface $user, string $token = null): SingleResourceResponseInterface {
+    private function returnApiTokenResponse(UserInterface $user, string $token = null): SingleResourceResponseInterface {
     /** @var ApiKeyInterface $apiKey */
     $apiKey = $this->generateOrGetApiKey($user, $token);
 
@@ -201,36 +201,36 @@ 
         ],
         'user' => $user,
     ]);
-  }
+    }
 
-  private function generateOrGetApiKey(UserInterface $user, $token): ?ApiKeyInterface {
+    private function generateOrGetApiKey(UserInterface $user, $token): ?ApiKeyInterface {
     $apiKey = null;
     if (null !== $token) {
-      $apiKey = $this->apiKeyRepository->getValidToken($token)->getQuery()->getOneOrNullResult();
+        $apiKey = $this->apiKeyRepository->getValidToken($token)->getQuery()->getOneOrNullResult();
     } else {
-      $validKeys = $this->apiKeyRepository->getValidTokenForUser($user)->getQuery()->getResult();
-      if (count($validKeys) > 0) {
+        $validKeys = $this->apiKeyRepository->getValidTokenForUser($user)->getQuery()->getResult();
+        if (count($validKeys) > 0) {
         $apiKey = reset($validKeys);
-      }
+        }
     }
 
     if (null === $apiKey) {
-      $apiKey = $this->apiKeyFactory->create($user, $token);
+        $apiKey = $this->apiKeyFactory->create($user, $token);
 
-      try {
+        try {
         $lock = $this->lockFactory->createLock(md5(json_encode(['type' => 'user_api_key', 'user' => $user->getId()])), 2);
         if (!$lock->acquire()) {
-          throw new RuntimeException('Other api key is created right now for this user');
+            throw new RuntimeException('Other api key is created right now for this user');
         }
         $this->apiKeyRepository->add($apiKey);
         $lock->release();
-      } catch (RuntimeException $e) {
+        } catch (RuntimeException $e) {
         sleep(2);
 
         return $this->generateOrGetApiKey($user, $token);
-      }
+        }
     }
 
     return $apiKey;
-  }
+    }
 }

src/SWP/Bundle/CoreBundle/Migrations/2021/01/Version20210112135542.php

Spacing +2 added lines, -2 removed lines

@@ -126,9 +126,9 @@
             return $data;
         }
 
-        $callback = function ($matches) {
+        $callback = function($matches) {
             $matches[2] = trim(preg_replace('/\s\s+/', ' ', $matches[2]));
-            return 's:' . mb_strlen($matches[2]) . ':"' . $matches[2] . '";';
+            return 's:'.mb_strlen($matches[2]).':"'.$matches[2].'";';
         };
 
         $data = preg_replace_callback('!s:(\d+):"(.*?)";!s', $callback, $data);

src/SWP/Bundle/CoreBundle/Command/FillArticleExtraTableCommand.php

Spacing +3 added lines, -3 removed lines

@@ -73,7 +73,7 @@ 
                     if (is_array($extraItem)) {
                         $extra = ArticleExtraEmbedField::newFromValue($key, $extraItem);
                     } else {
-                        $extra = ArticleExtraTextField::newFromValue($key, (string)$extraItem);
+                        $extra = ArticleExtraTextField::newFromValue($key, (string) $extraItem);
                     }
                     $extra->setArticle($article);
                     $this->entityManager->persist($extra);
@@ -113,9 +113,9 @@ 
             return $unserialized;
         }
 
-        $callback = function ($matches) {
+        $callback = function($matches) {
             $matches[2] = trim(preg_replace('/\s\s+/', ' ', $matches[2]));
-            return 's:' . mb_strlen($matches[2]) . ':"' . $matches[2] . '";';
+            return 's:'.mb_strlen($matches[2]).':"'.$matches[2].'";';
         };
 
         $fixedData = preg_replace_callback('!s:(\d+):"(.*?)";!s', $callback, $data);

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

Indentation +103 added lines, -103 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,51 +155,51 @@ 
             ]
         );
 
-      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:
             $updated = false;
             $contentListItem = $this->findByContentOr404($list, $contentId);
 
@@ -227,7 +227,7 @@ 
             $updatedArticles[$contentId] = $contentListItem->getContent();
 
             break;
-          case ContentListAction::ACTION_ADD:
+            case ContentListAction::ACTION_ADD:
             $this->ensureThereIsNoItemOnPositionOrThrow409(
                 $listId,
                 $position,
@@ -241,7 +241,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'));
@@ -250,17 +250,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,
@@ -272,13 +272,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,
@@ -286,13 +286,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,
@@ -300,29 +300,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.');
+        }
+    }
 }

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);