superdesk / web-publisher

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

Indentation +40 added lines, -40 removed lines

@@ -33,78 +33,78 @@ 
 
 class CurrentThemeController extends AbstractController {
 
-  private TenantAwareThemeContextInterface $tenantAwareThemeContext;
-  private FormFactoryInterface $formFactory;
-  private CachedTenantContextInterface $tenantContext;
-  private SettingsManagerInterface $settingsManager;
-  private ThemeLogoUploaderInterface $themeLogoUploader;
-
-  /**
-   * @param TenantAwareThemeContextInterface $tenantAwareThemeContext
-   * @param FormFactoryInterface $formFactory
-   * @param CachedTenantContextInterface $tenantContext
-   * @param SettingsManagerInterface $settingsManager
-   * @param ThemeLogoUploaderInterface $themeLogoUploader
-   */
-  public function __construct(TenantAwareThemeContextInterface $tenantAwareThemeContext,
-                              FormFactoryInterface             $formFactory,
-                              CachedTenantContextInterface     $tenantContext,
-                              SettingsManagerInterface         $settingsManager,
-                              ThemeLogoUploaderInterface       $themeLogoUploader) {
+    private TenantAwareThemeContextInterface $tenantAwareThemeContext;
+    private FormFactoryInterface $formFactory;
+    private CachedTenantContextInterface $tenantContext;
+    private SettingsManagerInterface $settingsManager;
+    private ThemeLogoUploaderInterface $themeLogoUploader;
+
+    /**
+     * @param TenantAwareThemeContextInterface $tenantAwareThemeContext
+     * @param FormFactoryInterface $formFactory
+     * @param CachedTenantContextInterface $tenantContext
+     * @param SettingsManagerInterface $settingsManager
+     * @param ThemeLogoUploaderInterface $themeLogoUploader
+     */
+    public function __construct(TenantAwareThemeContextInterface $tenantAwareThemeContext,
+                                FormFactoryInterface             $formFactory,
+                                CachedTenantContextInterface     $tenantContext,
+                                SettingsManagerInterface         $settingsManager,
+                                ThemeLogoUploaderInterface       $themeLogoUploader) {
     $this->tenantAwareThemeContext = $tenantAwareThemeContext;
     $this->formFactory = $formFactory;
     $this->tenantContext = $tenantContext;
     $this->settingsManager = $settingsManager;
     $this->themeLogoUploader = $themeLogoUploader;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/theme/logo_upload/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_theme_logo_2")
-   * @Route("/api/{version}/theme/logo_upload/{type}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_theme_logo")
-   */
-  public function uploadThemeLogoAction(Request $request,
+    /**
+     * @Route("/api/{version}/theme/logo_upload/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_theme_logo_2")
+     * @Route("/api/{version}/theme/logo_upload/{type}", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_upload_theme_logo")
+     */
+    public function uploadThemeLogoAction(Request $request,
                                         string  $type = ThemeLogoProviderInterface::SETTING_NAME_DEFAULT): SingleResourceResponseInterface {
     $themeContext = $this->tenantAwareThemeContext;
 
     if (null === ($theme = $themeContext->getTheme())) {
-      throw new \LogicException('Theme is not set!');
+        throw new \LogicException('Theme is not set!');
     }
 
     $form = $this->formFactory->createNamed('', ThemeLogoUploadType::class, $theme);
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $tenantContext = $this->tenantContext;
-      $currentTenant = $tenantContext->getTenant();
+        $tenantContext = $this->tenantContext;
+        $currentTenant = $tenantContext->getTenant();
 
-      try {
+        try {
         $settingsManager = $this->settingsManager;
         $setting = $settingsManager->get($type, ScopeContextInterface::SCOPE_THEME, $currentTenant);
         $theme->setLogoPath($setting);
         $themeLogoUploader = $this->themeLogoUploader;
         $themeLogoUploader->upload($theme);
-      } catch (\Exception $e) {
+        } catch (\Exception $e) {
         return new SingleResourceResponse(['message' => 'Could not upload logo.'], new ResponseContext(400));
-      }
+        }
 
-      $settingsManager = $this->settingsManager;
-      $setting = $settingsManager->set($type, $theme->getLogoPath(), ScopeContextInterface::SCOPE_THEME, $currentTenant);
+        $settingsManager = $this->settingsManager;
+        $setting = $settingsManager->set($type, $theme->getLogoPath(), ScopeContextInterface::SCOPE_THEME, $currentTenant);
 
-      return new SingleResourceResponse($setting, new ResponseContext(201));
+        return new SingleResourceResponse($setting, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/theme/settings/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_theme_settings_list")
-   */
-  public function listSettingsAction(): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/theme/settings/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_theme_settings_list")
+     */
+    public function listSettingsAction(): SingleResourceResponseInterface {
     $themeContext = $this->tenantAwareThemeContext;
 
     if (null === $themeContext->getTheme()) {
-      throw new \LogicException('Theme is not set!');
+        throw new \LogicException('Theme is not set!');
     }
 
     $tenantContext = $this->tenantContext;
@@ -112,5 +112,5 @@ 
     $settings = $settingsManager->getByScopeAndOwner(ScopeContextInterface::SCOPE_THEME, $tenantContext->getTenant());
 
     return new SingleResourceResponse($settings);
-  }
+    }
 }

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

Indentation +84 added lines, -84 removed lines

@@ -27,138 +27,138 @@
 use Symfony\Component\Routing\Annotation\Route;
 
 class StaticThemeAssetsController extends Controller {
-  /**
-   * Directory with assets inside theme.
-   */
-  const ASSETS_DIRECTORY = 'public';
-
-  private TenantAwareThemeContextInterface $tenantAwareThemeContext;
-  private ThemeLoaderInterface $themeLoader;
-  private ThemeRepositoryInterface $themeRepository;
-
-  /**
-   * @param TenantAwareThemeContextInterface $tenantAwareThemeContext
-   * @param ThemeLoaderInterface $themeLoader
-   * @param ThemeRepositoryInterface $themeRepository
-   */
-  public function __construct(TenantAwareThemeContextInterface $tenantAwareThemeContext,
-                              ThemeLoaderInterface             $themeLoader,
-                              ThemeRepositoryInterface         $themeRepository) {
+    /**
+     * Directory with assets inside theme.
+     */
+    const ASSETS_DIRECTORY = 'public';
+
+    private TenantAwareThemeContextInterface $tenantAwareThemeContext;
+    private ThemeLoaderInterface $themeLoader;
+    private ThemeRepositoryInterface $themeRepository;
+
+    /**
+     * @param TenantAwareThemeContextInterface $tenantAwareThemeContext
+     * @param ThemeLoaderInterface $themeLoader
+     * @param ThemeRepositoryInterface $themeRepository
+     */
+    public function __construct(TenantAwareThemeContextInterface $tenantAwareThemeContext,
+                                ThemeLoaderInterface             $themeLoader,
+                                ThemeRepositoryInterface         $themeRepository) {
     $this->tenantAwareThemeContext = $tenantAwareThemeContext;
     $this->themeLoader = $themeLoader;
     $this->themeRepository = $themeRepository;
-  }
+    }
 
 
-  /**
-   * @Route("/{fileName}.{fileExtension}", methods={"GET"}, name="static_theme_assets_root", requirements={"fileName": "sw|manifest|favicon|ads|OneSignalSDKWorker|OneSignalSDKUpdaterWorker|amp-web-push-helper-frame|amp-web-push-permission-dialog"})
-   * @Route("/public-{fileName}.{fileExtension}", methods={"GET"}, name="static_theme_assets_root_public", requirements={"fileName"=".+"})
-   * @Route("/public/{fileName}.{fileExtension}", methods={"GET"}, name="static_theme_assets_public", requirements={"fileName"=".+"})
-   */
-  public function rootAction($fileName, $fileExtension, ThemeHierarchyProviderInterface $themeHierarchyProvider) {
+    /**
+     * @Route("/{fileName}.{fileExtension}", methods={"GET"}, name="static_theme_assets_root", requirements={"fileName": "sw|manifest|favicon|ads|OneSignalSDKWorker|OneSignalSDKUpdaterWorker|amp-web-push-helper-frame|amp-web-push-permission-dialog"})
+     * @Route("/public-{fileName}.{fileExtension}", methods={"GET"}, name="static_theme_assets_root_public", requirements={"fileName"=".+"})
+     * @Route("/public/{fileName}.{fileExtension}", methods={"GET"}, name="static_theme_assets_public", requirements={"fileName"=".+"})
+     */
+    public function rootAction($fileName, $fileExtension, ThemeHierarchyProviderInterface $themeHierarchyProvider) {
     $themes = $themeHierarchyProvider->getThemeHierarchy(
         $this->tenantAwareThemeContext->getTheme()
     );
 
     $fileName = (null === $fileExtension) ? basename($fileName) : $fileName . '.' . $fileExtension;
     foreach ($themes as $theme) {
-      $filePath = $theme->getPath() . '/' . self::ASSETS_DIRECTORY . '/' . $fileName;
-      if (null !== $response = $this->handleFileLoading($filePath)) {
+        $filePath = $theme->getPath() . '/' . self::ASSETS_DIRECTORY . '/' . $fileName;
+        if (null !== $response = $this->handleFileLoading($filePath)) {
         return $response;
-      }
+        }
     }
 
     throw new NotFoundHttpException('File was not found.');
-  }
-
-  /**
-   * @Route("/themes/{type}/{themeName}/screenshots/{fileName}", methods={"GET"}, name="static_theme_screenshots", requirements={
-   *     "type": "organization|tenant"
-   * })
-   */
-  public function screenshotsAction(string $type, string $themeName, $fileName) {
+    }
+
+    /**
+     * @Route("/themes/{type}/{themeName}/screenshots/{fileName}", methods={"GET"}, name="static_theme_screenshots", requirements={
+     *     "type": "organization|tenant"
+     * })
+     */
+    public function screenshotsAction(string $type, string $themeName, $fileName) {
     if ('organization' === $type) {
-      $theme = $this->loadOrganizationTheme(str_replace('__', '/', $themeName));
+        $theme = $this->loadOrganizationTheme(str_replace('__', '/', $themeName));
     } elseif ('tenant' === $type) {
-      $theme = $this->loadTenantTheme(str_replace('__', '/', $themeName));
+        $theme = $this->loadTenantTheme(str_replace('__', '/', $themeName));
     } else {
-      throw new NotFoundHttpException('File was not found.');
+        throw new NotFoundHttpException('File was not found.');
     }
 
     $filePath = $theme->getPath() . '/screenshots/' . $fileName;
     if (null !== $response = $this->handleFileLoading($filePath)) {
-      return $response;
+        return $response;
     }
 
     throw new NotFoundHttpException('File was not found.');
-  }
-
-  /**
-   * @param $filePath
-   *
-   * @return Response
-   */
-  private function handleFileLoading($filePath) {
+    }
+
+    /**
+     * @param $filePath
+     *
+     * @return Response
+     */
+    private function handleFileLoading($filePath) {
     if (file_exists($filePath)) {
-      $response = new Response(file_get_contents($filePath));
-      $disposition = $response->headers->makeDisposition(
-          ResponseHeaderBag::DISPOSITION_INLINE,
-          basename($filePath)
-      );
-      $response->headers->set('Content-Disposition', $disposition);
-
-      try {
+        $response = new Response(file_get_contents($filePath));
+        $disposition = $response->headers->makeDisposition(
+            ResponseHeaderBag::DISPOSITION_INLINE,
+            basename($filePath)
+        );
+        $response->headers->set('Content-Disposition', $disposition);
+
+        try {
         $type = new Mime(new Read($filePath));
         $mime = str_replace('/x-', '/', Mime::getMimeFromExtension($type->getExtension()));
-      } catch (\Exception $e) {
+        } catch (\Exception $e) {
         $mime = 'text/plain';
-      }
+        }
 
-      $response->headers->set('Content-Type', $mime);
-      $response->setStatusCode(Response::HTTP_OK);
-      $response->setPublic();
-      $response->setMaxAge(3600);
-      $response->setSharedMaxAge(7200);
+        $response->headers->set('Content-Type', $mime);
+        $response->setStatusCode(Response::HTTP_OK);
+        $response->setPublic();
+        $response->setMaxAge(3600);
+        $response->setSharedMaxAge(7200);
 
-      return $response;
+        return $response;
+    }
     }
-  }
 
-  /**
-   * @return mixed
-   */
-  private function loadOrganizationTheme(string $themeName) {
+    /**
+     * @return mixed
+     */
+    private function loadOrganizationTheme(string $themeName) {
     $loadedThemes = $this->themeLoader->load();
 
     return $this->filterThemes($loadedThemes, $themeName);
-  }
+    }
 
-  /**
-   * @return mixed
-   */
-  private function loadTenantTheme(string $themeName) {
+    /**
+     * @return mixed
+     */
+    private function loadTenantTheme(string $themeName) {
     $loadedThemes = $this->themeRepository->findAll();
 
     return $this->filterThemes($loadedThemes, $themeName);
-  }
-
-  /**
-   * @param array $loadedThemes
-   *
-   * @return mixed
-   */
-  private function filterThemes($loadedThemes, string $themeName) {
+    }
+
+    /**
+     * @param array $loadedThemes
+     *
+     * @return mixed
+     */
+    private function filterThemes($loadedThemes, string $themeName) {
     $themes = array_filter(
         $loadedThemes,
         function ($element) use (&$themeName) {
-          return $element->getName() === $themeName;
+            return $element->getName() === $themeName;
         }
     );
 
     if (0 === count($themes)) {
-      throw new NotFoundHttpException(sprintf('Theme with name "%s" was not found in organization themes.', $themeName));
+        throw new NotFoundHttpException(sprintf('Theme with name "%s" was not found in organization themes.', $themeName));
     }
 
     return reset($themes);
-  }
+    }
 }

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

Indentation +96 added lines, -96 removed lines

@@ -42,30 +42,30 @@ 
 
 class TenantController extends FOSRestController {
 
-  private CachedTenantContextInterface $cachedTenantContext;
-  private EventDispatcherInterface $eventDispatcher;
-  private FormFactoryInterface $formFactory;
-  private TenantRepositoryInterface $tenantRepository;
-  private EntityManagerInterface $entityManager;
-  private SettingsManagerInterface $settingsManager;
-  private TenantFactoryInterface $tenantFactory;
-  private ArticleRepositoryInterface $articleRepository;
-
-  /**
-   * @param CachedTenantContextInterface $cachedTenantContext
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param FormFactoryInterface $formFactory
-   * @param TenantRepositoryInterface $tenantRepository
-   * @param EntityManagerInterface $entityManager
-   * @param SettingsManagerInterface $settingsManager
-   * @param TenantFactoryInterface $tenantFactory
-   * @param ArticleRepositoryInterface $articleRepository
-   */
-  public function __construct(CachedTenantContextInterface $cachedTenantContext,
-                              EventDispatcherInterface     $eventDispatcher, FormFactoryInterface $formFactory,
-                              TenantRepositoryInterface    $tenantRepository, EntityManagerInterface $entityManager,
-                              SettingsManagerInterface     $settingsManager, TenantFactoryInterface $tenantFactory,
-                              ArticleRepositoryInterface   $articleRepository) {
+    private CachedTenantContextInterface $cachedTenantContext;
+    private EventDispatcherInterface $eventDispatcher;
+    private FormFactoryInterface $formFactory;
+    private TenantRepositoryInterface $tenantRepository;
+    private EntityManagerInterface $entityManager;
+    private SettingsManagerInterface $settingsManager;
+    private TenantFactoryInterface $tenantFactory;
+    private ArticleRepositoryInterface $articleRepository;
+
+    /**
+     * @param CachedTenantContextInterface $cachedTenantContext
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param FormFactoryInterface $formFactory
+     * @param TenantRepositoryInterface $tenantRepository
+     * @param EntityManagerInterface $entityManager
+     * @param SettingsManagerInterface $settingsManager
+     * @param TenantFactoryInterface $tenantFactory
+     * @param ArticleRepositoryInterface $articleRepository
+     */
+    public function __construct(CachedTenantContextInterface $cachedTenantContext,
+                                EventDispatcherInterface     $eventDispatcher, FormFactoryInterface $formFactory,
+                                TenantRepositoryInterface    $tenantRepository, EntityManagerInterface $entityManager,
+                                SettingsManagerInterface     $settingsManager, TenantFactoryInterface $tenantFactory,
+                                ArticleRepositoryInterface   $articleRepository) {
     $this->cachedTenantContext = $cachedTenantContext;
     $this->eventDispatcher = $eventDispatcher;
     $this->formFactory = $formFactory;
@@ -74,32 +74,32 @@ 
     $this->settingsManager = $settingsManager;
     $this->tenantFactory = $tenantFactory;
     $this->articleRepository = $articleRepository;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/tenants/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_tenants")
-   */
-  public function listAction(Request $request) {
+    /**
+     * @Route("/api/{version}/tenants/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_tenants")
+     */
+    public function listAction(Request $request) {
     $tenants = $this->getTenantRepository()
         ->getPaginatedByCriteria($this->eventDispatcher, new Criteria(), $request->query->all('sorting'), new PaginationData($request));
     $responseContext = new ResponseContext();
     $responseContext->setSerializationGroups(['Default', 'api', 'details_api']);
 
     return new ResourcesListResponse($tenants, $responseContext);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/tenants/{code}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_tenant", requirements={"code"="[a-z0-9]+"})
-   */
-  public function getAction($code) {
+    /**
+     * @Route("/api/{version}/tenants/{code}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_tenant", requirements={"code"="[a-z0-9]+"})
+     */
+    public function getAction($code) {
     return new SingleResourceResponse($this->findOr404($code));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/tenants/{code}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_core_delete_tenant", requirements={"code"="[a-z0-9]+"})
-   */
-  public function deleteAction(Request $request, $code) {
+    /**
+     * @Route("/api/{version}/tenants/{code}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_core_delete_tenant", requirements={"code"="[a-z0-9]+"})
+     */
+    public function deleteAction(Request $request, $code) {
     $tenantContext = $this->cachedTenantContext;
     $eventDispatcher = $this->eventDispatcher;
     $currentTenant = $tenantContext->getTenant();
@@ -109,13 +109,13 @@ 
 
     $forceRemove = $request->query->has('force');
     if (!$forceRemove) {
-      $tenantContext->setTenant($tenant);
-      $eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_ENABLE);
-      $articlesRepository = $this->articleRepository;
-      $existingArticles = $articlesRepository->findAll();
-      if (0 !== \count($existingArticles)) {
+        $tenantContext->setTenant($tenant);
+        $eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_ENABLE);
+        $articlesRepository = $this->articleRepository;
+        $existingArticles = $articlesRepository->findAll();
+        if (0 !== \count($existingArticles)) {
         throw new ConflictHttpException('This tenant have articles attached to it.');
-      }
+        }
     }
 
     $repository->remove($tenant);
@@ -124,12 +124,12 @@ 
     $eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_ENABLE);
 
     return new SingleResourceResponse(null, new ResponseContext(204));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/tenants/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_create_tenant")
-   */
-  public function createAction(Request $request) {
+    /**
+     * @Route("/api/{version}/tenants/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_create_tenant")
+     */
+    public function createAction(Request $request) {
     $tenant = $this->tenantFactory->create();
     $tenantContext = $this->cachedTenantContext;
     $tenantObjectManager = $this->entityManager;
@@ -137,83 +137,83 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->ensureTenantDontExists($tenant->getDomainName(), $tenant->getSubdomain());
-      if (null === $tenant->getOrganization()) {
+        $this->ensureTenantDontExists($tenant->getDomainName(), $tenant->getSubdomain());
+        if (null === $tenant->getOrganization()) {
         $organization = $tenantObjectManager->merge($tenantContext->getTenant()->getOrganization());
         $tenant->setOrganization($organization);
-      }
-      $this->getTenantRepository()->add($tenant);
+        }
+        $this->getTenantRepository()->add($tenant);
 
-      return new SingleResourceResponse($tenant, new ResponseContext(201));
+        return new SingleResourceResponse($tenant, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/tenants/{code}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_tenant", requirements={"code"="[a-z0-9]+"})
-   */
-  public function updateAction(Request $request, $code) {
+    /**
+     * @Route("/api/{version}/tenants/{code}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_tenant", requirements={"code"="[a-z0-9]+"})
+     */
+    public function updateAction(Request $request, $code) {
     $tenant = $this->findOr404($code);
     $form = $this->formFactory->createNamed('', TenantType::class, $tenant, ['method' => $request->getMethod()]);
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $formData = $request->request->all();
-      $tenant->setUpdatedAt(new DateTime('now'));
-      $this->entityManager->flush();
+        $formData = $request->request->all();
+        $tenant->setUpdatedAt(new DateTime('now'));
+        $this->entityManager->flush();
 
-      $tenantContext = $this->cachedTenantContext;
-      $tenantContext->setTenant($tenant);
+        $tenantContext = $this->cachedTenantContext;
+        $tenantContext->setTenant($tenant);
 
-      $settingsManager = $this->settingsManager;
+        $settingsManager = $this->settingsManager;
 
-      if (array_key_exists('fbiaEnabled', $formData)) {
+        if (array_key_exists('fbiaEnabled', $formData)) {
         $settingsManager->set('fbia_enabled', (bool)$formData['fbiaEnabled'], ScopeContextInterface::SCOPE_TENANT, $tenant);
-      }
-      if (array_key_exists('paywallEnabled', $formData)) {
+        }
+        if (array_key_exists('paywallEnabled', $formData)) {
         $settingsManager->set('paywall_enabled', (bool)$formData['paywallEnabled'], ScopeContextInterface::SCOPE_TENANT, $tenant);
-      }
-      if (array_key_exists('defaultLanguage', $formData)) {
+        }
+        if (array_key_exists('defaultLanguage', $formData)) {
         $settingsManager->set('default_language', $formData['defaultLanguage'], ScopeContextInterface::SCOPE_TENANT, $tenant);
-      }
+        }
 
-      return new SingleResourceResponse($tenant);
+        return new SingleResourceResponse($tenant);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
-
-  /**
-   * @param string $code
-   *
-   * @return mixed|TenantInterface|null
-   * @throws NotFoundHttpException
-   *
-   */
-  private function findOr404($code) {
+    }
+
+    /**
+     * @param string $code
+     *
+     * @return mixed|TenantInterface|null
+     * @throws NotFoundHttpException
+     *
+     */
+    private function findOr404($code) {
     if (null === $tenant = $this->getTenantRepository()->findOneByCode($code)) {
-      throw $this->createNotFoundException(sprintf('Tenant with code "%s" was not found.', $code));
+        throw $this->createNotFoundException(sprintf('Tenant with code "%s" was not found.', $code));
     }
 
     return $tenant;
-  }
+    }
 
-  /**
-   * @return mixed|TenantInterface|null
-   */
-  private function ensureTenantDontExists(string $domain, string $subdomain = null) {
+    /**
+     * @return mixed|TenantInterface|null
+     */
+    private function ensureTenantDontExists(string $domain, string $subdomain = null) {
     if (null !== $tenant = $this->getTenantRepository()->findOneBySubdomainAndDomain($subdomain, $domain)) {
-      throw new ConflictHttpException('Tenant for this host already exists.');
+        throw new ConflictHttpException('Tenant for this host already exists.');
     }
 
     return $tenant;
-  }
+    }
 
-  /**
-   * @return object|\SWP\Bundle\MultiTenancyBundle\Doctrine\ORM\TenantRepository
-   */
-  private function getTenantRepository() {
+    /**
+     * @return object|\SWP\Bundle\MultiTenancyBundle\Doctrine\ORM\TenantRepository
+     */
+    private function getTenantRepository() {
     return $this->tenantRepository;
-  }
+    }
 }

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

Indentation +46 added lines, -46 removed lines

@@ -35,39 +35,39 @@ 
 
 class PublishDestinationController extends Controller {
 
-  private FormFactoryInterface $formFactory;
-  private EventDispatcherInterface $eventDispatcher;
-  private CachedTenantContextInterface $cachedTenantContext;
-  private RepositoryInterface $publishDestinationRepository;
-  private EntityManagerInterface $entityManager;
-  private FactoryInterface $publishDestinationFactory;
-
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param CachedTenantContextInterface $cachedTenantContext
-   * @param RepositoryInterface $publishDestinationRepository
-   * @param EntityManagerInterface $entityManager
-   * @param FactoryInterface $publishDestinationFactory
-   */
-  public function __construct(FormFactoryInterface         $formFactory,
-                              EventDispatcherInterface     $eventDispatcher,
-                              CachedTenantContextInterface $cachedTenantContext,
-                              RepositoryInterface          $publishDestinationRepository,
-                              EntityManagerInterface       $entityManager,
-                              FactoryInterface             $publishDestinationFactory) {
+    private FormFactoryInterface $formFactory;
+    private EventDispatcherInterface $eventDispatcher;
+    private CachedTenantContextInterface $cachedTenantContext;
+    private RepositoryInterface $publishDestinationRepository;
+    private EntityManagerInterface $entityManager;
+    private FactoryInterface $publishDestinationFactory;
+
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param CachedTenantContextInterface $cachedTenantContext
+     * @param RepositoryInterface $publishDestinationRepository
+     * @param EntityManagerInterface $entityManager
+     * @param FactoryInterface $publishDestinationFactory
+     */
+    public function __construct(FormFactoryInterface         $formFactory,
+                                EventDispatcherInterface     $eventDispatcher,
+                                CachedTenantContextInterface $cachedTenantContext,
+                                RepositoryInterface          $publishDestinationRepository,
+                                EntityManagerInterface       $entityManager,
+                                FactoryInterface             $publishDestinationFactory) {
     $this->formFactory = $formFactory;
     $this->eventDispatcher = $eventDispatcher;
     $this->cachedTenantContext = $cachedTenantContext;
     $this->publishDestinationRepository = $publishDestinationRepository;
     $this->entityManager = $entityManager;
     $this->publishDestinationFactory = $publishDestinationFactory;
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/organization/destinations/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_publishing_destination_create")
-   */
-  public function createAction(Request $request): SingleResourceResponse {
+    /**
+     * @Route("/api/{version}/organization/destinations/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_publishing_destination_create")
+     */
+    public function createAction(Request $request): SingleResourceResponse {
     $tenantContext = $this->cachedTenantContext;
 
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
@@ -78,28 +78,28 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $repository = $this->publishDestinationRepository;
-      /** @var PublishDestinationInterface $publishDestination */
-      $publishDestination = $repository->findOneByTenant($destination->getTenant());
-      if (null !== $publishDestination) {
+        $repository = $this->publishDestinationRepository;
+        /** @var PublishDestinationInterface $publishDestination */
+        $publishDestination = $repository->findOneByTenant($destination->getTenant());
+        if (null !== $publishDestination) {
         $repository->remove($publishDestination);
-      }
+        }
 
-      $currentOrganization->addPublishDestination($destination);
-      $this->entityManager->flush();
+        $currentOrganization->addPublishDestination($destination);
+        $this->entityManager->flush();
 
-      return new SingleResourceResponse($destination, new ResponseContext(200));
+        return new SingleResourceResponse($destination, new ResponseContext(200));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
-
-  /**
-   * @Route("/api/{version}/organization/destinations/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_publishing_destination_update", requirements={"id"="\d+"})
-   * @ParamConverter("publishDestination", class="SWP\Bundle\CoreBundle\Model\PublishDestination")
-   */
-  public function updateAction(Request                     $request,
-                               PublishDestinationInterface $publishDestination): SingleResourceResponse {
+    }
+
+    /**
+     * @Route("/api/{version}/organization/destinations/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_publishing_destination_update", requirements={"id"="\d+"})
+     * @ParamConverter("publishDestination", class="SWP\Bundle\CoreBundle\Model\PublishDestination")
+     */
+    public function updateAction(Request                     $request,
+                                PublishDestinationInterface $publishDestination): SingleResourceResponse {
     $objectManager = $this->entityManager;
 
     $form = $this->formFactory->createNamed('', PublishDestinationType::class, $publishDestination, [
@@ -108,12 +108,12 @@ 
 
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $objectManager->flush();
-      $objectManager->refresh($publishDestination);
+        $objectManager->flush();
+        $objectManager->refresh($publishDestination);
 
-      return new SingleResourceResponse($publishDestination);
+        return new SingleResourceResponse($publishDestination);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 }

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

Indentation +42 added lines, -42 removed lines

@@ -35,39 +35,39 @@ 
 
 class FbPageController extends AbstractController {
 
-  private FormFactoryInterface $formFactory;
-  private RepositoryInterface $facebookInstantArticlesFeedRepository;
-  private RepositoryInterface $facebookPageRepository;
-  private FactoryInterface $facebookPageFactory;
-  private EventDispatcherInterface $eventDispatcher;
-
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param RepositoryInterface $facebookInstantArticlesFeedRepository
-   * @param RepositoryInterface $facebookPageRepository
-   * @param FactoryInterface $facebookPageFactory
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(FormFactoryInterface $formFactory,
-                              RepositoryInterface  $facebookInstantArticlesFeedRepository,
-                              RepositoryInterface  $facebookPageRepository, FactoryInterface $facebookPageFactory,
-                              EventDispatcherInterface      $eventDispatcher) {
+    private FormFactoryInterface $formFactory;
+    private RepositoryInterface $facebookInstantArticlesFeedRepository;
+    private RepositoryInterface $facebookPageRepository;
+    private FactoryInterface $facebookPageFactory;
+    private EventDispatcherInterface $eventDispatcher;
+
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param RepositoryInterface $facebookInstantArticlesFeedRepository
+     * @param RepositoryInterface $facebookPageRepository
+     * @param FactoryInterface $facebookPageFactory
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(FormFactoryInterface $formFactory,
+                                RepositoryInterface  $facebookInstantArticlesFeedRepository,
+                                RepositoryInterface  $facebookPageRepository, FactoryInterface $facebookPageFactory,
+                                EventDispatcherInterface      $eventDispatcher) {
     $this->formFactory = $formFactory;
     $this->facebookInstantArticlesFeedRepository = $facebookInstantArticlesFeedRepository;
     $this->facebookPageRepository = $facebookPageRepository;
     $this->facebookPageFactory = $facebookPageFactory;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/facebook/pages/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_facebook_pages")
-   */
-  public function listAction(Request $request) {
+    /**
+     * @Route("/api/{version}/facebook/pages/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_list_facebook_pages")
+     */
+    public function listAction(Request $request) {
     $repository = $this->facebookPageRepository;
     $sort = $request->query->all('sorting');
     if (empty($sort)) {
-      $sort = ['id' => 'asc'];
+        $sort = ['id' => 'asc'];
     }
     $items = $repository->getPaginatedByCriteria(
         $this->eventDispatcher,
@@ -77,51 +77,51 @@ 
     );
 
     return new ResourcesListResponse($items);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/facebook/pages/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_create_facebook_pages")
-   */
-  public function createAction(Request $request) {
+    /**
+     * @Route("/api/{version}/facebook/pages/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_create_facebook_pages")
+     */
+    public function createAction(Request $request) {
     /* @var FacebookPage $feed */
     $page = $this->facebookPageFactory->create();
     $form = $this->formFactory->createNamed('', FacebookPageType::class, $page, ['method' => $request->getMethod()]);
 
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->checkIfPageExists($page);
-      $this->facebookPageRepository->add($page);
+        $this->checkIfPageExists($page);
+        $this->facebookPageRepository->add($page);
 
-      return new SingleResourceResponse($page, new ResponseContext(201));
+        return new SingleResourceResponse($page, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/facebook/pages/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_delete_facebook_pages")
-   */
-  public function deleteAction(int $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/facebook/pages/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_delete_facebook_pages")
+     */
+    public function deleteAction(int $id): SingleResourceResponseInterface {
     $repository = $this->facebookPageRepository;
     if (null === $page = $this->facebookPageRepository->findOneBy(['id' => $id])) {
-      throw new NotFoundHttpException('There is no Page with provided id!');
+        throw new NotFoundHttpException('There is no Page with provided id!');
     }
 
     if (null !== $feed = $this->facebookInstantArticlesFeedRepository->findOneBy(['facebookPage' => $id])) {
-      throw new ConflictHttpException(sprintf('This Page is used by Instant Articles Feed with id: %s!', $feed->getId()));
+        throw new ConflictHttpException(sprintf('This Page is used by Instant Articles Feed with id: %s!', $feed->getId()));
     }
 
     $repository->remove($page);
 
     return new SingleResourceResponse(null, new ResponseContext(204));
-  }
+    }
 
-  private function checkIfPageExists(PageInterface $page): void {
+    private function checkIfPageExists(PageInterface $page): void {
     if (null !== $this->facebookPageRepository->findOneBy([
             'pageId' => $page->getPageId(),
         ])
     ) {
-      throw new ConflictHttpException('This Page already exists!');
+        throw new ConflictHttpException('This Page already exists!');
+    }
     }
-  }
 }

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

Indentation +93 added lines, -93 removed lines

@@ -43,32 +43,32 @@ 
 
 class ContentListController extends AbstractController {
 
-  private ContentListRepositoryInterface $contentListRepository;
-  private ContentListItemRepositoryInterface $contentListItemRepository;
-  private ContentListServiceInterface $contentListService;
-  private FormFactoryInterface $formFactory;
-  private EntityManagerInterface $entityManager;
-  private EventDispatcherInterface $eventDispatcher;
-  private FactoryInterface $factory;
-
-  /**
-   * @param ContentListRepositoryInterface $contentListRepository
-   * @param ContentListItemRepositoryInterface $contentListItemRepository
-   * @param ContentListServiceInterface $contentListService
-   * @param FormFactoryInterface $formFactory
-   * @param EntityManagerInterface $entityManager
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param FactoryInterface $factory
-   */
-  public function __construct(
-      ContentListRepositoryInterface     $contentListRepository,
-      ContentListItemRepositoryInterface $contentListItemRepository,
-      ContentListServiceInterface        $contentListService,
-      FormFactoryInterface               $formFactory,
-      EntityManagerInterface             $entityManager,
-      EventDispatcherInterface           $eventDispatcher,
-      FactoryInterface                   $factory
-  ) {
+    private ContentListRepositoryInterface $contentListRepository;
+    private ContentListItemRepositoryInterface $contentListItemRepository;
+    private ContentListServiceInterface $contentListService;
+    private FormFactoryInterface $formFactory;
+    private EntityManagerInterface $entityManager;
+    private EventDispatcherInterface $eventDispatcher;
+    private FactoryInterface $factory;
+
+    /**
+     * @param ContentListRepositoryInterface $contentListRepository
+     * @param ContentListItemRepositoryInterface $contentListItemRepository
+     * @param ContentListServiceInterface $contentListService
+     * @param FormFactoryInterface $formFactory
+     * @param EntityManagerInterface $entityManager
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param FactoryInterface $factory
+     */
+    public function __construct(
+        ContentListRepositoryInterface     $contentListRepository,
+        ContentListItemRepositoryInterface $contentListItemRepository,
+        ContentListServiceInterface        $contentListService,
+        FormFactoryInterface               $formFactory,
+        EntityManagerInterface             $entityManager,
+        EventDispatcherInterface           $eventDispatcher,
+        FactoryInterface                   $factory
+    ) {
     $this->contentListRepository = $contentListRepository;
     $this->contentListItemRepository = $contentListItemRepository;
     $this->contentListService = $contentListService;
@@ -76,28 +76,28 @@ 
     $this->entityManager = $entityManager;
     $this->eventDispatcher = $eventDispatcher;
     $this->factory = $factory;
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_list_lists")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_list_lists")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $lists = $this->contentListRepository->getPaginatedByCriteria($this->eventDispatcher, new Criteria(), $request->query->all('sorting'), new PaginationData($request));
 
     return new ResourcesListResponse($lists);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_show_lists", requirements={"id"="\d+"})
-   */
-  public function getAction($id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_content_show_lists", requirements={"id"="\d+"})
+     */
+    public function getAction($id): SingleResourceResponseInterface {
     return new SingleResourceResponse($this->findOr404($id));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_content_create_lists")
-   */
-  public function createAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_content_create_lists")
+     */
+    public function createAction(Request $request): SingleResourceResponseInterface {
     /* @var ContentListInterface $contentList */
     $contentList = $this->factory->create();
     $form = $this->formFactory->createNamed('', ContentListType::class, $contentList, ['method' => $request->getMethod()]);
@@ -106,18 +106,18 @@ 
     $this->ensureContentListExists($contentList->getName());
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->contentListRepository->add($contentList);
+        $this->contentListRepository->add($contentList);
 
-      return new SingleResourceResponse($contentList, new ResponseContext(201));
+        return new SingleResourceResponse($contentList, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_content_update_lists", requirements={"id"="\d+"})
-   */
-  public function updateAction(Request $request, int $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_content_update_lists", requirements={"id"="\d+"})
+     */
+    public function updateAction(Request $request, int $id): SingleResourceResponseInterface {
     $objectManager = $this->entityManager;
     /** @var ContentListInterface $contentList */
     $contentList = $this->findOr404($id);
@@ -128,51 +128,51 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->eventDispatcher->dispatch(
-          new GenericEvent($contentList, ['filters' => $filters, 'previousLimit' => $listLimit]),
-          ContentListEvents::LIST_CRITERIA_CHANGE
-      );
+        $this->eventDispatcher->dispatch(
+            new GenericEvent($contentList, ['filters' => $filters, 'previousLimit' => $listLimit]),
+            ContentListEvents::LIST_CRITERIA_CHANGE
+        );
 
-      $objectManager->flush();
+        $objectManager->flush();
 
-      return new SingleResourceResponse($contentList);
+        return new SingleResourceResponse($contentList);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_content_delete_lists", requirements={"id"="\d+"})
-   */
-  public function deleteAction($id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_content_delete_lists", requirements={"id"="\d+"})
+     */
+    public function deleteAction($id): SingleResourceResponseInterface {
     $repository = $this->contentListRepository;
     $contentList = $this->findOr404($id);
 
     $repository->remove($contentList);
 
     return new SingleResourceResponse(null, new ResponseContext(204));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/lists/{id}", requirements={"id"="\w+"}, defaults={"version"="v2"}, methods={"LINK","UNLINK"}, name="swp_api_content_list_link_unlink")
-   */
-  public function linkUnlinkToContentListAction(Request $request, string $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/content/lists/{id}", requirements={"id"="\w+"}, defaults={"version"="v2"}, methods={"LINK","UNLINK"}, name="swp_api_content_list_link_unlink")
+     */
+    public function linkUnlinkToContentListAction(Request $request, string $id): SingleResourceResponseInterface {
     $objectManager = $this->entityManager;
     /** @var ContentListInterface $contentList */
     $contentList = $this->findOr404($id);
 
     $matched = false;
     foreach ($request->attributes->get('links', []) as $key => $objectArray) {
-      if (!is_array($objectArray)) {
+        if (!is_array($objectArray)) {
         continue;
-      }
+        }
 
-      $object = $objectArray['object'];
-      if ($object instanceof Exception) {
+        $object = $objectArray['object'];
+        if ($object instanceof Exception) {
         throw $object;
-      }
+        }
 
-      if ($object instanceof ArticleInterface) {
+        if ($object instanceof ArticleInterface) {
         $contentListItem = $this->contentListItemRepository
             ->findOneBy([
                 'contentList' => $contentList,
@@ -180,59 +180,59 @@ 
             ]);
 
         if ('LINK' === $request->getMethod()) {
-          $position = 0;
-          if (count($notConvertedLinks = RequestParser::getNotConvertedLinks($request->attributes->get('links'))) > 0) {
+            $position = 0;
+            if (count($notConvertedLinks = RequestParser::getNotConvertedLinks($request->attributes->get('links'))) > 0) {
             foreach ($notConvertedLinks as $link) {
-              if (isset($link['resourceType']) && 'position' === $link['resourceType']) {
+                if (isset($link['resourceType']) && 'position' === $link['resourceType']) {
                 $position = $link['resource'];
-              }
+                }
+            }
             }
-          }
 
-          if (false === $position && $contentListItem) {
+            if (false === $position && $contentListItem) {
             throw new ConflictHttpException('This content is already linked to Content List');
-          }
+            }
 
-          if (!$contentListItem) {
+            if (!$contentListItem) {
             $contentListItem = $this->contentListService->addArticleToContentList($contentList, $object, $position);
             $objectManager->persist($contentListItem);
-          } else {
+            } else {
             $contentListItem->setPosition($position);
-          }
+            }
 
-          $objectManager->flush();
+            $objectManager->flush();
         } elseif ('UNLINK' === $request->getMethod()) {
-          if ($contentListItem->getContentList() !== $contentList) {
+            if ($contentListItem->getContentList() !== $contentList) {
             throw new ConflictHttpException('Content is not linked to content list');
-          }
-          $objectManager->remove($contentListItem);
+            }
+            $objectManager->remove($contentListItem);
         }
 
         $matched = true;
 
         break;
-      }
+        }
     }
     if (false === $matched) {
-      throw new NotFoundHttpException('Any supported link object was not found');
+        throw new NotFoundHttpException('Any supported link object was not found');
     }
 
     $objectManager->flush();
 
     return new SingleResourceResponse($contentList, new ResponseContext(201));
-  }
+    }
 
-  private function findOr404($id) {
+    private function findOr404($id) {
     if (null === $list = $this->contentListRepository->find($id)) {
-      throw new NotFoundHttpException(sprintf('Content list with id "%s" was not found.', $id));
+        throw new NotFoundHttpException(sprintf('Content list with id "%s" was not found.', $id));
     }
 
     return $list;
-  }
+    }
 
-  private function ensureContentListExists($name) {
+    private function ensureContentListExists($name) {
     if (null !== $this->contentListRepository->findOneBy(['name' => $name])) {
-      throw new ConflictHttpException(sprintf('Content list named "%s" already exists!', $name));
+        throw new ConflictHttpException(sprintf('Content list named "%s" already exists!', $name));
+    }
     }
-  }
 }

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

Indentation +21 added lines, -21 removed lines

@@ -27,51 +27,51 @@
 
 class RedirectingController extends AbstractController {
 
-  private RouterInterface $router;
-  private ArticleRepositoryInterface $articleRepository;
+    private RouterInterface $router;
+    private ArticleRepositoryInterface $articleRepository;
 
-  /**
-   * @param RouterInterface $router
-   * @param ArticleRepositoryInterface $articleRepository
-   */
-  public function __construct(RouterInterface $router, ArticleRepositoryInterface $articleRepository) {
+    /**
+     * @param RouterInterface $router
+     * @param ArticleRepositoryInterface $articleRepository
+     */
+    public function __construct(RouterInterface $router, ArticleRepositoryInterface $articleRepository) {
     $this->router = $router;
     $this->articleRepository = $articleRepository;
-  }
+    }
 
 
-  public function redirectBasedOnExtraDataAction(Request $request, string $key, string $value): RedirectResponse {
+    public function redirectBasedOnExtraDataAction(Request $request, string $key, string $value): RedirectResponse {
     try {
-      $existingArticle = $this->articleRepository->getArticleByExtraData($key, $value)->getQuery()->getOneOrNullResult();
-      if (null === $existingArticle) {
+        $existingArticle = $this->articleRepository->getArticleByExtraData($key, $value)->getQuery()->getOneOrNullResult();
+        if (null === $existingArticle) {
         $existingArticle = $this->articleRepository->getArticleByPackageExtraData($key, $value)->getQuery()->getOneOrNullResult();
-      }
+        }
     } catch (NonUniqueResultException $e) {
-      $existingArticle = null;
+        $existingArticle = null;
     }
 
     if (null === $existingArticle || null === $existingArticle->getRoute()) {
-      throw $this->createNotFoundException('Article with provided data was not found.');
+        throw $this->createNotFoundException('Article with provided data was not found.');
     }
 
     return $this->redirect($this->generateArticleUrl($request, $existingArticle), 301);
-  }
+    }
 
-  public function redirectBasedOnSlugAction(Request $request, string $slug): RedirectResponse {
+    public function redirectBasedOnSlugAction(Request $request, string $slug): RedirectResponse {
     $existingArticle = $this->articleRepository->findOneBySlug($slug);
     if (null === $existingArticle || null === $existingArticle->getRoute()) {
-      throw $this->createNotFoundException('Article not found.');
+        throw $this->createNotFoundException('Article not found.');
     }
 
     return $this->redirect($this->generateArticleUrl($request, $existingArticle), 301);
-  }
+    }
 
-  private function generateArticleUrl(Request $request, ArticleInterface $article): string {
+    private function generateArticleUrl(Request $request, ArticleInterface $article): string {
     $parameters = ['slug' => $article->getSlug()];
     if ($request->query->has('amp')) {
-      $parameters['amp'] = 1;
+        $parameters['amp'] = 1;
     }
 
     return $this->router->generate($article->getRoute(), $parameters, UrlGeneratorInterface::ABSOLUTE_URL);
-  }
+    }
 }

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

Indentation +24 added lines, -24 removed lines

@@ -28,40 +28,40 @@ 
 
 class DefaultController extends AbstractController {
 
-  private CachedTenantContextInterface $tenantContext;
-  private MetaFactoryInterface $metaFactory;
-  private Context $templateEngineContext;
-  private RouteFactoryInterface $routeFactory;
+    private CachedTenantContextInterface $tenantContext;
+    private MetaFactoryInterface $metaFactory;
+    private Context $templateEngineContext;
+    private RouteFactoryInterface $routeFactory;
 
-  public function __construct(
-      CachedTenantContextInterface $tenantContext,
-      MetaFactoryInterface         $metaFactory,
-      Context                      $templateEngineContext,
-      RouteFactoryInterface        $routeFactory
-  ) {
+    public function __construct(
+        CachedTenantContextInterface $tenantContext,
+        MetaFactoryInterface         $metaFactory,
+        Context                      $templateEngineContext,
+        RouteFactoryInterface        $routeFactory
+    ) {
     $this->tenantContext = $tenantContext;
     $this->metaFactory = $metaFactory;
     $this->templateEngineContext = $templateEngineContext;
     $this->routeFactory = $routeFactory;
-  }
+    }
 
-  /**
-   * @Route("/", methods={"GET","POST"}, name="homepage")
-   */
-  public function indexAction(Request $request): Response {
+    /**
+     * @Route("/", methods={"GET","POST"}, name="homepage")
+     */
+    public function indexAction(Request $request): Response {
     /** @var TenantInterface $currentTenant */
     $currentTenant = $this->tenantContext->getTenant();
     $route = $currentTenant->getHomepage();
 
     if (null === $route) {
-      /** @var RouteInterface $route */
-      $route = $this->routeFactory->create();
-      $route->setStaticPrefix('/');
-      $route->setName('Homepage');
-      $route->setType('content');
-      $route->setTemplateName('index.html.twig');
-      $route->setCacheTimeInSeconds(360);
-      $request->attributes->set(DynamicRouter::ROUTE_KEY, $route);
+        /** @var RouteInterface $route */
+        $route = $this->routeFactory->create();
+        $route->setStaticPrefix('/');
+        $route->setName('Homepage');
+        $route->setType('content');
+        $route->setTemplateName('index.html.twig');
+        $route->setCacheTimeInSeconds(360);
+        $request->attributes->set(DynamicRouter::ROUTE_KEY, $route);
     }
 
     $this->templateEngineContext->setCurrentPage($this->metaFactory->create($route));
@@ -70,5 +70,5 @@ 
     $response->headers->set('Content-Type', 'text/html; charset=UTF-8');
 
     return $this->render('index.html.twig', [], $response);
-  }
+    }
 }

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

Indentation +69 added lines, -69 removed lines

@@ -45,39 +45,39 @@ 
 
 class PackageController extends Controller {
 
-  private FormFactoryInterface $formFactory;
-  private EventDispatcherInterface $eventDispatcher;
-  private CachedTenantContextInterface $cachedTenantContext;
-  private ArticlePublisherInterface $articlePublisher;
-  private PersisterRegistry $elasticaPersisterRegistry;
-  private PackageRepositoryInterface $packageRepository;
-
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param CachedTenantContextInterface $cachedTenantContext
-   * @param ArticlePublisherInterface $articlePublisher
-   * @param PersisterRegistry $elasticaPersisterRegistry
-   * @param PackageRepositoryInterface $packageRepository
-   */
-  public function __construct(FormFactoryInterface         $formFactory, EventDispatcherInterface $eventDispatcher,
-                              CachedTenantContextInterface $cachedTenantContext,
-                              ArticlePublisherInterface    $articlePublisher,
-                              PersisterRegistry            $elasticaPersisterRegistry,
-                              PackageRepositoryInterface   $packageRepository) {
+    private FormFactoryInterface $formFactory;
+    private EventDispatcherInterface $eventDispatcher;
+    private CachedTenantContextInterface $cachedTenantContext;
+    private ArticlePublisherInterface $articlePublisher;
+    private PersisterRegistry $elasticaPersisterRegistry;
+    private PackageRepositoryInterface $packageRepository;
+
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param CachedTenantContextInterface $cachedTenantContext
+     * @param ArticlePublisherInterface $articlePublisher
+     * @param PersisterRegistry $elasticaPersisterRegistry
+     * @param PackageRepositoryInterface $packageRepository
+     */
+    public function __construct(FormFactoryInterface         $formFactory, EventDispatcherInterface $eventDispatcher,
+                                CachedTenantContextInterface $cachedTenantContext,
+                                ArticlePublisherInterface    $articlePublisher,
+                                PersisterRegistry            $elasticaPersisterRegistry,
+                                PackageRepositoryInterface   $packageRepository) {
     $this->formFactory = $formFactory;
     $this->eventDispatcher = $eventDispatcher;
     $this->cachedTenantContext = $cachedTenantContext;
     $this->articlePublisher = $articlePublisher;
     $this->elasticaPersisterRegistry = $elasticaPersisterRegistry;
     $this->packageRepository = $packageRepository;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/packages/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_packages")
-   */
-  public function listAction(Request $request) {
+    /**
+     * @Route("/api/{version}/packages/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_packages")
+     */
+    public function listAction(Request $request) {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $tenantContext = $this->cachedTenantContext;
 
@@ -88,19 +88,19 @@ 
         ]), $request->query->all('sorting'), new PaginationData($request));
 
     return new ResourcesListResponse($packages);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_show_package", requirements={"id"="\d+"})
-   */
-  public function getAction(int $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/packages/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_show_package", requirements={"id"="\d+"})
+     */
+    public function getAction(int $id): SingleResourceResponseInterface {
     return new SingleResourceResponse($this->findOr404($id));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/{id}/publish/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_publish_package", requirements={"id"="\d+"})
-   */
-  public function publishAction(Request $request, int $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/packages/{id}/publish/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_publish_package", requirements={"id"="\d+"})
+     */
+    public function publishAction(Request $request, int $id): SingleResourceResponseInterface {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     /** @var PackageInterface $package */
     $package = $this->findOr404($id);
@@ -109,74 +109,74 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->articlePublisher->publish($package, $form->getData());
-      $this->elasticaPersisterRegistry->getPersister('swp_package')->replaceOne($package);
+        $this->articlePublisher->publish($package, $form->getData());
+        $this->elasticaPersisterRegistry->getPersister('swp_package')->replaceOne($package);
 
-      return new SingleResourceResponse(null, new ResponseContext(201));
+        return new SingleResourceResponse(null, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/{id}/unpublish/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_unpublish_package", requirements={"id"="\d+"})
-   */
-  public function unpublishAction(Request $request, int $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/packages/{id}/unpublish/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_unpublish_package", requirements={"id"="\d+"})
+     */
+    public function unpublishAction(Request $request, int $id): SingleResourceResponseInterface {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $package = $this->findOr404($id);
     $form = $this->formFactory->createNamed('', UnpublishFromTenantsType::class, null, ['method' => $request->getMethod()]);
 
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $formData = $form->getData();
-      /** @var Collection $tenants */
-      $tenants = $formData['tenants'];
-      $this->articlePublisher->unpublish($package, $tenants->toArray());
+        $formData = $form->getData();
+        /** @var Collection $tenants */
+        $tenants = $formData['tenants'];
+        $this->articlePublisher->unpublish($package, $tenants->toArray());
 
-      return new SingleResourceResponse(null, new ResponseContext(200));
+        return new SingleResourceResponse(null, new ResponseContext(200));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
-
-  /**
-   * @Route("/api/{version}/packages/{id}/", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_package", requirements={"id"="\d+"})
-   *
-   * @return SingleResourceResponse
-   */
-  public function updateAction(Request $request, int $id) {
+    }
+
+    /**
+     * @Route("/api/{version}/packages/{id}/", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_package", requirements={"id"="\d+"})
+     *
+     * @return SingleResourceResponse
+     */
+    public function updateAction(Request $request, int $id) {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $package = $this->findOr404($id);
     $form = $this->formFactory->createNamed('', PackageType::class, $package, ['method' => $request->getMethod()]);
 
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      if (ContentInterface::STATUS_CANCELED === $package->getPubStatus()) {
+        if (ContentInterface::STATUS_CANCELED === $package->getPubStatus()) {
         $package->setStatus(ContentInterface::STATUS_CANCELED);
-      }
-      $this->getPackageRepository()->flush();
+        }
+        $this->getPackageRepository()->flush();
 
-      return new SingleResourceResponse($package, new ResponseContext(200));
+        return new SingleResourceResponse($package, new ResponseContext(200));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @return object|null
-   */
-  private function findOr404(int $id) {
+    /**
+     * @return object|null
+     */
+    private function findOr404(int $id) {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $tenantContext = $this->cachedTenantContext;
 
     if (null === $package = $this->getPackageRepository()->findOneBy(['id' => $id, 'organization' => $tenantContext->getTenant()->getOrganization()])) {
-      throw new NotFoundHttpException('Package was not found.');
+        throw new NotFoundHttpException('Package was not found.');
     }
 
     return $package;
-  }
+    }
 
-  protected function getPackageRepository() {
+    protected function getPackageRepository() {
     return $this->packageRepository;
-  }
+    }
 }