From: Raphaël Gertz <git@rapsys.eu>
Date: Mon, 24 Oct 2022 06:59:54 +0000 (+0200)
Subject: Drop mailer interface
X-Git-Tag: 0.3.0~156
X-Git-Url: https://git.rapsys.eu/airbundle/commitdiff_plain/faae833b04055eb8e32208bde8aa3a319a9b1ccd?ds=inline

Drop mailer interface
Drop doctrine interface
New user view
New index view
Cleanup
---

diff --git a/Controller/DefaultController.php b/Controller/DefaultController.php
index e55d6e8..cfe5f58 100644
--- a/Controller/DefaultController.php
+++ b/Controller/DefaultController.php
@@ -12,15 +12,18 @@
 namespace Rapsys\AirBundle\Controller;
 
 use Symfony\Bridge\Twig\Mime\TemplatedEmail;
+use Symfony\Component\Filesystem\Exception\IOExceptionInterface;
+use Symfony\Component\Filesystem\Filesystem;
 use Symfony\Component\Form\FormError;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 use Symfony\Component\Mailer\Exception\TransportExceptionInterface;
-use Symfony\Component\Mailer\MailerInterface;
 use Symfony\Component\Mime\Address;
-use Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken;
+use Symfony\Component\Security\Core\Authentication\Token\AnonymousToken;
+use Symfony\Component\Security\Core\Exception\AccessDeniedException;
 
-use Rapsys\AirBundle\Entity\Civility;
+use Rapsys\AirBundle\Entity\Dance;
 use Rapsys\AirBundle\Entity\Location;
 use Rapsys\AirBundle\Entity\Session;
 use Rapsys\AirBundle\Entity\Snippet;
@@ -68,11 +71,10 @@ class DefaultController extends AbstractController {
 	 * @desc Send a contact mail to configured contact
 	 *
 	 * @param Request $request The request instance
-	 * @param MailerInterface $mailer The mailer instance
 	 *
 	 * @return Response The rendered view or redirection
 	 */
-	public function contact(Request $request, MailerInterface $mailer): Response {
+	public function contact(Request $request): Response {
 		//Set page
 		$this->context['title'] = $this->translator->trans('Contact');
 
@@ -88,9 +90,21 @@ class DefaultController extends AbstractController {
 			$this->translator->trans('calendar')
 		];
 
+		//Set data
+		$data = [];
+
+		//With user
+		if ($user = $this->getUser()) {
+			//Set data
+			$data = [
+				'name' => $user->getRecipientName(),
+				'mail' => $user->getMail()
+			];
+		}
+
 		//Create the form according to the FormType created previously.
 		//And give the proper parameters
-		$form = $this->createForm('Rapsys\AirBundle\Form\ContactType', null, [
+		$form = $this->createForm('Rapsys\AirBundle\Form\ContactType', $data, [
 			'action' => $this->generateUrl('rapsys_air_contact'),
 			'method' => 'POST'
 		]);
@@ -99,7 +113,7 @@ class DefaultController extends AbstractController {
 			// Refill the fields in case the form is not valid.
 			$form->handleRequest($request);
 
-			if ($form->isValid()) {
+			if ($form->isSubmitted() && $form->isValid()) {
 				//Get data
 				$data = $form->getData();
 
@@ -128,7 +142,7 @@ class DefaultController extends AbstractController {
 				//XXX: mail delivery may silently fail
 				try {
 					//Send message
-					$mailer->send($message);
+					$this->mailer->send($message);
 
 					//Redirect on the same route with sent=1 to cleanup form
 					return $this->redirectToRoute($request->get('_route'), ['sent' => 1]+$request->get('_route_params'));
@@ -155,11 +169,10 @@ class DefaultController extends AbstractController {
 	 * @desc Generate a dispute document
 	 *
 	 * @param Request $request The request instance
-	 * @param MailerInterface $mailer The mailer instance
 	 *
 	 * @return Response The rendered view or redirection
 	 */
-	public function dispute(Request $request, MailerInterface $mailer): Response {
+	public function dispute(Request $request): Response {
 		//Prevent non-guest to access here
 		$this->denyAccessUnlessGranted('ROLE_USER', null, $this->translator->trans('Unable to access this page without role %role%!', ['%role%' => $this->translator->trans('User')]));
 
@@ -250,7 +263,7 @@ class DefaultController extends AbstractController {
 #				//XXX: mail delivery may silently fail
 #				try {
 #					//Send message
-#					$mailer->send($message);
+#					$this->mailer->send($message);
 #
 #					//Redirect on the same route with sent=1 to cleanup form
 #					return $this->redirectToRoute($request->get('_route'), ['sent' => 1]+$request->get('_route_params'));
@@ -274,65 +287,118 @@ class DefaultController extends AbstractController {
 	/**
 	 * The index page
 	 *
-	 * @desc Display all granted sessions with an application or login form
+	 * Display session calendar
 	 *
 	 * @param Request $request The request instance
 	 * @return Response The rendered view
 	 */
 	public function index(Request $request): Response {
-		//Fetch doctrine
-		$doctrine = $this->getDoctrine();
+		//Add cities
+		$this->context['cities'] = $this->doctrine->getRepository(Location::class)->findCitiesAsArray($this->period);
 
-		//Set page
-		$this->context['title'] = $this->translator->trans('Argentine Tango in Paris');
+		//Add calendar
+		$this->context['calendar'] = $this->doctrine->getRepository(Session::class)->findAllByPeriodAsCalendarArray($this->period, !$this->isGranted('IS_AUTHENTICATED_REMEMBERED'));
 
-		//Set description
-		$this->context['description'] = $this->translator->trans('Outdoor Argentine Tango session calendar in Paris');
+		//Add dances
+		$this->context['dances'] = $this->doctrine->getRepository(Dance::class)->findNamesAsArray();
 
-		//Set keywords
-		$this->context['keywords'] = [
-			$this->translator->trans('Argentine Tango'),
-			$this->translator->trans('Paris'),
-			$this->translator->trans('outdoor'),
-			$this->translator->trans('calendar'),
-			$this->translator->trans('Libre Air')
-		];
+		//Set modified
+		$this->modified = max(array_map(function ($v) { return $v['modified']; }, array_merge($this->context['calendar'], $this->context['cities'], $this->context['dances'])));
 
-		//Set facebook type
-		//XXX: only valid for home page
-		$this->context['facebook']['metas']['og:type'] = 'website';
+		//Create response
+		$response = new Response();
+
+		//With logged user
+		if ($this->isGranted('IS_AUTHENTICATED_REMEMBERED')) {
+			//Set last modified
+			$response->setLastModified(new \DateTime('-1 year'));
+
+			//Set as private
+			$response->setPrivate();
+		//Without logged user
+		} else {
+			//Set etag
+			//XXX: only for public to force revalidation by last modified
+			$response->setEtag(md5(serialize(array_merge($this->context['calendar'], $this->context['cities'], $this->context['dances']))));
+
+			//Set last modified
+			$response->setLastModified($this->modified);
+
+			//Set as public
+			$response->setPublic();
+
+			//Without role and modification
+			if ($response->isNotModified($request)) {
+				//Return 304 response
+				return $response;
+			}
+		}
+
+		//With cities
+		if (!empty($this->context['cities'])) {
+			//Set locations
+			$locations = [];
+
+			//Iterate on each cities
+			foreach($this->context['cities'] as $city) {
+				//Iterate on each locations
+				foreach($city['locations'] as $location) {
+					//Add location
+					$locations[$location['id']] = $location;
+				}
+			}
+
+			//Add multi
+			$this->context['multimap'] = $this->map->getMultiMap($this->translator->trans('Libre Air cities sector map'), $this->modified->getTimestamp(), $locations);
+
+			//Set cities
+			$cities = array_map(function ($v) { return $v['in']; }, $this->context['cities']);
+
+			//Set dances
+			$dances = array_map(function ($v) { return $v['name']; }, $this->context['dances']);
+		} else {
+			//Set cities
+			$cities = [];
 
-		//Compute period
-		$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
-			new \DateTime(
-				$this->isGranted('IS_AUTHENTICATED_REMEMBERED')?'Monday this week + 3 week':'Monday this week + 2 week'
+			//Set dances
+			$dances = [];
+		}
+
+		//Set keywords
+		//TODO: use splice instead of that shit !!!
+		//TODO: handle smartly indoor and outdoor !!!
+		$this->context['keywords'] = array_values(
+			array_merge(
+				$dances,
+				$cities,
+				[
+					$this->translator->trans('indoor'),
+					$this->translator->trans('outdoor'),
+					$this->translator->trans('calendar'),
+					$this->translator->trans('Libre Air')
+				]
 			)
 		);
 
-		//Fetch calendar
-		$calendar = $doctrine->getRepository(Session::class)->fetchCalendarByDatePeriod($this->translator, $period, null, $request->get('session'), !$this->isGranted('IS_AUTHENTICATED_REMEMBERED'), $request->getLocale());
+		//Get textual cities
+		$cities = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($cities, 0, -1))], array_slice($cities, -1)), 'strlen'));
 
-		//Fetch locations
-		//XXX: we want to display all active locations anyway
-		$locations = $doctrine->getRepository(Location::class)->findTranslatedSortedByPeriod($this->translator, $period);
+		//Get textual dances
+		$dances = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($dances, 0, -1))], array_slice($dances, -1)), 'strlen'));
 
-		//Render the view
-		return $this->render('@RapsysAir/default/index.html.twig', ['calendar' => $calendar, 'locations' => $locations]+$this->context);
+		//Set title
+		$this->context['title'] = $this->translator->trans('%dances% %cities%', ['%dances%' => $dances, '%cities%' => $cities]);
 
-		//Set Cache-Control must-revalidate directive
-		//TODO: add a javascript forced refresh after 1h ? or header refresh ?
-		#$response->setPublic(true);
-		#$response->setMaxAge(300);
-		#$response->mustRevalidate();
-		##$response->setCache(['public' => true, 'max_age' => 300]);
+		//Set description
+		//TODO: handle french translation when city start with a A, change à in en !
+		$this->context['description'] = $this->translator->trans('%dances% indoor and outdoor calendar %cities%', ['%dances%' => $dances, '%cities%' => $cities]);
 
-		//Return the response
-		#return $response;
+		//Set facebook type
+		//XXX: only valid for home page
+		$this->context['facebook']['metas']['og:type'] = 'website';
+
+		//Render the view
+		return $this->render('@RapsysAir/default/index.html.twig', $this->context, $response);
 	}
 
 	/**
@@ -445,9 +511,6 @@ class DefaultController extends AbstractController {
 	 * @return Response The rendered view
 	 */
 	public function userIndex(Request $request): Response {
-		//Fetch doctrine
-		$doctrine = $this->getDoctrine();
-
 		//With admin role
 		if ($this->isGranted('ROLE_ADMIN')) {
 			//Set section
@@ -476,19 +539,7 @@ class DefaultController extends AbstractController {
 		$title = $this->translator->trans($this->config['site']['title']).' - '.$section;
 
 		//Fetch users
-		$users = $doctrine->getRepository(User::class)->findUserGroupedByTranslatedGroup($this->translator);
-
-		//Compute period
-		$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
-			new \DateTime(
-				$this->isGranted('IS_AUTHENTICATED_REMEMBERED')?'Monday this week + 3 week':'Monday this week + 2 week'
-			)
-		);
+		$users = $this->doctrine->getRepository(User::class)->findIndexByGroupId();
 
 		//With admin role
 		if ($this->isGranted('ROLE_ADMIN')) {
@@ -500,12 +551,8 @@ class DefaultController extends AbstractController {
 			$this->context['users'] = $users[$this->translator->trans('Senior')];
 		}
 
-		//Fetch locations
-		//XXX: we want to display all active locations anyway
-		$locations = $doctrine->getRepository(Location::class)->findTranslatedSortedByPeriod($this->translator, $period);
-
 		//Render the view
-		return $this->render('@RapsysAir/user/index.html.twig', ['title' => $title, 'section' => $section, 'locations' => $locations]+$this->context);
+		return $this->render('@RapsysAir/user/index.html.twig', ['title' => $title, 'section' => $section]+$this->context);
 	}
 
 	/**
@@ -518,218 +565,334 @@ class DefaultController extends AbstractController {
 	 *
 	 * @return Response The rendered view
 	 */
-	public function userView(Request $request, $id): Response {
-		//Fetch doctrine
-		$doctrine = $this->getDoctrine();
-
-		//Fetch user
-		if (empty($user = $doctrine->getRepository(User::class)->findOneById($id))) {
-			throw $this->createNotFoundException($this->translator->trans('Unable to find user: %id%', ['%id%' => $id]));
+	public function userView(Request $request, int $id, ?string $user): Response {
+		//Get user
+		if (empty($this->context['user'] = $this->doctrine->getRepository(User::class)->findOneByIdAsArray($id, $this->locale))) {
+			//Throw not found
+			throw new NotFoundHttpException($this->translator->trans('Unable to find user: %id%', ['%id%' => $id]));
 		}
 
-		//Get user token
-		$token = new UsernamePasswordToken($user, null, 'none', $user->getRoles());
-
-		//Check if guest
-		$isGuest = $this->get('rapsys_user.access_decision_manager')->decide($token, ['ROLE_GUEST']);
+		//Create token
+		$token = new AnonymousToken('', $this->context['user']['mail'], $this->context['user']['roles']);
 
 		//Prevent access when not admin, user is not guest and not currently logged user
-		if (!$this->isGranted('ROLE_ADMIN') && empty($isGuest) && $user != $this->getUser()) {
-			throw $this->createAccessDeniedException($this->translator->trans('Unable to access user: %id%', ['%id%' => $id]));
+		if (!($isAdmin = $this->isGranted('ROLE_ADMIN')) && !($isGuest = $this->decision->decide($token, ['ROLE_GUEST']))) {
+			//Throw access denied
+			throw new AccessDeniedException($this->translator->trans('Unable to access user: %id%', ['%id%' => $id]));
 		}
 
-		//Set section
-		$section = $user->getPseudonym();
+		//With invalid user slug
+		if ($this->context['user']['slug'] !== $user) {
+			//Redirect to cleaned url
+			return $this->redirectToRoute('rapsys_air_user_view', ['id' => $id, 'user' => $this->context['user']['slug']]);
+		}
 
-		//Set title
-		$title = $this->translator->trans($this->config['site']['title']).' - '.$section;
+		//Fetch calendar
+		$this->context['calendar'] = $this->doctrine->getRepository(Session::class)->findAllByPeriodAsCalendarArray($this->period, !$this->isGranted('IS_AUTHENTICATED_REMEMBERED'), null, null, $id);
 
-		//Set description
-		$this->context['description'] = $this->translator->trans('%pseudonym% outdoor Argentine Tango session calendar', [ '%pseudonym%' => $user->getPseudonym() ]);
+		//Get locations at less than 2 km
+		$this->context['locations'] = $this->doctrine->getRepository(Location::class)->findAllByUserIdAsArray($id, $this->period, 2);
 
-		//Set keywords
-		$this->context['keywords'] = [
-			$user->getPseudonym(),
-			$this->translator->trans('outdoor'),
-			$this->translator->trans('Argentine Tango'),
-			$this->translator->trans('calendar')
-		];
+		//Set ats
+		$ats = [];
 
-		//Compute period
-		$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
-			new \DateTime(
-				$this->isGranted('IS_AUTHENTICATED_REMEMBERED')?'Monday this week + 3 week':'Monday this week + 2 week'
-			)
-		);
-
-		//Fetch calendar
-		//TODO: highlight with current session route parameter
-		$calendar = $doctrine->getRepository(Session::class)->fetchUserCalendarByDatePeriod($this->translator, $period, $isGuest?$id:null, $request->get('session'), $request->getLocale());
-
-		//Fetch locations
-		//XXX: we want to display all active locations anyway
-		$locations = $doctrine->getRepository(Location::class)->findTranslatedSortedByPeriod($this->translator, $period, $id);
-
-		//Create user form for admin or current user
-		if ($this->isGranted('ROLE_ADMIN') || $user == $this->getUser()) {
-			//Create SnippetType form
-			$userForm = $this->createForm('Rapsys\AirBundle\Form\RegisterType', $user, [
-				//Set action
-				'action' => $this->generateUrl('rapsys_air_user_view', ['id' => $id]),
-				//Set the form attribute
-				'attr' => [ 'class' => 'col' ],
-				//Set civility class
-				'civility_class' => Civility::class,
-				//Disable mail
-				'mail' => $this->isGranted('ROLE_ADMIN'),
-				//Disable password
-				'password' => false
-			]);
-
-			//Init user to context
-			$this->context['forms']['user'] = $userForm->createView();
-
-			//Check if submitted
-			if ($request->isMethod('POST')) {
-				//Refill the fields in case the form is not valid.
-				$userForm->handleRequest($request);
-
-				//Handle invalid form
-				if (!$userForm->isSubmitted() || !$userForm->isValid()) {
-					//Render the view
-					return $this->render('@RapsysAir/user/view.html.twig', ['id' => $id, 'title' => $title, 'section' => $section, 'calendar' => $calendar, 'locations' => $locations]+$this->context);
-				}
+		//Set dances
+		$dances = [];
 
-				//Get data
-				$data = $userForm->getData();
+		//Set indoors
+		$indoors = [];
 
-				//Get manager
-				$manager = $doctrine->getManager();
+		//Set ins
+		$ins = [];
 
-				//Queue snippet save
-				$manager->persist($data);
+		//Set insides
+		$insides = [];
 
-				//Flush to get the ids
-				$manager->flush();
+		//Set locations
+		$locations = [];
 
-				//Add notice
-				$this->addFlash('notice', $this->translator->trans('User %id% updated', ['%id%' => $id]));
+		//Set types
+		$types = [];
 
-				//Extract and process referer
-				if ($referer = $request->headers->get('referer')) {
-					//Create referer request instance
-					$req = Request::create($referer);
+		//Iterate on each calendar
+		foreach($this->context['calendar'] as $date => $calendar) {
+			//Iterate on each session
+			foreach($calendar['sessions'] as $sessionId => $session) {
+				//Add dance
+				$dances[$session['application']['dance']['name']] = $session['application']['dance']['name'];
 
-					//Get referer path
-					$path = $req->getPathInfo();
+				//Add types
+				$types[$session['application']['dance']['type']] = lcfirst($session['application']['dance']['type']);
 
-					//Get referer query string
-					$query = $req->getQueryString();
+				//Add indoors
+				$indoors[$session['location']['indoor']?'indoor':'outdoor'] = $this->translator->trans($session['location']['indoor']?'indoor':'outdoor');
 
-					//Remove script name
-					$path = str_replace($request->getScriptName(), '', $path);
+				//Add insides
+				$insides[$session['location']['indoor']?'inside':'outside'] = $this->translator->trans($session['location']['indoor']?'inside':'outside');
 
-					//Try with referer path
-					try {
-						//Save old context
-						$oldContext = $this->router->getContext();
+				//Add ats
+				$ats[$session['location']['id']] = $session['location']['at'];
 
-						//Force clean context
-						//XXX: prevent MethodNotAllowedException because current context method is POST in onevendor/symfony/routing/Matcher/Dumper/CompiledUrlMatcherTrait.php+42
-						$this->router->setContext(new RequestContext());
+				//Add ins
+				$ins[$session['location']['id']] = $session['location']['in'];
 
-						//Retrieve route matching path
-						$route = $this->router->match($path);
+				//Session with application user id
+				if (!empty($session['application']['user']['id']) && $session['application']['user']['id'] == $id) {
+					//Add location
+					$locations[$session['location']['id']] = $session['location'];
+				}
+			}
+		}
 
-						//Reset context
-						$this->router->setContext($oldContext);
+		//Set modified
+		//XXX: dance modified is already computed inside calendar modified
+		$this->modified = max(array_merge([$this->context['user']['modified']], array_map(function ($v) { return $v['modified']; }, array_merge($this->context['calendar'], $this->context['locations']))));
 
-						//Clear old context
-						unset($oldContext);
+		//Create response
+		$response = new Response();
 
-						//Extract name
-						$name = $route['_route'];
+		//With logged user
+		if ($this->isGranted('IS_AUTHENTICATED_REMEMBERED')) {
+			//Set last modified
+			$response->setLastModified(new \DateTime('-1 year'));
 
-						//Remove route and controller from route defaults
-						unset($route['_route'], $route['_controller']);
+			//Set as private
+			$response->setPrivate();
+		//Without logged user
+		} else {
+			//Set etag
+			//XXX: only for public to force revalidation by last modified
+			$response->setEtag(md5(serialize(array_merge($this->context['user'], $this->context['calendar'], $this->context['locations']))));
 
-						//Check if user view route
-						if ($name == 'rapsys_air_user_view' && !empty($route['id'])) {
-							//Replace id
-							$route['id'] = $data->getId();
-						//Other routes
-						} else {
-							//Set user
-							$route['user'] = $data->getId();
-						}
+			//Set last modified
+			$response->setLastModified($this->modified);
 
-						//Generate url
-						return $this->redirectToRoute($name, $route);
-					//No route matched
-					} catch(MethodNotAllowedException|ResourceNotFoundException $e) {
-						//Unset referer to fallback to default route
-						unset($referer);
-					}
-				}
+			//Set as public
+			$response->setPublic();
 
-				//Redirect to cleanup the form
-				return $this->redirectToRoute('rapsys_air', ['user' => $data->getId()]);
+			//Without role and modification
+			if ($response->isNotModified($request)) {
+				//Return 304 response
+				return $response;
 			}
 		}
 
-		//Create snippet forms for role_guest
-		if ($this->isGranted('ROLE_ADMIN') || ($this->isGranted('ROLE_GUEST') && $user == $this->getUser())) {
-			//Fetch all user snippet
-			$snippets = $doctrine->getRepository(Snippet::class)->findByLocaleUserId($request->getLocale(), $id);
+		//Add multi map
+		$this->context['multimap'] = $this->map->getMultiMap($this->context['user']['multimap'], $this->modified->getTimestamp(), $this->context['locations']);
+
+		//Set keywords
+		$this->context['keywords'] = [
+			$this->context['user']['pseudonym'],
+			$this->translator->trans('calendar'),
+			$this->translator->trans('Libre Air')
+		];
 
-			//Rekey by location id
-			$snippets = array_reduce($snippets, function($carry, $item){$carry[$item->getLocation()->getId()] = $item; return $carry;}, []);
+		//Set cities
+		$cities = array_unique(array_map(function ($v) { return $v['city']; }, $locations));
 
-			//Init snippets to context
-			$this->context['forms']['snippets'] = [];
+		//Set titles
+		$titles = array_map(function ($v) { return $v['title']; }, $locations);
 
-			//Iterate on locations
-			foreach($locations as $locationId => $location) {
-				//Init snippet
-				$snippet = new Snippet();
+		//Insert dances in keywords
+		array_splice($this->context['keywords'], 1, 0, array_merge($types, $dances, $indoors, $insides, $titles, $cities));
+
+		//Deduplicate ins
+		$ins = array_unique($ins);
+
+		//Get textual dances
+		$dances = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($dances, 0, -1))], array_slice($dances, -1)), 'strlen'));
+
+		//Get textual types
+		$types = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($types, 0, -1))], array_slice($types, -1)), 'strlen'));
+
+		//Get textual indoors
+		$indoors = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($indoors, 0, -1))], array_slice($indoors, -1)), 'strlen'));
+
+		//Get textual ats
+		$ats = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($ats, 0, -1))], array_slice($ats, -1)), 'strlen'));
+
+		//Get textual ins
+		$ins = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($ins, 0, -1))], array_slice($ins, -1)), 'strlen'));
+
+		//Set title
+		$this->context['title'] = $this->translator->trans('%pseudonym% organizer', ['%pseudonym%' => $this->context['user']['pseudonym']]);
+
+		//With locations
+		if (!empty($locations)) {
+			//Set description
+			$this->context['description'] = ucfirst($this->translator->trans('%dances% %types% %indoors% calendar %ats% %ins% %pseudonym%', ['%dances%' => $dances, '%types%' => $types, '%indoors%' => $indoors, '%ats%' => $ats, '%ins%' => $ins, '%pseudonym%' => $this->translator->trans('by %pseudonym%', ['%pseudonym%' => $this->context['user']['pseudonym']])]));
+		//Without locations
+		} else {
+			//Set description
+			$this->context['description'] = $this->translator->trans('%pseudonym% calendar', ['%pseudonym%' => $this->context['user']['pseudonym']]);
+		}
 
-				//Set default locale
-				$snippet->setLocale($request->getLocale());
+		//Set user description
+		$this->context['locations_description'] = $this->translator->trans('Libre Air %pseudonym% location list', ['%pseudonym%' => $this->translator->trans('by %pseudonym%', ['%pseudonym%' => $this->context['user']['pseudonym']])]);
 
-				//Set default user
-				$snippet->setUser($user);
+		//Set alternates
+		$this->context['alternates'] += $this->context['user']['alternates'];
 
-				//Set default location
-				$snippet->setLocation($doctrine->getRepository(Location::class)->findOneById($locationId));
+		//Create snippet forms for role_guest
+		//TODO: optimize this call
+		if ($isAdmin || $isGuest && $this->getUser() && $this->context['user']['id'] == $this->getUser()->getId()) {
+			//Fetch all user snippet
+			$snippets = $this->doctrine->getRepository(Snippet::class)->findByUserIdLocaleIndexByLocationId($id, $this->locale);
 
+			//Get user
+			$user = $this->doctrine->getRepository(User::class)->findOneById($id);
+
+			//Iterate on locations
+			foreach($this->context['locations'] as $locationId => $location) {
 				//With existing snippet
-				if (!empty($snippets[$locationId])) {
-					$snippet = $snippets[$locationId];
-					$action = $this->generateUrl('rapsys_air_snippet_edit', ['id' => $snippet->getId()]);
-				//Without snippet
+				if (isset($snippets[$location['id']])) {
+					//Set existing in current
+					$current = $snippets[$location['id']];
+				//Without existing snippet
 				} else {
-					$action = $this->generateUrl('rapsys_air_snippet_add', ['location' => $locationId]);
+					//Init snippet
+					$current = new Snippet();
+
+					//Set default locale
+					$current->setLocale($this->locale);
+
+					//Set default user
+					$current->setUser($user);
+
+					//Set default location
+					$current->setLocation($this->doctrine->getRepository(Location::class)->findOneById($location['id']));
 				}
 
 				//Create SnippetType form
-				$form = $this->container->get('form.factory')->createNamed('snipped_'.$request->getLocale().'_'.$locationId, 'Rapsys\AirBundle\Form\SnippetType', $snippet, [
-					//Set the action
-					'action' => $action,
-					//Set the form attribute
-					'attr' => []
-				]);
+				$form = $this->factory->createNamed(
+					//Set form id
+					'snippet_'.$locationId.'_'.$id.'_'.$this->locale,
+					//Set form type
+					'Rapsys\AirBundle\Form\SnippetType',
+					//Set form data
+					$current
+				);
+
+				//Refill the fields in case of invalid form
+				$form->handleRequest($request);
+
+				//Handle submitted and valid form
+				//TODO: add a delete snippet ?
+				if ($form->isSubmitted() && $form->isValid()) {
+					//Get snippet
+					$snippet = $form->getData();
+
+					//Queue snippet save
+					$this->manager->persist($snippet);
+
+					//Flush to get the ids
+					$this->manager->flush();
+
+					//Add notice
+					$this->addFlash('notice', $this->translator->trans('Snippet for %user% %location% updated', ['%location%' => $location['at'], '%user%' => $this->context['user']['pseudonym']]));
+
+					//Redirect to cleaned url
+					return $this->redirectToRoute('rapsys_air_user_view', ['id' => $id, 'user' => $this->context['user']['slug']]);
+				}
 
 				//Add form to context
 				$this->context['forms']['snippets'][$locationId] = $form->createView();
+
+				//With location user source image
+				if (($isFile = is_file($source = $this->config['path'].'/location/'.$location['id'].'/'.$id.'.png')) && ($mtime = stat($source)['mtime'])) {
+					//Set location image
+					$this->context['locations'][$locationId]['image'] = $this->image->getThumb($location['miniature'], $mtime, $source);
+				}
+
+				//Create ImageType form
+				$form = $this->factory->createNamed(
+					//Set form id
+					'image_'.$locationId.'_'.$id,
+					//Set form type
+					'Rapsys\AirBundle\Form\ImageType',
+					//Set form data
+					[
+						//Set location
+						'location' => $location['id'],
+						//Set user
+						'user' => $id
+					],
+					//Set form attributes
+					[
+						//Enable delete with image
+						'delete' => isset($this->context['locations'][$locationId]['image'])
+					]
+				);
+
+				//Refill the fields in case of invalid form
+				$form->handleRequest($request);
+
+				//Handle submitted and valid form
+				if ($form->isSubmitted() && $form->isValid()) {
+					//With delete
+					if ($form->has('delete') && $form->get('delete')->isClicked()) {
+						//With source and mtime
+						if ($isFile && !empty($source) && !empty($mtime)) {
+							//Clear thumb
+							$this->image->remove($mtime, $source);
+
+							//Unlink file
+							unlink($this->config['path'].'/location/'.$location['id'].'/'.$id.'.png');
+
+							//Add notice
+							$this->addFlash('notice', $this->translator->trans('Image for %user% %location% deleted', ['%location%' => $location['at'], '%user%' => $this->context['user']['pseudonym']]));
+
+							//Redirect to cleaned url
+							return $this->redirectToRoute('rapsys_air_user_view', ['id' => $id, 'user' => $this->context['user']['slug']]);
+						}
+					}
+
+					//With image
+					if ($image = $form->get('image')->getData()) {
+						//Check source path
+						if (!is_dir($dir = dirname($source))) {
+							//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);
+							}
+						}
+
+						//Set source
+						$source = realpath($dir).'/'.basename($source);
+
+						//Create imagick object
+						$imagick = new \Imagick();
+
+						//Read image
+						$imagick->readImage($image->getRealPath());
+
+						//Save image
+						if (!$imagick->writeImage($source)) {
+							//Throw error
+							throw new \Exception(sprintf('Unable to write image "%s"', $source));
+						}
+
+						//Add notice
+						$this->addFlash('notice', $this->translator->trans('Image for %user% %location% updated', ['%location%' => $location['at'], '%user%' => $this->context['user']['pseudonym']]));
+
+						//Redirect to cleaned url
+						return $this->redirectToRoute('rapsys_air_user_view', ['id' => $id, 'user' => $this->context['user']['slug']]);
+					}
+				}
+
+				//Add form to context
+				$this->context['forms']['images'][$locationId] = $form->createView();
 			}
 		}
 
 		//Render the view
-		return $this->render('@RapsysAir/user/view.html.twig', ['id' => $id, 'title' => $title, 'section' => $section, 'calendar' => $calendar, 'locations' => $locations]+$this->context);
+		return $this->render('@RapsysAir/user/view.html.twig', ['id' => $id]+$this->context);
 	}
 }