superdesk / web-publisher

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

Indentation +66 added lines, -66 removed lines

@@ -34,87 +34,87 @@ 
 use SWP\Component\Common\Exception\UnexpectedTypeException;
 
 final class DownloadArticleAuthorAvatarListener {
-  /**
-   * @var MediaManagerInterface
-   */
-  private $authorMediaManager;
-
-  /**
-   * @var EntityManagerInterface
-   */
-  private $entityManager;
-
-  /**
-   * @var string
-   */
-  private $cacheDirectory;
-
-  public function __construct(EntityManagerInterface $entityManager, $cacheDirectory) {
+    /**
+     * @var MediaManagerInterface
+     */
+    private $authorMediaManager;
+
+    /**
+     * @var EntityManagerInterface
+     */
+    private $entityManager;
+
+    /**
+     * @var string
+     */
+    private $cacheDirectory;
+
+    public function __construct(EntityManagerInterface $entityManager, $cacheDirectory) {
     $this->entityManager = $entityManager;
     $this->cacheDirectory = $cacheDirectory;
-  }
+    }
 
-  /**
-   * @param MediaManagerInterface $authorMediaManager
-   */
-  public function setAuthorMediaManager(MediaManagerInterface $authorMediaManager) {
+    /**
+     * @param MediaManagerInterface $authorMediaManager
+     */
+    public function setAuthorMediaManager(MediaManagerInterface $authorMediaManager) {
     $this->authorMediaManager = $authorMediaManager;
-  }
+    }
 
-  /**
-   * @param GenericEvent $event
-   */
-  public function processAuthors(GenericEvent $event): void {
+    /**
+     * @param GenericEvent $event
+     */
+    public function processAuthors(GenericEvent $event): void {
     $package = $event->getSubject();
 
     if (!$package instanceof PackageInterface) {
-      throw new UnexpectedTypeException($package, PackageInterface::class);
+        throw new UnexpectedTypeException($package, PackageInterface::class);
     }
 
     $authors = [];
     /** @var ArticleAuthorInterface $packageAuthor */
     foreach ($package->getAuthors() as $packageAuthor) {
-      $authors[] = $this->handle($packageAuthor);
+        $authors[] = $this->handle($packageAuthor);
     }
     $package->setAuthors(new ArrayCollection($authors));
-  }
-
-  /**
-   * @param ArticleAuthorInterface $object
-   *
-   * @return ArticleAuthorInterface
-   */
-  private function handle(ArticleAuthorInterface $object): ArticleAuthorInterface {
+    }
+
+    /**
+     * @param ArticleAuthorInterface $object
+     *
+     * @return ArticleAuthorInterface
+     */
+    private function handle(ArticleAuthorInterface $object): ArticleAuthorInterface {
     if (null !== $object->getAvatarUrl()) {
-      $filesystem = new Filesystem();
-      $pathParts = \pathinfo($object->getAvatarUrl());
-      $assetId = \sha1($pathParts['filename']);
-      if (null !== $object->getSlug()) {
+        $filesystem = new Filesystem();
+        $pathParts = \pathinfo($object->getAvatarUrl());
+        $assetId = \sha1($pathParts['filename']);
+        if (null !== $object->getSlug()) {
         $assetId = $object->getSlug() . '_' . $assetId;
-      }
-      $existingAvatar = $this->entityManager->getRepository(Image::class)->findBy(['assetId' => $assetId]);
-      if (\count($existingAvatar) > 0) {
+        }
+        $existingAvatar = $this->entityManager->getRepository(Image::class)->findBy(['assetId' => $assetId]);
+        if (\count($existingAvatar) > 0) {
         $object->setAvatarUrl((string)\reset($existingAvatar));
 
         return $object;
-      }
+        }
 
-      try {
+        try {
         $tempDirectory = $this->cacheDirectory . \DIRECTORY_SEPARATOR . 'downloaded_avatars';
         $tempLocation = $tempDirectory . \DIRECTORY_SEPARATOR . \sha1($assetId . date('his'));
         if (!$filesystem->exists($tempDirectory)) {
-          $filesystem->mkdir($tempDirectory);
+            $filesystem->mkdir($tempDirectory);
         }
 
         $avatarUrl = $object->getAvatarUrl();
         if (strpos($avatarUrl, "http://") === 0 || strpos($avatarUrl, "https://") === 0) {
-          self::downloadFile($avatarUrl, $tempLocation);
+            self::downloadFile($avatarUrl, $tempLocation);
         } else {
-          $file = \file_get_contents($avatarUrl);
-          if (false === $file) {
+            $file = \file_get_contents($avatarUrl);
+            if (false === $file) {
             throw new \Exception('File can\'t be downloaded');
-          }
-          $filesystem->dumpFile($tempLocation, $file);
+            }
+            $filesystem->dumpFile($tempLocation, $file);
         }
 
         $uploadedFile = new UploadedFile($tempLocation,
@@ -123,29 +123,29 @@ 
             \filesize($tempLocation),
             true
         );
-      } catch (\Exception $e) {
+        } catch (\Exception $e) {
         return $object;
-      }
-      /** @var Image $image */
-      $image = $this->authorMediaManager->handleUploadedFile($uploadedFile, $assetId);
-      $avatar = $this->createAuthorMedia($object, $image);
-      $this->entityManager->persist($avatar);
-      $this->entityManager->persist($image);
-
-      $object->setAvatar($avatar);
-      $object->setAvatarUrl((string)$image);
+        }
+        /** @var Image $image */
+        $image = $this->authorMediaManager->handleUploadedFile($uploadedFile, $assetId);
+        $avatar = $this->createAuthorMedia($object, $image);
+        $this->entityManager->persist($avatar);
+        $this->entityManager->persist($image);
+
+        $object->setAvatar($avatar);
+        $object->setAvatarUrl((string)$image);
     }
 
     return $object;
-  }
+    }
 
-  private function createAuthorMedia(ArticleAuthorInterface $articleAuthor, Image $image): AuthorMediaInterface {
+    private function createAuthorMedia(ArticleAuthorInterface $articleAuthor, Image $image): AuthorMediaInterface {
     return new AuthorMedia('avatar', $articleAuthor, $image);
-  }
+    }
 
-  private static function downloadFile($url, $location) {
+    private static function downloadFile($url, $location) {
     $handlerStack = HandlerStack::create(new CurlHandler());
     $client = new Client(['handler' => $handlerStack]);
     $client->request('GET', $url, ['sink' => $location]);
-  }
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -90,18 +90,18 @@ 
       $pathParts = \pathinfo($object->getAvatarUrl());
       $assetId = \sha1($pathParts['filename']);
       if (null !== $object->getSlug()) {
-        $assetId = $object->getSlug() . '_' . $assetId;
+        $assetId = $object->getSlug().'_'.$assetId;
       }
       $existingAvatar = $this->entityManager->getRepository(Image::class)->findBy(['assetId' => $assetId]);
       if (\count($existingAvatar) > 0) {
-        $object->setAvatarUrl((string)\reset($existingAvatar));
+        $object->setAvatarUrl((string) \reset($existingAvatar));
 
         return $object;
       }
 
       try {
-        $tempDirectory = $this->cacheDirectory . \DIRECTORY_SEPARATOR . 'downloaded_avatars';
-        $tempLocation = $tempDirectory . \DIRECTORY_SEPARATOR . \sha1($assetId . date('his'));
+        $tempDirectory = $this->cacheDirectory.\DIRECTORY_SEPARATOR.'downloaded_avatars';
+        $tempLocation = $tempDirectory.\DIRECTORY_SEPARATOR.\sha1($assetId.date('his'));
         if (!$filesystem->exists($tempDirectory)) {
           $filesystem->mkdir($tempDirectory);
         }
@@ -133,7 +133,7 @@ 
       $this->entityManager->persist($image);
 
       $object->setAvatar($avatar);
-      $object->setAvatarUrl((string)$image);
+      $object->setAvatarUrl((string) $image);
     }
 
     return $object;

src/SWP/Bundle/UserBundle/Controller/ResetPasswordController.php

Indentation +70 added lines, -70 removed lines

@@ -36,88 +36,88 @@ 
  * @Route("/reset-password")
  */
 class ResetPasswordController extends AbstractController {
-  use ResetPasswordControllerTrait;
+    use ResetPasswordControllerTrait;
 
-  private ResetPasswordHelperInterface $resetPasswordHelper;
-  private EntityManagerInterface  $entityManager;
+    private ResetPasswordHelperInterface $resetPasswordHelper;
+    private EntityManagerInterface  $entityManager;
 
-  /**
-   * @param ResetPasswordHelperInterface $resetPasswordHelper
-   * @param EntityManagerInterface $entityManager
-   */
-  public function __construct(ResetPasswordHelperInterface $resetPasswordHelper,
-                              EntityManagerInterface       $entityManager) {
+    /**
+     * @param ResetPasswordHelperInterface $resetPasswordHelper
+     * @param EntityManagerInterface $entityManager
+     */
+    public function __construct(ResetPasswordHelperInterface $resetPasswordHelper,
+                                EntityManagerInterface       $entityManager) {
     $this->resetPasswordHelper = $resetPasswordHelper;
     $this->entityManager = $entityManager;
-  }
+    }
 
 
-  /**
-   * Display & process form to request a password reset.
-   *
-   * @Route("", name="swp_user_forgot_password_request")
-   */
-  public function request(Request $request, MailerInterface $mailer): Response {
+    /**
+     * Display & process form to request a password reset.
+     *
+     * @Route("", name="swp_user_forgot_password_request")
+     */
+    public function request(Request $request, MailerInterface $mailer): Response {
     $form = $this->createForm(ResetPasswordRequestFormType::class);
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      return $this->processSendingPasswordResetEmail(
-          $form->get('email')->getData(),
-          $mailer
-      );
+        return $this->processSendingPasswordResetEmail(
+            $form->get('email')->getData(),
+            $mailer
+        );
     }
 
     return $this->render('@SWPUser/reset_password/request.html.twig', [
         'requestForm' => $form->createView(),
     ]);
-  }
-
-  /**
-   * Confirmation page after a user has requested a password reset.
-   *
-   * @Route("/check-email", name="swp_user_check_email")
-   */
-  public function checkEmail(): Response {
+    }
+
+    /**
+     * Confirmation page after a user has requested a password reset.
+     *
+     * @Route("/check-email", name="swp_user_check_email")
+     */
+    public function checkEmail(): Response {
     // We prevent users from directly accessing this page
     if (!$this->canCheckEmail()) {
-      return $this->redirectToRoute('swp_user_forgot_password_request');
+        return $this->redirectToRoute('swp_user_forgot_password_request');
     }
 
     return $this->render('@SWPUser/reset_password/check_email.html.twig', [
         'tokenLifetime' => $this->resetPasswordHelper->getTokenLifetime(),
     ]);
-  }
-
-  /**
-   * Validates and process the reset URL that the user clicked in their email.
-   *
-   * @Route("/reset/{token}", name="swp_user_reset_password")
-   */
-  public function reset(Request $request, UserPasswordEncoderInterface $passwordEncoder,
+    }
+
+    /**
+     * Validates and process the reset URL that the user clicked in their email.
+     *
+     * @Route("/reset/{token}", name="swp_user_reset_password")
+     */
+    public function reset(Request $request, UserPasswordEncoderInterface $passwordEncoder,
                         string  $token = null): Response {
     if ($token) {
-      // We store the token in session and remove it from the URL, to avoid the URL being
-      // loaded in a browser and potentially leaking the token to 3rd party JavaScript.
-      $this->storeTokenInSession($token);
+        // We store the token in session and remove it from the URL, to avoid the URL being
+        // loaded in a browser and potentially leaking the token to 3rd party JavaScript.
+        $this->storeTokenInSession($token);
 
-      return $this->redirectToRoute('swp_user_reset_password');
+        return $this->redirectToRoute('swp_user_reset_password');
     }
 
     $token = $this->getTokenFromSession();
     if (null === $token) {
-      throw $this->createNotFoundException('No reset password token found in the URL or in the session.');
+        throw $this->createNotFoundException('No reset password token found in the URL or in the session.');
     }
 
     try {
-      $user = $this->resetPasswordHelper->validateTokenAndFetchUser($token);
+        $user = $this->resetPasswordHelper->validateTokenAndFetchUser($token);
     } catch (ResetPasswordExceptionInterface $e) {
-      $this->addFlash('reset_password_error', sprintf(
-          'There was a problem validating your reset request - %s',
-          $e->getReason()
-      ));
+        $this->addFlash('reset_password_error', sprintf(
+            'There was a problem validating your reset request - %s',
+            $e->getReason()
+        ));
 
-      return $this->redirectToRoute('swp_user_forgot_password_request');
+        return $this->redirectToRoute('swp_user_forgot_password_request');
     }
 
     // The token is valid; allow the user to change their password.
@@ -125,30 +125,30 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      // A password reset token should be used only once, remove it.
-      $this->resetPasswordHelper->removeResetRequest($token);
+        // A password reset token should be used only once, remove it.
+        $this->resetPasswordHelper->removeResetRequest($token);
 
-      // Encode the plain password, and set it.
-      $encodedPassword = $passwordEncoder->encodePassword(
-          $user,
-          $form->get('plainPassword')->getData()
-      );
+        // Encode the plain password, and set it.
+        $encodedPassword = $passwordEncoder->encodePassword(
+            $user,
+            $form->get('plainPassword')->getData()
+        );
 
-      $user->setPassword($encodedPassword);
-      $this->entityManager->flush();
+        $user->setPassword($encodedPassword);
+        $this->entityManager->flush();
 
-      // The session is cleaned up after the password has been changed.
-      $this->cleanSessionAfterReset();
+        // The session is cleaned up after the password has been changed.
+        $this->cleanSessionAfterReset();
 
-      return $this->redirectToRoute('homepage');
+        return $this->redirectToRoute('homepage');
     }
 
     return $this->render('@SWPUser/reset_password/reset.html.twig', [
         'resetForm' => $form->createView(),
     ]);
-  }
+    }
 
-  private function processSendingPasswordResetEmail(string $emailFormData, MailerInterface $mailer): RedirectResponse {
+    private function processSendingPasswordResetEmail(string $emailFormData, MailerInterface $mailer): RedirectResponse {
     $user = $this->entityManager->getRepository(User::class)->findOneBy([
         'email' => $emailFormData,
     ]);
@@ -158,22 +158,22 @@ 
 
     // Do not reveal whether a user account was found or not.
     if (!$user) {
-      return $this->redirectToRoute('swp_user_check_email');
+        return $this->redirectToRoute('swp_user_check_email');
     }
 
     try {
-      $resetToken = $this->resetPasswordHelper->generateResetToken($user);
+        $resetToken = $this->resetPasswordHelper->generateResetToken($user);
     } catch (ResetPasswordExceptionInterface $e) {
-      $this->addFlash('reset_password_error', sprintf(
-          'There was a problem handling your password reset request - %s',
-          $e->getReason()
-      ));
+        $this->addFlash('reset_password_error', sprintf(
+            'There was a problem handling your password reset request - %s',
+            $e->getReason()
+        ));
 
-      return $this->redirectToRoute('swp_user_check_email');
+        return $this->redirectToRoute('swp_user_check_email');
     }
 
     $mailer->sendResetPasswordEmail($user, $resetToken);
 
     return $this->redirectToRoute('swp_user_check_email');
-  }
+    }
 }

src/SWP/Bundle/FixturesBundle/WebTestCase.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
       unset($server['HTTP_Authorization']);
     }
 
-    if(!array_key_exists("HTTP_HOST", $server)) {
+    if (!array_key_exists("HTTP_HOST", $server)) {
       $server['HTTP_HOST'] = 'localhost';
     }
 

Indentation +28 added lines, -28 removed lines

@@ -23,71 +23,71 @@ 
 use Symfony\Component\Cache\Adapter\ArrayAdapter;
 
 class WebTestCase extends BaseWebTestCase {
-  protected ?AbstractDatabaseTool $databaseTool;
-  private static ?KernelBrowser $client = null;
+    protected ?AbstractDatabaseTool $databaseTool;
+    private static ?KernelBrowser $client = null;
 
-  public function setUp(): void {
+    public function setUp(): void {
     if (self::$client == null) {
-      self::$client = parent::createClient();
+        self::$client = parent::createClient();
     }
     $this->databaseTool = static::getContainer()->get(DatabaseToolCollection::class)->get();
-  }
+    }
 
-  /**
-   * Initializes database.
-   */
-  protected function initDatabase() {
+    /**
+     * Initializes database.
+     */
+    protected function initDatabase() {
     $this->clearMetadataCache();
-  }
+    }
 
-  /**
-   * Clears metadata cache of the various cache drivers.
-   */
-  private function clearMetadataCache() {
+    /**
+     * Clears metadata cache of the various cache drivers.
+     */
+    private function clearMetadataCache() {
     $entityManager = $this->getContainer()->get('doctrine.orm.entity_manager');
 
     $cacheDriver = $entityManager->getConfiguration()->getMetadataCacheImpl();
 
     if (!$cacheDriver instanceof ArrayAdapter) {
-      $cacheDriver->deleteAll();
+        $cacheDriver->deleteAll();
+    }
     }
-  }
 
-  protected function tearDown(): void {
+    protected function tearDown(): void {
     unset($this->databaseTool);
     $reflection = new \ReflectionObject($this);
     foreach ($reflection->getProperties() as $prop) {
-      if (!$prop->isStatic() && 0 !== strpos($prop->getDeclaringClass()->getName(), 'PHPUnit_')) {
+        if (!$prop->isStatic() && 0 !== strpos($prop->getDeclaringClass()->getName(), 'PHPUnit_')) {
         $prop->setAccessible(true);
         if ($prop->name == 'databaseTool') {
-          continue;
+            continue;
         }
         $prop->setValue($this, null);
-      }
+        }
     }
     parent::tearDown();
-  }
+    }
 
-  protected function loadCustomFixtures(array $fixtures) {
+    protected function loadCustomFixtures(array $fixtures) {
     $env = $this->getContainer()->getParameter('test_env');
 
     $registry = new FixtureRegistry();
     $registry->setEnvironment($env);
 
     return $this->databaseTool->loadFixtures($registry->getFixtures($fixtures))->getReferenceRepository();
-  }
+    }
 
-  public static function createClient(array $options = [], array $server = []) {
+    public static function createClient(array $options = [], array $server = []) {
     if (!array_key_exists('HTTP_Authorization', $server)) {
-      $server['HTTP_Authorization'] = base64_encode('test_token:');
+        $server['HTTP_Authorization'] = base64_encode('test_token:');
     }
 
     if (null === $server['HTTP_Authorization']) {
-      unset($server['HTTP_Authorization']);
+        unset($server['HTTP_Authorization']);
     }
 
     if(!array_key_exists("HTTP_HOST", $server)) {
-      $server['HTTP_HOST'] = 'localhost';
+        $server['HTTP_HOST'] = 'localhost';
     }
 
     $newClient = clone self::$client;
@@ -95,5 +95,5 @@ 
     $newClient->getKernel()->shutdown();
     $newClient->getKernel()->boot();
     return $newClient;
-  }
+    }
 }

src/SWP/Bundle/MultiTenancyBundle/Context/TenantContext.php

Indentation +37 added lines, -37 removed lines

@@ -28,62 +28,62 @@
  * Class TenantContext.
  */
 class TenantContext implements TenantContextInterface {
-  /**
-   * @var TenantInterface
-   */
-  protected $tenant;
+    /**
+     * @var TenantInterface
+     */
+    protected $tenant;
 
-  /**
-   * @var TenantResolverInterface
-   */
-  protected $tenantResolver;
+    /**
+     * @var TenantResolverInterface
+     */
+    protected $tenantResolver;
 
-  /**
-   * @var RequestStack
-   */
-  protected $requestStack;
+    /**
+     * @var RequestStack
+     */
+    protected $requestStack;
 
-  /**
-   * @var EventDispatcherInterface
-   */
-  protected $dispatcher;
+    /**
+     * @var EventDispatcherInterface
+     */
+    protected $dispatcher;
 
-  /**
-   * TenantContext constructor.
-   */
-  public function __construct(TenantResolverInterface  $tenantResolver, RequestStack $requestStack,
-                              EventDispatcherInterface $dispatcher) {
+    /**
+     * TenantContext constructor.
+     */
+    public function __construct(TenantResolverInterface  $tenantResolver, RequestStack $requestStack,
+                                EventDispatcherInterface $dispatcher) {
     $this->tenantResolver = $tenantResolver;
     $this->requestStack = $requestStack;
     $this->dispatcher = $dispatcher;
-  }
+    }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function getTenant() {
+    /**
+     * {@inheritdoc}
+     */
+    public function getTenant() {
     if ($this->tenant !== null)
-      return $this->tenant;
+        return $this->tenant;
 
     $currentRequest = $this->requestStack->getCurrentRequest();
     if (null !== $currentRequest && false !== strpos($currentRequest->getRequestUri(), '_profiler')) {
-      $profilerTenant = new Tenant();
-      $profilerTenant->setDomainName($currentRequest->getHost());
-      $this->setTenant($profilerTenant);
-      return $profilerTenant;
+        $profilerTenant = new Tenant();
+        $profilerTenant->setDomainName($currentRequest->getHost());
+        $this->setTenant($profilerTenant);
+        return $profilerTenant;
     }
 
     $newTenant = $this->tenantResolver->resolve($currentRequest ? $currentRequest->getHost() : null);
     $this->setTenant($newTenant);
     return $newTenant;
-  }
+    }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function setTenant(TenantInterface $tenant) {
+    /**
+     * {@inheritdoc}
+     */
+    public function setTenant(TenantInterface $tenant) {
     $this->tenant = $tenant;
 
     $this->dispatcher->dispatch(new GenericEvent($tenant), MultiTenancyEvents::TENANT_SET);
-  }
+    }
 }

Braces +3 added lines, -2 removed lines

@@ -62,8 +62,9 @@
    * {@inheritdoc}
    */
   public function getTenant() {
-    if ($this->tenant !== null)
-      return $this->tenant;
+    if ($this->tenant !== null) {
+          return $this->tenant;
+    }
 
     $currentRequest = $this->requestStack->getCurrentRequest();
     if (null !== $currentRequest && false !== strpos($currentRequest->getRequestUri(), '_profiler')) {

src/SWP/Bundle/ContentListBundle/Doctrine/ORM/ContentListItemRepository.php

Indentation +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
             return $paginator->paginate(
                 $queryBuilder,
                 (int)$criteria->get('firstResult', 0),
-                 (int)$criteria->get('maxResults', RepositoryInterface::MAX_RESULTS)
+                    (int)$criteria->get('maxResults', RepositoryInterface::MAX_RESULTS)
             );
         }
 

Spacing +3 added lines, -3 removed lines

@@ -77,12 +77,12 @@
             $paginator = new Paginator($eventDispatcher);
             return $paginator->paginate(
                 $queryBuilder,
-                (int)$criteria->get('firstResult', 0),
-                 (int)$criteria->get('maxResults', RepositoryInterface::MAX_RESULTS)
+                (int) $criteria->get('firstResult', 0),
+                 (int) $criteria->get('maxResults', RepositoryInterface::MAX_RESULTS)
             );
         }
 
-        return $this->getPaginator( $eventDispatcher,$queryBuilder, $paginationData);
+        return $this->getPaginator($eventDispatcher, $queryBuilder, $paginationData);
     }
 
     public function getCountByCriteria(Criteria $criteria): int

src/SWP/Bundle/CoreBundle/Theme/Locator/OrganizationThemesRecursiveFileLocator.php

Indentation +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     private function getPaths() : array
     {
-      return [$this->themeUploader->getAvailableThemesPath()];
+        return [$this->themeUploader->getAvailableThemesPath()];
     }
 
     /**

src/SWP/Bundle/CoreBundle/Pagination/Paginator.php

Indentation +40 added lines, -40 removed lines

@@ -25,62 +25,62 @@ 
 use Symfony\Component\HttpFoundation\RequestStack;
 
 class Paginator implements PaginatorInterface {
-  private EventDispatcherInterface $eventDispatcher;
-  private ?RequestStack $requestStack;
-
-  private array $defaultOptions = [
-      self::PAGE_PARAMETER_NAME => 'page',
-      self::SORT_FIELD_PARAMETER_NAME => 'sort',
-      self::SORT_DIRECTION_PARAMETER_NAME => 'direction',
-      self::FILTER_FIELD_PARAMETER_NAME => 'filterParam',
-      self::FILTER_VALUE_PARAMETER_NAME => 'filterValue',
-      self::DISTINCT => true,
-      self::PAGE_OUT_OF_RANGE => self::PAGE_OUT_OF_RANGE_IGNORE,
-      self::DEFAULT_LIMIT => self::DEFAULT_LIMIT_VALUE,
-  ];
-
-
-  public function __construct(EventDispatcherInterface $eventDispatcher, ?RequestStack $requestStack = null) {
+    private EventDispatcherInterface $eventDispatcher;
+    private ?RequestStack $requestStack;
+
+    private array $defaultOptions = [
+        self::PAGE_PARAMETER_NAME => 'page',
+        self::SORT_FIELD_PARAMETER_NAME => 'sort',
+        self::SORT_DIRECTION_PARAMETER_NAME => 'direction',
+        self::FILTER_FIELD_PARAMETER_NAME => 'filterParam',
+        self::FILTER_VALUE_PARAMETER_NAME => 'filterValue',
+        self::DISTINCT => true,
+        self::PAGE_OUT_OF_RANGE => self::PAGE_OUT_OF_RANGE_IGNORE,
+        self::DEFAULT_LIMIT => self::DEFAULT_LIMIT_VALUE,
+    ];
+
+
+    public function __construct(EventDispatcherInterface $eventDispatcher, ?RequestStack $requestStack = null) {
     $this->eventDispatcher = $eventDispatcher;
     $this->requestStack = $requestStack;
-  }
+    }
 
-  public function setDefaultPaginatorOptions(array $options): void {
+    public function setDefaultPaginatorOptions(array $options): void {
     $this->defaultOptions = \array_merge($this->defaultOptions, $options);
-  }
-
-  /**
-   * {@inheritdoc}
-   *
-   * IMPORTANT: This method expects OFFSET value instead PAGE
-   */
-  public function paginate($target, int $offset = 1, $limit = 10, array $options = array()) : PaginationInterface {
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * IMPORTANT: This method expects OFFSET value instead PAGE
+     */
+    public function paginate($target, int $offset = 1, $limit = 10, array $options = array()) : PaginationInterface {
     $limit = intval(abs($limit));
     if (!$limit) {
-      throw new \LogicException('Invalid item per page number, must be a positive number');
+        throw new \LogicException('Invalid item per page number, must be a positive number');
     }
 
     $page = intval(round($offset / $limit), 10);
     if ($page < 1) {
-      $page = 1;
+        $page = 1;
     }
 
     $options = \array_merge($this->defaultOptions, $options);
 
     // normalize default sort field
     if (isset($options[PaginatorInterface::DEFAULT_SORT_FIELD_NAME]) && is_array($options[PaginatorInterface::DEFAULT_SORT_FIELD_NAME])) {
-      $options[PaginatorInterface::DEFAULT_SORT_FIELD_NAME] = implode('+', $options[PaginatorInterface::DEFAULT_SORT_FIELD_NAME]);
+        $options[PaginatorInterface::DEFAULT_SORT_FIELD_NAME] = implode('+', $options[PaginatorInterface::DEFAULT_SORT_FIELD_NAME]);
     }
 
     $request = null === $this->requestStack ? Request::createFromGlobals() : $this->requestStack->getCurrentRequest();
 
     // default sort field and direction are set based on options (if available)
     if (isset($options[self::DEFAULT_SORT_FIELD_NAME]) && !$request->query->has($options[self::SORT_FIELD_PARAMETER_NAME])) {
-      $request->query->set($options[self::SORT_FIELD_PARAMETER_NAME], $options[self::DEFAULT_SORT_FIELD_NAME]);
+        $request->query->set($options[self::SORT_FIELD_PARAMETER_NAME], $options[self::DEFAULT_SORT_FIELD_NAME]);
 
-      if (!$request->query->has($options[PaginatorInterface::SORT_DIRECTION_PARAMETER_NAME])) {
+        if (!$request->query->has($options[PaginatorInterface::SORT_DIRECTION_PARAMETER_NAME])) {
         $request->query->set($options[PaginatorInterface::SORT_DIRECTION_PARAMETER_NAME], $options[PaginatorInterface::DEFAULT_SORT_DIRECTION] ?? 'asc');
-      }
+        }
     }
 
     // before pagination start
@@ -92,20 +92,20 @@ 
     $itemsEvent->target = &$target;
     $this->eventDispatcher->dispatch($itemsEvent, 'knp_pager.items');
     if (!$itemsEvent->isPropagationStopped()) {
-      throw new \RuntimeException('One of listeners must count and slice given target');
+        throw new \RuntimeException('One of listeners must count and slice given target');
     }
     if ($page > ceil($itemsEvent->count / $limit)) {
-      $pageOutOfRangeOption = $options[PaginatorInterface::PAGE_OUT_OF_RANGE] ?? $this->defaultOptions[PaginatorInterface::PAGE_OUT_OF_RANGE];
-      if ($pageOutOfRangeOption === PaginatorInterface::PAGE_OUT_OF_RANGE_FIX && $itemsEvent->count > 0) {
+        $pageOutOfRangeOption = $options[PaginatorInterface::PAGE_OUT_OF_RANGE] ?? $this->defaultOptions[PaginatorInterface::PAGE_OUT_OF_RANGE];
+        if ($pageOutOfRangeOption === PaginatorInterface::PAGE_OUT_OF_RANGE_FIX && $itemsEvent->count > 0) {
         // replace page number out of range with max page
         return $this->paginate($target, (int) ceil($itemsEvent->count / $limit), $limit, $options);
-      }
-      if ($pageOutOfRangeOption === self::PAGE_OUT_OF_RANGE_THROW_EXCEPTION && $page > 1) {
+        }
+        if ($pageOutOfRangeOption === self::PAGE_OUT_OF_RANGE_THROW_EXCEPTION && $page > 1) {
         throw new PageNumberOutOfRangeException(
             sprintf('Page number: %d is out of range.', $page),
             (int) ceil($itemsEvent->count / $limit)
         );
-      }
+        }
     }
 
     // pagination initialization event
@@ -114,7 +114,7 @@ 
     $paginationEvent->options = &$options;
     $this->eventDispatcher->dispatch($paginationEvent, 'knp_pager.pagination');
     if (!$paginationEvent->isPropagationStopped()) {
-      throw new \RuntimeException('One of listeners must create pagination view');
+        throw new \RuntimeException('One of listeners must create pagination view');
     }
     // pagination class can be different, with different rendering methods
     $paginationView = $paginationEvent->getPagination();
@@ -130,5 +130,5 @@ 
     $this->eventDispatcher->dispatch($afterEvent, 'knp_pager.after');
 
     return $paginationView;
-  }
+    }
 }

src/SWP/Bundle/CoreBundle/Service/FacebookInstantArticlesService.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
             $name = "";
             $params = ['slug' => $article->getSlug()];
             DecoratingRoutingExtension::setupParams($article->getRoute(), $name, $params);
-            $url = $this->urlGenerator->generate($name, $params , UrlGeneratorInterface::ABSOLUTE_URL);
+            $url = $this->urlGenerator->generate($name, $params, UrlGeneratorInterface::ABSOLUTE_URL);
 
             return $this->getClient($feed)->removeArticle($url);
         }

src/BypassFinalHook.php

Indentation +3 added lines, -3 removed lines

@@ -6,8 +6,8 @@
 
 final class BypassFinalHook implements BeforeTestHook
 {
-  public function executeBeforeTest(string $test): void
-  {
+    public function executeBeforeTest(string $test): void
+    {
     BypassFinals::enable();
-  }
+    }
 }