superdesk / web-publisher

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

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

Indentation +70 added lines, -70 removed lines

@@ -37,83 +37,83 @@ 
 
 class MenuController extends AbstractFOSRestController {
 
-  private FormFactoryInterface $formFactory;
-  private EventDispatcherInterface $eventDispatcher;
-  private MenuItemRepositoryInterface $menuItemRepository;
-  private MenuItemManager $menuItemManager;
-  private MenuFactoryInterface $menuFactory;
-
-  /**
-   * @param FormFactoryInterface $formFactory
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param MenuItemRepositoryInterface $menuItemRepository
-   * @param MenuItemManager $menuItemManager
-   * @param MenuFactoryInterface $menuFactory
-   */
-  public function __construct(FormFactoryInterface        $formFactory, EventDispatcherInterface $eventDispatcher,
-                              MenuItemRepositoryInterface $menuItemRepository, MenuItemManager $menuItemManager,
-                              MenuFactoryInterface        $menuFactory) {
+    private FormFactoryInterface $formFactory;
+    private EventDispatcherInterface $eventDispatcher;
+    private MenuItemRepositoryInterface $menuItemRepository;
+    private MenuItemManager $menuItemManager;
+    private MenuFactoryInterface $menuFactory;
+
+    /**
+     * @param FormFactoryInterface $formFactory
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param MenuItemRepositoryInterface $menuItemRepository
+     * @param MenuItemManager $menuItemManager
+     * @param MenuFactoryInterface $menuFactory
+     */
+    public function __construct(FormFactoryInterface        $formFactory, EventDispatcherInterface $eventDispatcher,
+                                MenuItemRepositoryInterface $menuItemRepository, MenuItemManager $menuItemManager,
+                                MenuFactoryInterface        $menuFactory) {
     $this->formFactory = $formFactory;
     $this->eventDispatcher = $eventDispatcher;
     $this->menuItemRepository = $menuItemRepository;
     $this->menuItemManager = $menuItemManager;
     $this->menuFactory = $menuFactory;
-  }
+    }
 
 
-  /**
-   * @Route("/api/{version}/menus/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_menu")
-   */
-  public function listAction(Request $request): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/menus/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_menu")
+     */
+    public function listAction(Request $request): ResourcesListResponseInterface {
     $menuRepository = $this->menuItemRepository;
     $page = $request->query->get('page', 1);
     $limit = $request->query->get('limit', 10);
 
     return new ResourcesListResponse($menuRepository->findRootNodes($this->eventDispatcher, $page, $limit));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/menus/{id}/children/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_children_menu")
-   */
-  public function listChildrenAction($id): ResourcesListResponseInterface {
+    /**
+     * @Route("/api/{version}/menus/{id}/children/", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_list_children_menu")
+     */
+    public function listChildrenAction($id): ResourcesListResponseInterface {
     $menuRepository = $this->menuItemRepository;
 
     $menus = $menuRepository->findChildrenAsTree($this->eventDispatcher, $this->findOr404($id));
 
     return new ResourcesListResponse($menus);
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/menus/{id}/move/", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_move_menu", requirements={"id"="\d+"})
-   */
-  public function moveAction(Request $request, $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/menus/{id}/move/", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_move_menu", requirements={"id"="\d+"})
+     */
+    public function moveAction(Request $request, $id): SingleResourceResponseInterface {
     $menuItem = $this->findOr404($id);
     $form = $this->formFactory->createNamed('', MenuItemMoveType::class, [], ['method' => $request->getMethod()]);
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $menuItemManager = $this->menuItemManager;
-      $formData = $form->getData();
+        $menuItemManager = $this->menuItemManager;
+        $formData = $form->getData();
 
-      $menuItemManager->move($menuItem, $formData['parent'], $formData['position']);
+        $menuItemManager->move($menuItem, $formData['parent'], $formData['position']);
 
-      return new SingleResourceResponse($menuItem);
+        return new SingleResourceResponse($menuItem);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/menus/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_menu")
-   */
-  public function getAction($id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/menus/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"GET"}, name="swp_api_core_get_menu")
+     */
+    public function getAction($id): SingleResourceResponseInterface {
     return new SingleResourceResponse($this->findOr404($id));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/menus/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_create_menu")
-   */
-  public function createAction(Request $request): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/menus/", options={"expose"=true}, defaults={"version"="v2"}, methods={"POST"}, name="swp_api_core_create_menu")
+     */
+    public function createAction(Request $request): SingleResourceResponseInterface {
     /* @var MenuItemInterface $menu */
     $menu = $this->menuFactory->create();
     $form = $this->formFactory->createNamed('', MenuType::class, $menu, ['method' => $request->getMethod()]);
@@ -121,20 +121,20 @@ 
     $form->handleRequest($request);
 
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->menuItemManager->update($menu);
-      $this->menuItemRepository->add($menu);
-      $this->eventDispatcher->dispatch(new GenericEvent($menu), MenuEvents::MENU_CREATED);
+        $this->menuItemManager->update($menu);
+        $this->menuItemRepository->add($menu);
+        $this->eventDispatcher->dispatch(new GenericEvent($menu), MenuEvents::MENU_CREATED);
 
-      return new SingleResourceResponse($menu, new ResponseContext(201));
+        return new SingleResourceResponse($menu, new ResponseContext(201));
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/menus/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_core_delete_menu")
-   */
-  public function deleteAction(int $id) {
+    /**
+     * @Route("/api/{version}/menus/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"DELETE"}, name="swp_api_core_delete_menu")
+     */
+    public function deleteAction(int $id) {
     $repository = $this->menuItemRepository;
     $menu = $this->findOr404($id);
 
@@ -142,35 +142,35 @@ 
     $this->eventDispatcher->dispatch(new GenericEvent($menu), MenuEvents::MENU_DELETED);
 
     return new SingleResourceResponse(null, new ResponseContext(204));
-  }
+    }
 
-  /**
-   * @Route("/api/{version}/menus/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_menu")
-   */
-  public function updateAction(Request $request, $id): SingleResourceResponseInterface {
+    /**
+     * @Route("/api/{version}/menus/{id}", options={"expose"=true}, defaults={"version"="v2"}, methods={"PATCH"}, name="swp_api_core_update_menu")
+     */
+    public function updateAction(Request $request, $id): SingleResourceResponseInterface {
     $menu = $this->findOr404($id);
     $parent = $menu->getParent();
     $form = $this->formFactory->createNamed('', MenuType::class, $menu, ['method' => $request->getMethod()]);
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
-      $this->menuItemManager->update($menu);
-      if($parent !== $menu->getParent()) {
-          $this->menuItemManager->update($parent);
-      }
-      $this->menuItemRepository->flush();
-      $this->eventDispatcher->dispatch(new GenericEvent($menu), MenuEvents::MENU_UPDATED);
-      return new SingleResourceResponse($menu);
+        $this->menuItemManager->update($menu);
+        if($parent !== $menu->getParent()) {
+            $this->menuItemManager->update($parent);
+        }
+        $this->menuItemRepository->flush();
+        $this->eventDispatcher->dispatch(new GenericEvent($menu), MenuEvents::MENU_UPDATED);
+        return new SingleResourceResponse($menu);
     }
 
     return new SingleResourceResponse($form, new ResponseContext(400));
-  }
+    }
 
-  private function findOr404($id): MenuItemInterface {
+    private function findOr404($id): MenuItemInterface {
     /* @var MenuItemInterface $menu */
     if (null === $menu = $this->menuItemRepository->findOneBy(['id' => $id])) {
-      throw new NotFoundHttpException('Menu item was not found.');
+        throw new NotFoundHttpException('Menu item was not found.');
     }
 
     return $menu;
-  }
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -154,7 +154,7 @@
     $form->handleRequest($request);
     if ($form->isSubmitted() && $form->isValid()) {
       $this->menuItemManager->update($menu);
-      if($parent !== $menu->getParent()) {
+      if ($parent !== $menu->getParent()) {
           $this->menuItemManager->update($parent);
       }
       $this->menuItemRepository->flush();

src/SWP/Bundle/CoreBundle/Twig/MenuExtension.php

Indentation +32 added lines, -32 removed lines

@@ -21,58 +21,58 @@
 use Twig\TwigFunction;
 
 class MenuExtension extends KnpMenuExtension {
-  /**
-   * {@inheritdoc}
-   */
-  public function getMy($menu, array $path = [], array $options = []): ?ItemInterface {
+    /**
+     * {@inheritdoc}
+     */
+    public function getMy($menu, array $path = [], array $options = []): ?ItemInterface {
     try {
-      return parent::get($menu, $path, $options);
+        return parent::get($menu, $path, $options);
     } catch (\InvalidArgumentException $e) {
-      return null;
+        return null;
+    }
     }
-  }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function render($menu, array $options = [], $renderer = null): string {
+    /**
+     * {@inheritdoc}
+     */
+    public function render($menu, array $options = [], $renderer = null): string {
     try {
-      return parent::render($menu, $options, $renderer);
+        return parent::render($menu, $options, $renderer);
     } catch (\InvalidArgumentException $e) {
-      // allow to render empty value
-      return "";
+        // allow to render empty value
+        return "";
+    }
     }
-  }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function getBreadcrumbsArrayMy($menu, $subItem = null): ?array {
+    /**
+     * {@inheritdoc}
+     */
+    public function getBreadcrumbsArrayMy($menu, $subItem = null): ?array {
     try {
-      return parent::getBreadcrumbsArray($menu, $subItem);
+        return parent::getBreadcrumbsArray($menu, $subItem);
     } catch (\InvalidArgumentException $e) {
-      // allow to render empty value
-      return null;
+        // allow to render empty value
+        return null;
+    }
     }
-  }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function getCurrentItemMy($menu): ?ItemInterface {
+    /**
+     * {@inheritdoc}
+     */
+    public function getCurrentItemMy($menu): ?ItemInterface {
     try {
-      return parent::getCurrentItem($menu);
+        return parent::getCurrentItem($menu);
     } catch (\InvalidArgumentException $e) {
-      return null;
+        return null;
+    }
     }
-  }
 
-  public function getFunctions(): array {
+    public function getFunctions(): array {
     return [
         new TwigFunction('knp_menu_get', [$this, 'getMy']),
         new TwigFunction('knp_menu_render', [$this, 'render'], ['is_safe' => ['html']]),
         new TwigFunction('knp_menu_get_breadcrumbs_array', [$this, 'getBreadcrumbsArrayMy']),
         new TwigFunction('knp_menu_get_current_item', [$this, 'getCurrentItemMy']),
     ];
-  }
+    }
 }