Render a dropdown menu for external task providers

2017-10-12 14:01:39 -07:00 · 2017-10-12 14:01:39 -07:00 · 4bb422b41f
parent ad6f898134
commit 4bb422b41f
32 changed files with 109 additions and 3 deletions
--- a/app/Core/ExternalTask/ExternalTaskManager.php
+++ b/app/Core/ExternalTask/ExternalTaskManager.php
@ -55,4 +55,14 @@ class ExternalTaskManager

        return array();
    }
+
+    /**
+     * Get all providers
+     *
+     * @return ExternalTaskProviderInterface[]
+     */
+    public function getProviders()
+    {
+        return $this->providers;
+    }
 }
--- a/app/Core/ExternalTask/ExternalTaskProviderInterface.php
+++ b/app/Core/ExternalTask/ExternalTaskProviderInterface.php
@ -18,6 +18,22 @@ interface ExternalTaskProviderInterface
     */
    public function getName();

+    /**
+     * Get provider icon
+     *
+     * @access public
+     * @return string
+     */
+    public function getIcon();
+
+    /**
+     * Get label for adding a new task
+     *
+     * @access public
+     * @return string
+     */
+    public function getMenuAddLabel();
+
    /**
     * Retrieve task from external system or cache
     *
--- a/app/Helper/TaskHelper.php
+++ b/app/Helper/TaskHelper.php
@ -256,4 +256,58 @@ class TaskHelper extends Base
        $html .= '</ul></div></small>';
        return $html;
    }
+
+    public function getNewBoardTaskButton(array $swimlane, array $column)
+    {
+        $html = '<div class="board-add-icon">';
+        $providers = $this->externalTaskManager->getProviders();
+
+        if (empty($providers)) {
+            $html .= $this->helper->modal->largeIcon(
+                'plus',
+                t('Add a new task'),
+                'TaskCreationController',
+                'show', array(
+                    'project_id'  => $column['project_id'],
+                    'column_id'   => $column['id'],
+                    'swimlane_id' => $swimlane['id'],
+                )
+            );
+        } else {
+            $html .= '<div class="dropdown">';
+            $html .= '<a href="#" class="dropdown-menu"><i class="fa fa-plus" aria-hidden="true"></i></a><ul>';
+
+            $link = $this->helper->modal->large(
+                'plus',
+                t('Add a new Kanboard task'),
+                'TaskCreationController',
+                'show', array(
+                    'project_id'  => $column['project_id'],
+                    'column_id'   => $column['id'],
+                    'swimlane_id' => $swimlane['id'],
+                )
+            );
+
+            $html .= '<li>'.$link.'</li>';
+
+            foreach ($providers as $provider) {
+                $link = $this->helper->url->link(
+                    $provider->getMenuAddLabel(),
+                    'ExternalTaskCreationController',
+                    'step1',
+                    array('project_id' => $column['project_id'], 'swimlane_id' => $swimlane['id'], 'column_id' => $column['id'], 'provider_name' => $provider->getName()),
+                    false,
+                    'js-modal-large'
+                );
+
+                $html .= '<li>'.$provider->getIcon().' '.$link.'</li>';
+            }
+
+            $html .= '</ul></div>';
+        }
+
+        $html .= '</div>';
+
+        return $html;
+    }
 }
--- a/app/Locale/bs_BA/translations.php
+++ b/app/Locale/bs_BA/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/ca_ES/translations.php
+++ b/app/Locale/ca_ES/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/cs_CZ/translations.php
+++ b/app/Locale/cs_CZ/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/da_DK/translations.php
+++ b/app/Locale/da_DK/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/de_DE/translations.php
+++ b/app/Locale/de_DE/translations.php
@ -1340,4 +1340,5 @@ return array(
    'Write one subject by line.' => 'Schreibe ein Betreff pro Zeile.',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/el_GR/translations.php
+++ b/app/Locale/el_GR/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/es_ES/translations.php
+++ b/app/Locale/es_ES/translations.php
@ -1340,4 +1340,5 @@ return array(
    'Write one subject by line.' => 'Escriba un asunto por línea.',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/fi_FI/translations.php
+++ b/app/Locale/fi_FI/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/fr_FR/translations.php
+++ b/app/Locale/fr_FR/translations.php
@ -1340,4 +1340,5 @@ return array(
    'Write one subject by line.' => 'Écrivez un sujet par ligne.',
    'Create another link' => 'Créer un autre lien',
    'BRL - Brazilian Real' => 'BRL - Real brésilien',
+    'Add a new Kanboard task' => 'Ajouter une nouvelle tâche Kanboard',
 );
--- a/app/Locale/hr_HR/translations.php
+++ b/app/Locale/hr_HR/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/hu_HU/translations.php
+++ b/app/Locale/hu_HU/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/id_ID/translations.php
+++ b/app/Locale/id_ID/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/it_IT/translations.php
+++ b/app/Locale/it_IT/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/ja_JP/translations.php
+++ b/app/Locale/ja_JP/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/ko_KR/translations.php
+++ b/app/Locale/ko_KR/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/my_MY/translations.php
+++ b/app/Locale/my_MY/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/nb_NO/translations.php
+++ b/app/Locale/nb_NO/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/nl_NL/translations.php
+++ b/app/Locale/nl_NL/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/pl_PL/translations.php
+++ b/app/Locale/pl_PL/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/pt_BR/translations.php
+++ b/app/Locale/pt_BR/translations.php
@ -1340,4 +1340,5 @@ return array(
    'Write one subject by line.' => 'Escreva um assunto por linha.',
    'Create another link' => 'Criar outro link',
    'BRL - Brazilian Real' => 'BRL - Real Brasileiro',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/pt_PT/translations.php
+++ b/app/Locale/pt_PT/translations.php
@ -1340,4 +1340,5 @@ return array(
    'Write one subject by line.' => 'Escreva um assunto por linha.',
    'Create another link' => 'Criar outro link',
    'BRL - Brazilian Real' => 'BRL - Real Brasileiro',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/ru_RU/translations.php
+++ b/app/Locale/ru_RU/translations.php
@ -1340,4 +1340,5 @@ return array(
    'Write one subject by line.' => 'Записываются по одной теме на строку.',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/sr_Latn_RS/translations.php
+++ b/app/Locale/sr_Latn_RS/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/sv_SE/translations.php
+++ b/app/Locale/sv_SE/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/th_TH/translations.php
+++ b/app/Locale/th_TH/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/tr_TR/translations.php
+++ b/app/Locale/tr_TR/translations.php
@ -1340,4 +1340,5 @@ return array(
    'Write one subject by line.' => 'Her satıra bir konu yazınız',
    'Create another link' => 'Başka bağlantı oluştur',
    'BRL - Brazilian Real' => 'BRL - Brezilya Reali',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/vi_VN/translations.php
+++ b/app/Locale/vi_VN/translations.php
@ -1346,4 +1346,5 @@ return array(
    'Write one subject by line.' => 'Viết một chủ đề theo dòng.',
    'Create another link' => 'Tạo liên kết khác',
    'BRL - Brazilian Real' => 'BRL - Brazilian Real',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Locale/zh_CN/translations.php
+++ b/app/Locale/zh_CN/translations.php
@ -1340,4 +1340,5 @@ return array(
    // 'Write one subject by line.' => '',
    // 'Create another link' => '',
    // 'BRL - Brazilian Real' => '',
+    // 'Add a new Kanboard task' => '',
 );
--- a/app/Template/board/table_column.php
+++ b/app/Template/board/table_column.php
@ -16,9 +16,7 @@
        <!-- column in expanded mode -->
        <div class="board-column-expanded">
            <?php if (! $not_editable && $this->projectRole->canCreateTaskInColumn($column['project_id'], $column['id'])): ?>
-                <div class="board-add-icon">
-                    <?= $this->modal->largeIcon('plus', t('Add a new task'), 'TaskCreationController', 'show', array('project_id' => $column['project_id'], 'column_id' => $column['id'], 'swimlane_id' => $swimlane['id'])) ?>
-                </div>
+                <?= $this->task->getNewBoardTaskButton($swimlane, $column) ?>
            <?php endif ?>

            <?php if ($swimlane['nb_swimlanes'] > 1 && ! empty($column['column_nb_tasks'])): ?>