superdesk / web-publisher

src/SWP/Bundle/CoreBundle/Context/CachedTenantContext.php

Indentation +13 added lines, -13 removed lines

@@ -24,43 +24,43 @@
 use SWP\Component\MultiTenancy\Model\TenantInterface;
 
 class CachedTenantContext extends TenantContext implements CachedTenantContextInterface {
-  private $resolvedTenants = [];
+    private $resolvedTenants = [];
 
-  public function getTenant(): ?TenantInterface {
+    public function getTenant(): ?TenantInterface {
     $currentRequest = $this->requestStack->getCurrentRequest();
     if ($currentRequest && $this->requestStack->getCurrentRequest()->attributes->get('exception') instanceof TenantNotFoundException) {
-      return null;
+        return null;
     }
 
     if ($currentRequest === null) {
-      return $this->tenant;
+        return $this->tenant;
     }
 
     $cacheKey = self::getCacheKey($currentRequest->getHost());
     if (!array_key_exists($cacheKey, $this->resolvedTenants) || $this->resolvedTenants[$cacheKey] instanceof TenantInterface) {
-      $this->resolvedTenants[$cacheKey] = parent::getTenant();
+        $this->resolvedTenants[$cacheKey] = parent::getTenant();
     } else {
-      $this->tenant = $this->resolvedTenants[$cacheKey];
+        $this->tenant = $this->resolvedTenants[$cacheKey];
     }
 
     return $this->tenant;
-  }
+    }
 
-  public function setTenant(TenantInterface $tenant): void {
+    public function setTenant(TenantInterface $tenant): void {
     parent::setTenant($tenant);
     $this->dispatcher->dispatch(new GenericEvent(), MultiTenancyEvents::TENANTABLE_ENABLE);
 
     $this->resolvedTenants[self::getCacheKey(
         $tenant->getSubdomain() ? $tenant->getSubdomain() . '.' . $tenant->getDomainName() : $tenant->getDomainName()
     )] = $tenant;
-  }
+    }
 
-  public function reset(): void {
+    public function reset(): void {
     $this->tenant = null;
     $this->resolvedTenants = [];
-  }
+    }
 
-  private static function getCacheKey(string $host): string {
+    private static function getCacheKey(string $host): string {
     return md5('tenant_cache__' . $host);
-  }
+    }
 }

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

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

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/Twig/DecoratingRoutingExtension.php

Indentation +33 added lines, -33 removed lines

@@ -36,88 +36,88 @@
 use Twig\TwigFunction;
 
 final class DecoratingRoutingExtension extends AbstractExtension {
-  private RoutingExtension $routingExtension;
+    private RoutingExtension $routingExtension;
 
-  public function __construct(RoutingExtension $routingExtension) {
+    public function __construct(RoutingExtension $routingExtension) {
     $this->routingExtension = $routingExtension;
-  }
+    }
 
-  public function getPath($name, $parameters = [], $relative = false): ?string {
+    public function getPath($name, $parameters = [], $relative = false): ?string {
     if ($name == null) {
-      return null;
+        return null;
     }
 
     if (is_object($name)) {
-      $object = $name;
-      $name = null;
-      self::setupParams($object, $name, $parameters);
+        $object = $name;
+        $name = null;
+        self::setupParams($object, $name, $parameters);
     }
 
     try {
-      return $this->routingExtension->getPath($name, $parameters, $relative);
+        return $this->routingExtension->getPath($name, $parameters, $relative);
     } catch (RouteNotFoundException|MissingMandatoryParametersException|InvalidParameterException $e) {
-      // allow empty path
+        // allow empty path
     }
 
     return null;
-  }
+    }
 
-  public function getUrl($name, $parameters = [], $schemeRelative = false): ?string {
+    public function getUrl($name, $parameters = [], $schemeRelative = false): ?string {
     if ($name == null) {
-      return null;
+        return null;
     }
 
     if (is_object($name)) {
-      $object = $name;
-      $name = null;
-      self::setupParams($object, $name, $parameters);
+        $object = $name;
+        $name = null;
+        self::setupParams($object, $name, $parameters);
     }
 
     try {
-      return $this->routingExtension->getUrl($name, $parameters, $schemeRelative);
+        return $this->routingExtension->getUrl($name, $parameters, $schemeRelative);
     } catch (RouteNotFoundException|MissingMandatoryParametersException|InvalidParameterException $e) {
-      // allow empty url
+        // allow empty url
     }
 
     return null;
-  }
+    }
 
-  public static function setupParams(object $object, &$name, &$parameters) {
+    public static function setupParams(object $object, &$name, &$parameters) {
     $name = RouteObjectInterface::OBJECT_BASED_ROUTE_NAME;
     $parameters[RouteObjectInterface::ROUTE_OBJECT] = $object;
 
     if ($object instanceof Meta) {
-      $values = $object->getValues();
-      if (($values instanceof ArticleMediaInterface || $values instanceof ImageRenditionInterface)) {
+        $values = $object->getValues();
+        if (($values instanceof ArticleMediaInterface || $values instanceof ImageRenditionInterface)) {
         $name = MediaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
 
-      if ($values instanceof ArticleSeoMediaInterface) {
+        if ($values instanceof ArticleSeoMediaInterface) {
         $name = SeoMediaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
 
-      if ($values instanceof AuthorMediaInterface) {
+        if ($values instanceof AuthorMediaInterface) {
         $name = ArticleAuthorMediaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
 
-      if ($values instanceof ArticleInterface || $values instanceof RouteInterface) {
+        if ($values instanceof ArticleInterface || $values instanceof RouteInterface) {
         $name = MetaRouter::OBJECT_BASED_ROUTE_NAME;
         return;
-      }
+        }
     }
 
     if ($object instanceof RouteInterface || $object instanceof ArticleInterface) {
-      $name = MetaRouter::OBJECT_BASED_ROUTE_NAME;
+        $name = MetaRouter::OBJECT_BASED_ROUTE_NAME;
+    }
     }
-  }
 
-  public function getFunctions(): array {
+    public function getFunctions(): array {
     return [
         new TwigFunction('url', [$this, 'getUrl'], ['is_safe_callback' => [$this->routingExtension, 'isUrlGenerationSafe']]),
         new TwigFunction('path', [$this, 'getPath'], ['is_safe_callback' => [$this->routingExtension, 'isUrlGenerationSafe']]),
     ];
-  }
+    }
 }

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

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