X-Git-Url: https://git.rapsys.eu/userbundle/blobdiff_plain/e76a1923e813fdf1fd24115ad0f976d2df58cc6f..5305ca069746c39ab9271fdf4ee9e14287237c21:/Controller/AbstractController.php?ds=sidebyside diff --git a/Controller/AbstractController.php b/Controller/AbstractController.php index cdbbf12..d6f34a2 100644 --- a/Controller/AbstractController.php +++ b/Controller/AbstractController.php @@ -16,14 +16,20 @@ use Doctrine\Persistence\ManagerRegistry; 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\HttpFoundation\Request; 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\PasswordHasher\Hasher\UserPasswordHasherInterface; -use Symfony\Contracts\Translation\TranslatorInterface; +use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface; +use Symfony\Component\Security\Core\User\UserInterface; use Symfony\Contracts\Service\ServiceSubscriberInterface; +use Symfony\Contracts\Translation\TranslatorInterface; +use Twig\Environment; use Rapsys\PackBundle\Util\SluggerUtil; @@ -35,42 +41,95 @@ use Rapsys\UserBundle\RapsysUserBundle; * {@inheritdoc} */ abstract class AbstractController extends BaseAbstractController implements ServiceSubscriberInterface { - ///Config array - protected $config; + /** + * Config array + */ + protected array $config; - ///Context array - protected $context; + /** + * Context array + */ + protected array $context; - ///ManagerRegistry - protected $doctrine; + /** + * Limit integer + */ + protected int $limit; - ///UserPasswordHasherInterface - protected $hasher; + /** + * Locale string + */ + protected string $locale; - ///LoggerInterface - protected $logger; + /** + * Page integer + */ + protected int $page; - ///MailerInterface - protected $mailer; + /** + * AuthorizationCheckerInterface instance + */ + protected AuthorizationCheckerInterface $checker; - ///EntityManagerInterface - protected $manager; + /** + * ManagerRegistry instance + */ + protected ManagerRegistry $doctrine; - ///Router instance - protected $router; + /** + * UserPasswordHasherInterface instance + */ + protected UserPasswordHasherInterface $hasher; + + /** + * 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 - protected $slugger; + /** + * Slugger util instance + */ + protected SluggerUtil $slugger; - ///Translator instance - protected $translator; + /** + * Translator instance + */ + protected TranslatorInterface $translator; - ///Locale - protected $locale; + /** + * Twig\Environment instance + */ + protected Environment $twig; /** * Abstract constructor * + * @param AuthorizationCheckerInterface $checker The checker instance * @param ContainerInterface $container The container instance * @param ManagerRegistry $doctrine The doctrine instance * @param UserPasswordHasherInterface $hasher The password hasher instance @@ -78,14 +137,20 @@ abstract class AbstractController extends BaseAbstractController implements Serv * @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, ManagerRegistry $doctrine, UserPasswordHasherInterface $hasher, LoggerInterface $logger, MailerInterface $mailer, EntityManagerInterface $manager, RouterInterface $router, SluggerUtil $slugger, RequestStack $stack, TranslatorInterface $translator) { + 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()); + //Set checker + $this->checker = $checker; + //Set container $this->container = $container; @@ -98,6 +163,9 @@ abstract class AbstractController extends BaseAbstractController implements Serv //Set logger $this->logger = $logger; + //Set limit + $this->limit = $limit; + //Set mailer $this->mailer = $mailer; @@ -107,20 +175,31 @@ abstract class AbstractController extends BaseAbstractController implements Serv //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 - $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 = []; @@ -131,10 +210,12 @@ abstract class AbstractController extends BaseAbstractController implements Serv 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]); } + //Append tree $translates = array_replace_recursive($translates, $tmp); } @@ -142,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) { - //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 @@ -234,12 +309,12 @@ abstract class AbstractController extends BaseAbstractController implements Serv $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 - 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); } @@ -255,10 +330,10 @@ abstract class AbstractController extends BaseAbstractController implements Serv //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 - $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), @@ -267,9 +342,9 @@ abstract class AbstractController extends BaseAbstractController implements Serv } //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 - $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), @@ -284,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} * @@ -292,7 +412,6 @@ abstract class AbstractController extends BaseAbstractController implements Serv public static function getSubscribedServices(): array { //Return subscribed services return [ - 'service_container' => ContainerInterface::class, 'doctrine' => ManagerRegistry::class, 'doctrine.orm.default_entity_manager' => EntityManagerInterface::class, 'logger' => LoggerInterface::class, @@ -300,7 +419,10 @@ abstract class AbstractController extends BaseAbstractController implements Serv 'rapsys_pack.slugger_util' => SluggerUtil::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 ]; }