superdesk / web-publisher

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

Indentation +14 added lines, -14 removed lines

@@ -28,22 +28,22 @@ 
 
 class ArticleSourceController extends AbstractController {
 
-  private EntityRepository $entityRepository;
-  private EventDispatcherInterface $eventDispatcher;
-
-  /**
-   * @param EntityRepository $entityRepository
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(EntityRepository $entityRepository, EventDispatcherInterface $eventDispatcher) {
+    private EntityRepository $entityRepository;
+    private EventDispatcherInterface $eventDispatcher;
+
+    /**
+     * @param EntityRepository $entityRepository
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(EntityRepository $entityRepository, EventDispatcherInterface $eventDispatcher) {
     $this->entityRepository = $entityRepository;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/content/sources/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_article_sources")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/content/sources/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_article_sources")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $sorting = $request->query->all('sorting');
     $lists = $this->entityRepository->getPaginatedByCriteria(
         $this->eventDispatcher,
@@ -52,5 +52,5 @@ 
         new PaginationData($request));
 
     return new ResourcesListResponse($lists);
-  }
+    }
 }

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/RelatedArticleOrganizationController.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
    * @Route("/api/{version}/packages/{id}/related/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_packages_related_articles", requirements={"id"="\d+"})
    */
   public function getRelatedAction(string $id) {
-    $package = $this->findOr404((int)$id);
+    $package = $this->findOr404((int) $id);
 
     $relatedArticlesList = $this->getRelated($package);
 
@@ -88,7 +88,7 @@ 
   }
 
   private function getRelated(PackageInterface $package): RelatedArticleList {
-    $relatedItemsGroups = $package->getItems()->filter(static function ($group) {
+    $relatedItemsGroups = $package->getItems()->filter(static function($group) {
       return ItemInterface::TYPE_TEXT === $group->getType();
     });
 

Indentation +48 added lines, -48 removed lines

@@ -35,102 +35,102 @@
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class RelatedArticleOrganizationController extends Controller {
-  private EventDispatcherInterface $eventDispatcher;
-  private DataTransformerInterface $dataTransformer;
-  private CachedTenantContextInterface $cachedTenantContext;
-  private PackageRepositoryInterface $packageRepository;
-  private ArticleRepositoryInterface $articleRepository;
-  private TenantRepositoryInterface $tenantRepository;
-
-  /**
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param DataTransformerInterface $dataTransformer
-   * @param CachedTenantContextInterface $cachedTenantContext
-   * @param PackageRepositoryInterface $packageRepository
-   * @param ArticleRepositoryInterface $articleRepository
-   * @param TenantRepositoryInterface $tenantRepository
-   */
-  public function __construct(EventDispatcherInterface     $eventDispatcher,
-                              DataTransformerInterface     $dataTransformer,
-                              CachedTenantContextInterface $cachedTenantContext,
-                              PackageRepositoryInterface   $packageRepository,
-                              ArticleRepositoryInterface   $articleRepository,
-                              TenantRepositoryInterface    $tenantRepository) {
+    private EventDispatcherInterface $eventDispatcher;
+    private DataTransformerInterface $dataTransformer;
+    private CachedTenantContextInterface $cachedTenantContext;
+    private PackageRepositoryInterface $packageRepository;
+    private ArticleRepositoryInterface $articleRepository;
+    private TenantRepositoryInterface $tenantRepository;
+
+    /**
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param DataTransformerInterface $dataTransformer
+     * @param CachedTenantContextInterface $cachedTenantContext
+     * @param PackageRepositoryInterface $packageRepository
+     * @param ArticleRepositoryInterface $articleRepository
+     * @param TenantRepositoryInterface $tenantRepository
+     */
+    public function __construct(EventDispatcherInterface     $eventDispatcher,
+                                DataTransformerInterface     $dataTransformer,
+                                CachedTenantContextInterface $cachedTenantContext,
+                                PackageRepositoryInterface   $packageRepository,
+                                ArticleRepositoryInterface   $articleRepository,
+                                TenantRepositoryInterface    $tenantRepository) {
     $this->eventDispatcher = $eventDispatcher;
     $this->dataTransformer = $dataTransformer;
     $this->cachedTenantContext = $cachedTenantContext;
     $this->packageRepository = $packageRepository;
     $this->articleRepository = $articleRepository;
     $this->tenantRepository = $tenantRepository;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/organization/articles/related/", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_organization_related_articles")
-   */
-  public function postAction(Request $request) {
+    /**
+     * @Route("/api/{version}/organization/articles/related/", methods={"POST"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_organization_related_articles")
+     */
+    public function postAction(Request $request) {
     $content = $request->getContent();
     $package = $this->dataTransformer->transform($content);
     $relatedArticlesList = $this->getRelated($package);
 
     return new SingleResourceResponse($relatedArticlesList);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/packages/{id}/related/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_packages_related_articles", requirements={"id"="\d+"})
-   */
-  public function getRelatedAction(string $id) {
+    /**
+     * @Route("/api/{version}/packages/{id}/related/", methods={"GET"}, options={"expose"=true}, defaults={"version"="v2"}, name="swp_api_core_packages_related_articles", requirements={"id"="\d+"})
+     */
+    public function getRelatedAction(string $id) {
     $package = $this->findOr404((int)$id);
 
     $relatedArticlesList = $this->getRelated($package);
 
     return new SingleResourceResponse($relatedArticlesList);
-  }
+    }
 
-  private function getRelated(PackageInterface $package): RelatedArticleList {
+    private function getRelated(PackageInterface $package): RelatedArticleList {
     $relatedItemsGroups = $package->getItems()->filter(static function ($group) {
-      return ItemInterface::TYPE_TEXT === $group->getType();
+        return ItemInterface::TYPE_TEXT === $group->getType();
     });
 
     $relatedArticlesList = new RelatedArticleList();
 
     if (null === $package || (null !== $package && 0 === \count($relatedItemsGroups))) {
-      return $relatedArticlesList;
+        return $relatedArticlesList;
     }
 
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $articleRepository = $this->articleRepository;
 
     foreach ($relatedItemsGroups as $item) {
-      if (null === ($existingArticles = $articleRepository->findBy(['code' => $item->getGuid()]))) {
+        if (null === ($existingArticles = $articleRepository->findBy(['code' => $item->getGuid()]))) {
         continue;
-      }
+        }
 
-      $tenants = [];
-      foreach ($existingArticles as $existingArticle) {
+        $tenants = [];
+        foreach ($existingArticles as $existingArticle) {
         $tenantCode = $existingArticle->getTenantCode();
         $tenant = $this->tenantRepository->findOneByCode($tenantCode);
 
         $tenants[] = $tenant;
-      }
+        }
 
-      $relatedArticleListItem = new RelatedArticleListItem();
-      $relatedArticleListItem->setTenants($tenants);
-      $relatedArticleListItem->setTitle($item->getHeadline());
+        $relatedArticleListItem = new RelatedArticleListItem();
+        $relatedArticleListItem->setTenants($tenants);
+        $relatedArticleListItem->setTitle($item->getHeadline());
 
-      $relatedArticlesList->addRelatedArticleItem($relatedArticleListItem);
+        $relatedArticlesList->addRelatedArticleItem($relatedArticleListItem);
     }
 
     return $relatedArticlesList;
-  }
+    }
 
-  private function findOr404(int $id): PackageInterface {
+    private function findOr404(int $id): PackageInterface {
     $this->eventDispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_DISABLE);
     $tenantContext = $this->cachedTenantContext;
     if (null === $package = $this->packageRepository->findOneBy(['id' => $id, 'organization' => $tenantContext->getTenant()->getOrganization()])) {
-      throw new NotFoundHttpException('Package was not found.');
+        throw new NotFoundHttpException('Package was not found.');
     }
 
     return $package;
-  }
+    }
 }

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

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
    * @Route("/api/{version}/webhooks/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_webhook")
    */
   public function listAction(Request $request): ResourcesListResponseInterface {
-    return $this->listWebhooks($this->eventDispatcher,$this->webhookRepository, $request);
+    return $this->listWebhooks($this->eventDispatcher, $this->webhookRepository, $request);
   }
 
   /**

Indentation +47 added lines, -47 removed lines

@@ -28,77 +28,77 @@
 use FOS\RestBundle\Controller\Annotations\Route;
 
 class WebhookController extends AbstractAPIController {
-  private WebhookRepositoryInterface $webhookRepository;
-  private FormFactoryInterface $formFactory;
-  private FactoryInterface $webhookFactory;
-  private EntityManagerInterface $entityManager;
-  private EventDispatcherInterface $eventDispatcher;
+    private WebhookRepositoryInterface $webhookRepository;
+    private FormFactoryInterface $formFactory;
+    private FactoryInterface $webhookFactory;
+    private EntityManagerInterface $entityManager;
+    private EventDispatcherInterface $eventDispatcher;
 
-  /**
-   * @param WebhookRepositoryInterface $webhookRepository
-   * @param FormFactoryInterface $formFactory
-   * @param FactoryInterface $webhookFactory
-   * @param EntityManagerInterface $entityManager
-   * @param EventDispatcherInterface $eventDispatcher
-   */
-  public function __construct(WebhookRepositoryInterface $webhookRepository, FormFactoryInterface $formFactory,
-                              FactoryInterface           $webhookFactory, EntityManagerInterface $entityManager,
-                              EventDispatcherInterface            $eventDispatcher) {
+    /**
+     * @param WebhookRepositoryInterface $webhookRepository
+     * @param FormFactoryInterface $formFactory
+     * @param FactoryInterface $webhookFactory
+     * @param EntityManagerInterface $entityManager
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(WebhookRepositoryInterface $webhookRepository, FormFactoryInterface $formFactory,
+                                FactoryInterface           $webhookFactory, EntityManagerInterface $entityManager,
+                                EventDispatcherInterface            $eventDispatcher) {
     $this->webhookRepository = $webhookRepository;
     $this->formFactory = $formFactory;
     $this->webhookFactory = $webhookFactory;
     $this->entityManager = $entityManager;
     $this->eventDispatcher = $eventDispatcher;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/webhooks/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_webhook")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/webhooks/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_webhook")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     return $this->listWebhooks($this->eventDispatcher,$this->webhookRepository, $request);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/webhooks/{id}", requirements={"id"="\d+"}, options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_webhook")
-   *
-   * @ParamConverter("webhook", class="SWP\Bundle\WebhookBundle\Model\Webhook")
-   */
-  public function getAction(WebhookInterface $webhook): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/webhooks/{id}", requirements={"id"="\d+"}, options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_webhook")
+     *
+     * @ParamConverter("webhook", class="SWP\Bundle\WebhookBundle\Model\Webhook")
+     */
+    public function getAction(WebhookInterface $webhook): SingleResourceResponseInterface {
     return $this->getSingleWebhook($webhook);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/webhooks/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_create_webhook")
-   */
-  public function createAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/webhooks/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_create_webhook")
+     */
+    public function createAction(Request $request): SingleResourceResponseInterface {
     $ruleRepository = $this->webhookRepository;
     $ruleFactory = $this->webhookFactory;
     $formFactory = $this->formFactory;
 
     return $this->createWebhook($ruleRepository, $ruleFactory, $request, $formFactory);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/webhooks/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_core_delete_webhook", requirements={"id"="\d+"})
-   *
-   * @ParamConverter("webhook", class="SWP\Bundle\WebhookBundle\Model\Webhook")
-   */
-  public function deleteAction(WebhookInterface $webhook): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/webhooks/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_core_delete_webhook", requirements={"id"="\d+"})
+     *
+     * @ParamConverter("webhook", class="SWP\Bundle\WebhookBundle\Model\Webhook")
+     */
+    public function deleteAction(WebhookInterface $webhook): SingleResourceResponseInterface {
     $webhookRepository = $this->webhookRepository;
 
     return $this->deleteWebhook($webhookRepository, $webhook);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/webhooks/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_webhook", requirements={"id"="\d+"})
-   *
-   * @ParamConverter("webhook", class="SWP\Bundle\WebhookBundle\Model\Webhook")
-   */
-  public function updateAction(Request $request, WebhookInterface $webhook): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/webhooks/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_webhook", requirements={"id"="\d+"})
+     *
+     * @ParamConverter("webhook", class="SWP\Bundle\WebhookBundle\Model\Webhook")
+     */
+    public function updateAction(Request $request, WebhookInterface $webhook): SingleResourceResponseInterface {
     $objectManager = $this->entityManager;
     $formFactory = $this->formFactory;
 
     return $this->updateWebhook($objectManager, $request, $webhook, $formFactory);
-  }
+    }
 }

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

Spacing +4 added lines, -4 removed lines

@@ -60,9 +60,9 @@ 
         $this->tenantAwareThemeContext->getTheme()
     );
 
-    $fileName = (null === $fileExtension) ? basename($fileName) : $fileName . '.' . $fileExtension;
+    $fileName = (null === $fileExtension) ? basename($fileName) : $fileName.'.'.$fileExtension;
     foreach ($themes as $theme) {
-      $filePath = $theme->getPath() . '/' . self::ASSETS_DIRECTORY . '/' . $fileName;
+      $filePath = $theme->getPath().'/'.self::ASSETS_DIRECTORY.'/'.$fileName;
       if (null !== $response = $this->handleFileLoading($filePath)) {
         return $response;
       }
@@ -85,7 +85,7 @@ 
       throw new NotFoundHttpException('File was not found.');
     }
 
-    $filePath = $theme->getPath() . '/screenshots/' . $fileName;
+    $filePath = $theme->getPath().'/screenshots/'.$fileName;
     if (null !== $response = $this->handleFileLoading($filePath)) {
       return $response;
     }
@@ -150,7 +150,7 @@ 
   private function filterThemes($loadedThemes, string $themeName) {
     $themes = array_filter(
         $loadedThemes,
-        function ($element) use (&$themeName) {
+        function($element) use (&$themeName) {
           return $element->getName() === $themeName;
         }
     );

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

Spacing +2 added lines, -2 removed lines

@@ -169,10 +169,10 @@
       $settingsManager = $this->settingsManager;
 
       if (array_key_exists('fbiaEnabled', $formData)) {
-        $settingsManager->set('fbia_enabled', (bool)$formData['fbiaEnabled'], ScopeContextInterface::SCOPE_TENANT, $tenant);
+        $settingsManager->set('fbia_enabled', (bool) $formData['fbiaEnabled'], ScopeContextInterface::SCOPE_TENANT, $tenant);
       }
       if (array_key_exists('paywallEnabled', $formData)) {
-        $settingsManager->set('paywall_enabled', (bool)$formData['paywallEnabled'], ScopeContextInterface::SCOPE_TENANT, $tenant);
+        $settingsManager->set('paywall_enabled', (bool) $formData['paywallEnabled'], ScopeContextInterface::SCOPE_TENANT, $tenant);
       }
       if (array_key_exists('defaultLanguage', $formData)) {
         $settingsManager->set('default_language', $formData['defaultLanguage'], ScopeContextInterface::SCOPE_TENANT, $tenant);

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