superdesk / web-publisher

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/ElasticSearchBundle/Controller/Api/AuthorSearchController.php

Indentation +16 added lines, -16 removed lines

@@ -26,27 +26,27 @@ 
 use Symfony\Component\Routing\Annotation\Route;
 
 class AuthorSearchController extends AbstractController {
-  private RepositoryManagerInterface $repositoryManager;
-  private PaginatorInterface $paginator;
-  private string $authorClassName;
+    private RepositoryManagerInterface $repositoryManager;
+    private PaginatorInterface $paginator;
+    private string $authorClassName;
 
-  /**
-   * @param RepositoryManagerInterface $repositoryManager
-   * @param PaginatorInterface $paginator
-   * @param string $authorClassName
-   */
-  public function __construct(RepositoryManagerInterface $repositoryManager, PaginatorInterface $paginator,
-                              string                     $authorClassName) {
+    /**
+     * @param RepositoryManagerInterface $repositoryManager
+     * @param PaginatorInterface $paginator
+     * @param string $authorClassName
+     */
+    public function __construct(RepositoryManagerInterface $repositoryManager, PaginatorInterface $paginator,
+                                string                     $authorClassName) {
     $this->repositoryManager = $repositoryManager;
     $this->paginator = $paginator;
     $this->authorClassName = $authorClassName;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/authors/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_authors")
-   */
-  public function searchAction(Request $request): ResourcesListResponse {
+    /**
+     * @Route("/api/{version}/authors/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_authors")
+     */
+    public function searchAction(Request $request): ResourcesListResponse {
     $criteria = Criteria::fromQueryParameters(
         $request->query->get('term', ''),
         [
@@ -87,5 +87,5 @@ 
     );
 
     return new ResourcesListResponse($pagination, $responseContext);
-  }
+    }
 }

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

Indentation +51 added lines, -51 removed lines

@@ -28,35 +28,35 @@ 
 
 class BodyListener
 {
-  private $decoderProvider;
-  private $throwExceptionOnUnsupportedContentType;
-  private $defaultFormat;
-  private $arrayNormalizer;
-  private $normalizeForms;
-
-  public function __construct(
-      DecoderProviderInterface $decoderProvider,
-      bool $throwExceptionOnUnsupportedContentType = false,
-      ArrayNormalizerInterface $arrayNormalizer = null,
-      bool $normalizeForms = false
-  ) {
+    private $decoderProvider;
+    private $throwExceptionOnUnsupportedContentType;
+    private $defaultFormat;
+    private $arrayNormalizer;
+    private $normalizeForms;
+
+    public function __construct(
+        DecoderProviderInterface $decoderProvider,
+        bool $throwExceptionOnUnsupportedContentType = false,
+        ArrayNormalizerInterface $arrayNormalizer = null,
+        bool $normalizeForms = false
+    ) {
     $this->decoderProvider = $decoderProvider;
     $this->throwExceptionOnUnsupportedContentType = $throwExceptionOnUnsupportedContentType;
     $this->arrayNormalizer = $arrayNormalizer;
     $this->normalizeForms = $normalizeForms;
-  }
+    }
 
-  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;
-  }
+    }
 }

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 {

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

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

Indentation +83 added lines, -83 removed lines

@@ -50,33 +50,33 @@ 
 
 class AnalyticsExportController extends AbstractController {
 
-  protected CacheInterface $cacheProvider;
-  protected RepositoryInterface $analyticsReportRepository;
-  protected Filesystem $filesystem;
-  protected CsvReportFileLocationResolver $csvReportFileLocationResolver;
-  protected CachedTenantContextInterface $cachedTenantContext;
-  protected RouteRepositoryInterface $routeRepository;
-  private MessageBusInterface $messageBus;
-  private FormFactoryInterface $formFactory;
-  private EventDispatcherInterface $eventDispatcher;
-
-  /**
-   * @param CacheInterface $cacheProvider
-   * @param RepositoryInterface $analyticsReportRepository
-   * @param Filesystem $filesystem
-   * @param CsvReportFileLocationResolver $csvReportFileLocationResolver
-   * @param CachedTenantContextInterface $cachedTenantContext
-   * @param RouteRepositoryInterface $routeRepository
-   * @param MessageBusInterface $messageBus
-   * @param FormFactoryInterface $formFactory
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(CacheInterface                $cacheProvider,
-                              RepositoryInterface           $analyticsReportRepository, Filesystem $filesystem,
-                              CsvReportFileLocationResolver $csvReportFileLocationResolver,
-                              CachedTenantContextInterface  $cachedTenantContext,
-                              RouteRepositoryInterface      $routeRepository, MessageBusInterface $messageBus,
-                              FormFactoryInterface          $formFactory, EventDispatcherInterface $eventDispatcher) {
+    protected CacheInterface $cacheProvider;
+    protected RepositoryInterface $analyticsReportRepository;
+    protected Filesystem $filesystem;
+    protected CsvReportFileLocationResolver $csvReportFileLocationResolver;
+    protected CachedTenantContextInterface $cachedTenantContext;
+    protected RouteRepositoryInterface $routeRepository;
+    private MessageBusInterface $messageBus;
+    private FormFactoryInterface $formFactory;
+    private EventDispatcherInterface $eventDispatcher;
+
+    /**
+     * @param CacheInterface $cacheProvider
+     * @param RepositoryInterface $analyticsReportRepository
+     * @param Filesystem $filesystem
+     * @param CsvReportFileLocationResolver $csvReportFileLocationResolver
+     * @param CachedTenantContextInterface $cachedTenantContext
+     * @param RouteRepositoryInterface $routeRepository
+     * @param MessageBusInterface $messageBus
+     * @param FormFactoryInterface $formFactory
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(CacheInterface                $cacheProvider,
+                                RepositoryInterface           $analyticsReportRepository, Filesystem $filesystem,
+                                CsvReportFileLocationResolver $csvReportFileLocationResolver,
+                                CachedTenantContextInterface  $cachedTenantContext,
+                                RouteRepositoryInterface      $routeRepository, MessageBusInterface $messageBus,
+                                FormFactoryInterface          $formFactory, EventDispatcherInterface $eventDispatcher) {
     $this->cacheProvider = $cacheProvider;
     $this->analyticsReportRepository = $analyticsReportRepository;
     $this->filesystem = $filesystem;
@@ -86,15 +86,15 @@ 
     $this->messageBus = $messageBus;
     $this->formFactory = $formFactory;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/export/analytics/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_analytics_export_post")
-   *
-   * @throws \Exception
-   */
-  public function post(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/export/analytics/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_analytics_export_post")
+     *
+     * @throws \Exception
+     */
+    public function post(Request $request): SingleResourceResponseInterface {
     /** @var UserInterface $currentlyLoggedInUser */
     $currentlyLoggedInUser = $this->getUser();
 
@@ -105,46 +105,46 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $data = $form->getData();
+        $data = $form->getData();
 
-      $analyticsReport = new AnalyticsReport();
-      $analyticsReport->setAssetId($fileName);
-      $analyticsReport->setFileExtension('csv');
-      $analyticsReport->setUser($currentlyLoggedInUser);
+        $analyticsReport = new AnalyticsReport();
+        $analyticsReport->setAssetId($fileName);
+        $analyticsReport->setFileExtension('csv');
+        $analyticsReport->setUser($currentlyLoggedInUser);
 
-      $exportAnalytics = new ExportAnalytics(
-          $data['start'],
-          $data['end'],
-          $this->cachedTenantContext->getTenant()->getCode(),
-          $fileName,
-          $currentlyLoggedInUser->getEmail(),
-          !empty($data['routes']) ? $this->toIds($data['routes']) : [],
-          !empty($data['authors']) ? $this->toIds($data['authors']) : [],
-          $data['term'] ?? ''
-      );
+        $exportAnalytics = new ExportAnalytics(
+            $data['start'],
+            $data['end'],
+            $this->cachedTenantContext->getTenant()->getCode(),
+            $fileName,
+            $currentlyLoggedInUser->getEmail(),
+            !empty($data['routes']) ? $this->toIds($data['routes']) : [],
+            !empty($data['authors']) ? $this->toIds($data['authors']) : [],
+            $data['term'] ?? ''
+        );
 
-      $filters = $this->processFilters(
-          $exportAnalytics->getFilters(),
-          !empty($data['routes']) ? $data['routes'] : [],
-          !empty($data['authors']) ? $data['authors'] : []
-      );
+        $filters = $this->processFilters(
+            $exportAnalytics->getFilters(),
+            !empty($data['routes']) ? $data['routes'] : [],
+            !empty($data['authors']) ? $data['authors'] : []
+        );
 
-      $analyticsReport->setFilters($filters);
+        $analyticsReport->setFilters($filters);
 
-      $this->analyticsReportRepository->add($analyticsReport);
+        $this->analyticsReportRepository->add($analyticsReport);
 
-      $this->messageBus->dispatch($exportAnalytics);
+        $this->messageBus->dispatch($exportAnalytics);
 
-      return new SingleResourceResponse(['status' => 'OK'], new ResponseContext(201));
+        return new SingleResourceResponse(['status' => 'OK'], new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/export/analytics/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_analytics_reports")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/export/analytics/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_list_analytics_reports")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $sorting = $request->query->all('sorting');
     $reports = $this->analyticsReportRepository->getPaginatedByCriteria(
         $this->eventDispatcher,
@@ -154,21 +154,21 @@ 
     );
 
     return new ResourcesListResponse($reports);
-  }
+    }
 
-  /**
-   * @Route("/analytics/export/{fileName}", methods={"GET"}, options={"expose"=true}, requirements={"mediaId"=".+"}, name="swp_export_analytics_download")
-   */
-  public function downloadFile(string $fileName): Response {
+    /**
+     * @Route("/analytics/export/{fileName}", methods={"GET"}, options={"expose"=true}, requirements={"mediaId"=".+"}, name="swp_export_analytics_download")
+     */
+    public function downloadFile(string $fileName): Response {
     $cacheKey = md5(serialize(['analytics_report', $fileName]));
 
     $analyticsReport = $this->cacheProvider->get($cacheKey, function () use ($fileName) {
-      /* @var AnalyticsReportInterface|null $analyticsReport */
-      return $this->analyticsReportRepository->findOneBy(['assetId' => $fileName]);
+        /* @var AnalyticsReportInterface|null $analyticsReport */
+        return $this->analyticsReportRepository->findOneBy(['assetId' => $fileName]);
     });
 
     if (null === $analyticsReport) {
-      throw new NotFoundHttpException('Report file was not found.');
+        throw new NotFoundHttpException('Report file was not found.');
     }
 
     $response = new Response();
@@ -187,30 +187,30 @@ 
     $response->setContent($this->filesystem->read($this->csvReportFileLocationResolver->getMediaBasePath() . '/' . $analyticsReport->getAssetId()));
 
     return $response;
-  }
+    }
 
-  private function toIds(array $items): array {
+    private function toIds(array $items): array {
     $ids = [];
     foreach ($items as $item) {
-      foreach ($item as $entity) {
+        foreach ($item as $entity) {
         if (!$entity instanceof PersistableInterface) {
-          continue;
+            continue;
         }
 
         $ids[] = $entity->getId();
-      }
+        }
     }
 
     return $ids;
-  }
+    }
 
-  private function processFilters(array $filters, array $routes, array $authors): array {
+    private function processFilters(array $filters, array $routes, array $authors): array {
     $routeNames = [];
 
     foreach ($routes as $route) {
-      foreach ($route as $entity) {
+        foreach ($route as $entity) {
         $routeNames[] = $entity->getName();
-      }
+        }
     }
 
     $filters['routes'] = $routeNames;
@@ -218,13 +218,13 @@ 
     $authorNames = [];
     /** @var ArticleAuthorInterface $author */
     foreach ($authors as $author) {
-      foreach ($author as $entity) {
+        foreach ($author as $entity) {
         $authorNames[] = $entity->getName();
-      }
+        }
     }
 
     $filters['authors'] = $authorNames;
 
     return $filters;
-  }
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -99,7 +99,7 @@ 
     $currentlyLoggedInUser = $this->getUser();
 
     $now = PublisherDateTime::getCurrentDateTime();
-    $fileName = 'analytics-' . $now->format('Y-m-d-H:i:s') . '.csv';
+    $fileName = 'analytics-'.$now->format('Y-m-d-H:i:s').'.csv';
 
     $form = $this->formFactory->createNamed('', ExportAnalyticsType::class, null, ['method' => $request->getMethod()]);
     $form->handleRequest($request);
@@ -162,7 +162,7 @@ 
   public function downloadFile(string $fileName): Response {
     $cacheKey = md5(serialize(['analytics_report', $fileName]));
 
-    $analyticsReport = $this->cacheProvider->get($cacheKey, function () use ($fileName) {
+    $analyticsReport = $this->cacheProvider->get($cacheKey, function() use ($fileName) {
       /* @var AnalyticsReportInterface|null $analyticsReport */
       return $this->analyticsReportRepository->findOneBy(['assetId' => $fileName]);
     });
@@ -184,7 +184,7 @@ 
     $response->setMaxAge(63072000);
     $response->setSharedMaxAge(63072000);
     $response->setLastModified($analyticsReport->getUpdatedAt() ?: $analyticsReport->getCreatedAt());
-    $response->setContent($this->filesystem->read($this->csvReportFileLocationResolver->getMediaBasePath() . '/' . $analyticsReport->getAssetId()));
+    $response->setContent($this->filesystem->read($this->csvReportFileLocationResolver->getMediaBasePath().'/'.$analyticsReport->getAssetId()));
 
     return $response;
   }

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

Indentation +26 added lines, -26 removed lines

@@ -29,28 +29,28 @@ 
 use Symfony\Component\Routing\Annotation\Route;
 
 class SlideshowController extends Controller {
-  private ArticleRepositoryInterface $articleRepository;
-  private SlideshowRepositoryInterface $slideshowRepository;
-  private EventDispatcherInterface $eventDispatcher;
+    private ArticleRepositoryInterface $articleRepository;
+    private SlideshowRepositoryInterface $slideshowRepository;
+    private EventDispatcherInterface $eventDispatcher;
 
-  /**
-   * @param ArticleRepositoryInterface $articleRepository
-   * @param SlideshowRepositoryInterface $slideshowRepository
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(ArticleRepositoryInterface   $articleRepository,
-                              SlideshowRepositoryInterface $slideshowRepository,
-                              EventDispatcherInterface     $eventDispatcher) {
+    /**
+     * @param ArticleRepositoryInterface $articleRepository
+     * @param SlideshowRepositoryInterface $slideshowRepository
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(ArticleRepositoryInterface   $articleRepository,
+                                SlideshowRepositoryInterface $slideshowRepository,
+                                EventDispatcherInterface     $eventDispatcher) {
     $this->articleRepository = $articleRepository;
     $this->slideshowRepository = $slideshowRepository;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/content/slideshows/{articleId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_slideshows_list")
-   */
-  public function listAction(Request $request, string $articleId): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/content/slideshows/{articleId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_slideshows_list")
+     */
+    public function listAction(Request $request, string $articleId): ResourcesListResponseInterface {
     $repository = $this->slideshowRepository;
 
     $article = $this->findArticleOr404($articleId);
@@ -60,29 +60,29 @@ 
     ]), $request->query->all('sorting'), new PaginationData($request));
 
     return new ResourcesListResponse($slideshows);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/slideshows/{articleId}/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_get_slideshow", requirements={"id"="\d+"})
-   */
-  public function getAction($id, string $articleId): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/slideshows/{articleId}/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_get_slideshow", requirements={"id"="\d+"})
+     */
+    public function getAction($id, string $articleId): SingleResourceResponseInterface {
     $article = $this->findArticleOr404($articleId);
 
     if (null === $list = $this->slideshowRepository->findOneBy([
             'id' => $id,
             'article' => $article,
         ])) {
-      throw new NotFoundHttpException(sprintf('Slideshow with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('Slideshow with id "%s" was not found.', $id));
     }
 
     return new SingleResourceResponse($list);
-  }
+    }
 
-  private function findArticleOr404($id) {
+    private function findArticleOr404($id) {
     if (null === $article = $this->articleRepository->findOneById($id)) {
-      throw new NotFoundHttpException(sprintf('Article with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('Article with id "%s" was not found.', $id));
     }
 
     return $article;
-  }
+    }
 }