Fix coalesce warning

[airbundle] / Controller / LocationController.php

diff --git a/Controller/LocationController.php b/Controller/LocationController.php
index ce735583bfd92e5c0e0690b12b3219dca60f0f55..9ee9471af6ceda4badb0f3086cdfd35a3d025727 100644 (file)
--- a/Controller/LocationController.php
+++ b/Controller/LocationController.php
@@ -1,277 +1,279 @@
-<?php
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of the Rapsys AirBundle package.
+ *
+ * (c) Raphaël Gertz <symfony@rapsys.eu>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */

 
 namespace Rapsys\AirBundle\Controller;
 
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 
 namespace Rapsys\AirBundle\Controller;
 
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;

-use Symfony\Component\Routing\RequestContext;
-use Symfony\Component\Routing\Exception\MethodNotAllowedException;
-use Symfony\Component\Routing\Exception\ResourceNotFoundException;
-use Rapsys\AirBundle\Entity\Slot;
-use Rapsys\AirBundle\Entity\Session;
+
+use Rapsys\AirBundle\Entity\Dance;

 use Rapsys\AirBundle\Entity\Location;
 use Rapsys\AirBundle\Entity\Location;

+use Rapsys\AirBundle\Entity\Session;

 
 

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

 class LocationController extends DefaultController {
        /**
 class LocationController extends DefaultController {
        /**

-        * Add location
-        *
-        * @desc Persist location in database
+        * List all cities

         *
         * @param Request $request The request instance
         *
         * @param Request $request The request instance

-        *
-        * @return Response The rendered view or redirection
-        *
-        * @throws \RuntimeException When user has not at least admin role
+        * @return Response The rendered view

         */
         */

-       public function add(Request $request) {
-               //Prevent non-guest to access here
-               $this->denyAccessUnlessGranted('ROLE_ADMIN', null, $this->translator->trans('Unable to access this page without role %role%!', ['%role%' => $this->translator->trans('Admin')]));
-
-               //Create LocationType form
-               $form = $this->createForm('Rapsys\AirBundle\Form\LocationType', null, [
-                       //Set the action
-                       'action' => $this->generateUrl('rapsys_air_location_add'),
-                       //Set the form attribute
-                       'attr' => []
-               ]);
-
-               //Refill the fields in case of invalid form
-               $form->handleRequest($request);
-
-               //Handle invalid form
-               if (!$form->isSubmitted() || !$form->isValid()) {
-                       //Set section
-                       $section = $this->translator->trans('Location add');
-
-                       //Set title
-                       $title = $this->translator->trans($this->config['site']['title']).' - '.$section;
-
-                       //Render the view
-                       return $this->render('@RapsysAir/location/add.html.twig', ['title' => $title, 'section' => $section, 'form' => $form->createView()]+$this->context);
-               }
-
-               //Get doctrine
-               $doctrine = $this->getDoctrine();
-
-               //Get manager
-               $manager = $doctrine->getManager();
+       public function cities(Request $request): Response {
+               //Add cities
+               $this->context['cities'] = $this->doctrine->getRepository(Location::class)->findCitiesAsArray($this->period, 0);

 
 

-               //Get location
-               $location = $form->getData();
+               //Add dances
+               $this->context['dances'] = $this->doctrine->getRepository(Dance::class)->findNamesAsArray();

 
 

-               //Set created
-               $location->setCreated(new \DateTime('now'));
+               //Create response
+               $response = new Response();

 
 

-               //Set updated
-               $location->setUpdated(new \DateTime('now'));
+               //Set modified
+               $this->modified = max(array_map(function ($v) { return $v['modified']; }, array_merge($this->context['cities'], $this->context['dances'])));

 
 

-               //Queue location save
-               $manager->persist($location);
-
-               //Flush to get the ids
-               $manager->flush();
-
-               //Add notice
-               $this->addFlash('notice', $this->translator->trans('Location %id% created', ['%id%' => $location->getId()]));
-
-               //Extract and process referer
-               if ($referer = $request->headers->get('referer')) {
-                       //Create referer request instance
-                       $req = Request::create($referer);
-
-                       //Get referer path
-                       $path = $req->getPathInfo();
-
-                       //Get referer query string
-                       $query = $req->getQueryString();
-
-                       //Remove script name
-                       $path = str_replace($request->getScriptName(), '', $path);
-
-                       //Try with referer path
-                       try {
-                               //Save old context
-                               $oldContext = $this->router->getContext();
-
-                               //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 city multi
+               foreach($this->context['cities'] as $id => $city) {
+                       //Add city multi
+                       $this->context['cities'][$id]['multimap'] = $this->map->getMultiMap($city['multimap'], $this->modified->getTimestamp(), $city['locations']);
+               }

 
 

-                               //Retrieve route matching path
-                               $route = $this->router->match($path);
+               //With logged user
+               if ($this->checker->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['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;
+                       }
+               }

 
 

-                               //Reset context
-                               $this->router->setContext($oldContext);
+               //Set section
+               $this->context['title']['page'] = $this->translator->trans('Libre Air cities');

 
 

-                               //Clear old context
-                               unset($oldContext);
+               //Set description
+               $this->context['description'] = $this->translator->trans('Libre Air city list');

 
 

-                               //Extract name
-                               $name = $route['_route'];
+               //Set cities
+               $cities = array_map(function ($v) { return $v['in']; }, $this->context['cities']);

 
 

-                               //Remove route and controller from route defaults
-                               unset($route['_route'], $route['_controller']);
+               //Set dances
+               $dances = array_map(function ($v) { return $v['name']; }, $this->context['dances']);

 
 

-                               //Check if location view route
-                               if ($name == 'rapsys_air_location_view' && !empty($route['id'])) {
-                                       //Replace id
-                                       $route['id'] = $location->getId();
-                               //Other routes
-                               } else {
-                                       //Set location
-                                       $route['location'] = $location->getId();
-                               }
+               //Set indoors
+               $indoors = array_reduce($this->context['cities'], function ($c, $v) { return array_merge($c, $v['indoors']); }, []);

 
 

-                               //Generate url
-                               return $this->redirectToRoute($name, $route);
-                       //No route matched
-                       } catch(MethodNotAllowedException|ResourceNotFoundException $e) {
-                               //Unset referer to fallback to default route
-                               unset($referer);
-                       }
-               }
+               //Set keywords
+               $this->context['keywords'] = array_values(
+                       array_merge(
+                               [
+                                       $this->translator->trans('Cities'),
+                                       $this->translator->trans('City list'),
+                                       $this->translator->trans('Listing'),
+                               ],
+                               $cities,
+                               $indoors,
+                               [
+                                       $this->translator->trans('calendar'),
+                                       $this->translator->trans('Libre Air')
+                               ]
+                       )
+               );

 
 

-               //Redirect to cleanup the form
-               return $this->redirectToRoute('rapsys_air', ['location' => $location->getId()]);
+               //Render the view
+               return $this->render('@RapsysAir/location/cities.html.twig', $this->context, $response);

        }
 
        /**
        }
 
        /**

-        * Edit location
+        * Display city

         *
         *

-        * @desc Persist location in database
+        * @todo XXX: TODO: add <link rel="prev|next" for sessions or classes ? />
+        * @todo XXX: TODO: like described in: https://www.alsacreations.com/article/lire/1400-attribut-rel-relations.html#xnf-rel-attribute
+        * @todo XXX: TODO: or here: http://microformats.org/wiki/existing-rel-values#HTML5_link_type_extensions

         *
         * @param Request $request The request instance
         *
         * @param Request $request The request instance

-        *
-        * @return Response The rendered view or redirection
-        *
-        * @throws \RuntimeException When user has not at least guest role
+        * @param float $latitude The city latitude
+        * @param float $longitude The city longitude
+        * @return Response The rendered view

         */
         */

-       public function edit(Request $request, $id) {
-               //Prevent non-admin to access here
-               $this->denyAccessUnlessGranted('ROLE_ADMIN', null, $this->translator->trans('Unable to access this page without role %role%!', ['%role%' => $this->translator->trans('Admin')]));
-
-               //Get doctrine
-               $doctrine = $this->getDoctrine();
-
-               //Get location
-               if (empty($location = $doctrine->getRepository(Location::class)->findOneById($id))) {
-                       throw $this->createNotFoundException($this->translator->trans('Unable to find location: %id%', ['%id%' => $id]));
+       public function city(Request $request, float $latitude, float $longitude, string $city): Response {
+               //Get city
+               if (!($this->context['city'] = $this->doctrine->getRepository(Location::class)->findCityByLatitudeLongitudeAsArray(floatval($latitude), floatval($longitude)))) {
+                       throw $this->createNotFoundException($this->translator->trans('Unable to find city: %latitude%,%longitude%', ['%latitude%' => $latitude, '%longitude%' => $longitude]));

                }
 
                }
 

-               //Create LocationType form
-               $form = $this->createForm('Rapsys\AirBundle\Form\LocationType', $location, [
-                       //Set the action
-                       'action' => $this->generateUrl('rapsys_air_location_edit', ['id' => $id]),
-                       //Set the form attribute
-                       'attr' => []
-               ]);
-
-               //Refill the fields in case of invalid form
-               $form->handleRequest($request);
+               //Add calendar
+               $this->context['calendar'] = $this->doctrine->getRepository(Session::class)->findAllByPeriodAsCalendarArray($this->period, !$this->checker->isGranted('IS_AUTHENTICATED_REMEMBERED'), floatval($latitude), floatval($longitude));

 
 

-               //Handle invalid form
-               if (!$form->isSubmitted() || !$form->isValid()) {
-                       //Set section
-                       $section = $this->translator->trans('Location %id%', ['%id%' => $id]);
+               //Set dances
+               $this->context['dances'] = [];

 
 

-                       //Set title
-                       $title = $this->translator->trans($this->config['site']['title']).' - '.$section;
-
-                       //Render the view
-                       return $this->render('@RapsysAir/location/edit.html.twig', ['id' => $id, 'title' => $title, 'section' => $section, 'form' => $form->createView()]+$this->context);
+               //Iterate on each calendar
+               foreach($this->context['calendar'] as $date => $calendar) {
+                       //Iterate on each session
+                       foreach($calendar['sessions'] as $sessionId => $session) {
+                               //Session with application dance
+                               if (!empty($session['application']['dance'])) {
+                                       //Add dance
+                                       $this->context['dances'][$session['application']['dance']['id']] = $session['application']['dance'];
+                               }
+                       }

                }
 
                }
 

-               //Get manager
-               $manager = $doctrine->getManager();
+               //Add locations
+               $this->context['locations'] = $this->doctrine->getRepository(Location::class)->findAllByLatitudeLongitudeAsArray(floatval($latitude), floatval($longitude), $this->period, 0);

 
 

-               //Set updated
-               $location->setUpdated(new \DateTime('now'));
+               //Set modified
+               //XXX: dance modified is already computed inside calendar modified
+               $this->modified = max(array_merge([$this->context['city']['updated']], array_map(function ($v) { return $v['modified']; }, array_merge($this->context['calendar'], $this->context['locations']))));

 
 

-               //Queue location save
-               $manager->persist($location);
+               //Create response
+               $response = new Response();

 
 

-               //Flush to get the ids
-               $manager->flush();
+               //With logged user
+               if ($this->checker->isGranted('IS_AUTHENTICATED_REMEMBERED')) {
+                       //Set last modified
+                       $response->setLastModified(new \DateTime('-1 year'));

 
 

-               //Add notice
-               $this->addFlash('notice', $this->translator->trans('Location %id% updated', ['%id%' => $id]));
+                       //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['city'], $this->context['dances'], $this->context['calendar'], $this->context['locations']))));

 
 

-               //Extract and process referer
-               if ($referer = $request->headers->get('referer')) {
-                       //Create referer request instance
-                       $req = Request::create($referer);
+                       //Set last modified
+                       $response->setLastModified($this->modified);

 
 

-                       //Get referer path
-                       $path = $req->getPathInfo();
+                       //Set as public
+                       $response->setPublic();

 
 

-                       //Get referer query string
-                       $query = $req->getQueryString();
-
-                       //Remove script name
-                       $path = str_replace($request->getScriptName(), '', $path);
-
-                       //Try with referer path
-                       try {
-                               //Save old context
-                               $oldContext = $this->router->getContext();
+                       //Without role and modification
+                       if ($response->isNotModified($request)) {
+                               //Return 304 response
+                               return $response;
+                       }
+               }

 
 

-                               //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 multi
+               $this->context['multimap'] = $this->map->getMultiMap($this->context['city']['multimap'], $this->modified->getTimestamp(), $this->context['locations']);

 
 

-                               //Retrieve route matching path
-                               $route = $this->router->match($path);
+               //Set keywords
+               $this->context['keywords'] = [
+                       $this->context['city']['city'],
+                       $this->translator->trans('Indoor'),
+                       $this->translator->trans('Outdoor'),
+                       $this->translator->trans('Calendar'),
+                       $this->translator->trans('Libre Air')
+               ];

 
 

-                               //Reset context
-                               $this->router->setContext($oldContext);
+               //With context dances
+               if (!empty($this->context['dances'])) {
+                       //Set dances
+                       $dances = array_map(function ($v) { return $v['name']; }, $this->context['dances']);

 
 

-                               //Clear old context
-                               unset($oldContext);
+                       //Insert dances in keywords
+                       array_splice($this->context['keywords'], 1, 0, $dances);

 
 

-                               //Extract name
-                               $name = $route['_route'];
+                       //Get textual dances
+                       $dances = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($dances, 0, -1))], array_slice($dances, -1)), 'strlen'));

 
 

-                               //Remove route and controller from route defaults
-                               unset($route['_route'], $route['_controller']);
+                       //Set title
+                       $this->context['title']['page'] = $this->translator->trans('%dances% %city%', ['%dances%' => $dances, '%city%' => $this->context['city']['in']]);

 
 

-                               //Check if location view route
-                               if ($name == 'rapsys_air_location_view' && !empty($route['id'])) {
-                                       //Replace id
-                                       $route['id'] = $location->getId();
-                               //Other routes
-                               } else {
-                                       //Set location
-                                       $route['location'] = $location->getId();
-                               }
+                       //Set description
+                       $this->context['description'] = $this->translator->trans('%dances% indoor and outdoor calendar %city%', ['%dances%' => $dances, '%city%' => $this->context['city']['in']]);
+               } else {
+                       //Set title
+                       $this->context['title']['page'] = $this->translator->trans('Dance %city%', ['%city%' => $this->context['city']['in']]);

 
 

-                               //Generate url
-                               return $this->redirectToRoute($name, $route);
-                       //No route matched
-                       } catch(MethodNotAllowedException|ResourceNotFoundException $e) {
-                               //Unset referer to fallback to default route
-                               unset($referer);
-                       }
+                       //Set description
+                       $this->context['description'] = $this->translator->trans('Indoor and outdoor dance calendar %city%', ['%city%' => $this->context['city']['in']]);

                }
 
                }
 

-               //Redirect to cleanup the form
-               return $this->redirectToRoute('rapsys_air', ['location' => $location->getId()]);
+               //Set locations description
+               $this->context['locations_description'] = $this->translator->trans('Libre Air location list %city%', ['%city%' => $this->context['city']['in']]);
+
+               //Render the view
+               return $this->render('@RapsysAir/location/city.html.twig', $this->context, $response);

        }
 
        /**
         * List all locations
         *
        }
 
        /**
         * List all locations
         *

-        * @desc Display all locations
+        * Display all locations

         *
         * @param Request $request The request instance
         *
         * @return Response The rendered view
         */
        public function index(Request $request): Response {
         *
         * @param Request $request The request instance
         *
         * @return Response The rendered view
         */
        public function index(Request $request): Response {

-               //Fetch doctrine
-               $doctrine = $this->getDoctrine();
+               //Get locations
+               $this->context['locations'] = $this->doctrine->getRepository(Location::class)->findAllAsArray($this->period);
+
+               //Set modified
+               $this->modified = max(array_map(function ($v) { return $v['updated']; }, $this->context['locations']));
+
+               //Create response
+               $response = new Response();
+
+               //With logged user
+               if ($this->checker->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($this->context['locations'])));
+
+                       //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;
+                       }
+               }

 
 

-               //Set section
-               $section = $this->translator->trans('Libre Air locations');
+               //Add multi map
+               $this->context['multimap'] = $this->map->getMultiMap($this->translator->trans('Libre Air locations sector map'), $this->modified->getTimestamp(), $this->context['locations']);
+
+               //Set title
+               $this->context['title']['page'] = $this->translator->trans('Libre Air locations');

 
                //Set description
                $this->context['description'] = $this->translator->trans('Libre Air location list');
 
                //Set description
                $this->context['description'] = $this->translator->trans('Libre Air location list');


@@ -284,126 +286,227 @@ class LocationController extends DefaultController {
                        $this->translator->trans('Libre Air')
                ];
 
                        $this->translator->trans('Libre Air')
                ];
 

-               //Set title
-               $title = $this->translator->trans($this->config['site']['title']).' - '.$section;
-
-               //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'
-                       )
-               );
-

                //Create location forms for role_admin
                //Create location forms for role_admin

-               if ($this->isGranted('ROLE_ADMIN')) {
+               if ($this->checker->isGranted('ROLE_ADMIN')) {

                        //Fetch all locations
                        //Fetch all locations

-                       $locations = $doctrine->getRepository(Location::class)->findAll();
-
-                       //Rekey by id
-                       $locations = array_reduce($locations, function($carry, $item){$carry[$item->getId()] = $item; return $carry;}, []);
+                       $locations = $this->doctrine->getRepository(Location::class)->findAll();

 
                        //Init locations to context
                        $this->context['forms']['locations'] = [];
 
                        //Iterate on locations
 
                        //Init locations to context
                        $this->context['forms']['locations'] = [];
 
                        //Iterate on locations

-                       foreach($locations as $locationId => $location) {
+                       foreach($this->context['locations'] as $id => $location) {

                                //Create LocationType form
                                //Create LocationType form

-                               $form = $this->createForm('Rapsys\AirBundle\Form\LocationType', $location, [
-                                       //Set the action
-                                       'action' => $this->generateUrl('rapsys_air_location_edit', ['id' => $location->getId()]),
-                                       //Set the form attribute
-                                       'attr' => [],
-                                       //Set block prefix
-                                       //TODO: make this shit works to prevent label collision
-                                       //XXX: see https://stackoverflow.com/questions/8703016/adding-a-prefix-to-a-form-label-for-translation
-                                       'label_prefix' => 'location_'.$locationId
-                               ]);
+                               $form = $this->factory->createNamed(
+                                       //Set form id
+                                       'locations_'.$id,
+                                       //Set form type
+                                       'Rapsys\AirBundle\Form\LocationType',
+                                       //Set form data
+                                       $locations[$location['id']],
+                                       //Set the form attributes
+                                       ['attr' => []]
+                               );
+
+                               //Refill the fields in case of invalid form
+                               $form->handleRequest($request);
+
+                               //Handle valid form
+                               if ($form->isSubmitted() && $form->isValid()) {
+                                       //Get data
+                                       $data = $form->getData();
+
+                                       //Set updated
+                                       $data->setUpdated(new \DateTime('now'));
+
+                                       //Queue location save
+                                       $this->manager->persist($data);
+
+                                       //Flush to get the ids
+                                       $this->manager->flush();
+
+                                       //Add notice
+                                       $this->addFlash('notice', $this->translator->trans('Location %id% updated', ['%id%' => $location['id']]));
+
+                                       //Redirect to cleanup the form
+                                       return $this->redirectToRoute('rapsysair_location', ['location' => $location['id']]);
+                               }

 
                                //Add form to context
 
                                //Add form to context

-                               $this->context['forms']['locations'][$locationId] = $form->createView();
+                               $this->context['forms']['locations'][$id] = $form->createView();

                        }
 
                        //Create LocationType form
                        }
 
                        //Create LocationType form

-                       $form = $this->createForm('Rapsys\AirBundle\Form\LocationType', null, [
-                               //Set the action
-                               'action' => $this->generateUrl('rapsys_air_location_add'),
-                               //Set the form attribute
-                               'attr' => [ 'class' => 'col' ]
-                       ]);
+                       $form = $this->factory->createNamed(
+                               //Set form id
+                               'locations',
+                               //Set form type
+                               'Rapsys\AirBundle\Form\LocationType',
+                               //Set form data
+                               new Location(),
+                               //Set the form attributes
+                               ['attr' => ['class' => 'col']]
+                       );
+
+                       //Refill the fields in case of invalid form
+                       $form->handleRequest($request);
+
+                       //Handle valid form
+                       if ($form->isSubmitted() && $form->isValid()) {
+                               //Get data
+                               $data = $form->getData();
+
+                               //Queue location save
+                               $this->manager->persist($data);
+
+                               //Flush to get the ids
+                               $this->manager->flush();
+
+                               //Add notice
+                               $this->addFlash('notice', $this->translator->trans('Location created'));
+
+                               //Redirect to cleanup the form
+                               return $this->redirectToRoute('rapsysair_location', ['location' => $data->getId()]);
+                       }

 
                        //Add form to context
                        $this->context['forms']['location'] = $form->createView();
                }
 
 
                        //Add form to context
                        $this->context['forms']['location'] = $form->createView();
                }
 

-               //Fetch locations
-               //XXX: we want to display all active locations anyway
-               $locations = $doctrine->getRepository(Location::class)->findTranslatedSortedByPeriod($this->translator, $period);
-

                //Render the view
                //Render the view

-               return $this->render('@RapsysAir/location/index.html.twig', ['title' => $title, 'section' => $section, 'locations' => $locations]+$this->context);
+               return $this->render('@RapsysAir/location/index.html.twig', $this->context);

        }
 
        /**
         * List all sessions for the location
         *
        }
 
        /**
         * List all sessions for the location
         *

-        * @desc Display all sessions for the location with an application or login form
+        * Display all sessions for the location with an application or login form
+        *
+        * @TODO: add location edit form ???

         *
         * @param Request $request The request instance
         * @param int $id The location id
         *
         * @param Request $request The request instance
         * @param int $id The location id

+        * @param ?string $location The location slug

         *
         * @return Response The rendered view
         */
         *
         * @return Response The rendered view
         */

-       public function view(Request $request, $id): Response {
-               //Fetch doctrine
-               $doctrine = $this->getDoctrine();
-
-               //Fetch location
-               if (empty($location = $doctrine->getRepository(Location::class)->findOneById($id))) {
+       public function view(Request $request, int $id, ?string $location): Response {
+               //Without location
+               if (empty($this->context['location'] = $this->doctrine->getRepository(Location::class)->findOneByIdAsArray($id, $this->locale))) {
+                       //Throw 404

                        throw $this->createNotFoundException($this->translator->trans('Unable to find location: %id%', ['%id%' => $id]));
                }
 
                        throw $this->createNotFoundException($this->translator->trans('Unable to find location: %id%', ['%id%' => $id]));
                }
 

-               //Set section
-               $section = $this->translator->trans('Argentine Tango at '.$location);
+               //With invalid slug
+               if ($location !== $this->context['location']['slug']) {
+                       //Redirect on correctly spelled location
+                       return $this->redirectToRoute('rapsysair_location_view', ['id' => $this->context['location']['id'], 'location' => $this->context['location']['slug']], Response::HTTP_MOVED_PERMANENTLY);
+               }

 
 

-               //Set description
-               $this->context['description'] = $this->translator->trans('Outdoor Argentine Tango session calendar %location%', [ '%location%' => $this->translator->trans('at '.$location) ]);
+               //Fetch calendar
+               $this->context['calendar'] = $this->doctrine->getRepository(Session::class)->findAllByPeriodAsCalendarArray($this->period, !$this->checker->isGranted('IS_AUTHENTICATED_REMEMBERED'), $this->context['location']['latitude'], $this->context['location']['longitude']);
+
+               //Set dances
+               $this->context['dances'] = [];
+
+               //Iterate on each calendar
+               foreach($this->context['calendar'] as $date => $calendar) {
+                       //Iterate on each session
+                       foreach($calendar['sessions'] as $sessionId => $session) {
+                               //Session with application dance
+                               if (!empty($session['application']['dance'])) {
+                                       //Add dance
+                                       $this->context['dances'][$session['application']['dance']['id']] = $session['application']['dance'];
+                               }
+                       }
+               }
+
+               //Get locations at less than 2 km
+               $this->context['locations'] = $this->doctrine->getRepository(Location::class)->findAllByLatitudeLongitudeAsArray($this->context['location']['latitude'], $this->context['location']['longitude'], $this->period, 2);
+
+               //Set modified
+               //XXX: dance modified is already computed inside calendar modified
+               $this->modified = max(array_merge([$this->context['location']['modified']], array_map(function ($v) { return $v['modified']; }, array_merge($this->context['calendar'], $this->context['locations']))));
+
+               //Create response
+               $response = new Response();
+
+               //With logged user
+               if ($this->checker->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['location'], $this->context['calendar'], $this->context['locations']))));
+
+                       //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;
+                       }
+               }
+
+               //Add multi map
+               $this->context['multimap'] = $this->map->getMultiMap($this->context['location']['multimap'], $this->modified->getTimestamp(), $this->context['locations']);

 
                //Set keywords
                $this->context['keywords'] = [
 
                //Set keywords
                $this->context['keywords'] = [

-                       $this->translator->trans($location),
-                       $this->translator->trans('outdoor'),
-                       $this->translator->trans('Argentine Tango'),
-                       $this->translator->trans('calendar')
+                       $this->context['location']['title'],
+                       $this->context['location']['city']['title'],
+                       $this->translator->trans($this->context['location']['indoor']?'Indoor':'Outdoor'),
+                       $this->translator->trans('Calendar'),
+                       $this->translator->trans('Libre Air')

                ];
 
                ];
 

-               //Set title
-               $title = $this->translator->trans($this->config['site']['title']).' - '.$section;
-
-               //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'
-                       )
-               );
+               //With dances
+               if (!empty($this->context['dances'])) {
+                       //Set dances
+                       $dances = array_map(function ($v) { return $v['name']; }, $this->context['dances']);

 
 

-               //Fetch calendar
-               $calendar = $doctrine->getRepository(Session::class)->fetchCalendarByDatePeriod($this->translator, $period, $id, $request->get('session'), !$this->isGranted('IS_AUTHENTICATED_REMEMBERED'), $request->getLocale());
+                       //Insert dances in keywords
+                       array_splice($this->context['keywords'], 2, 0, $dances);
+
+                       //Get textual dances
+                       $dances = implode($this->translator->trans(' and '), array_filter(array_merge([implode(', ', array_slice($dances, 0, -1))], array_slice($dances, -1)), 'strlen'));
+
+                       //Set title
+                       $this->context['title']['page'] = $this->translator->trans('%dances% %location%', ['%dances%' => $dances, '%location%' => $this->context['location']['atin']]);
+
+                       //Set description
+                       $this->context['description'] = $this->translator->trans('%dances% indoor and outdoor calendar %location%', ['%dances%' => $dances, '%location%' => $this->context['location']['at']]);
+               //Without dances
+               } else {
+                       //Set title
+                       $this->context['title']['page'] = $this->translator->trans('Dance %location%', ['%location%' => $this->context['location']['atin']]);
+
+                       //Set description
+                       $this->context['description'] = $this->translator->trans('Indoor and outdoor dance calendar %location%', ['%location%' => $this->context['location']['at']]);
+               }
+
+               //Set locations description
+               $this->context['locations_description'] = $this->translator->trans('Libre Air location list %location% %city%', ['%location%' => $this->context['location']['around'], '%city%' => $this->context['location']['city']['in']]);
+
+               //Set locations link
+               $this->context['locations_link'] = $this->context['location']['city']['link'];
+
+               //Set locations title
+               $this->context['locations_title'] = $this->context['location']['city']['title'].' ('.$this->context['location']['city']['id'].')';

 
 

-               //Fetch locations
-               //XXX: we want to display all active locations anyway
-               $locations = $doctrine->getRepository(Location::class)->findTranslatedSortedByPeriod($this->translator, $period);
+               //Set alternates
+               $this->context['alternates'] += $this->context['location']['alternates'];

 
                //Render the view
 
                //Render the view

-               return $this->render('@RapsysAir/location/view.html.twig', ['id' => $id, 'title' => $title, 'section' => $section, 'calendar' => $calendar, 'locations' => $locations]+$this->context);
+               return $this->render('@RapsysAir/location/view.html.twig', $this->context, $response);

        }
 }
        }
 }