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/MultiTenancyBundle/Context/TenantContext.php

Indentation +45 added lines, -45 removed lines

@@ -27,76 +27,76 @@
  * Class TenantContext.
  */
 class TenantContext implements TenantContextInterface {
-  /**
-   * @var TenantInterface
-   */
-  protected $tenant;
-
-  /**
-   * @var TenantResolverInterface
-   */
-  protected $tenantResolver;
-
-  /**
-   * @var RequestStack
-   */
-  protected $requestStack;
-
-  /**
-   * @var EventDispatcherInterface
-   */
-  protected $dispatcher;
-
-  /**
-   * TenantContext constructor.
-   */
-  public function __construct(TenantResolverInterface  $tenantResolver, RequestStack $requestStack,
-                              EventDispatcherInterface $dispatcher) {
+    /**
+     * @var TenantInterface
+     */
+    protected $tenant;
+
+    /**
+     * @var TenantResolverInterface
+     */
+    protected $tenantResolver;
+
+    /**
+     * @var RequestStack
+     */
+    protected $requestStack;
+
+    /**
+     * @var EventDispatcherInterface
+     */
+    protected $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 (null === $this->tenant || !$this->isHostSame()) {
-      $currentRequest = $this->requestStack->getCurrentRequest();
+        $currentRequest = $this->requestStack->getCurrentRequest();
 
-      if (null !== $currentRequest && false !== strpos($currentRequest->getRequestUri(), '_profiler')) {
+        if (null !== $currentRequest && false !== strpos($currentRequest->getRequestUri(), '_profiler')) {
         $profilerTenant = new Tenant();
         $profilerTenant->setDomainName($currentRequest->getHost());
         $this->setTenant($profilerTenant);
 
         return $this->tenant;
-      }
+        }
 
-      $this->setTenant($this->tenantResolver->resolve(
-          $currentRequest ? $currentRequest->getHost() : null
-      ));
+        $this->setTenant($this->tenantResolver->resolve(
+            $currentRequest ? $currentRequest->getHost() : null
+        ));
     }
 
     return $this->tenant;
-  }
+    }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function setTenant(TenantInterface $tenant) {
+    /**
+     * {@inheritdoc}
+     */
+    public function setTenant(TenantInterface $tenant) {
     $this->tenant = $tenant;
 
     $this->dispatcher->dispatch(new GenericEvent($tenant), MultiTenancyEvents::TENANT_SET);
-  }
+    }
 
-  protected function isHostSame() {
+    protected function isHostSame() {
     if ($this->tenant == null) {
-      return false;
+        return false;
     }
 
     $currentRequest = $this->requestStack->getCurrentRequest();
     $host = $currentRequest !== null ? $currentRequest->getHost() : 'localhost';
     $tenantHost = $this->tenant->getSubdomain() ? $this->tenant->getSubdomain() . '.' . $this->tenant->getDomainName() : $this->tenant->getDomainName();
     return $host === $tenantHost;
-  }
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
 
     $currentRequest = $this->requestStack->getCurrentRequest();
     $host = $currentRequest !== null ? $currentRequest->getHost() : 'localhost';
-    $tenantHost = $this->tenant->getSubdomain() ? $this->tenant->getSubdomain() . '.' . $this->tenant->getDomainName() : $this->tenant->getDomainName();
+    $tenantHost = $this->tenant->getSubdomain() ? $this->tenant->getSubdomain().'.'.$this->tenant->getDomainName() : $this->tenant->getDomainName();
     return $host === $tenantHost;
   }
 }

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

Indentation +28 added lines, -28 removed lines

@@ -23,75 +23,75 @@
 use Symfony\Bundle\FrameworkBundle\KernelBrowser;
 
 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');
     /** @var ClearableCache $cacheDriver */
     $cacheDriver = $entityManager->getConfiguration()->getMetadataCacheImpl();
 
     if (!$cacheDriver instanceof ArrayCache) {
-      $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;
     $newClient->setServerParameters($server);
     return $newClient;
-  }
+    }
 }

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