From b0ee67f8b5eea8b6bd56fdc416b3e2185106dee5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien.fastre@champs-libres.coop>
Date: Fri, 8 Jun 2018 17:49:35 +0200
Subject: [PATCH] handle menu with new menuBuilder api

---
 .../ChillActivityExtension.php                |  1 +
 Menu/MenuBuilder.php                          | 88 +++++++++++++++++++
 Resources/config/routing/activity.yml         | 10 ---
 Resources/config/services/menu.yml            |  8 ++
 4 files changed, 97 insertions(+), 10 deletions(-)
 create mode 100644 Menu/MenuBuilder.php
 create mode 100644 Resources/config/services/menu.yml

diff --git a/DependencyInjection/ChillActivityExtension.php b/DependencyInjection/ChillActivityExtension.php
index 844e13355..ab0059155 100644
--- a/DependencyInjection/ChillActivityExtension.php
+++ b/DependencyInjection/ChillActivityExtension.php
@@ -52,6 +52,7 @@ class ChillActivityExtension extends Extension implements PrependExtensionInterf
         $loader->load('services/export.yml');
         $loader->load('services/repositories.yml');
         $loader->load('services/fixtures.yml');
+        $loader->load('services/menu.yml');
     }
     
     public function prepend(ContainerBuilder $container)
diff --git a/Menu/MenuBuilder.php b/Menu/MenuBuilder.php
new file mode 100644
index 000000000..f11aa28bc
--- /dev/null
+++ b/Menu/MenuBuilder.php
@@ -0,0 +1,88 @@
+<?php
+/*
+ * 
+ */
+namespace Chill\ActivityBundle\Menu;
+
+use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
+use Chill\MainBundle\Security\Authorization\AuthorizationHelper;
+use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
+use Symfony\Component\Translation\TranslatorInterface;
+use Knp\Menu\MenuItem;
+use Symfony\Component\Security\Core\Role\Role;
+use Chill\ActivityBundle\Security\Authorization\ActivityVoter;
+
+/**
+ * 
+ *
+ * @author Julien Fastré <julien.fastre@champs-libres.coop>
+ */
+class MenuBuilder implements LocalMenuBuilderInterface
+{
+    /**
+     *
+     * @var TokenStorageInterface
+     */
+    protected $tokenStorage;
+    
+    /**
+     *
+     * @var TranslatorInterface
+     */
+    protected $translator;
+    
+    /**
+     *
+     * @var AuthorizationHelper
+     */
+    protected $authorizationHelper;
+    
+    public function __construct(
+        TokenStorageInterface $tokenStorage, 
+        TranslatorInterface $translator, 
+        AuthorizationHelper $authorizationHelper
+    ) {
+        $this->tokenStorage = $tokenStorage;
+        $this->translator = $translator;
+        $this->authorizationHelper = $authorizationHelper;
+    }
+
+    
+    public function buildMenu($menuId, MenuItem $menu, array $parameters)
+    {
+        /* @var $person \Chill\PersonBundle\Entity\Person */
+        $person  = $parameters['person'];
+        $user    = $this->tokenStorage->getToken()->getUser();
+        $roleSee = new Role(ActivityVoter::SEE);
+        $roleAdd = new Role(ActivityVoter::CREATE);
+        
+        if ($this->authorizationHelper->userHasAccess($user, $person, $roleSee)) {
+            $menu->addChild($this->translator->trans('Activity list'), [
+                    'route' => 'chill_activity_activity_list',
+                    'routeParameters' => [
+                        'person_id' => $person->getId()
+                    ]
+                ])
+                ->setExtras([
+                    'order' => 201
+                ]);
+        }
+        
+        if ($this->authorizationHelper->userHasAccess($user, $person, $roleAdd)) {
+            $menu->addChild($this->translator->trans('Add a new activity'), [
+                    'route' => 'chill_activity_activity_new',
+                    'routeParameters' => [
+                        'person_id' => $person->getId()
+                    ]
+                ])
+                ->setExtras([
+                    'order' => 200
+                ]);
+        }
+    }
+
+    public static function getMenuIds(): array
+    {
+        return [ 'person' ];
+    }
+}
diff --git a/Resources/config/routing/activity.yml b/Resources/config/routing/activity.yml
index 6d287eec7..5b49b93e5 100644
--- a/Resources/config/routing/activity.yml
+++ b/Resources/config/routing/activity.yml
@@ -1,11 +1,6 @@
 chill_activity_activity_list:
     path:     /{_locale}/person/{person_id}/activity/
     defaults: { _controller: "ChillActivityBundle:Activity:list" }
-    options:
-        menus:
-            person:
-                order: 201
-                label: Activity list
 
 chill_activity_activity_show:
     path:     /{_locale}/person/{person_id}/activity/{id}/show
@@ -14,11 +9,6 @@ chill_activity_activity_show:
 chill_activity_activity_new:
     path:     /{_locale}/person/{person_id}/activity/new
     defaults: { _controller: "ChillActivityBundle:Activity:new" }
-    options:
-        menus:
-            person:
-                order: 200
-                label: Add a new activity
 
 chill_activity_activity_create:
     path:     /{_locale}/person/{person_id}/activity/create
diff --git a/Resources/config/services/menu.yml b/Resources/config/services/menu.yml
new file mode 100644
index 000000000..2a0434996
--- /dev/null
+++ b/Resources/config/services/menu.yml
@@ -0,0 +1,8 @@
+services:
+    Chill\ActivityBundle\Menu\MenuBuilder:
+        arguments:
+            $authorizationHelper: '@Chill\MainBundle\Security\Authorization\AuthorizationHelper'
+            $tokenStorage: '@Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface'
+            $translator: '@Symfony\Component\Translation\TranslatorInterface'
+        tags:
+            - { name: 'chill.menu_builder' }