7 files changed, 378 insertions, 7 deletions

diff --git a/src/Wallabag/GroupBundle/Controller/ManageController.php b/src/Wallabag/GroupBundle/Controller/ManageController.php
index 31a0c7b1..7214d899 100644
--- a/src/Wallabag/GroupBundle/Controller/ManageController.php
+++ b/src/Wallabag/GroupBundle/Controller/ManageController.php
@@ -2,6 +2,7 @@
 
 namespace Wallabag\GroupBundle\Controller;
 
+use Pagerfanta\Adapter\ArrayAdapter;
 use Pagerfanta\Adapter\DoctrineORMAdapter;
 use Pagerfanta\Exception\OutOfRangeCurrentPageException;
 use Pagerfanta\Pagerfanta;
@@ -15,6 +16,7 @@ use Wallabag\GroupBundle\Entity\Group;
 use Wallabag\GroupBundle\Entity\UserGroup;
 use Wallabag\GroupBundle\Form\GroupType;
 use Wallabag\GroupBundle\Form\NewGroupType;
+use Wallabag\GroupBundle\Form\UserGroupType;
 use Wallabag\UserBundle\Entity\User;
 
 /**
@@ -210,4 +212,186 @@ class ManageController extends Controller
         }
         throw $this->createAccessDeniedException();
     }
+
+        /**
+         * @Route("/join/{group}", name="group_join")
+         * @param Group $group
+         * @return Response
+         */
+        public function joinGroupAction(Group $group): Response
+        {
+                $em = $this->getDoctrine()->getManager();
+
+                if ($group->getAcceptSystem() === Group::ACCESS_PASSWORD) {
+                        return $this->redirectToRoute('group_password', ['group' => $group->getId()]);
+                }
+                $this->getUser()->addAGroup($group, $group->getDefaultRole());
+
+                $em->flush();
+
+                return $this->redirect($this->generateUrl('group_index'), 302);
+        }
+
+        /**
+         * @Route("/manage/{group}/{page}", name="group-manage", defaults={"page" = "1"})
+         * @param Group $group
+         * @return Response
+         */
+        public function manageGroupUsersAction(Group $group, int $page): Response
+        {
+                if ($this->getUser()->getGroupRoleForUser($group) < Group::ROLE_MANAGE_USERS) {
+                        $this->createAccessDeniedException();
+                }
+
+                $members = $this->get('wallabag_user.user_repository')->findGroupMembers($group->getId());
+
+                $pagerAdapter = new DoctrineORMAdapter($members->getQuery(), true, false);
+                $pagerFanta = new Pagerfanta($pagerAdapter);
+                $pagerFanta->setMaxPerPage(9);
+
+                try {
+                        $pagerFanta->setCurrentPage($page);
+                } catch (OutOfRangeCurrentPageException $e) {
+                        if ($page > 1) {
+                                return $this->redirect($this->generateUrl('groups', ['page' => $pagerFanta->getNbPages()]), 302);
+                        }
+                }
+
+                return $this->render('WallabagGroupBundle:Manage:manage.html.twig', [
+                        'members' => $pagerFanta,
+                        'group' => $group,
+                        'currentPage' => $page,
+                ]);
+        }
+
+        /**
+         * @Route("/leave/{group}", name="group_leave")
+         * @param Group $group
+         * @return Response
+         */
+        public function leaveGroupAction(Group $group): Response
+        {
+                $logger = $this->get('logger');
+                $em = $this->getDoctrine()->getManager();
+                $removeGroup = false;
+
+                if ($this->getUser()->getGroupRoleForUser($group) == Group::ROLE_ADMIN) {
+                        $logger->info('User ' . $this->getUser()->getUsername() . ' is the admin for group ' . $group->getName());
+                        $newUser = $group->getUsers()->first();
+                        $newUser->setGroupRole($group, Group::ROLE_ADMIN);
+                        $logger->info('The new admin for group ' . $group->getName() . ' is user ' . $newUser->getUsername());
+                }
+
+                if ($group->getUsers()->count() <= 1) {
+                        $logger->info('User ' . $this->getUser()->getUsername() . ' was the last one on the group ' . $group->getName() . ' so it will be deleted');
+                        $removeGroup = true;
+                }
+
+                $logger->info('Removing user ' . $this->getUser()->getUsername() . ' from group ' . $group->getName());
+                $em->remove($this->getUser()->getUserGroupFromGroup($group));
+
+                if ($removeGroup) {
+                        $logger->info("Removing group " . $group->getName() . " as it doesn't contains users anymore");
+                        $em->remove($group);
+                }
+
+                $em->flush();
+                return $this->redirect($this->generateUrl('groups'), 302);
+        }
+
+        /**
+         * @Route("/requests/{group}/{page}", name="group-requests", defaults={"page" = "1"})
+         * @param Request $request
+         * @param int $page
+         * @return Response
+         */
+        public function showRequestsAction(Group $group, int $page): Response
+        {
+                if ($this->getUser()->getGroupRoleForUser($group) < Group::ROLE_MANAGE_USERS) {
+                        $this->createAccessDeniedException();
+                }
+
+                $requests = $group->getRequests();
+                $pagerAdapter = new ArrayAdapter($requests->toArray());
+
+                $pagerFanta = new Pagerfanta($pagerAdapter);
+                $pagerFanta->setMaxPerPage(9);
+
+                try {
+                        $pagerFanta->setCurrentPage($page);
+                } catch (OutOfRangeCurrentPageException $e) {
+                        if ($page > 1) {
+                                return $this->redirect($this->generateUrl('groups', ['page' => $pagerFanta->getNbPages()]), 302);
+                        }
+                }
+
+                return $this->render('WallabagGroupBundle:Manage:requests.html.twig', [
+                        'requests' => $pagerFanta,
+                        'group' => $group,
+                        'currentPage' => $page,
+                ]);
+        }
+
+        /**
+         * @Route("/activate/{group}/{user}/{accept}", name="group-activate", requirements={"accept" = "\d+"})
+         * @param Group $group
+         * @param User $user
+         * @param $accept
+         * @return Response
+         */
+        public function postRequestAction(Group $group, User $user, $accept): Response
+        {
+                if (!$this->getUser() < Group::ROLE_MANAGE_USERS) {
+                        $this->createAccessDeniedException("You don't have the rights to do this");
+                }
+
+                $em = $this->getDoctrine()->getManager();
+
+                $accept = $accept == 1;
+                $user->getUserGroupFromGroup($group)->setAccepted($accept);
+                if (!$accept) {
+                        $em->remove($user->getUserGroupFromGroup($group));
+                }
+
+                $em->flush();
+
+                return $this->redirectToRoute('group_index');
+        }
+
+        /**
+         * @Route("/user-edit/{group}/{user}", name="group-user-edit")
+         * @param Request $request
+         * @param Group $group
+         * @param User $user
+         * @return Response
+         */
+        public function editGroupUsersAction(Request $request, Group $group, User $user): Response
+        {
+                if ($this->getUser()->getGroupRoleForUser($group) < Group::ROLE_MANAGE_USERS) {
+                        $this->createAccessDeniedException();
+                }
+
+                $groupUser = $user->getUserGroupFromGroup($group);
+                $editForm = $this->createForm(UserGroupType::class, $groupUser);
+                $editForm->handleRequest($request);
+
+                if ($editForm->isSubmitted() && $editForm->isValid()) {
+                        $em = $this->getDoctrine()->getManager();
+                        $em->persist($groupUser);
+                        $em->flush();
+
+                        $this->get('session')->getFlashBag()->add(
+                                'notice',
+                                $this->get('translator')->trans('flashes.group.notice.user.edited', ['%user%' => $user->getUsername(), '%group%' => $group->getName()])
+                        );
+
+                        return $this->redirectToRoute('group-manage', ['group' => $group->getId()]);
+                }
+
+                return $this->render('WallabagGroupBundle:Manage:edit_user.html.twig', array(
+                        'user' => $user,
+                        'group' => $group,
+                        'edit_form' => $editForm->createView(),
+                ));
+        }
 }
diff --git a/src/Wallabag/GroupBundle/Form/UserGroupType.php b/src/Wallabag/GroupBundle/Form/UserGroupType.php
index e8f814e4..326fa682 100644
--- a/src/Wallabag/GroupBundle/Form/UserGroupType.php
+++ b/src/Wallabag/GroupBundle/Form/UserGroupType.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace Wallabag\GroupBundle\Form\Type;
+namespace Wallabag\GroupBundle\Form;
 
 use Wallabag\GroupBundle\Entity\Group;
 use Symfony\Component\Form\AbstractType;
@@ -23,7 +23,7 @@ class UserGroupType extends AbstractType
                 'choices' => [
                     'group.roles.readonly' => Group::ROLE_READ_ONLY,
                     'group.roles.write' => Group::ROLE_WRITE,
-                    'group.roles.manage_prez' => Group::ROLE_MANAGE_PREZ,
+                    'group.roles.manage_entries' => Group::ROLE_MANAGE_ENTRIES,
                     'group.roles.manage_users' => Group::ROLE_MANAGE_USERS,
                     'group.roles.admin' => Group::ROLE_ADMIN,
                 ],
@@ -40,7 +40,7 @@ class UserGroupType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults(array(
-            'data_class' => 'Strut\StrutBundle\Entity\UserGroup',
+            'data_class' => 'Wallabag\GroupBundle\Entity\UserGroup',
         ));
     }
 }
diff --git a/src/Wallabag/GroupBundle/Resources/config/services.yml b/src/Wallabag/GroupBundle/Resources/config/services.yml
index 76e8427a..618f7ed7 100644
--- a/src/Wallabag/GroupBundle/Resources/config/services.yml
+++ b/src/Wallabag/GroupBundle/Resources/config/services.yml
@@ -1,3 +1,8 @@
 services:
     sha256salted_encoder:
-          class: Strut\StrutBundle\Service\Sha256Salted
\ No newline at end of file
+          class: Wallabag\GroupBundle\Service\Sha256Salted
+
+    wallabag_core.filter.type.entry:
+            class: Wallabag\GroupBundle\Form\UserGroupType
+            tags:
+                - { name: form.type }
\ No newline at end of file
diff --git a/src/Wallabag/GroupBundle/Resources/views/Manage/edit_user.html.twig b/src/Wallabag/GroupBundle/Resources/views/Manage/edit_user.html.twig
new file mode 100644
index 00000000..b7771690
--- /dev/null
+++ b/src/Wallabag/GroupBundle/Resources/views/Manage/edit_user.html.twig
@@ -0,0 +1,26 @@
+{% extends "WallabagCoreBundle::layout.html.twig" %}
+
+{% block title %}{{ 'group.edit_user.title'|trans }}{% endblock %}
+
+{% block content %}
+
+<div class="row">
+    <div class="col-md-6">
+
+        {{ form_start(edit_form) }}
+        {{ form_errors(edit_form) }}
+
+        <div class="form-group">
+            {{ form_label(edit_form.role) }}
+            {{ form_errors(edit_form.role) }}
+            {{ form_widget(edit_form.role) }}
+        </div>
+
+        {{ form_widget(edit_form.save, {'attr': {'class': 'btn waves-effect'}}) }}
+        <a class="btn btn-default" href="{{ path('group-manage', {'group': group.id}) }}">{{ 'group.edit_user.cancel' | trans }}</a>
+        {{ form_widget(edit_form._token) }}
+        {{ form_end(edit_form) }}
+    </div>
+</div>
+
+{% endblock %}
diff --git a/src/Wallabag/GroupBundle/Resources/views/Manage/index.html.twig b/src/Wallabag/GroupBundle/Resources/views/Manage/index.html.twig
index 01466e52..35a70400 100644
--- a/src/Wallabag/GroupBundle/Resources/views/Manage/index.html.twig
+++ b/src/Wallabag/GroupBundle/Resources/views/Manage/index.html.twig
@@ -12,7 +12,7 @@
                         <p class="help">{{ 'group.description'|trans|raw }}</p>
 
                         {% if groups is not empty %}
-                            <table class="bordered">
+                            <table class="bordered responsive-table">
                                 <thead>
                                     <tr>
                                         <th>{{ 'group.form.name_label'|trans }}</th>
@@ -24,9 +24,32 @@
                                 {% for group in groups %}
                                     <tr>
                                         <td>{{ group.name }}</td>
-                                        <td></td>
                                         <td>
-                                            <a href="{{ path('group_edit', { 'id': group.id }) }}">{{ 'group.list.edit_action'|trans }}</a>
+                                            {% if app.user in group.users and app.user.inGroup(group) %}
+                                                {{ 'group.user.inGroup' | transchoice(group.users | length) }}
+                                            {% else %}
+                                                {{ 'group.user.notInGroup' | transchoice(group.users | length) }}
+                                            {% endif %}
+                                        </td>
+                                        <td>
+                                            {% if app.user in group.users and app.user.inGroup(group) %}
+                                                {% if app.user.getGroupRoleForUser(group) >= 5 %}
+                                                    <a href="{{ path('group_edit', { 'id': group.id }) }}" class="waves-effect waves-light btn tooltipped" data-delay="50" data-position="bottom" data-tooltip="{{ 'group.list.edit_action'|trans }}"><i class="material-icons">edit</i></a>
+                                                    <a href="{{ path('group-manage', { 'group': group.id }) }}" class="waves-effect waves-light btn tooltipped" data-delay="50" data-position="bottom" data-tooltip="{{ 'group.manage.label'|trans }}"><i class="material-icons">settings_applications</i></a>
+                                                {% endif %}
+                                                <a href="{{ path('group-presentations', { 'group': group.id }) }}" class="waves-effect waves-light btn tooltipped" data-delay="50" data-position="bottom" data-tooltip="{{ 'group.entries.label'|trans }}"><i class="material-icons">view_list</i></a>
+                                                <a href="{{ path('group_leave', { 'group': group.id }) }}" class="waves-effect waves-light btn red tooltipped" data-delay="50" data-position="bottom" data-tooltip="{{ 'group.leave.label'|trans }}"><i class="material-icons">clear</i></a>
+                                            {% elseif app.user in group.users and not app.user.inGroup(group) %}
+                                                {{ 'group.requests.sent' | trans }}
+                                            {% else %}
+                                                <a href="{{ path('group_join', { 'group': group.id }) }}" class="waves-effect waves-light btn tooltipped" data-tooltip="{{ 'group.join.label'|trans }}" data-delay="50" data-position="bottom">
+                                                    {% if group.acceptSystem == 3 %}
+                                                        <i class="material-icons md-18 vertical-align-middle" data-toggle="tooltip" data-placement="bottom" title="{{ 'group.join'|trans }}">lock</i>
+                                                    {% else %}
+                                                        <i class="material-icons" data-toggle="tooltip" data-placement="bottom" title="{{ 'group.join'|trans }}">input</i>
+                                                    {% endif %}
+                                                </a>
+                                            {% endif %}
                                         </td>
                                     </tr>
                                 {% endfor %}
diff --git a/src/Wallabag/GroupBundle/Resources/views/Manage/manage.html.twig b/src/Wallabag/GroupBundle/Resources/views/Manage/manage.html.twig
new file mode 100644
index 00000000..2226b299
--- /dev/null
+++ b/src/Wallabag/GroupBundle/Resources/views/Manage/manage.html.twig
@@ -0,0 +1,102 @@
+{% extends "WallabagCoreBundle::layout.html.twig" %}
+
+{% block title %}{{ 'group.manage.title' |trans({ '%group%': group.name }) }}{% endblock %}
+
+{% block content %}
+
+<div class="row">
+    <div class="input-field col s12">
+        <p>
+            {{ 'group.form.role_label' |trans }}:
+            <em>
+                {% if group.defaultRole == 1 %}
+                    {{ 'group.roles.readonly' | trans }}
+                {% elseif group.defaultRole == 2 %}
+                    {{ 'group.roles.write' | trans }}
+                {% elseif group.defaultRole == 3 %}
+                    {{ 'group.roles.manage_prez' | trans }}
+                {% elseif group.defaultRole == 5 %}
+                    {{ 'group.roles.manage_users' | trans }}
+                {% elseif group.defaultRole == 10 %}
+                    {{ 'group.roles.admin' | trans }}
+                {% else %}
+                    {{ 'group.roles.unknown' | trans }}
+                {% endif %}
+            </em>
+        </p>
+        <p>
+            {{ 'group.form.access_label' | trans }}:
+            <em>
+                {% if group.acceptSystem == 1 %}
+                    {{ 'group.access.open' | trans }}
+                {% elseif group.acceptSystem == 2 %}
+                    {{ 'group.access.request' | trans }}
+                {% elseif group.acceptSystem == 3 %}
+                    {{ 'group.access.password' | trans }}
+                {% elseif group.acceptSystem == 4 %}
+                    {{ 'group.access.invitation' | trans }}
+                {% elseif group.acceptSystem == 10 %}
+                    {{ 'group.access.hidden' | trans }}
+                {% else %}
+                    {{ 'group.access.unknown' | trans }}
+                {% endif %}
+            </em>
+        </p>
+        <a href="{{ path('group_edit', {'id' : group.id}) }}" class="btn waves-effect">{{ 'group.edit_group' | trans }}</a>
+        {% if group.acceptSystem == 2 %}
+            <a href="{{ path('group-requests', {'group' : group.id}) }}" class="btn waves-effect">{{ 'group.requests.list' | trans }} <span class="badge">{{ group.getRequests() | length }}</span></a>
+        {% elseif group.acceptSystem >= 4 %}
+            <a href="{{ path('group-invitations', {'group' : group.id}) }}" class="btn waves-effect">{{ 'group.edit.invitations' | trans }} <span class="badge">{{ group.getInvited() | length }}</span></a>
+        {% endif %}
+        <a href="{{ path('group_delete', { 'id': group.id }) }}" class="btn waves-effect red"><i class="material-icons md-18" data-toggle="tooltip" data-placement="bottom" title="{{ 'group.form.delete'|trans }}">delete</i></a>
+
+        {% if members | length > 1 %}
+            <table class="table">
+                <thead>
+                <tr>
+                    <th>{{ 'group.members.name'|trans }}</th>
+                    <th>{{ 'group.members.role' | trans }}</th>
+                    <th>{{ 'group.members.action'|trans }}</th>
+                </tr>
+                </thead>
+
+                <tbody>
+                {% for user in members %}
+                    {% if user != app.user %}
+                        <tr>
+                            <td>{{ user.username }}</td>
+                            <td>
+                                {% if user.getGroupRoleForUser(group) == 1 %}
+                                    {{ 'group.roles.readonly' | trans }}
+                                {% elseif user.getGroupRoleForUser(group) == 2 %}
+                                    {{ 'group.roles.write' | trans }}
+                                {% elseif user.getGroupRoleForUser(group) == 3 %}
+                                    {{ 'group.roles.manage_prez' | trans }}
+                                {% elseif user.getGroupRoleForUser(group) == 5 %}
+                                    {{ 'group.roles.manage_users' | trans }}
+                                {% elseif user.getGroupRoleForUser(group) == 10 %}
+                                    {{ 'group.roles.admin' | trans }}
+                                {% else %}
+                                    {{ 'group.roles.unknown' | trans }}
+                                {% endif %}
+                            </td>
+                            <td>
+                                <a href="{{ path('group-user-edit', { 'group': group.id, 'user': user.id }) }}" class="btn waves-effect"><i class="material-icons md-18" data-toggle="tooltip" data-placement="bottom" title="{{ 'group.members.edit'|trans }}">edit</i></a>
+                                <a class="btn waves-effect red" href="{{ path('group-user-exclude', { 'group': group.id, 'user': user.id }) }}"><i class="material-icons md-18" data-toggle="tooltip" data-placement="bottom" title="{{ 'group.members.exclude'|trans }}">clear</i></a>
+                            </td>
+                        </tr>
+                    {% endif %}
+                {% endfor %}
+                </tbody>
+            </table>
+        {% else %}
+                <p>{{ 'group.members.empty' | trans }}</p>
+        {% endif %}
+        <div class="pull-right">
+            {% if members.getNbPages > 1 %}
+                {{ pagerfanta(members, 'twitter_bootstrap3', {'proximity': 1}) }}
+            {% endif %}
+        </div>
+    </div>
+</div>
+{% endblock %}
diff --git a/src/Wallabag/GroupBundle/Resources/views/Manage/requests.html.twig b/src/Wallabag/GroupBundle/Resources/views/Manage/requests.html.twig
new file mode 100644
index 00000000..36d069ce
--- /dev/null
+++ b/src/Wallabag/GroupBundle/Resources/views/Manage/requests.html.twig
@@ -0,0 +1,31 @@
+{% extends "WallabagCoreBundle::layout.html.twig" %}
+
+{% block title %}{{ 'group.requests.list'|trans }}{% endblock %}
+
+{% block content %}
+<div class="row">
+    <div class="col s12">
+
+        <table class="table">
+            <thead>
+            <tr>
+                <th>{{ 'group.requests.username'|trans }}</th>
+                <th>{{ 'group.requests.action'|trans }}</th>
+            </tr>
+            </thead>
+
+            <tbody>
+            {% for request in requests %}
+                <tr>
+                    <td>{{ request.username }}</td>
+                    <td>
+                        <a class="btn waves-effect" href="{{ path('group-activate', {'group': group.id, 'user': request.id, 'accept': 1}) }}"><i class="material-icons">check</i></a>
+                        <a class="btn waves-effect red" href="{{ path('group-activate', {'group': group.id, 'user': request.id, 'accept': 2}) }}"><i class="material-icons">close</i></a>
+                    </td>
+                </tr>
+            {% endfor %}
+            </tbody>
+        </table>
+    </div>
+</div>
+{% endblock %}