superdesk / web-publisher

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

Indentation +23 added lines, -23 removed lines

@@ -13,21 +13,21 @@ 
 use Symfony\Component\Routing\Annotation\Route;
 
 class ExternalOauthController extends Controller {
-  public const PUBLISHER_JWT_COOKIE = 'publisher_jwt';
+    public const PUBLISHER_JWT_COOKIE = 'publisher_jwt';
 
-  private ClientRegistry $clientRegistry;
+    private ClientRegistry $clientRegistry;
 
-  /**
-   * @param ClientRegistry $clientRegistry
-   */
-  public function __construct(ClientRegistry $clientRegistry) {
+    /**
+     * @param ClientRegistry $clientRegistry
+     */
+    public function __construct(ClientRegistry $clientRegistry) {
     $this->clientRegistry = $clientRegistry;
-  }
+    }
 
-  /**
-   * @Route("/connect/oauth", name="connect_oauth_start")
-   */
-  public function connectAction(Request $request): Response {
+    /**
+     * @Route("/connect/oauth", name="connect_oauth_start")
+     */
+    public function connectAction(Request $request): Response {
     $referer = $request->headers->get('referer');
 
     return $this->clientRegistry
@@ -35,20 +35,20 @@ 
         ->redirect([
             'openid', 'email', 'profile',
         ], ['state' => $referer]);
-  }
-
-  /**
-   * This is where the user is redirected after being succesfully authenticated by the OAuth server.
-   *
-   * @Route("/connect/oauth/check", name="connect_oauth_check")
-   */
-  public function connectCheckAction(Request $request, JWTTokenManagerInterface $jwtTokenManager): Response {
+    }
+
+    /**
+     * This is where the user is redirected after being succesfully authenticated by the OAuth server.
+     *
+     * @Route("/connect/oauth/check", name="connect_oauth_check")
+     */
+    public function connectCheckAction(Request $request, JWTTokenManagerInterface $jwtTokenManager): Response {
     // If we didn't log in, something went wrong. Throw an exception!
     if (!($user = $this->getUser())) {
-      $response = $this->render('bundles/TwigBundle/Exception/error403.html.twig');
-      $response->setStatusCode(403);
+        $response = $this->render('bundles/TwigBundle/Exception/error403.html.twig');
+        $response->setStatusCode(403);
 
-      return $response;
+        return $response;
     }
 
     $state = $request->query->get('state');
@@ -57,5 +57,5 @@ 
     $response->headers->setCookie(Cookie::create(self::PUBLISHER_JWT_COOKIE, $jwtTokenManager->create($user)));
 
     return $response;
-  }
+    }
 }

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

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
     $route = $this->findRouteOr404($routeId);
 
     /** @var string $content */
-    $content = (string)$request->getContent();
+    $content = (string) $request->getContent();
     $dispatcher = $this->eventDispatcher;
     $package = $this->dataTransformer->transform($content);
     $dispatcher->dispatch(new GenericEvent($package), Events::SWP_VALIDATION);

Indentation +81 added lines, -81 removed lines

@@ -47,36 +47,36 @@ 
 
 class PackagePreviewController extends Controller {
 
-  private EventDispatcherInterface $eventDispatcher;
-  private DataTransformerInterface $dataTransformer;
-  private PackageRepositoryInterface $packageRepository;
-  private RouteRepositoryInterface $routeRepository;
-  private RepositoryInterface $packagePreviewTokenRepository;
-  private PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory;
-  private TemplateNameResolverInterface $templateNameResolver;
-  private ArticlePreviewerInterface $articlePreviewer;
-  private ArticlePreviewContextInterface $articlePreviewContext;
-
-  /**
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param DataTransformerInterface $dataTransformer
-   * @param PackageRepositoryInterface $packageRepository
-   * @param RouteRepositoryInterface $routeRepository
-   * @param RepositoryInterface $packagePreviewTokenRepository
-   * @param PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory
-   * @param TemplateNameResolverInterface $templateNameResolver
-   * @param ArticlePreviewerInterface $articlePreviewer
-   * @param ArticlePreviewContextInterface $articlePreviewContext
-   */
-  public function __construct(EventDispatcherInterface            $eventDispatcher,
-                              DataTransformerInterface            $dataTransformer,
-                              PackageRepositoryInterface          $packageRepository,
-                              RouteRepositoryInterface            $routeRepository,
-                              RepositoryInterface                 $packagePreviewTokenRepository,
-                              PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory,
-                              TemplateNameResolverInterface       $templateNameResolver,
-                              ArticlePreviewerInterface           $articlePreviewer,
-                              ArticlePreviewContextInterface      $articlePreviewContext) {
+    private EventDispatcherInterface $eventDispatcher;
+    private DataTransformerInterface $dataTransformer;
+    private PackageRepositoryInterface $packageRepository;
+    private RouteRepositoryInterface $routeRepository;
+    private RepositoryInterface $packagePreviewTokenRepository;
+    private PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory;
+    private TemplateNameResolverInterface $templateNameResolver;
+    private ArticlePreviewerInterface $articlePreviewer;
+    private ArticlePreviewContextInterface $articlePreviewContext;
+
+    /**
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param DataTransformerInterface $dataTransformer
+     * @param PackageRepositoryInterface $packageRepository
+     * @param RouteRepositoryInterface $routeRepository
+     * @param RepositoryInterface $packagePreviewTokenRepository
+     * @param PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory
+     * @param TemplateNameResolverInterface $templateNameResolver
+     * @param ArticlePreviewerInterface $articlePreviewer
+     * @param ArticlePreviewContextInterface $articlePreviewContext
+     */
+    public function __construct(EventDispatcherInterface            $eventDispatcher,
+                                DataTransformerInterface            $dataTransformer,
+                                PackageRepositoryInterface          $packageRepository,
+                                RouteRepositoryInterface            $routeRepository,
+                                RepositoryInterface                 $packagePreviewTokenRepository,
+                                PackagePreviewTokenFactoryInterface $packagePreviewTokenFactory,
+                                TemplateNameResolverInterface       $templateNameResolver,
+                                ArticlePreviewerInterface           $articlePreviewer,
+                                ArticlePreviewContextInterface      $articlePreviewContext) {
     $this->eventDispatcher = $eventDispatcher;
     $this->dataTransformer = $dataTransformer;
     $this->packageRepository = $packageRepository;
@@ -86,13 +86,13 @@ 
     $this->templateNameResolver = $templateNameResolver;
     $this->articlePreviewer = $articlePreviewer;
     $this->articlePreviewContext = $articlePreviewContext;
-  }
+    }
 
 
-  /**
-   * @Route("/preview/package/{routeId}/{id}", options={"expose"=true}, requirements={"id"="\d+", "routeId"="\d+", "token"=".+"}, methods={"GET"}, name="swp_package_preview")
-   */
-  public function previewAction(int $routeId, $id) {
+    /**
+     * @Route("/preview/package/{routeId}/{id}", options={"expose"=true}, requirements={"id"="\d+", "routeId"="\d+", "token"=".+"}, methods={"GET"}, name="swp_package_preview")
+     */
+    public function previewAction(int $routeId, $id) {
     /** @var RouteInterface $route */
     $route = $this->findRouteOr404($routeId);
     /** @var PackageInterface $package */
@@ -106,22 +106,22 @@ 
     $this->eventDispatcher->dispatch(new GenericEvent($articlePreview), ArticleEvents::PREVIEW);
 
     if (null !== ($url = $articlePreview->getPreviewUrl())) {
-      return new RedirectResponse($url);
+        return new RedirectResponse($url);
     }
 
     $route = $this->ensureRouteTemplateExists($route, $article);
 
     try {
-      return $this->render($route->getArticlesTemplateName());
+        return $this->render($route->getArticlesTemplateName());
     } catch (\Exception $e) {
-      throw $this->createNotFoundException(sprintf('Template for route with id "%d" (%s) not found!', $route->getId(), $route->getName()));
+        throw $this->createNotFoundException(sprintf('Template for route with id "%d" (%s) not found!', $route->getId(), $route->getName()));
+    }
     }
-  }
 
-  /**
-   * @FOSRoute("/api/{version}/preview/package/generate_token/{routeId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_preview_package_token", requirements={"routeId"="\d+"})
-   */
-  public function generateTokenAction(Request $request, int $routeId): SingleResourceResponseInterface {
+    /**
+     * @FOSRoute("/api/{version}/preview/package/generate_token/{routeId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_preview_package_token", requirements={"routeId"="\d+"})
+     */
+    public function generateTokenAction(Request $request, int $routeId): SingleResourceResponseInterface {
     $route = $this->findRouteOr404($routeId);
 
     /** @var string $content */
@@ -134,27 +134,27 @@ 
     $existingPreviewToken = $tokenRepository->findOneBy(['route' => $route]);
 
     if (null === $existingPreviewToken) {
-      $packagePreviewToken = $this->packagePreviewTokenFactory->createTokenizedWith($route, $content);
+        $packagePreviewToken = $this->packagePreviewTokenFactory->createTokenizedWith($route, $content);
 
-      $tokenRepository->persist($packagePreviewToken);
-      $tokenRepository->flush();
+        $tokenRepository->persist($packagePreviewToken);
+        $tokenRepository->flush();
 
-      return $this->returnResponseWithPreviewUrl($packagePreviewToken);
+        return $this->returnResponseWithPreviewUrl($packagePreviewToken);
     }
 
     $this->updatePackagePreviewTokenBody($content, $existingPreviewToken);
 
     return $this->returnResponseWithPreviewUrl($existingPreviewToken);
-  }
+    }
 
-  /**
-   * @Route("/preview/publish/package/{token}", options={"expose"=true}, requirements={"token"=".+"}, methods={"GET"}, name="swp_package_preview_publish")
-   */
-  public function publishPreviewAction(string $token) {
+    /**
+     * @Route("/preview/publish/package/{token}", options={"expose"=true}, requirements={"token"=".+"}, methods={"GET"}, name="swp_package_preview_publish")
+     */
+    public function publishPreviewAction(string $token) {
     $existingPreviewToken = $this->packagePreviewTokenRepository->findOneBy(['token' => $token]);
 
     if (null === $existingPreviewToken) {
-      throw $this->createNotFoundException(sprintf('Token %s is not valid.', $token));
+        throw $this->createNotFoundException(sprintf('Token %s is not valid.', $token));
     }
 
     $article = $this->getArticleForPreview($existingPreviewToken);
@@ -162,17 +162,17 @@ 
     $route = $this->ensureRouteTemplateExists($route, $article);
 
     return $this->renderTemplateOr404($route);
-  }
+    }
 
-  private function updatePackagePreviewTokenBody(string $content, PackagePreviewTokenInterface $packagePreviewToken) {
+    private function updatePackagePreviewTokenBody(string $content, PackagePreviewTokenInterface $packagePreviewToken) {
     if (md5($content) !== md5($packagePreviewToken->getBody())) {
-      $packagePreviewToken->setBody($content);
+        $packagePreviewToken->setBody($content);
 
-      $this->packagePreviewTokenRepository->flush();
+        $this->packagePreviewTokenRepository->flush();
+    }
     }
-  }
 
-  private function returnResponseWithPreviewUrl(PackagePreviewTokenInterface $packagePreviewToken): SingleResourceResponseInterface {
+    private function returnResponseWithPreviewUrl(PackagePreviewTokenInterface $packagePreviewToken): SingleResourceResponseInterface {
     $article = $this->getArticleForPreview($packagePreviewToken);
     $articlePreview = new ArticlePreview();
     $articlePreview->setArticle($article);
@@ -182,19 +182,19 @@ 
     $url = $articlePreview->getPreviewUrl();
 
     if (null === $url) {
-      $url = $this->generateUrl(
-          'swp_package_preview_publish',
-          ['token' => $packagePreviewToken->getToken()],
-          UrlGeneratorInterface::ABSOLUTE_URL
-      );
+        $url = $this->generateUrl(
+            'swp_package_preview_publish',
+            ['token' => $packagePreviewToken->getToken()],
+            UrlGeneratorInterface::ABSOLUTE_URL
+        );
     }
 
     return new SingleResourceResponse([
         'preview_url' => $url,
     ]);
-  }
+    }
 
-  private function getArticleForPreview(PackagePreviewTokenInterface $packagePreviewToken): ArticleInterface {
+    private function getArticleForPreview(PackagePreviewTokenInterface $packagePreviewToken): ArticleInterface {
     $dispatcher = $this->eventDispatcher;
     $package = $this->dataTransformer->transform($packagePreviewToken->getBody());
     $dispatcher->dispatch(new GenericEvent($package), Events::SWP_VALIDATION);
@@ -205,40 +205,40 @@ 
     $articlePreviewContext->setIsPreview(true);
 
     return $articlePreviewer->preview($package, $packagePreviewToken->getRoute());
-  }
+    }
 
-  private function renderTemplateOr404(RouteInterface $route): Response {
+    private function renderTemplateOr404(RouteInterface $route): Response {
     try {
-      return $this->render($templateName = $route->getArticlesTemplateName());
+        return $this->render($templateName = $route->getArticlesTemplateName());
     } catch (\InvalidArgumentException | LoaderError $e) {
-      throw $this->createNotFoundException(sprintf('Template %s for route with id "%d" (%s) not found!', $templateName, $route->getId(), $route->getName()));
+        throw $this->createNotFoundException(sprintf('Template %s for route with id "%d" (%s) not found!', $templateName, $route->getId(), $route->getName()));
+    }
     }
-  }
 
-  private function ensureRouteTemplateExists(RouteInterface $route, ArticleInterface $article): RouteInterface {
+    private function ensureRouteTemplateExists(RouteInterface $route, ArticleInterface $article): RouteInterface {
     if (null === $route->getArticlesTemplateName()) {
-      $templateNameResolver = $this->templateNameResolver;
-      $route->setArticlesTemplateName($templateNameResolver->resolve($article));
+        $templateNameResolver = $this->templateNameResolver;
+        $route->setArticlesTemplateName($templateNameResolver->resolve($article));
     }
 
     return $route;
-  }
+    }
 
-  private function findRouteOr404(int $id): RouteInterface {
+    private function findRouteOr404(int $id): RouteInterface {
     /** @var RouteInterface $route */
     if (null === ($route = $this->routeRepository->findOneBy(['id' => $id]))) {
-      throw $this->createNotFoundException(sprintf('Route with id: "%s" not found!', $id));
+        throw $this->createNotFoundException(sprintf('Route with id: "%s" not found!', $id));
     }
 
     return $route;
-  }
+    }
 
-  private function findPackageOr404(string $id): PackageInterface {
+    private function findPackageOr404(string $id): PackageInterface {
     /** @var PackageInterface $package */
     if (null === ($package = $this->packageRepository->findOneBy(['id' => $id]))) {
-      throw $this->createNotFoundException(sprintf('Package with id: "%s" not found!', $id));
+        throw $this->createNotFoundException(sprintf('Package with id: "%s" not found!', $id));
     }
 
     return $package;
-  }
+    }
 }

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

Indentation +41 added lines, -41 removed lines

@@ -35,38 +35,38 @@ 
 
 class FbApplicationController extends Controller {
 
-  private FormFactoryInterface $formFactory;
-  private RepositoryInterface $facebookAppRepository;
-  private RepositoryInterface $facebookPageRepository;
-  private Factory $facebookAppFactory;
-  private EventDispatcherInterface $eventDispatcher;
-
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param RepositoryInterface $facebookAppRepository
-   * @param RepositoryInterface $facebookPageRepository
-   * @param Factory $facebookAppFactory
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(FormFactoryInterface     $formFactory, RepositoryInterface $facebookAppRepository,
-                              RepositoryInterface      $facebookPageRepository, Factory $facebookAppFactory,
-                              EventDispatcherInterface $eventDispatcher) {
+    private FormFactoryInterface $formFactory;
+    private RepositoryInterface $facebookAppRepository;
+    private RepositoryInterface $facebookPageRepository;
+    private Factory $facebookAppFactory;
+    private EventDispatcherInterface $eventDispatcher;
+
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param RepositoryInterface $facebookAppRepository
+     * @param RepositoryInterface $facebookPageRepository
+     * @param Factory $facebookAppFactory
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(FormFactoryInterface     $formFactory, RepositoryInterface $facebookAppRepository,
+                                RepositoryInterface      $facebookPageRepository, Factory $facebookAppFactory,
+                                EventDispatcherInterface $eventDispatcher) {
     $this->formFactory = $formFactory;
     $this->facebookAppRepository = $facebookAppRepository;
     $this->facebookPageRepository = $facebookPageRepository;
     $this->facebookAppFactory = $facebookAppFactory;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/facebook/applications/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_facebook_applications")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/facebook/applications/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_facebook_applications")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $repository = $this->facebookAppRepository;
     $sort = $request->query->all('sorting');
     if (empty($sort)) {
-      $sort = ['id' => 'asc'];
+        $sort = ['id' => 'asc'];
     }
     $items = $repository->getPaginatedByCriteria(
         $this->eventDispatcher,
@@ -76,51 +76,51 @@ 
     );
 
     return new ResourcesListResponse($items);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/facebook/applications/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_create_facebook_applications")
-   */
-  public function createAction(Request $request) {
+    /**
+     * @Route("/api/{version}/facebook/applications/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_create_facebook_applications")
+     */
+    public function createAction(Request $request) {
     /* @var FacebookApplication $feed */
     $application = $this->facebookAppFactory->create();
     $form = $this->formFactory->createNamed('', FacebookApplicationType::class, $application, ['method' => $request->getMethod()]);
 
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->checkIfApplicationExists($application);
-      $this->facebookAppRepository->add($application);
+        $this->checkIfApplicationExists($application);
+        $this->facebookAppRepository->add($application);
 
-      return new SingleResourceResponse($application, new ResponseContext(201));
+        return new SingleResourceResponse($application, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/facebook/applications/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_delete_facebook_applications")
-   */
-  public function deleteAction($id) {
+    /**
+     * @Route("/api/{version}/facebook/applications/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_delete_facebook_applications")
+     */
+    public function deleteAction($id) {
     $repository = $this->facebookAppRepository;
     if (null === $application = $repository->findOneBy(['id' => $id])) {
-      throw new NotFoundHttpException('There is no Application with provided id!');
+        throw new NotFoundHttpException('There is no Application with provided id!');
     }
 
     if (null !== $page = $this->facebookPageRepository->findOneBy(['id' => $id])) {
-      throw new ConflictHttpException(sprintf('This Application is used by page with id: %s!', $page->getId()));
+        throw new ConflictHttpException(sprintf('This Application is used by page with id: %s!', $page->getId()));
     }
 
     $repository->remove($application);
 
     return new SingleResourceResponse(null, new ResponseContext(204));
-  }
+    }
 
-  private function checkIfApplicationExists(ApplicationInterface $application) {
+    private function checkIfApplicationExists(ApplicationInterface $application) {
     if (null !== $this->facebookAppRepository->findOneBy([
             'appId' => $application->getAppId(),
         ])
     ) {
-      throw new ConflictHttpException('This Application already exists!');
+        throw new ConflictHttpException('This Application already exists!');
+    }
     }
-  }
 }

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

Indentation +79 added lines, -79 removed lines

@@ -37,98 +37,98 @@ 
 use Symfony\Component\Security\Core\User\UserProviderInterface;
 
 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->loadUserByUsername($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'
@@ -136,18 +136,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']);
@@ -157,20 +157,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);
 
@@ -181,36 +181,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/Controller/ThemesController.php

Spacing +3 added lines, -3 removed lines

@@ -83,8 +83,8 @@ 
     $tenantCode = $tenant->getCode();
     $themes = array_filter(
         $themeRepository->findAll(),
-        static function ($element) use (&$tenantCode) {
-          if (strpos($element->getName(), ThemeHelper::SUFFIX_SEPARATOR . $tenantCode)) {
+        static function($element) use (&$tenantCode) {
+          if (strpos($element->getName(), ThemeHelper::SUFFIX_SEPARATOR.$tenantCode)) {
             return true;
           }
         }
@@ -112,7 +112,7 @@ 
       } catch (\Exception $e) {
         return new SingleResourceResponse(['message' => $e->getMessage()], new ResponseContext(400));
       }
-      $themeConfig = json_decode(file_get_contents($themePath . DIRECTORY_SEPARATOR . 'theme.json'), true);
+      $themeConfig = json_decode(file_get_contents($themePath.DIRECTORY_SEPARATOR.'theme.json'), true);
 
       return new SingleResourceResponse($themeConfig, new ResponseContext(201));
     }

Indentation +51 added lines, -51 removed lines

@@ -36,34 +36,34 @@ 
 
 class ThemesController extends Controller {
 
-  private CachedTenantContextInterface $cachedTenantContext;
-  private FormFactoryInterface $formFactory;
-  private ThemeLoader $themeLoader;
-  private ThemeServiceInterface $themeService;
-  private ThemeUploaderInterface $themeUploader;
-
-  /**
-   * @param CachedTenantContextInterface $cachedTenantContext
-   * @param FormFactoryInterface $formFactory
-   * @param ThemeLoader $themeLoader
-   * @param ThemeServiceInterface $themeService
-   * @param ThemeUploaderInterface $themeUploader
-   */
-  public function __construct(CachedTenantContextInterface $cachedTenantContext, FormFactoryInterface $formFactory,
-                              ThemeLoader         $themeLoader, ThemeServiceInterface $themeService,
-                              ThemeUploaderInterface       $themeUploader) {
+    private CachedTenantContextInterface $cachedTenantContext;
+    private FormFactoryInterface $formFactory;
+    private ThemeLoader $themeLoader;
+    private ThemeServiceInterface $themeService;
+    private ThemeUploaderInterface $themeUploader;
+
+    /**
+     * @param CachedTenantContextInterface $cachedTenantContext
+     * @param FormFactoryInterface $formFactory
+     * @param ThemeLoader $themeLoader
+     * @param ThemeServiceInterface $themeService
+     * @param ThemeUploaderInterface $themeUploader
+     */
+    public function __construct(CachedTenantContextInterface $cachedTenantContext, FormFactoryInterface $formFactory,
+                                ThemeLoader         $themeLoader, ThemeServiceInterface $themeService,
+                                ThemeUploaderInterface       $themeUploader) {
     $this->cachedTenantContext = $cachedTenantContext;
     $this->formFactory = $formFactory;
     $this->themeLoader = $themeLoader;
     $this->themeService = $themeService;
     $this->themeUploader = $themeUploader;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/organization/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_available_themes")
-   */
-  public function listAvailableAction(): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/organization/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_available_themes")
+     */
+    public function listAvailableAction(): ResourcesListResponseInterface {
     $themeLoader = $this->themeLoader;
     $themes = $themeLoader->load();
     $pagination = new SlidingPagination();
@@ -73,21 +73,21 @@ 
     $pagination->setTotalItemCount(count($themes));
 
     return new ResourcesListResponse($pagination);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_tenant_themes")
-   */
-  public function listInstalledAction(ThemeRepositoryInterface $themeRepository): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_tenant_themes")
+     */
+    public function listInstalledAction(ThemeRepositoryInterface $themeRepository): ResourcesListResponseInterface {
     /** @var TenantInterface $tenant */
     $tenant = $this->cachedTenantContext->getTenant();
     $tenantCode = $tenant->getCode();
     $themes = array_filter(
         $themeRepository->findAll(),
         static function ($element) use (&$tenantCode) {
-          if (strpos($element->getName(), ThemeHelper::SUFFIX_SEPARATOR . $tenantCode)) {
+            if (strpos($element->getName(), ThemeHelper::SUFFIX_SEPARATOR . $tenantCode)) {
             return true;
-          }
+            }
         }
     );
 
@@ -98,46 +98,46 @@ 
     $pagination->setTotalItemCount(count($themes));
 
     return new ResourcesListResponse($pagination);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/organization/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_theme")
-   */
-  public function uploadThemeAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/organization/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_theme")
+     */
+    public function uploadThemeAction(Request $request): SingleResourceResponseInterface {
     $form = $this->formFactory->createNamed('', ThemeUploadType::class, []);
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $formData = $form->getData();
-      $themeUploader = $this->themeUploader;
+        $formData = $form->getData();
+        $themeUploader = $this->themeUploader;
 
-      try {
+        try {
         $themePath = $themeUploader->upload($formData['file']);
-      } catch (\Exception $e) {
+        } catch (\Exception $e) {
         return new SingleResourceResponse(['message' => $e->getMessage()], new ResponseContext(400));
-      }
-      $themeConfig = json_decode(file_get_contents($themePath . DIRECTORY_SEPARATOR . 'theme.json'), true);
+        }
+        $themeConfig = json_decode(file_get_contents($themePath . DIRECTORY_SEPARATOR . 'theme.json'), true);
 
-      return new SingleResourceResponse($themeConfig, new ResponseContext(201));
+        return new SingleResourceResponse($themeConfig, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_install_theme")
-   */
-  public function installThemeAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/themes/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_install_theme")
+     */
+    public function installThemeAction(Request $request): SingleResourceResponseInterface {
     $form = $this->formFactory->createNamed('', ThemeInstallType::class, []);
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $formData = $form->getData();
-      $themeService = $this->themeService;
-      [$sourceDir, $themeDir] = $themeService->getDirectoriesForTheme($formData['name']);
-      $themeService->installAndProcessGeneratedData($sourceDir, $themeDir, $formData['processGeneratedData']);
+        $formData = $form->getData();
+        $themeService = $this->themeService;
+        [$sourceDir, $themeDir] = $themeService->getDirectoriesForTheme($formData['name']);
+        $themeService->installAndProcessGeneratedData($sourceDir, $themeDir, $formData['processGeneratedData']);
 
-      return new SingleResourceResponse(['status' => 'installed'], new ResponseContext(201));
+        return new SingleResourceResponse(['status' => 'installed'], new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 }

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

Indentation +33 added lines, -33 removed lines

@@ -29,50 +29,50 @@
 
 class ThemeLogoController extends Controller {
 
-  private Filesystem $filesystem;
-  private ThemeLogoUploaderInterface $themeLogoUploader;
-  private CacheInterface $cacheInterface;
+    private Filesystem $filesystem;
+    private ThemeLogoUploaderInterface $themeLogoUploader;
+    private CacheInterface $cacheInterface;
 
-  /**
-   * @param Filesystem $filesystem
-   * @param ThemeLogoUploaderInterface $themeLogoUploader
-   * @param CacheInterface $cacheInterface
-   */
-  public function __construct(Filesystem     $filesystem, ThemeLogoUploaderInterface $themeLogoUploader,
-                              CacheInterface $cacheInterface) {
+    /**
+     * @param Filesystem $filesystem
+     * @param ThemeLogoUploaderInterface $themeLogoUploader
+     * @param CacheInterface $cacheInterface
+     */
+    public function __construct(Filesystem     $filesystem, ThemeLogoUploaderInterface $themeLogoUploader,
+                                CacheInterface $cacheInterface) {
     $this->filesystem = $filesystem;
     $this->themeLogoUploader = $themeLogoUploader;
     $this->cacheInterface = $cacheInterface;
-  }
+    }
 
-  /**
-   * @Route("/theme_logo/{id}", options={"expose"=true}, requirements={"id"=".+"}, methods={"GET"}, name="swp_theme_logo_get")
-   */
-  public function getLogoAction(string $id) {
+    /**
+     * @Route("/theme_logo/{id}", options={"expose"=true}, requirements={"id"=".+"}, methods={"GET"}, name="swp_theme_logo_get")
+     */
+    public function getLogoAction(string $id) {
     $cacheKey = md5(serialize(['upload', $id]));
     return $this->cacheInterface->get($cacheKey, function (CacheItemInterface $item, &$save) use ($id) {
-      $item->expiresAfter(63072000);
+        $item->expiresAfter(63072000);
 
-      $fileSystem = $this->filesystem;
-      $themeLogoUploader = $this->themeLogoUploader;
-      $id = $themeLogoUploader->getThemeLogoUploadPath($id);
-      $file = $fileSystem->has($id);
-      if (!$file) {
+        $fileSystem = $this->filesystem;
+        $themeLogoUploader = $this->themeLogoUploader;
+        $id = $themeLogoUploader->getThemeLogoUploadPath($id);
+        $file = $fileSystem->has($id);
+        if (!$file) {
         $save = false;
         throw new NotFoundHttpException('File was not found.');
-      }
+        }
 
-      $path = $fileSystem->get($id)->getPath();
-      $response = new Response();
-      $disposition = $response->headers->makeDisposition(ResponseHeaderBag::DISPOSITION_INLINE, pathinfo($path, PATHINFO_BASENAME));
-      $response->headers->set('Content-Disposition', $disposition);
-      $response->headers->set('Content-Type', Mime::getMimeFromExtension($path));
-      $response->setPublic();
-      $response->setMaxAge(63072000);
-      $response->setSharedMaxAge(63072000);
-      $response->setContent($fileSystem->read($path));
+        $path = $fileSystem->get($id)->getPath();
+        $response = new Response();
+        $disposition = $response->headers->makeDisposition(ResponseHeaderBag::DISPOSITION_INLINE, pathinfo($path, PATHINFO_BASENAME));
+        $response->headers->set('Content-Disposition', $disposition);
+        $response->headers->set('Content-Type', Mime::getMimeFromExtension($path));
+        $response->setPublic();
+        $response->setMaxAge(63072000);
+        $response->setSharedMaxAge(63072000);
+        $response->setContent($fileSystem->read($path));
 
-      return $response;
+        return $response;
     });
-  }
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
    */
   public function getLogoAction(string $id) {
     $cacheKey = md5(serialize(['upload', $id]));
-    return $this->cacheInterface->get($cacheKey, function (CacheItemInterface $item, &$save) use ($id) {
+    return $this->cacheInterface->get($cacheKey, function(CacheItemInterface $item, &$save) use ($id) {
       $item->expiresAfter(63072000);
 
       $fileSystem = $this->filesystem;

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

Indentation +23 added lines, -23 removed lines

@@ -29,32 +29,32 @@ 
 
 class FbiaArticleController extends Controller {
 
-  private FacebookInstantArticlesArticleRepository $facebookInstantArticlesArticleRepository;
-  private FacebookInstantArticlesService $facebookInstantArticlesService;
-  private EventDispatcherInterface $eventDispatcher;
+    private FacebookInstantArticlesArticleRepository $facebookInstantArticlesArticleRepository;
+    private FacebookInstantArticlesService $facebookInstantArticlesService;
+    private EventDispatcherInterface $eventDispatcher;
 
-  /**
-   * @param FacebookInstantArticlesArticleRepository $facebookInstantArticlesArticleRepository
-   * @param FacebookInstantArticlesService $facebookInstantArticlesService
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(FacebookInstantArticlesArticleRepository $facebookInstantArticlesArticleRepository,
-                              FacebookInstantArticlesService           $facebookInstantArticlesService,
-                              EventDispatcherInterface                 $eventDispatcher) {
+    /**
+     * @param FacebookInstantArticlesArticleRepository $facebookInstantArticlesArticleRepository
+     * @param FacebookInstantArticlesService $facebookInstantArticlesService
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(FacebookInstantArticlesArticleRepository $facebookInstantArticlesArticleRepository,
+                                FacebookInstantArticlesService           $facebookInstantArticlesService,
+                                EventDispatcherInterface                 $eventDispatcher) {
     $this->facebookInstantArticlesArticleRepository = $facebookInstantArticlesArticleRepository;
     $this->facebookInstantArticlesService = $facebookInstantArticlesService;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/facebook/instantarticles/articles/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_facebook_instant_articles_articles")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/facebook/instantarticles/articles/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_facebook_instant_articles_articles")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $repository = $this->facebookInstantArticlesArticleRepository;
     $sort = $request->query->all('sorting');
     if (empty($sort)) {
-      $sort = ['createdAt' => 'desc'];
+        $sort = ['createdAt' => 'desc'];
     }
     $items = $repository->getPaginatedByCriteria(
         $this->eventDispatcher,
@@ -64,15 +64,15 @@ 
     );
 
     return new ResourcesListResponse($items);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/facebook/instantarticles/articles/{submissionId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_facebook_instant_articles_articles_update")
-   */
-  public function updateSubmissionAction(string $submissionId): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/facebook/instantarticles/articles/{submissionId}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_facebook_instant_articles_articles_update")
+     */
+    public function updateSubmissionAction(string $submissionId): SingleResourceResponseInterface {
     $instantArticlesService = $this->facebookInstantArticlesService;
     $instantArticle = $instantArticlesService->updateSubmissionStatus($submissionId);
 
     return new SingleResourceResponse($instantArticle);
-  }
+    }
 }

src/SWP/Bundle/CoreBundle/Resolver/ArticleResolver.php

Indentation +11 added lines, -11 removed lines

@@ -43,29 +43,29 @@
     {
         $collectionRouteCacheKey = md5('route_'.$url);
         return $this->cacheProvider->get($collectionRouteCacheKey, function (ItemInterface $item, &$save) use ($url) {
-          try {
+            try {
             $route = $this->matcher->match($this->getFragmentFromUrl($url, 'path'));
-          } catch(ResourceNotFoundException $e) {
+            } catch(ResourceNotFoundException $e) {
             $save = false;
             return null;
-          }
-          if(!isset($route['_article_meta'])) {
+            }
+            if(!isset($route['_article_meta'])) {
             $save = false;
             return null;
-          }
+            }
 
-          if(!($route['_article_meta'] instanceof Meta)) {
+            if(!($route['_article_meta'] instanceof Meta)) {
             $save = false;
             return null;
-          }
+            }
 
-          $values = $route['_article_meta']->getValues();
-          if(!($values instanceof ArticleInterface)) {
+            $values = $route['_article_meta']->getValues();
+            if(!($values instanceof ArticleInterface)) {
             $save = false;
             return null;
-          }
+            }
 
-          return $values;
+            return $values;
         });
     }
 

Spacing +5 added lines, -5 removed lines

@@ -42,25 +42,25 @@
     public function resolve(string $url): ?ArticleInterface
     {
         $collectionRouteCacheKey = md5('route_'.$url);
-        return $this->cacheProvider->get($collectionRouteCacheKey, function (ItemInterface $item, &$save) use ($url) {
+        return $this->cacheProvider->get($collectionRouteCacheKey, function(ItemInterface $item, &$save) use ($url) {
           try {
             $route = $this->matcher->match($this->getFragmentFromUrl($url, 'path'));
-          } catch(ResourceNotFoundException $e) {
+          } catch (ResourceNotFoundException $e) {
             $save = false;
             return null;
           }
-          if(!isset($route['_article_meta'])) {
+          if (!isset($route['_article_meta'])) {
             $save = false;
             return null;
           }
 
-          if(!($route['_article_meta'] instanceof Meta)) {
+          if (!($route['_article_meta'] instanceof Meta)) {
             $save = false;
             return null;
           }
 
           $values = $route['_article_meta']->getValues();
-          if(!($values instanceof ArticleInterface)) {
+          if (!($values instanceof ArticleInterface)) {
             $save = false;
             return null;
           }

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

Spacing +6 added lines, -6 removed lines

@@ -125,10 +125,10 @@ 
         if (null !== $existingPackage) {
             $existingPackage = $this->packageHydrator->hydrate($package, $existingPackage);
 
-            $this->eventDispatcher->dispatch( new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PRE_UPDATE]), Events::PACKAGE_PRE_UPDATE);
+            $this->eventDispatcher->dispatch(new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PRE_UPDATE]), Events::PACKAGE_PRE_UPDATE);
             $this->packageObjectManager->flush();
-            $this->eventDispatcher->dispatch( new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_POST_UPDATE]), Events::PACKAGE_POST_UPDATE);
-            $this->eventDispatcher->dispatch( new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
+            $this->eventDispatcher->dispatch(new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_POST_UPDATE]), Events::PACKAGE_POST_UPDATE);
+            $this->eventDispatcher->dispatch(new GenericEvent($existingPackage, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
             $this->packageObjectManager->flush();
 
             $this->reset();
@@ -137,10 +137,10 @@ 
             return;
         }
 
-        $this->eventDispatcher->dispatch( new GenericEvent($package, ['eventName' => Events::PACKAGE_PRE_CREATE]), Events::PACKAGE_PRE_CREATE);
+        $this->eventDispatcher->dispatch(new GenericEvent($package, ['eventName' => Events::PACKAGE_PRE_CREATE]), Events::PACKAGE_PRE_CREATE);
         $this->packageRepository->add($package);
-        $this->eventDispatcher->dispatch( new GenericEvent($package, ['eventName' => Events::PACKAGE_POST_CREATE]), Events::PACKAGE_POST_CREATE);
-        $this->eventDispatcher->dispatch( new GenericEvent($package, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
+        $this->eventDispatcher->dispatch(new GenericEvent($package, ['eventName' => Events::PACKAGE_POST_CREATE]), Events::PACKAGE_POST_CREATE);
+        $this->eventDispatcher->dispatch(new GenericEvent($package, ['eventName' => Events::PACKAGE_PROCESSED]), Events::PACKAGE_PROCESSED);
         $this->packageObjectManager->flush();
 
         $this->logger->info(sprintf('Package %s was created', $package->getGuid()));