X-Git-Url: https://git.rapsys.eu/airbundle/blobdiff_plain/bc28bb466c3b8f25bdee997270890297d6086a00..11f4f833ffd015afd0b06e4d5f542a1f81b0a1c9:/Controller/AbstractController.php?ds=sidebyside

diff --git a/Controller/AbstractController.php b/Controller/AbstractController.php
index 6fbac61..8a10b87 100644
--- a/Controller/AbstractController.php
+++ b/Controller/AbstractController.php
@@ -1,7 +1,7 @@
 <?php declare(strict_types=1);
 
 /*
- * This file is part of the Rapsys UserBundle package.
+ * This file is part of the Rapsys AirBundle package.
  *
  * (c) Raphaël Gertz <symfony@rapsys.eu>
  *
@@ -11,37 +11,44 @@
 
 namespace Rapsys\AirBundle\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 Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
 use Symfony\Component\Asset\PackageInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\Filesystem\Exception\IOExceptionInterface;
 use Symfony\Component\Filesystem\Filesystem;
+use Symfony\Component\Form\FormFactoryInterface;
 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Mailer\MailerInterface;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 use Symfony\Component\Routing\RouterInterface;
-use Symfony\Component\Translation\TranslatorInterface;
+use Symfony\Component\Security\Core\Authorization\AccessDecisionManagerInterface;
+use Symfony\Contracts\Translation\TranslatorInterface;
 use Symfony\Contracts\Service\ServiceSubscriberInterface;
 
+use Rapsys\AirBundle\Entity\Dance;
 use Rapsys\AirBundle\Entity\Location;
 use Rapsys\AirBundle\Entity\Slot;
 use Rapsys\AirBundle\Entity\User;
 use Rapsys\AirBundle\RapsysAirBundle;
 
+use Rapsys\PackBundle\Util\FacebookUtil;
+use Rapsys\PackBundle\Util\ImageUtil;
+use Rapsys\PackBundle\Util\MapUtil;
+use Rapsys\PackBundle\Util\SluggerUtil;
+
 /**
- *
  * Provides common features needed in controllers.
  *
  * {@inheritdoc}
  */
 abstract class AbstractController extends BaseAbstractController implements ServiceSubscriberInterface {
-	use ControllerTrait {
-		//Rename render as baseRender
-		render as protected baseRender;
-	}
+	///AuthorizationCheckerInterface instance
+	protected $checker;
 
 	///Config array
 	protected $config;
@@ -52,39 +59,186 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 	///Context array
 	protected $context;
 
+	///AccessDecisionManagerInterface instance
+	protected $decision;
+
+	///ManagerRegistry instance
+	protected $doctrine;
+
+	///FacebookUtil instance
+	protected $facebook;
+
+	///FormFactoryInterface instance
+	protected $factory;
+
+	///Image util instance
+	protected $image;
+
+	///Locale string
+	protected $locale;
+
+	///MailerInterface instance
+	protected $mailer;
+
+	///EntityManagerInterface instance
+	protected $manager;
+
+	///Map util instance
+	protected $map;
+
+	///Modified DateTime
+	protected $modified;
+
+	///PackageInterface instance
+	protected $package;
+
+	///DatePeriod instance
+	protected $period;
+
+	///Request instance
+	protected $request;
+
+	///Route string
+	protected $route;
+
+	///Route params array
+	protected $routeParams;
+
 	///Router instance
 	protected $router;
 
+	///Slugger util instance
+	protected $slugger;
+
+	///RequestStack instance
+	protected $stack;
+
 	///Translator instance
 	protected $translator;
 
 	/**
-	 * Common constructor
-	 *
-	 * Stores container, router and translator interfaces
-	 * Stores config
-	 * Prepares context tree
+	 * Abstract constructor
 	 *
+	 * @param AuthorizationCheckerInterface $checker The container instance
 	 * @param ContainerInterface $container The container instance
+	 * @param AccessDecisionManagerInterface $decision The decision instance
+	 * @param ManagerRegistry $doctrine The doctrine instance
+	 * @param FacebookUtil $facebook The facebook instance
+	 * @param FormFactoryInterface $factory The factory instance
+	 * @param ImageUtil $image The image instance
+	 * @param MailerInterface $mailer The mailer instance
+	 * @param EntityManagerInterface $manager The manager instance
+	 * @param MapUtil $map The map instance
+	 * @param PackageInterface $package The package instance
+	 * @param RouterInterface $router The router instance
+	 * @param SluggerUtil $slugger The slugger instance
+	 * @param RequestStack $stack The stack instance
+	 * @param TranslatorInterface $translator The translator instance
+	 *
+	 * @TODO move all that stuff to setSlugger('@slugger') setters with a calls: [ setSlugger: [ '@slugger' ] ] to unbload classes ???
+	 * @TODO add a calls: [ ..., prepare: ['@???'] ] that do all the logic that can't be done in constructor because various things are not available
 	 */
-	public function __construct(ContainerInterface $container) {
+	public function __construct(AuthorizationCheckerInterface $checker, ContainerInterface $container, AccessDecisionManagerInterface $decision, ManagerRegistry $doctrine, FacebookUtil $facebook, FormFactoryInterface $factory, ImageUtil $image, MailerInterface $mailer, EntityManagerInterface $manager, MapUtil $map, PackageInterface $package, RouterInterface $router, SluggerUtil $slugger, RequestStack $stack, TranslatorInterface $translator) {
+		//Set checker
+		$this->checker = $checker;
+
 		//Retrieve config
 		$this->config = $container->getParameter(RapsysAirBundle::getAlias());
 
 		//Set the container
 		$this->container = $container;
 
-		//Set the router
-		$this->router = $container->get('router');
+		//Set decision
+		$this->decision = $decision;
+
+		//Set doctrine
+		$this->doctrine = $doctrine;
+
+		//Set facebook
+		$this->facebook = $facebook;
+
+		//Set factory
+		$this->factory = $factory;
+
+		//Set image
+		$this->image = $image;
+
+		//Set mailer
+		$this->mailer = $mailer;
+
+		//Set manager
+		$this->manager = $manager;
+
+		//Set map
+		$this->map = $map;
+
+		//Set package
+		$this->package = $package;
+
+		//Set period
+		$this->period = new \DatePeriod(
+			//Start from first monday of week
+			new \DateTime('Monday this week'),
+			//Iterate on each day
+			new \DateInterval('P1D'),
+			//End with next sunday and 4 weeks
+			//XXX: we can't use isGranted here as AuthenticatedVoter deny access because user is likely not authenticated yet :'(
+			new \DateTime('Monday this week + 2 week')
+		);
+
+
+		//Set router
+		$this->router = $router;
+
+		//Set slugger
+		$this->slugger = $slugger;
+
+		//Set stack
+		$this->stack = $stack;
+
+		//Set translator
+		$this->translator = $translator;
+
+		//Get main request
+		$this->request = $this->stack->getMainRequest();
+
+		//Get current locale
+		$this->locale = $this->request->getLocale();
+
+		//Set canonical
+		$canonical = null;
+
+		//Set alternates
+		$alternates = [];
+
+		//Set route
+		//TODO: default to not found route ???
+		//TODO: pour une url not found, cet attribut n'est pas défini, comment on fait ???
+		//XXX: on génère une route bidon par défaut ???
+		$this->route = $this->request->attributes->get('_route');
 
-		//Set the translator
-		$this->translator = $container->get('translator');
+		//Set route params
+		$this->routeParams = $this->request->attributes->get('_route_params');
+
+		//With route and routeParams
+		if ($this->route !== null && $this->routeParams !== null) {
+			//Set canonical
+			$canonical = $this->router->generate($this->route, $this->routeParams, UrlGeneratorInterface::ABSOLUTE_URL);
+
+			//Set alternates
+			$alternates = [
+				substr($this->locale, 0, 2) => [
+					'absolute' => $canonical
+				]
+			];
+		}
 
 		//Set the context
 		$this->context = [
 			'description' => null,
 			'section' => null,
 			'title' => null,
+			'locale' => str_replace('_', '-', $this->locale),
 			'contact' => [
 				'title' => $this->translator->trans($this->config['contact']['title']),
 				'mail' => $this->config['contact']['mail']
@@ -98,427 +252,70 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 			],
 			'site' => [
 				'donate' => $this->config['site']['donate'],
-				'ico' => $this->config['site']['ico'],
+				'icon' => $this->config['site']['icon'],
 				'logo' => $this->config['site']['logo'],
 				'png' => $this->config['site']['png'],
-				'svg' => $this->config['site']['svg'],
-				'title' => $this->translator->trans($this->config['site']['title']),
+				'title' => $title = $this->translator->trans($this->config['site']['title']),
 				'url' => $this->router->generate($this->config['site']['url'])
 			],
-			'canonical' => null,
-			'alternates' => [],
+			'canonical' => $canonical,
+			'alternates' => $alternates,
 			'facebook' => [
-				'prefixes' => [
-					'og' => 'http://ogp.me/ns#',
-					'fb' => 'http://ogp.me/ns/fb#'
-				],
 				'metas' => [
 					'og:type' => 'article',
-					'og:site_name' => $this->translator->trans($this->config['site']['title']),
+					'og:site_name' => $title,
+					'og:url' => $canonical,
 					#'fb:admins' => $this->config['facebook']['admins'],
 					'fb:app_id' => $this->config['facebook']['apps']
 				],
-				'texts' => []
+				'texts' => [
+					$this->translator->trans($this->config['site']['title']) => [
+						'font' => 'irishgrover',
+						'size' => 110
+					]
+				]
 			],
 			'forms' => []
 		];
 	}
 
-	/**
-	 * Return the facebook image
-	 *
-	 * @desc Generate image in jpeg format or load it from cache
-	 *
-	 * @param string $pathInfo The request path info
-	 * @param array $parameters The image parameters
-	 * @return array The image array
-	 */
-	protected function getFacebookImage(string $pathInfo, array $parameters = []): array {
-		//Get asset package
-		//XXX: require asset package to be public
-		$package = $this->container->get('rapsys_pack.path_package');
-
-		//Set texts
-		$texts = $parameters['texts'] ?? [];
-
-		//Set default source
-		$source = $parameters['source'] ?? 'png/facebook.png';
-
-		//Set default source
-		$updated = $parameters['updated'] ?? strtotime('last week');
-
-		//Set source path
-		$src = $this->config['path']['public'].'/'.$source;
-
-		//Set cache path
-		//XXX: remove extension and store as png anyway
-		$cache = $this->config['path']['cache'].'/facebook/'.substr($source, 0, strrpos($source, '.')).'.'.$this->config['facebook']['width'].'x'.$this->config['facebook']['height'].'.png';
-
-		//Set destination path
-		//XXX: format <public>/facebook<pathinfo>.jpeg
-		//XXX: was <public>/facebook/<controller>/<action>.<locale>.jpeg
-		$dest = $this->config['path']['public'].'/facebook'.$pathInfo.'.jpeg';
-
-		//With up to date generated image
-		if (
-			is_file($dest) &&
-			($stat = stat($dest)) &&
-			$stat['mtime'] >= $updated
-		) {
-			//Get image size
-			list ($width, $height) = getimagesize($dest);
-
-			//Iterate each text
-			foreach($texts as $text => $data) {
-				//With canonical text
-				if (!empty($data['canonical'])) {
-					//Prevent canonical to finish in alt
-					unset($texts[$text]);
-				}
-			}
-
-			//Return image data
-			return [
-				'og:image' => $package->getAbsoluteUrl('@RapsysAir/facebook/'.$stat['mtime'].$pathInfo.'.jpeg'),
-				'og:image:alt' => str_replace("\n", ' ', implode(' - ', array_keys($texts))),
-				'og:image:height' => $height,
-				'og:image:width' => $width
-			];
-		//With image candidate
-		} elseif (is_file($src)) {
-			//Create image object
-			$image = new \Imagick();
-
-			//With cache image
-			if (is_file($cache)) {
-				//Read image
-				$image->readImage($cache);
-			//Without we generate it
-			} else {
-				//Check target directory
-				if (!is_dir($dir = dirname($cache))) {
-					//Create filesystem object
-					$filesystem = new Filesystem();
-
-					try {
-						//Create dir
-						//XXX: set as 0775, symfony umask (0022) will reduce rights (0755)
-						$filesystem->mkdir($dir, 0775);
-					} catch (IOExceptionInterface $e) {
-						//Throw error
-						throw new \Exception(sprintf('Output directory "%s" do not exists and unable to create it', $dir), 0, $e);
-					}
-				}
-
-				//Read image
-				$image->readImage($src);
-
-				//Crop using aspect ratio
-				//XXX: for better result upload image directly in aspect ratio :)
-				$image->cropThumbnailImage($this->config['facebook']['width'], $this->config['facebook']['height']);
-
-				//Strip image exif data and properties
-				$image->stripImage();
-
-				//Save cache image
-				if (!$image->writeImage($cache)) {
-					//Throw error
-					throw new \Exception(sprintf('Unable to write image "%s"', $cache));
-				}
-			}
-			//Check target directory
-			if (!is_dir($dir = dirname($dest))) {
-				//Create filesystem object
-				$filesystem = new Filesystem();
-
-				try {
-					//Create dir
-					//XXX: set as 0775, symfony umask (0022) will reduce rights (0755)
-					$filesystem->mkdir($dir, 0775);
-				} catch (IOExceptionInterface $e) {
-					//Throw error
-					throw new \Exception(sprintf('Output directory "%s" do not exists and unable to create it', $dir), 0, $e);
-				}
-			}
-
-			//Get image width
-			$width = $image->getImageWidth();
-
-			//Get image height
-			$height = $image->getImageHeight();
-
-			//Create draw
-			$draw = new \ImagickDraw();
-
-			//Set stroke antialias
-			$draw->setStrokeAntialias(true);
-
-			//Set text antialias
-			$draw->setTextAntialias(true);
-
-			//Set font aliases
-			$fonts = [
-				'irishgrover' => $this->config['path']['public'].'/ttf/irishgrover.v10.ttf',
-				'droidsans' => $this->config['path']['public'].'/ttf/droidsans.regular.ttf',
-				'dejavusans' => $this->config['path']['public'].'/ttf/dejavusans.2.37.ttf',
-				'labelleaurore' => $this->config['path']['public'].'/ttf/labelleaurore.v10.ttf'
-			];
-
-			//Set align aliases
-			$aligns = [
-				'left' => \Imagick::ALIGN_LEFT,
-				'center' => \Imagick::ALIGN_CENTER,
-				'right' => \Imagick::ALIGN_RIGHT
-			];
-
-			//Set default font
-			$defaultFont = 'dejavusans';
-
-			//Set default align
-			$defaultAlign = 'center';
-
-			//Set default size
-			$defaultSize = 60;
-
-			//Set default stroke
-			$defaultStroke = '#00c3f9';
-
-			//Set default width
-			$defaultWidth = 15;
-
-			//Set default fill
-			$defaultFill = 'white';
-
-			//Init counter
-			$i = 1;
-
-			//Set text count
-			$count = count($texts);
-
-			//Draw each text stroke
-			foreach($texts as $text => $data) {
-				//Set font
-				$draw->setFont($fonts[$data['font']??$defaultFont]);
-
-				//Set font size
-				$draw->setFontSize($data['size']??$defaultSize);
-
-				//Set stroke width
-				$draw->setStrokeWidth($data['width']??$defaultWidth);
-
-				//Set text alignment
-				$draw->setTextAlignment($align = ($aligns[$data['align']??$defaultAlign]));
-
-				//Get font metrics
-				$metrics = $image->queryFontMetrics($draw, $text);
-
-				//Without y
-				if (empty($data['y'])) {
-					//Position verticaly each text evenly
-					$texts[$text]['y'] = $data['y'] = (($height + 100) / (count($texts) + 1) * $i) - 50;
-				}
-
-				//Without x
-				if (empty($data['x'])) {
-					if ($align == \Imagick::ALIGN_CENTER) {
-						$texts[$text]['x'] = $data['x'] = $width/2;
-					} elseif ($align == \Imagick::ALIGN_LEFT) {
-						$texts[$text]['x'] = $data['x'] = 50;
-					} elseif ($align == \Imagick::ALIGN_RIGHT) {
-						$texts[$text]['x'] = $data['x'] = $width - 50;
-					}
-				}
-
-				//Center verticaly
-				//XXX: add ascender part then center it back by half of textHeight
-				//TODO: maybe add a boundingbox ???
-				$texts[$text]['y'] = $data['y'] += $metrics['ascender'] - $metrics['textHeight']/2;
-
-				//Set stroke color
-				$draw->setStrokeColor(new \ImagickPixel($data['stroke']??$defaultStroke));
-
-				//Set fill color
-				$draw->setFillColor(new \ImagickPixel($data['stroke']??$defaultStroke));
-
-				//Add annotation
-				$draw->annotation($data['x'], $data['y'], $text);
-
-				//Increase counter
-				$i++;
-			}
-
-			//Create stroke object
-			$stroke = new \Imagick();
-
-			//Add new image
-			$stroke->newImage($width, $height, new \ImagickPixel('transparent'));
-
-			//Draw on image
-			$stroke->drawImage($draw);
-
-			//Blur image
-			//XXX: blur the stroke canvas only
-			$stroke->blurImage(5,3);
-
-			//Set opacity to 0.5
-			//XXX: see https://www.php.net/manual/en/image.evaluateimage.php
-			$stroke->evaluateImage(\Imagick::EVALUATE_DIVIDE, 1.5, \Imagick::CHANNEL_ALPHA);
-
-			//Compose image
-			$image->compositeImage($stroke, \Imagick::COMPOSITE_OVER, 0, 0);
-
-			//Clear stroke
-			$stroke->clear();
-
-			//Destroy stroke
-			unset($stroke);
-
-			//Clear draw
-			$draw->clear();
-
-			//Set text antialias
-			$draw->setTextAntialias(true);
-
-			//Draw each text
-			foreach($texts as $text => $data) {
-				//Set font
-				$draw->setFont($fonts[$data['font']??$defaultFont]);
-
-				//Set font size
-				$draw->setFontSize($data['size']??$defaultSize);
-
-				//Set text alignment
-				$draw->setTextAlignment($aligns[$data['align']??$defaultAlign]);
-
-				//Set fill color
-				$draw->setFillColor(new \ImagickPixel($data['fill']??$defaultFill));
-
-				//Add annotation
-				$draw->annotation($data['x'], $data['y'], $text);
-
-				//With canonical text
-				if (!empty($data['canonical'])) {
-					//Prevent canonical to finish in alt
-					unset($texts[$text]);
-				}
-			}
-
-			//Draw on image
-			$image->drawImage($draw);
-
-			//Strip image exif data and properties
-			$image->stripImage();
-
-			//Set image format
-			$image->setImageFormat('jpeg');
-
-			//Save image
-			if (!$image->writeImage($dest)) {
-				//Throw error
-				throw new \Exception(sprintf('Unable to write image "%s"', $dest));
-			}
-
-			//Get dest stat
-			$stat = stat($dest);
-
-			//Return image data
-			return [
-				'og:image' => $package->getAbsoluteUrl('@RapsysAir/facebook/'.$stat['mtime'].$pathInfo.'.jpeg'),
-				'og:image:alt' => str_replace("\n", ' ', implode(' - ', array_keys($texts))),
-				'og:image:height' => $height,
-				'og:image:width' => $width
-			];
-		}
-
-		//Return empty array without image
-		return [];
-	}
-
 	/**
 	 * Renders a view
 	 *
 	 * {@inheritdoc}
 	 */
 	protected function render(string $view, array $parameters = [], Response $response = null): Response {
-		//Get request stack
-		$stack = $this->container->get('request_stack');
-
-		//Get current request
-		$request = $stack->getCurrentRequest();
-
-		//Get current locale
-		$locale = $request->getLocale();
-
-		//Set locale
-		$parameters['locale'] = str_replace('_', '-', $locale);
-
-		//Get context path
-		$pathInfo = $this->router->getContext()->getPathInfo();
-
-		//Iterate on locales excluding current one
-		foreach($this->config['locales'] as $current) {
-			//Set titles
-			$titles = [];
-
-			//Iterate on other locales
-			foreach(array_diff($this->config['locales'], [$current]) as $other) {
-				$titles[$other] = $this->translator->trans($this->config['languages'][$current], [], null, $other);
-			}
-
-			//Retrieve route matching path
-			$route = $this->router->match($pathInfo);
-
-			//Get route name
-			$name = $route['_route'];
-
-			//Unset route name
-			unset($route['_route']);
-
-			//With current locale
-			if ($current == $locale) {
-				//Set locale locales context
-				$parameters['canonical'] = $this->router->generate($name, ['_locale' => $current]+$route, UrlGeneratorInterface::ABSOLUTE_URL);
-			} else {
-				//Set locale locales context
-				$parameters['alternates'][str_replace('_', '-', $current)] = [
-					'absolute' => $this->router->generate($name, ['_locale' => $current]+$route, UrlGeneratorInterface::ABSOLUTE_URL),
-					'relative' => $this->router->generate($name, ['_locale' => $current]+$route),
-					'title' => implode('/', $titles),
-					'translated' => $this->translator->trans($this->config['languages'][$current], [], null, $current)
-				];
-			}
-
-			//Add shorter locale
-			if (empty($parameters['alternates'][$shortCurrent = substr($current, 0, 2)])) {
-				//Set locale locales context
-				$parameters['alternates'][$shortCurrent] = [
-					'absolute' => $this->router->generate($name, ['_locale' => $current]+$route, UrlGeneratorInterface::ABSOLUTE_URL),
-					'relative' => $this->router->generate($name, ['_locale' => $current]+$route),
-					'title' => implode('/', $titles),
-					'translated' => $this->translator->trans($this->config['languages'][$current], [], null, $current)
-				];
-			}
-		}
-
 		//Create application form for role_guest
 		if ($this->isGranted('ROLE_GUEST')) {
 			//Without application form
 			if (empty($parameters['forms']['application'])) {
-				//Fetch doctrine
-				$doctrine = $this->get('doctrine');
+				//Get favorites dances
+				$danceFavorites = $this->doctrine->getRepository(Dance::class)->findByUserId($this->getUser()->getId());
+
+				//Set dance default
+				$danceDefault = !empty($danceFavorites)?current($danceFavorites):null;
 
 				//Get favorites locations
-				$locationFavorites = $doctrine->getRepository(Location::class)->findByUserId($this->getUser()->getId());
+				$locationFavorites = $this->doctrine->getRepository(Location::class)->findByUserId($this->getUser()->getId());
 
 				//Set location default
-				$locationDefault = current($locationFavorites);
+				$locationDefault = !empty($locationFavorites)?current($locationFavorites):null;
 
 				//With admin
-				if ($this->isGranted('ROLE_ADMIN')) {
+				if ($this->checker->isGranted('ROLE_ADMIN')) {
+					//Get dances
+					$dances = $this->doctrine->getRepository(Dance::class)->findAll();
+
 					//Get locations
-					$locations = $doctrine->getRepository(Location::class)->findAll();
+					$locations = $this->doctrine->getRepository(Location::class)->findAll();
 				//Without admin
 				} else {
+					//Restrict to favorite dances
+					$dances = $danceFavorites;
+
+					//Reset favorites
+					$danceFavorites = [];
+
 					//Restrict to favorite locations
 					$locations = $locationFavorites;
 
@@ -526,8 +323,24 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 					$locationFavorites = [];
 				}
 
+				//With session application dance id
+				if (!empty($parameters['session']['application']['dance']['id'])) {
+					//Iterate on each dance
+					foreach($dances as $dance) {
+						//Found dance
+						if ($dance->getId() == $parameters['session']['application']['dance']['id']) {
+							//Set dance as default
+							$danceDefault = $dance;
+
+							//Stop search
+							break;
+						}
+					}
+				}
+
 				//With session location id
 				//XXX: set in session controller
+				//TODO: with new findAll that key by id, it should be as simple as isset($locations[$id]) ?
 				if (!empty($parameters['session']['location']['id'])) {
 					//Iterate on each location
 					foreach($locations as $location) {
@@ -548,6 +361,12 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 					'action' => $this->generateUrl('rapsys_air_application_add'),
 					//Set the form attribute
 					'attr' => [ 'class' => 'col' ],
+					//Set dance choices
+					'dance_choices' => $dances,
+					//Set dance default
+					'dance_default' => $danceDefault,
+					//Set dance favorites
+					'dance_favorites' => $danceFavorites,
 					//Set location choices
 					'location_choices' => $locations,
 					//Set location default
@@ -555,21 +374,24 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 					//Set location favorites
 					'location_favorites' => $locationFavorites,
 					//With user
-					'user' => $this->isGranted('ROLE_ADMIN'),
+					'user' => $this->checker->isGranted('ROLE_ADMIN'),
 					//Set user choices
-					'user_choices' => $doctrine->getRepository(User::class)->findAllWithTranslatedGroupAndCivility($this->translator),
+					'user_choices' => $this->doctrine->getRepository(User::class)->findIndexByGroupPseudonym(),
 					//Set default user to current
 					'user_default' => $this->getUser()->getId(),
 					//Set to session slot or evening by default
 					//XXX: default to Evening (3)
-					'slot_default' => $doctrine->getRepository(Slot::class)->findOneById($parameters['session']['slot']['id']??3)
+					'slot_default' => $this->doctrine->getRepository(Slot::class)->findOneById($parameters['session']['slot']['id']??3)
 				]);
 
 				//Add form to context
 				$parameters['forms']['application'] = $application->createView();
 			}
+		}/*
+		#XXX: removed because it fucks up the seo by displaying register and login form instead of content
+		#XXX: until we find a better way, removed !!!
 		//Create login form for anonymous
-		} elseif (!$this->isGranted('IS_AUTHENTICATED_REMEMBERED')) {
+		elseif (!$this->checker->isGranted('IS_AUTHENTICATED_REMEMBERED')) {
 			//Create LoginType form
 			$login = $this->createForm('Rapsys\UserBundle\Form\LoginType', null, [
 				//Set the action
@@ -603,18 +425,15 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 				'phone' => false
 			];
 
-			//Get slugger
-			$slugger = $this->container->get('rapsys_pack.slugger_util');
-
 			//Create RegisterType form
 			$register = $this->createForm('Rapsys\AirBundle\Form\RegisterType', null, $field+[
 				//Set the action
 				'action' => $this->generateUrl(
 					'rapsys_user_register',
 					[
-						'mail' => $smail = $slugger->short(''),
-						'field' => $sfield = $slugger->serialize($field),
-						'hash' => $slugger->hash($smail.$sfield)
+						'mail' => $smail = $this->slugger->short(''),
+						'field' => $sfield = $this->slugger->serialize($field),
+						'hash' => $this->slugger->hash($smail.$sfield)
 					]
 				),
 				//Set the form attribute
@@ -623,26 +442,71 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 
 			//Add form to context
 			$parameters['forms']['register'] = $register->createView();
+		}*/
+
+		//Without alternates
+		if (count($parameters['alternates']) <= 1) {
+			//Set routeParams
+			$routeParams = $this->routeParams;
+
+			//Iterate on locales excluding current one
+			foreach($this->config['locales'] as $locale) {
+				//With current locale
+				if ($locale !== $this->locale) {
+					//Set titles
+					$titles = [];
+
+					//Set route params locale
+					$routeParams['_locale'] = $locale;
+
+					//Iterate on other locales
+					foreach(array_diff($this->config['locales'], [$locale]) as $other) {
+						//Set other locale title
+						$titles[$other] = $this->translator->trans($this->config['languages'][$locale], [], null, $other);
+					}
+
+					//Set locale locales context
+					$parameters['alternates'][str_replace('_', '-', $locale)] = [
+						'absolute' => $this->router->generate($this->route, $routeParams, UrlGeneratorInterface::ABSOLUTE_URL),
+						'relative' => $this->router->generate($this->route, $routeParams),
+						'title' => implode('/', $titles),
+						'translated' => $this->translator->trans($this->config['languages'][$locale], [], null, $locale)
+					];
+
+					//Add shorter locale
+					if (empty($parameters['alternates'][$shortCurrent = substr($locale, 0, 2)])) {
+						//Set locale locales context
+						$parameters['alternates'][$shortCurrent] = $parameters['alternates'][str_replace('_', '-', $locale)];
+					}
+				}
+			}
 		}
 
 		//With page infos and without facebook texts
-		if (empty($parameters['facebook']['texts']) && !empty($parameters['site']['title']) && !empty($parameters['title']) && !empty($parameters['canonical'])) {
-			//Set facebook image
-			$parameters['facebook']['texts'] = [
-				$parameters['site']['title'] => [
-					'font' => 'irishgrover',
-					'size' => 110
-				],
+		if (count($parameters['facebook']['texts']) <= 1 && isset($parameters['title']) && isset($this->route) && isset($this->routeParams)) {
+			//Append facebook image texts
+			$parameters['facebook']['texts'] += [
 				$parameters['title'] => [
 					'align' => 'left'
-				],
-				$parameters['canonical'] => [
+				]/*XXX: same problem as url, too long :'(,
+				$parameters['description'] => [
 					'align' => 'right',
 					'canonical' => true,
 					'font' => 'labelleaurore',
 					'size' => 50
-				]
+				]*/
 			];
+
+			/*With short path info
+			We don't add this stupid url in image !!!
+			if (strlen($pathInfo = $this->router->generate($this->route, $this->routeParams)) <= 64) {
+				 => [
+					'align' => 'right',
+					'canonical' => true,
+					'font' => 'labelleaurore',
+					'size' => 50
+				 ]
+			}*/
 		}
 
 		//With canonical
@@ -664,9 +528,9 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 		}
 
 		//With locale
-		if (!empty($locale)) {
+		if (!empty($this->locale)) {
 			//Set facebook locale
-			$parameters['facebook']['metas']['og:locale'] = $locale;
+			$parameters['facebook']['metas']['og:locale'] = $this->locale;
 
 			//With alternates
 			//XXX: locale change when fb_locale=xx_xx is provided is done in FacebookSubscriber
@@ -683,13 +547,13 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 		}
 
 		//Without facebook image defined and texts
-		if (empty($parameters['facebook']['metas']['og:image']) && !empty($parameters['facebook']['texts'])) {
+		if (empty($parameters['facebook']['metas']['og:image']) && !empty($this->request) && !empty($parameters['facebook']['texts']) && !empty($this->modified)) {
 			//Get facebook image
-			$parameters['facebook']['metas'] += $this->getFacebookImage($pathInfo, $parameters['facebook']);
+			$parameters['facebook']['metas'] += $this->facebook->getImage($this->request->getPathInfo(), $parameters['facebook']['texts'], $this->modified->getTimestamp());
 		}
 
 		//Call parent method
-		return $this->baseRender($view, $parameters, $response);
+		return parent::render($view, $parameters, $response);
 	}
 
 	/**
@@ -700,11 +564,20 @@ abstract class AbstractController extends BaseAbstractController implements Serv
 	public static function getSubscribedServices(): array {
 		//Return subscribed services
 		return [
-			//'logger' => LoggerInterface::class,
 			'doctrine' => ManagerRegistry::class,
+			'doctrine.orm.default_entity_manager' => EntityManagerInterface::class,
+			'form.factory' => FormFactoryInterface::class,
+			'mailer.mailer' => MailerInterface::class,
+			'rapsys_air.facebook_util' => FacebookUtil::class,
+			'rapsys_pack.image_util' => ImageUtil::class,
+			'rapsys_pack.map_util' => MapUtil::class,
 			'rapsys_pack.path_package' => PackageInterface::class,
+			'rapsys_pack.slugger_util' => SluggerUtil::class,
+			'rapsys_user.access_decision_manager' => AccessDecisionManagerInterface::class,
 			'request_stack' => RequestStack::class,
 			'router' => RouterInterface::class,
+			'security.authorization_checker' => AuthorizationCheckerInterface::class,
+			'service_container' => ContainerInterface::class,
 			'translator' => TranslatorInterface::class
 		];
 	}