From 22c5e32def94560881ad9ec032158cd570be44f4 Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Sat, 5 Mar 2016 17:40:49 -0500
Subject: Improve automatic action creation

---
 app/Template/action/event.php  |  25 --------
 app/Template/action/index.php  | 130 +++++++++++++++++++----------------------
 app/Template/action/params.php |  49 ----------------
 3 files changed, 60 insertions(+), 144 deletions(-)
 delete mode 100644 app/Template/action/event.php
 delete mode 100644 app/Template/action/params.php

(limited to 'app/Template/action')

diff --git a/app/Template/action/event.php b/app/Template/action/event.php
deleted file mode 100644
index f4f12db3..00000000
--- a/app/Template/action/event.php
+++ /dev/null
@@ -1,25 +0,0 @@
-<div class="page-header">
-    <h2><?= t('Automatic actions for the project "%s"', $project['name']) ?></h2>
-</div>
-
-<h3><?= t('Choose an event') ?></h3>
-<form method="post" action="<?= $this->url->href('action', 'params', array('project_id' => $project['id'])) ?>">
-
-    <?= $this->form->csrf() ?>
-
-    <?= $this->form->hidden('project_id', $values) ?>
-    <?= $this->form->hidden('action_name', $values) ?>
-
-    <?= $this->form->label(t('Event'), 'event_name') ?>
-    <?= $this->form->select('event_name', $events, $values) ?>
-
-    <div class="form-help">
-        <?= t('When the selected event occurs execute the corresponding action.') ?>
-    </div>
-
-    <div class="form-actions">
-        <button type="submit" class="btn btn-blue"><?= t('Next step') ?></button>
-        <?= t('or') ?>
-        <?= $this->url->link(t('cancel'), 'action', 'index', array('project_id' => $project['id'])) ?>
-    </div>
-</form>
\ No newline at end of file
diff --git a/app/Template/action/index.php b/app/Template/action/index.php
index 1cc14782..63d63887 100644
--- a/app/Template/action/index.php
+++ b/app/Template/action/index.php
@@ -3,79 +3,69 @@
     <ul>
         <li>
             <i class="fa fa-plus fa-fw"></i>
+            <?= $this->url->link(t('Add a new action'), 'ActionCreation', 'create', array('project_id' => $project['id']), false, 'popover') ?>
+        </li>
+        <li>
+            <i class="fa fa-copy fa-fw"></i>
             <?= $this->url->link(t('Import from another project'), 'ActionProject', 'project', array('project_id' => $project['id']), false, 'popover') ?>
         </li>
     </ul>
 </div>
 
-<?php if (! empty($actions)): ?>
-
-<h3><?= t('Defined actions') ?></h3>
-<table>
-    <tr>
-        <th><?= t('Automatic actions') ?></th>
-        <th><?= t('Action parameters') ?></th>
-        <th><?= t('Action') ?></th>
-    </tr>
-
-    <?php foreach ($actions as $action): ?>
-    <tr>
-        <td>
-            <ul>
-                <li>
-                    <?= t('Event name') ?> =
-                    <strong><?= $this->text->in($action['event_name'], $available_events) ?></strong>
-                </li>
-                <li>
-                    <?= t('Action name') ?> =
-                    <strong><?= $this->text->in($action['action_name'], $available_actions) ?></strong>
-                </li>
-            <ul>
-        </td>
-        <td>
-            <ul>
-            <?php foreach ($action['params'] as $param_name => $param_value): ?>
-                <li>
-                    <?= $this->text->in($param_name, $available_params[$action['action_name']]) ?> =
-                    <strong>
-                    <?php if ($this->text->contains($param_name, 'column_id')): ?>
-                        <?= $this->text->in($param_value, $columns_list) ?>
-                    <?php elseif ($this->text->contains($param_name, 'user_id')): ?>
-                        <?= $this->text->in($param_value, $users_list) ?>
-                    <?php elseif ($this->text->contains($param_name, 'project_id')): ?>
-                        <?= $this->text->in($param_value, $projects_list) ?>
-                    <?php elseif ($this->text->contains($param_name, 'color_id')): ?>
-                        <?= $this->text->in($param_value, $colors_list) ?>
-                    <?php elseif ($this->text->contains($param_name, 'category_id')): ?>
-                        <?= $this->text->in($param_value, $categories_list) ?>
-                    <?php elseif ($this->text->contains($param_name, 'link_id')): ?>
-                        <?= $this->text->in($param_value, $links_list) ?>
-                    <?php else: ?>
-                        <?= $this->text->e($param_value) ?>
-                    <?php endif ?>
-                    </strong>
-                </li>
-            <?php endforeach ?>
-            </ul>
-        </td>
-        <td>
-            <?= $this->url->link(t('Remove'), 'action', 'confirm', array('project_id' => $project['id'], 'action_id' => $action['id']), false, 'popover') ?>
-        </td>
-    </tr>
-    <?php endforeach ?>
-</table>
-
-<?php endif ?>
-
-<h3><?= t('Add an action') ?></h3>
-<form method="post" action="<?= $this->url->href('action', 'event', array('project_id' => $project['id'])) ?>" class="listing">
-    <?= $this->form->csrf() ?>
-    <?= $this->form->hidden('project_id', $values) ?>
-
-    <?= $this->form->label(t('Action'), 'action_name') ?>
-    <?= $this->form->select('action_name', $available_actions, $values) ?>
+<?php if (empty($actions)): ?>
+    <p class="alert"><?= t('There is no action at the moment.') ?></p>
+<?php else: ?>
+    <table>
+        <tr>
+            <th><?= t('Automatic actions') ?></th>
+            <th><?= t('Action parameters') ?></th>
+            <th><?= t('Action') ?></th>
+        </tr>
 
-    <div class="form-actions">
-        <button type="submit" class="btn btn-blue"><?= t('Next step') ?></button>
-    </div>
-</form>
\ No newline at end of file
+        <?php foreach ($actions as $action): ?>
+        <tr>
+            <td>
+                <ul>
+                    <li>
+                        <?= t('Event name') ?> =
+                        <strong><?= $this->text->in($action['event_name'], $available_events) ?></strong>
+                    </li>
+                    <li>
+                        <?= t('Action name') ?> =
+                        <strong><?= $this->text->in($action['action_name'], $available_actions) ?></strong>
+                    </li>
+                <ul>
+            </td>
+            <td>
+                <ul>
+                <?php foreach ($action['params'] as $param_name => $param_value): ?>
+                    <li>
+                        <?= $this->text->in($param_name, $available_params[$action['action_name']]) ?> =
+                        <strong>
+                        <?php if ($this->text->contains($param_name, 'column_id')): ?>
+                            <?= $this->text->in($param_value, $columns_list) ?>
+                        <?php elseif ($this->text->contains($param_name, 'user_id')): ?>
+                            <?= $this->text->in($param_value, $users_list) ?>
+                        <?php elseif ($this->text->contains($param_name, 'project_id')): ?>
+                            <?= $this->text->in($param_value, $projects_list) ?>
+                        <?php elseif ($this->text->contains($param_name, 'color_id')): ?>
+                            <?= $this->text->in($param_value, $colors_list) ?>
+                        <?php elseif ($this->text->contains($param_name, 'category_id')): ?>
+                            <?= $this->text->in($param_value, $categories_list) ?>
+                        <?php elseif ($this->text->contains($param_name, 'link_id')): ?>
+                            <?= $this->text->in($param_value, $links_list) ?>
+                        <?php else: ?>
+                            <?= $this->text->e($param_value) ?>
+                        <?php endif ?>
+                        </strong>
+                    </li>
+                <?php endforeach ?>
+                </ul>
+            </td>
+            <td>
+                <?= $this->url->link(t('Remove'), 'action', 'confirm', array('project_id' => $project['id'], 'action_id' => $action['id']), false, 'popover') ?>
+            </td>
+        </tr>
+        <?php endforeach ?>
+    </table>
+<?php endif ?>
\ No newline at end of file
diff --git a/app/Template/action/params.php b/app/Template/action/params.php
deleted file mode 100644
index 99e9206f..00000000
--- a/app/Template/action/params.php
+++ /dev/null
@@ -1,49 +0,0 @@
-<div class="page-header">
-    <h2><?= t('Automatic actions for the project "%s"', $project['name']) ?></h2>
-</div>
-
-<h3><?= t('Define action parameters') ?></h3>
-<form method="post" action="<?= $this->url->href('action', 'create', array('project_id' => $project['id'])) ?>" autocomplete="off">
-
-    <?= $this->form->csrf() ?>
-
-    <?= $this->form->hidden('project_id', $values) ?>
-    <?= $this->form->hidden('event_name', $values) ?>
-    <?= $this->form->hidden('action_name', $values) ?>
-
-    <?php foreach ($action_params as $param_name => $param_desc): ?>
-
-        <?php if ($this->text->contains($param_name, 'column_id')): ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->select('params['.$param_name.']', $columns_list, $values) ?>
-        <?php elseif ($this->text->contains($param_name, 'user_id')): ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->select('params['.$param_name.']', $users_list, $values) ?>
-        <?php elseif ($this->text->contains($param_name, 'project_id')): ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->select('params['.$param_name.']', $projects_list, $values) ?>
-        <?php elseif ($this->text->contains($param_name, 'color_id')): ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->select('params['.$param_name.']', $colors_list, $values) ?>
-        <?php elseif ($this->text->contains($param_name, 'category_id')): ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->select('params['.$param_name.']', $categories_list, $values) ?>
-        <?php elseif ($this->text->contains($param_name, 'link_id')): ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->select('params['.$param_name.']', $links_list, $values) ?>
-        <?php elseif ($this->text->contains($param_name, 'duration')): ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->number('params['.$param_name.']', $values) ?>
-        <?php else: ?>
-            <?= $this->form->label($param_desc, $param_name) ?>
-            <?= $this->form->text('params['.$param_name.']', $values) ?>
-        <?php endif ?>
-
-    <?php endforeach ?>
-
-    <div class="form-actions">
-        <button type="submit" class="btn btn-blue"><?= t('Save') ?></button>
-        <?= t('or') ?>
-        <?= $this->url->link(t('cancel'), 'action', 'index', array('project_id' => $project['id'])) ?>
-    </div>
-</form>
\ No newline at end of file
-- 
cgit v1.2.3