From 3eec6b72e64cc920e11b0efade6ea74ed30eb0ab Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Sun, 31 Jan 2016 12:02:55 -0500
Subject: Move subtask form elements to subtask helper

---
 app/Template/subtask/create.php | 15 ++++-----------
 app/Template/subtask/edit.php   | 19 +++++--------------
 2 files changed, 9 insertions(+), 25 deletions(-)

(limited to 'app/Template/subtask')

diff --git a/app/Template/subtask/create.php b/app/Template/subtask/create.php
index 82e378f5..e237619e 100644
--- a/app/Template/subtask/create.php
+++ b/app/Template/subtask/create.php
@@ -5,22 +5,15 @@
 <form method="post" action="<?= $this->url->href('subtask', 'save', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?>" autocomplete="off">
 
     <?= $this->form->csrf() ?>
-
     <?= $this->form->hidden('task_id', $values) ?>
-
-    <?= $this->form->label(t('Title'), 'title') ?>
-    <?= $this->form->text('title', $values, $errors, array('required', 'autofocus', 'maxlength="255"')) ?><br/>
-
-    <?= $this->form->label(t('Assignee'), 'user_id') ?>
-    <?= $this->form->select('user_id', $users_list, $values, $errors) ?><br/>
-
-    <?= $this->form->label(t('Original estimate'), 'time_estimated') ?>
-    <?= $this->form->numeric('time_estimated', $values, $errors) ?> <?= t('hours') ?><br/>
+    <?= $this->subtask->selectTitle($values, $errors, array('autofocus')) ?>
+    <?= $this->subtask->selectAssignee($users_list, $values, $errors) ?>
+    <?= $this->subtask->selectTimeEstimated($values, $errors) ?>
 
     <?= $this->form->checkbox('another_subtask', t('Create another sub-task'), 1, isset($values['another_subtask']) && $values['another_subtask'] == 1) ?>
 
     <div class="form-actions">
-        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
+        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue">
         <?= t('or') ?>
         <?= $this->url->link(t('cancel'), 'task', 'show', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?>
     </div>
diff --git a/app/Template/subtask/edit.php b/app/Template/subtask/edit.php
index 2e583069..ca028108 100644
--- a/app/Template/subtask/edit.php
+++ b/app/Template/subtask/edit.php
@@ -5,24 +5,15 @@
 <form method="post" action="<?= $this->url->href('subtask', 'update', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'subtask_id' => $subtask['id'])) ?>" autocomplete="off">
 
     <?= $this->form->csrf() ?>
-
     <?= $this->form->hidden('id', $values) ?>
     <?= $this->form->hidden('task_id', $values) ?>
-
-    <?= $this->form->label(t('Title'), 'title') ?>
-    <?= $this->form->text('title', $values, $errors, array('required', 'autofocus', 'maxlength="255"')) ?><br/>
-
-    <?= $this->form->label(t('Assignee'), 'user_id') ?>
-    <?= $this->form->select('user_id', $users_list, $values, $errors) ?><br/>
-
-    <?= $this->form->label(t('Original estimate'), 'time_estimated') ?>
-    <?= $this->form->numeric('time_estimated', $values, $errors) ?> <?= t('hours') ?><br/>
-
-    <?= $this->form->label(t('Time spent'), 'time_spent') ?>
-    <?= $this->form->numeric('time_spent', $values, $errors) ?> <?= t('hours') ?><br/>
+    <?= $this->subtask->selectTitle($values, $errors, array('autofocus')) ?>
+    <?= $this->subtask->selectAssignee($users_list, $values, $errors) ?>
+    <?= $this->subtask->selectTimeEstimated($values, $errors) ?>
+    <?= $this->subtask->selectTimeSpent($values, $errors) ?>
 
     <div class="form-actions">
-        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
+        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue">
         <?= t('or') ?>
         <?= $this->url->link(t('cancel'), 'task', 'show', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?>
     </div>
-- 
cgit v1.2.3