Use register inherited forms

[userbundle] / Controller / AbstractController.php

diff --git a/Controller/AbstractController.php b/Controller/AbstractController.php
index c29690398f86942ef71b04f13751a769f6c4c621..d6f34a2936c475a3296d6d7fc2cb5d83937cb8eb 100644 (file)
--- a/Controller/AbstractController.php
+++ b/Controller/AbstractController.php
@@ -11,15 +11,27 @@
 
 namespace Rapsys\UserBundle\Controller;
 
 
 namespace Rapsys\UserBundle\Controller;
 

+use Doctrine\ORM\EntityManagerInterface;
+use Doctrine\Persistence\ManagerRegistry;

 use Psr\Log\LoggerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController as BaseAbstractController;
 use Symfony\Bundle\FrameworkBundle\Controller\ControllerTrait;
 use Psr\Log\LoggerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController as BaseAbstractController;
 use Symfony\Bundle\FrameworkBundle\Controller\ControllerTrait;

+use Symfony\Bundle\SecurityBundle\Security;

 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;

+use Symfony\Component\HttpFoundation\Request;

 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpFoundation\RequestStack;

+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Mailer\MailerInterface;
+use Symfony\Component\PasswordHasher\Hasher\UserPasswordHasherInterface;

 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 use Symfony\Component\Routing\RouterInterface;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 use Symfony\Component\Routing\RouterInterface;

-use Symfony\Component\Translation\TranslatorInterface;
+use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
+use Symfony\Component\Security\Core\User\UserInterface;

 use Symfony\Contracts\Service\ServiceSubscriberInterface;
 use Symfony\Contracts\Service\ServiceSubscriberInterface;

+use Symfony\Contracts\Translation\TranslatorInterface;
+use Twig\Environment;
+
+use Rapsys\PackBundle\Util\SluggerUtil;

 
 use Rapsys\UserBundle\RapsysUserBundle;
 
 
 use Rapsys\UserBundle\RapsysUserBundle;
 


@@ -29,62 +41,165 @@ use Rapsys\UserBundle\RapsysUserBundle;
  * {@inheritdoc}
  */
 abstract class AbstractController extends BaseAbstractController implements ServiceSubscriberInterface {
  * {@inheritdoc}
  */
 abstract class AbstractController extends BaseAbstractController implements ServiceSubscriberInterface {

-       use ControllerTrait {
-               //Rename render as baseRender
-               render as protected baseRender;
-       }
+       /**
+        * Config array
+        */
+       protected array $config;
+
+       /**
+        * Context array
+        */
+       protected array $context;

 
 

-       ///Config array
-       protected $config;
+       /**
+        * Limit integer
+        */
+       protected int $limit;

 
 

-       ///ContainerInterface instance
-       protected $container;
+       /**
+        * Locale string
+        */
+       protected string $locale;

 
 

-       ///Context array
-       protected $context;
+       /**
+        * Page integer
+        */
+       protected int $page;

 
 

-       ///Router instance
-       protected $router;
+       /**
+        * AuthorizationCheckerInterface instance
+        */
+       protected AuthorizationCheckerInterface $checker;

 
 

-       ///Translator instance
-       protected $translator;
+       /**
+        * ManagerRegistry instance
+        */
+       protected ManagerRegistry $doctrine;

 
 

-       ///Locale
-       protected $locale;
+       /**
+        * UserPasswordHasherInterface instance
+        */
+       protected UserPasswordHasherInterface $hasher;

 
        /**
 
        /**

-        * Common constructor
-        *
-        * Stores container, router and translator interfaces
-        * Stores config
-        * Prepares context tree
+        * LoggerInterface instance
+        */
+       protected LoggerInterface $logger;
+
+       /**
+        * MailerInterface instance
+        */
+       protected MailerInterface $mailer;
+
+       /**
+        * EntityManagerInterface instance
+        */
+       protected EntityManagerInterface $manager;
+
+       /**
+        * Request instance
+        */
+       protected Request $request;
+
+       /**
+        * Router instance
+        */
+       protected RouterInterface $router;
+
+       /**
+        * Security instance
+        */
+       protected Security $security;
+
+       /**
+        * Slugger util instance
+        */
+       protected SluggerUtil $slugger;
+
+       /**
+        * Translator instance
+        */
+       protected TranslatorInterface $translator;
+
+       /**
+        * Twig\Environment instance
+        */
+       protected Environment $twig;
+
+       /**
+        * Abstract constructor

         *
         *

+        * @param AuthorizationCheckerInterface $checker The checker instance

         * @param ContainerInterface $container The container instance
         * @param ContainerInterface $container The container instance

+        * @param ManagerRegistry $doctrine The doctrine instance
+        * @param UserPasswordHasherInterface $hasher The password hasher instance
+        * @param LoggerInterface $logger The logger instance
+        * @param MailerInterface $mailer The mailer instance
+        * @param EntityManagerInterface $manager The manager instance
+        * @param RouterInterface $router The router instance
+        * @param Security $security The security instance
+        * @param SluggerUtil $slugger The slugger instance
+        * @param RequestStack $stack The stack instance
+        * @param TranslatorInterface $translator The translator instance
+        * @param Environment $twig The twig environment instance
+        * @param integer $limit The page limit

         */
         */

-       public function __construct(ContainerInterface $container) {
+       public function __construct(AuthorizationCheckerInterface $checker, ContainerInterface $container, ManagerRegistry $doctrine, UserPasswordHasherInterface $hasher, LoggerInterface $logger, MailerInterface $mailer, EntityManagerInterface $manager, RouterInterface $router, Security $security, SluggerUtil $slugger, RequestStack $stack, TranslatorInterface $translator, Environment $twig, int $limit = 5) {

                //Retrieve config
                $this->config = $container->getParameter(RapsysUserBundle::getAlias());
 
                //Retrieve config
                $this->config = $container->getParameter(RapsysUserBundle::getAlias());
 

-               //Set the container
+               //Set checker
+               $this->checker = $checker;
+
+               //Set container

                $this->container = $container;
 
                $this->container = $container;
 

-               //Set the router
-               $this->router = $container->get('router');
+               //Set doctrine
+               $this->doctrine = $doctrine;
+
+               //Set hasher
+               $this->hasher = $hasher;
+
+               //Set logger
+               $this->logger = $logger;

 
 

-               //Set the translator
-               $this->translator = $container->get('translator');
+               //Set limit
+               $this->limit = $limit;

 
 

-               //Get request stack
-               $stack = $this->container->get('request_stack');
+               //Set mailer
+               $this->mailer = $mailer;
+
+               //Set manager
+               $this->manager = $manager;
+
+               //Set router
+               $this->router = $router;
+
+               //Set security
+               $this->security = $security;
+
+               //Set slugger
+               $this->slugger = $slugger;
+
+               //Set translator
+               $this->translator = $translator;
+
+               //Set twig
+               $this->twig = $twig;

 
                //Get current request
 
                //Get current request

-               $request = $stack->getCurrentRequest();
+               $this->request = $stack->getCurrentRequest();

 
 

-               //Get current locale
-               $this->locale = $request->getLocale();
+               //Get current page
+               $this->page = (int) $this->request->query->get('page');

 
 

-               //Set locale
-               $this->config['context']['locale'] = str_replace('_', '-', $this->locale);
+               //With negative page
+               if ($this->page < 0) {
+                       $this->page = 0;
+               }
+
+               //Get current locale
+               $this->locale = $this->request->getLocale();

 
                //Set translate array
                $translates = [];
 
                //Set translate array
                $translates = [];


@@ -95,10 +210,12 @@ abstract class AbstractController extends BaseAbstractController implements Serv
                        foreach($this->config['translate'] as $translate) {
                                //Set tmp
                                $tmp = null;
                        foreach($this->config['translate'] as $translate) {
                                //Set tmp
                                $tmp = null;

+

                                //Iterate on keys
                                foreach(array_reverse(explode('.', $translate)) as $curkey) {
                                        $tmp = array_combine([$curkey], [$tmp]);
                                }
                                //Iterate on keys
                                foreach(array_reverse(explode('.', $translate)) as $curkey) {
                                        $tmp = array_combine([$curkey], [$tmp]);
                                }

+

                                //Append tree
                                $translates = array_replace_recursive($translates, $tmp);
                        }
                                //Append tree
                                $translates = array_replace_recursive($translates, $tmp);
                        }


@@ -106,12 +223,6 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 
                //Inject every requested route in view and mail context
                foreach($this->config as $tag => $current) {
 
                //Inject every requested route in view and mail context
                foreach($this->config as $tag => $current) {

-                       //Look for entry with title subkey
-                       if (!empty($current['title'])) {
-                               //Translate title value
-                               $this->config[$tag]['title'] = $this->translator->trans($current['title']);
-                       }
-

                        //Look for entry with route subkey
                        if (!empty($current['route'])) {
                                //Generate url for both view and mail
                        //Look for entry with route subkey
                        if (!empty($current['route'])) {
                                //Generate url for both view and mail


@@ -198,12 +309,12 @@ abstract class AbstractController extends BaseAbstractController implements Serv
                                                        $pathInfo = $this->router->getContext()->getPathInfo();
 
                                                        //Iterate on locales excluding current one
                                                        $pathInfo = $this->router->getContext()->getPathInfo();
 
                                                        //Iterate on locales excluding current one

-                                                       foreach($this->config['locales'] as $locale) {
+                                                       foreach(($locales = array_keys($this->config['languages'])) as $locale) {

                                                                //Set titles
                                                                $titles = [];
 
                                                                //Iterate on other locales
                                                                //Set titles
                                                                $titles = [];
 
                                                                //Iterate on other locales

-                                                               foreach(array_diff($this->config['locales'], [$locale]) as $other) {
+                                                               foreach(array_diff($locales, [$locale]) as $other) {

                                                                        $titles[$other] = $this->translator->trans($this->config['languages'][$locale], [], null, $other);
                                                                }
 
                                                                        $titles[$other] = $this->translator->trans($this->config['languages'][$locale], [], null, $other);
                                                                }
 


@@ -219,10 +330,10 @@ abstract class AbstractController extends BaseAbstractController implements Serv
                                                                //With current locale
                                                                if ($locale == $this->locale) {
                                                                        //Set locale locales context
                                                                //With current locale
                                                                if ($locale == $this->locale) {
                                                                        //Set locale locales context

-                                                                       $this->config[$tag][$view]['context']['canonical'] = $this->router->generate($name, ['_locale' => $locale]+$route, UrlGeneratorInterface::ABSOLUTE_URL);
+                                                                       $this->config[$tag][$view]['context']['head']['canonical'] = $this->router->generate($name, ['_locale' => $locale]+$route, UrlGeneratorInterface::ABSOLUTE_URL);

                                                                } else {
                                                                        //Set locale locales context
                                                                } else {
                                                                        //Set locale locales context

-                                                                       $this->config[$tag][$view]['context']['alternates'][$locale] = [
+                                                                       $this->config[$tag][$view]['context']['head']['alternates'][$locale] = [

                                                                                'absolute' => $this->router->generate($name, ['_locale' => $locale]+$route, UrlGeneratorInterface::ABSOLUTE_URL),
                                                                                'relative' => $this->router->generate($name, ['_locale' => $locale]+$route),
                                                                                'title' => implode('/', $titles),
                                                                                'absolute' => $this->router->generate($name, ['_locale' => $locale]+$route, UrlGeneratorInterface::ABSOLUTE_URL),
                                                                                'relative' => $this->router->generate($name, ['_locale' => $locale]+$route),
                                                                                'title' => implode('/', $titles),


@@ -231,9 +342,9 @@ abstract class AbstractController extends BaseAbstractController implements Serv
                                                                }
 
                                                                //Add shorter locale
                                                                }
 
                                                                //Add shorter locale

-                                                               if (empty($this->config[$tag][$view]['context']['alternates'][$slocale = substr($locale, 0, 2)])) {
+                                                               if (empty($this->config[$tag][$view]['context']['head']['alternates'][$slocale = substr($locale, 0, 2)])) {

                                                                        //Add shorter locale
                                                                        //Add shorter locale

-                                                                       $this->config[$tag][$view]['context']['alternates'][$slocale] = [
+                                                                       $this->config[$tag][$view]['context']['head']['alternates'][$slocale] = [

                                                                                'absolute' => $this->router->generate($name, ['_locale' => $locale]+$route, UrlGeneratorInterface::ABSOLUTE_URL),
                                                                                'relative' => $this->router->generate($name, ['_locale' => $locale]+$route),
                                                                                'title' => implode('/', $titles),
                                                                                'absolute' => $this->router->generate($name, ['_locale' => $locale]+$route, UrlGeneratorInterface::ABSOLUTE_URL),
                                                                                'relative' => $this->router->generate($name, ['_locale' => $locale]+$route),
                                                                                'title' => implode('/', $titles),


@@ -248,6 +359,51 @@ abstract class AbstractController extends BaseAbstractController implements Serv
                }
        }
 
                }
        }
 

+       /**
+        * Renders a view
+        *
+        * {@inheritdoc}
+        */
+       protected function render(string $view, array $parameters = [], Response $response = null): Response {
+               //Create response when null
+               $response ??= new Response();
+
+               //With empty head locale
+               if (empty($parameters['head']['locale'])) {
+                       //Set head locale
+                       $parameters['head']['locale'] = $this->locale;
+               }
+
+               //With empty head title and section
+               if (empty($parameters['head']['title']) && !empty($parameters['section'])) {
+                       //Set head title
+                       $parameters['head']['title'] = implode(' - ', [$parameters['title'], $parameters['section'], $parameters['head']['site']]);
+               //With empty head title
+               } elseif (empty($parameters['head']['title'])) {
+                       //Set head title
+                       $parameters['head']['title'] = implode(' - ', [$parameters['title'], $parameters['head']['site']]);
+               }
+
+               //Call twig render method
+               $content = $this->twig->render($view, $parameters);
+
+               //Invalidate OK response on invalid form
+               if (200 === $response->getStatusCode()) {
+                       foreach ($parameters as $v) {
+                               if ($v instanceof FormInterface && $v->isSubmitted() && !$v->isValid()) {
+                                       $response->setStatusCode(422);
+                                       break;
+                               }
+                       }
+               }
+
+               //Store content in response
+               $response->setContent($content);
+
+               //Return response
+               return $response;
+       }
+

        /**
         * {@inheritdoc}
         *
        /**
         * {@inheritdoc}
         *


@@ -256,9 +412,17 @@ abstract class AbstractController extends BaseAbstractController implements Serv
        public static function getSubscribedServices(): array {
                //Return subscribed services
                return [
        public static function getSubscribedServices(): array {
                //Return subscribed services
                return [

+                       'doctrine' => ManagerRegistry::class,
+                       'doctrine.orm.default_entity_manager' => EntityManagerInterface::class,

                        'logger' => LoggerInterface::class,
                        'logger' => LoggerInterface::class,

+                       'mailer.mailer' => MailerInterface::class,
+                       'rapsys_pack.slugger_util' => SluggerUtil::class,

                        'request_stack' => RequestStack::class,
                        'router' => RouterInterface::class,
                        'request_stack' => RequestStack::class,
                        'router' => RouterInterface::class,

+                       'security.authorization_checker' => AuthorizationCheckerInterface::class,
+                       'security' => Security::class,
+                       'security.user_password_hasher' => UserPasswordHasherInterface::class,
+                       'service_container' => ContainerInterface::class,

                        'translator' => TranslatorInterface::class
                ];
        }
                        'translator' => TranslatorInterface::class
                ];
        }