superdesk / web-publisher

src/SWP/Bundle/CoreBundle/Migrations/Version20180314091410.php

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     /**
      * @param Schema $schema
      */
-   public function postUp(Schema $schema) : void
+    public function postUp(Schema $schema) : void
     {
         $entityManager = $this->container->get('doctrine.orm.default_entity_manager');
         $query = $entityManager

src/SWP/Component/ContentList/Repository/ContentListItemRepositoryInterface.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
     public function getSortedItems(Criteria $criteria, array $sorting = [], array $groupValues = []): QueryBuilder;
 
-    public function getPaginatedByCriteria(EventDispatcherInterface $eventDispatcher,Criteria $criteria, array $sorting = [], PaginationData $paginationData = null);
+    public function getPaginatedByCriteria(EventDispatcherInterface $eventDispatcher, Criteria $criteria, array $sorting = [], PaginationData $paginationData = null);
 
     public function getCountByCriteria(Criteria $criteria): int;
 

src/SWP/Bundle/ElasticSearchBundle/Controller/Api/ArticleSearchController.php

Spacing +6 added lines, -6 removed lines

@@ -106,15 +106,15 @@
 
   protected function createAdditionalCriteria(Request $request): array {
     return [
-        'routes' => array_filter((array)$request->query->get('route', [])),
-        'statuses' => array_filter((array)$request->query->get('status', [])),
-        'authors' => array_filter((array)$request->query->get('author', [])),
+        'routes' => array_filter((array) $request->query->get('route', [])),
+        'statuses' => array_filter((array) $request->query->get('status', [])),
+        'authors' => array_filter((array) $request->query->get('author', [])),
         'publishedBefore' => $request->query->has('published_before') ? new \DateTime($request->query->get('published_before')) : null,
         'publishedAfter' => $request->query->has('published_after') ? new \DateTime($request->query->get('published_after')) : null,
         'publishedAt' => $request->query->get('published_at'),
-        'sources' => array_filter((array)$request->query->get('source', [])),
-        'metadata' => array_filter((array)$request->query->get('metadata', [])),
-        'keywords' => array_filter((array)$request->query->get('keywords', [])),
+        'sources' => array_filter((array) $request->query->get('source', [])),
+        'metadata' => array_filter((array) $request->query->get('metadata', [])),
+        'keywords' => array_filter((array) $request->query->get('keywords', [])),
     ];
   }
 }

Indentation +24 added lines, -24 removed lines

@@ -28,25 +28,25 @@ 
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class ArticleSearchController extends AbstractController {
-  private $extraFields; //
-  private CachedTenantContext $cachedTenantContext; // swp_multi_tenancy.tenant_context
-  private PaginatorInterface $paginator; // knp_paginator
-
-  /**
-   * @param $extraFields
-   * @param CachedTenantContext $cachedTenantContext
-   * @param PaginatorInterface $paginator
-   */
-  public function __construct($extraFields, CachedTenantContext $cachedTenantContext, PaginatorInterface $paginator) {
+    private $extraFields; //
+    private CachedTenantContext $cachedTenantContext; // swp_multi_tenancy.tenant_context
+    private PaginatorInterface $paginator; // knp_paginator
+
+    /**
+     * @param $extraFields
+     * @param CachedTenantContext $cachedTenantContext
+     * @param PaginatorInterface $paginator
+     */
+    public function __construct($extraFields, CachedTenantContext $cachedTenantContext, PaginatorInterface $paginator) {
     $this->extraFields = $extraFields;
     $this->cachedTenantContext = $cachedTenantContext;
     $this->paginator = $paginator;
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/articles/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_list_articles")
-   */
-  public function searchAction(Request $request, RepositoryManagerInterface $repositoryManager) {
+    /**
+     * @Route("/api/{version}/content/articles/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_content_list_articles")
+     */
+    public function searchAction(Request $request, RepositoryManagerInterface $repositoryManager) {
     $criteria = $this->createCriteriaFrom($request);
     $extraFields = $this->extraFields;
 
@@ -69,16 +69,16 @@ 
     $responseContext->setSerializationGroups($this->getSerializationGroups());
 
     return new ResourcesListResponse($pagination, $responseContext);
-  }
+    }
 
-  private function createCriteriaFrom(Request $request): Criteria {
+    private function createCriteriaFrom(Request $request): Criteria {
     return Criteria::fromQueryParameters(
         $request->query->get('term', ''),
         array_merge($this->createDefaultCriteria($request), $this->createAdditionalCriteria($request)
         ));
-  }
+    }
 
-  private function createDefaultCriteria(Request $request): array {
+    private function createDefaultCriteria(Request $request): array {
     $currentTenant = $this->cachedTenantContext->getTenant();
 
     return [
@@ -87,9 +87,9 @@ 
         'limit' => $request->query->get('limit', 10),
         'tenantCode' => $currentTenant->getCode(),
     ];
-  }
+    }
 
-  protected function getSerializationGroups(): array {
+    protected function getSerializationGroups(): array {
     return [
         'Default',
         'api_articles_list',
@@ -102,9 +102,9 @@ 
         'api_tenant_list',
         'api_articles_statistics_list',
     ];
-  }
+    }
 
-  protected function createAdditionalCriteria(Request $request): array {
+    protected function createAdditionalCriteria(Request $request): array {
     return [
         'routes' => array_filter((array)$request->query->get('route', [])),
         'statuses' => array_filter((array)$request->query->get('status', [])),
@@ -116,5 +116,5 @@ 
         'metadata' => array_filter((array)$request->query->get('metadata', [])),
         'keywords' => array_filter((array)$request->query->get('keywords', [])),
     ];
-  }
+    }
 }

src/SWP/Bundle/ElasticSearchBundle/Controller/Api/PackageSearchController.php

Indentation +16 added lines, -16 removed lines

@@ -32,27 +32,27 @@ 
 
 class PackageSearchController extends AbstractController {
 
-  private PaginatorInterface $paginator;
-  private EventDispatcherInterface $eventDispatcher;
-  private CachedTenantContext $cachedTenantContext;
+    private PaginatorInterface $paginator;
+    private EventDispatcherInterface $eventDispatcher;
+    private CachedTenantContext $cachedTenantContext;
 
-  /**
-   * @param PaginatorInterface $paginator
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param CachedTenantContext $cachedTenantContext
-   */
-  public function __construct(PaginatorInterface  $paginator, EventDispatcherInterface $eventDispatcher,
-                              CachedTenantContext $cachedTenantContext) {
+    /**
+     * @param PaginatorInterface $paginator
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param CachedTenantContext $cachedTenantContext
+     */
+    public function __construct(PaginatorInterface  $paginator, EventDispatcherInterface $eventDispatcher,
+                                CachedTenantContext $cachedTenantContext) {
     $this->paginator = $paginator;
     $this->eventDispatcher = $eventDispatcher;
     $this->cachedTenantContext = $cachedTenantContext;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/packages/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_packages")
-   */
-  public function searchAction(Request $request, RepositoryManagerInterface $repositoryManager) {
+    /**
+     * @Route("/api/{version}/packages/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_packages")
+     */
+    public function searchAction(Request $request, RepositoryManagerInterface $repositoryManager) {
     $this->eventDispatcher->dispatch( new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $currentTenant = $this->cachedTenantContext->getTenant();
 
@@ -106,5 +106,5 @@ 
     );
 
     return new ResourcesListResponse($pagination, $responseContext);
-  }
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -53,7 +53,7 @@ 
    * @Route("/api/{version}/packages/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_packages")
    */
   public function searchAction(Request $request, RepositoryManagerInterface $repositoryManager) {
-    $this->eventDispatcher->dispatch( new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
+    $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $currentTenant = $this->cachedTenantContext->getTenant();
 
     $criteria = Criteria::fromQueryParameters(
@@ -62,14 +62,14 @@ 
             'page' => $request->query->get('page'),
             'sort' => $request->query->get('sorting'),
             'limit' => $request->query->get('limit', 10),
-            'authors' => array_filter((array)$request->query->get('author', [])),
+            'authors' => array_filter((array) $request->query->get('author', [])),
             'publishedBefore' => $request->query->get('published_before'),
             'publishedAfter' => $request->query->get('published_after'),
             'organization' => $currentTenant->getOrganization()->getId(),
-            'sources' => array_filter((array)$request->query->get('source', [])),
-            'tenants' => array_filter((array)$request->query->get('tenant', [])),
-            'routes' => array_filter((array)$request->query->get('route', [])),
-            'statuses' => array_filter((array)$request->query->get('status', [])),
+            'sources' => array_filter((array) $request->query->get('source', [])),
+            'tenants' => array_filter((array) $request->query->get('tenant', [])),
+            'routes' => array_filter((array) $request->query->get('route', [])),
+            'statuses' => array_filter((array) $request->query->get('status', [])),
             'language' => $request->query->get('language', ''),
         ]
     );

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

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 
       $request->request = new ParameterBag($data);
 
-      if(!empty($request->files->all())) {
+      if (!empty($request->files->all())) {
         $data = $request->files->all();
 
         try {

Indentation +51 added lines, -51 removed lines

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

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

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 use SWP\Bundle\CoreBundle\Model\OrganizationInterface;
 use SWP\Component\MultiTenancy\Context\TenantContextInterface;
 use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\HttpKernel\Event\RequestEvent ;
+use Symfony\Component\HttpKernel\Event\RequestEvent;
 
 class SecuredContentPushListener
 {

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

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 namespace SWP\Bundle\CoreBundle\EventListener;
 
 use SWP\Bundle\CoreBundle\Detection\DeviceDetectionInterface;
-use Symfony\Component\HttpKernel\Event\RequestEvent ;
+use Symfony\Component\HttpKernel\Event\RequestEvent;
 use Symfony\Component\HttpKernel\HttpKernelInterface;
 
 class CurrentDeviceListener

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

Indentation +18 added lines, -18 removed lines

@@ -28,24 +28,24 @@ 
 
 class SubscriptionController extends Controller {
 
-  private SubscriptionsProviderInterface $subscriptionsProvider;
-  private RepositoryInterface $userRepository;
-
-  /**
-   * @param SubscriptionsProviderInterface $subscriptionsProvider
-   * @param RepositoryInterface $userRepository
-   */
-  public function __construct(SubscriptionsProviderInterface $subscriptionsProvider,
-                              RepositoryInterface            $userRepository) {
+    private SubscriptionsProviderInterface $subscriptionsProvider;
+    private RepositoryInterface $userRepository;
+
+    /**
+     * @param SubscriptionsProviderInterface $subscriptionsProvider
+     * @param RepositoryInterface $userRepository
+     */
+    public function __construct(SubscriptionsProviderInterface $subscriptionsProvider,
+                                RepositoryInterface            $userRepository) {
     $this->subscriptionsProvider = $subscriptionsProvider;
     $this->userRepository = $userRepository;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/subscriptions/{userId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_paywall_list_subscriptions", requirements={"id"="\d+"})
-   */
-  public function getAction(Request $request, int $userId) {
+    /**
+     * @Route("/api/{version}/subscriptions/{userId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_paywall_list_subscriptions", requirements={"id"="\d+"})
+     */
+    public function getAction(Request $request, int $userId) {
     $subscriptionsProvider = $this->subscriptionsProvider;
 
     $filters = [
@@ -58,15 +58,15 @@ 
     $subscriptions = $subscriptionsProvider->getSubscriptions($user, $filters);
 
     return new SingleResourceResponse($subscriptions);
-  }
+    }
 
-  private function findOr404(int $id) {
+    private function findOr404(int $id) {
     $user = $this->userRepository->findOneById($id);
 
     if (!$user instanceof UserInterface) {
-      throw new NotFoundHttpException(sprintf('User with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('User with id "%s" was not found.', $id));
     }
 
     return $user;
-  }
+    }
 }

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

Indentation +35 added lines, -35 removed lines

@@ -29,40 +29,40 @@ 
 
 class ExternalDataController extends AbstractController {
 
-  private PackageRepositoryInterface $packageRepository;
-  private FactoryInterface $factory;
-
-  /**
-   * @param PackageRepositoryInterface $packageRepository
-   * @param FactoryInterface $factory
-   */
-  public function __construct(PackageRepositoryInterface $packageRepository, FactoryInterface $factory) {
+    private PackageRepositoryInterface $packageRepository;
+    private FactoryInterface $factory;
+
+    /**
+     * @param PackageRepositoryInterface $packageRepository
+     * @param FactoryInterface $factory
+     */
+    public function __construct(PackageRepositoryInterface $packageRepository, FactoryInterface $factory) {
     $this->packageRepository = $packageRepository;
     $this->factory = $factory;
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/extra/{slug}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PUT"}, name="swp_api_core_add_extra_data")
-   */
-  public function setAction(Request $request, string $slug): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/packages/extra/{slug}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PUT"}, name="swp_api_core_add_extra_data")
+     */
+    public function setAction(Request $request, string $slug): SingleResourceResponseInterface {
     $packageRepository = $this->packageRepository;
     $existingPackage = $packageRepository->findOneBy(['slugline' => $slug]);
     if (null === $existingPackage) {
-      throw new NotFoundHttpException(sprintf('Package with slug %s was not found', $slug));
+        throw new NotFoundHttpException(sprintf('Package with slug %s was not found', $slug));
     }
 
     $externalData = $existingPackage->getExternalData();
     if (null === $externalData) {
-      foreach ($externalData as $data) {
+        foreach ($externalData as $data) {
         $packageRepository->remove($data);
-      }
-      $packageRepository->flush();
+        }
+        $packageRepository->flush();
     }
 
     if (null !== $request->getContent()) {
-      $validJson = self::getArrayFromJson($request->getContent());
-      $responseData = [];
-      foreach ($validJson as $key => $value) {
+        $validJson = self::getArrayFromJson($request->getContent());
+        $responseData = [];
+        foreach ($validJson as $key => $value) {
         /** @var ExternalDataInterface $externalData */
         $externalData = $this->factory->create();
         $externalData->setKey($key);
@@ -70,45 +70,45 @@ 
         $externalData->setPackage($existingPackage);
         $packageRepository->persist($externalData);
         $responseData[$externalData->getKey()] = $externalData->getValue();
-      }
-      $packageRepository->flush();
+        }
+        $packageRepository->flush();
 
-      return new SingleResourceResponse($responseData, new ResponseContext(200));
+        return new SingleResourceResponse($responseData, new ResponseContext(200));
     }
 
     return new SingleResourceResponse(['Provided request content is not valid JSON'], new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/extra/{slug}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_extra_data")
-   */
-  public function getAction(string $slug): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/packages/extra/{slug}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_extra_data")
+     */
+    public function getAction(string $slug): SingleResourceResponseInterface {
     $packageRepository = $this->packageRepository;
     $existingPackage = $packageRepository->findOneBy(['slugline' => $slug]);
     if (null === $existingPackage) {
-      throw new NotFoundHttpException(sprintf('package with slug %s was not found', $slug));
+        throw new NotFoundHttpException(sprintf('package with slug %s was not found', $slug));
     }
 
     $externalData = $existingPackage->getExternalData();
     if (null === $externalData) {
-      return new SingleResourceResponse([], new ResponseContext(200));
+        return new SingleResourceResponse([], new ResponseContext(200));
     }
 
     $responseData = [];
     /** @var ExternalDataInterface $data */
     foreach ($externalData as $data) {
-      $responseData[$data->getKey()] = $data->getValue();
+        $responseData[$data->getKey()] = $data->getValue();
     }
 
     return new SingleResourceResponse($responseData, new ResponseContext(200));
-  }
+    }
 
-  private static function getArrayFromJson($jsonString) {
+    private static function getArrayFromJson($jsonString) {
     $jsonArray = json_decode($jsonString, true);
     if (null === $jsonArray || JSON_ERROR_NONE !== json_last_error()) {
-      throw new \UnexpectedValueException('Provided request content is not valid JSON');
+        throw new \UnexpectedValueException('Provided request content is not valid JSON');
     }
 
     return $jsonArray;
-  }
+    }
 }