Fix wrong dropdown menu in group members list

2017-05-12 18:24:58 -04:00 · 2017-05-12 18:24:58 -04:00 · 740b0ac1a5
parent eab2d872a8
commit 740b0ac1a5
30 changed files with 53 additions and 3 deletions
--- a/1
+++ b/1
@ -12,6 +12,7 @@ Bug fixes:

 * Ensure project tags are removed when the project is removed
 * Avoid PHP notice when regenerating API token for a user
+* Fix wrong dropdown menu in group members list

 Version 1.0.43 (April 30, 2017)
 -------------------------------
--- a/app/Locale/bs_BA/translations.php
+++ b/app/Locale/bs_BA/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/ca_ES/translations.php
+++ b/app/Locale/ca_ES/translations.php
@ -1333,4 +1333,5 @@ return array(
    'Only moving task between those columns is permitted for tasks assigned to the current user' => 'Only moving task between those columns is permitted for tasks assigned to the current user',
    '[DUPLICATE]' => '[DUPLICATE]',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/cs_CZ/translations.php
+++ b/app/Locale/cs_CZ/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/da_DK/translations.php
+++ b/app/Locale/da_DK/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/de_DE/translations.php
+++ b/app/Locale/de_DE/translations.php
@ -1333,4 +1333,5 @@ return array(
    'Only moving task between those columns is permitted for tasks assigned to the current user' => 'Das Bewegen einer Aufgabe zwischen diesen Spalten ist nur für Aufgaben zulässig, die dem aktuellen Benutzer zugewiesen sind',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/el_GR/translations.php
+++ b/app/Locale/el_GR/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/es_ES/translations.php
+++ b/app/Locale/es_ES/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/fi_FI/translations.php
+++ b/app/Locale/fi_FI/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/fr_FR/translations.php
+++ b/app/Locale/fr_FR/translations.php
@ -1333,4 +1333,5 @@ return array(
    'Only moving task between those columns is permitted for tasks assigned to the current user' => 'Seulement le déplacement des tâches entre ces colonnes est autorisé pour l\'utilisateur courant',
    '[DUPLICATE]' => '[COPIE]',
    'DKK - Danish Krona' => 'DKK - Couronne danoise',
+    'Remove user from group' => 'Supprimer cet utilisateur du groupe',
 );
--- a/app/Locale/hr_HR/translations.php
+++ b/app/Locale/hr_HR/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/hu_HU/translations.php
+++ b/app/Locale/hu_HU/translations.php
@ -1333,4 +1333,5 @@ return array(
    'Only moving task between those columns is permitted for tasks assigned to the current user' => 'Az aktuális felhasználóhoz rendelt feladatok csak az engedélyezett oszlopok között mozgathatóak',
    '[DUPLICATE]' => '[Duplikált]',
    'DKK - Danish Krona' => 'Dán korona',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/id_ID/translations.php
+++ b/app/Locale/id_ID/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/it_IT/translations.php
+++ b/app/Locale/it_IT/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/ja_JP/translations.php
+++ b/app/Locale/ja_JP/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/ko_KR/translations.php
+++ b/app/Locale/ko_KR/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/my_MY/translations.php
+++ b/app/Locale/my_MY/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/nb_NO/translations.php
+++ b/app/Locale/nb_NO/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/nl_NL/translations.php
+++ b/app/Locale/nl_NL/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/pl_PL/translations.php
+++ b/app/Locale/pl_PL/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/pt_BR/translations.php
+++ b/app/Locale/pt_BR/translations.php
@ -1333,4 +1333,5 @@ return array(
    'Only moving task between those columns is permitted for tasks assigned to the current user' => 'Somente movimentação de tarefas entre aquelas colunas são permitidas para tarefas designadas para o usuário atual',
    '[DUPLICATE]' => '[DUPLICADO]',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/pt_PT/translations.php
+++ b/app/Locale/pt_PT/translations.php
@ -1333,4 +1333,5 @@ return array(
    'Only moving task between those columns is permitted for tasks assigned to the current user' => 'Apenas é permitido mover a tarefa entre essas colunas para tarefas atribuídas ao utilizador atual',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/ru_RU/translations.php
+++ b/app/Locale/ru_RU/translations.php
@ -1333,4 +1333,5 @@ return array(
    'Only moving task between those columns is permitted for tasks assigned to the current user' => 'Текущий пользователь может перемещать назначенные ему задачи только между этими колонками',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/sr_Latn_RS/translations.php
+++ b/app/Locale/sr_Latn_RS/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/sv_SE/translations.php
+++ b/app/Locale/sv_SE/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/th_TH/translations.php
+++ b/app/Locale/th_TH/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/tr_TR/translations.php
+++ b/app/Locale/tr_TR/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Locale/zh_CN/translations.php
+++ b/app/Locale/zh_CN/translations.php
@ -1333,4 +1333,5 @@ return array(
    // 'Only moving task between those columns is permitted for tasks assigned to the current user' => '',
    // '[DUPLICATE]' => '',
    // 'DKK - Danish Krona' => '',
+    // 'Remove user from group' => '',
 );
--- a/app/Template/group/user_dropdown.php
+++ b/app/Template/group/user_dropdown.php
@ -0,0 +1,11 @@
+<div class="dropdown">
+    <a href="#" class="dropdown-menu dropdown-menu-link-icon"><strong>#<?= $user['id'] ?> <i class="fa fa-caret-down"></i></strong></a>
+    <ul>
+        <li>
+            <?= $this->url->icon('user', t('View profile'), 'UserViewController', 'show', array('user_id' => $user['id'])) ?>
+        </li>
+        <li>
+            <?= $this->modal->medium('trash', t('Remove user from group'), 'GroupListController', 'dissociate', array('group_id' => $user['group_id'], 'user_id' => $user['id'])) ?>
+        </li>
+    </ul>
+</div>
--- a/app/Template/group/users.php
+++ b/app/Template/group/users.php
@ -13,9 +13,20 @@
            <?= $this->render('user_list/header', array('paginator' => $paginator)) ?>
            <?php foreach ($paginator->getCollection() as $user): ?>
                <div class="table-list-row table-border-left">
-                    <?= $this->render('user_list/user_title', array(
-                        'user' => $user,
-                    )) ?>
+                    <div>
+                        <?= $this->render('group/user_dropdown', array('user' => $user)) ?>
+                        <span class="table-list-title <?= $user['is_active'] == 0 ? 'status-closed' : '' ?>">
+                            <?= $this->avatar->small(
+                                $user['id'],
+                                $user['username'],
+                                $user['name'],
+                                $user['email'],
+                                $user['avatar_path'],
+                                'avatar-inline'
+                            ) ?>
+                            <?= $this->url->link($this->text->e($user['name'] ?: $user['username']), 'UserViewController', 'show', array('user_id' => $user['id'])) ?>
+                        </span>
+                    </div>

                    <?= $this->render('user_list/user_details', array(
                        'user' => $user,