diff options
author | Frederic Guillot <fred@kanboard.net> | 2015-02-16 22:04:41 -0500 |
---|---|---|
committer | Frederic Guillot <fred@kanboard.net> | 2015-02-16 22:04:41 -0500 |
commit | 697e661ef5bc17c0639a2bfed08307f8e113816d (patch) | |
tree | f6a01556cbd147922c7ea6f1e3d8b02898b369a0 | |
parent | 14ba39be8d78761fb322e0e0860c6344eebc2153 (diff) | |
parent | 32cd7c15085023b8526e8fba22d5fbad7d6b8679 (diff) |
Merge pull-request #629
-rw-r--r-- | app/Controller/Comment.php | 22 | ||||
-rw-r--r-- | app/Template/board/task.php | 1 | ||||
-rw-r--r-- | app/Template/comment/create.php | 8 |
3 files changed, 26 insertions, 5 deletions
diff --git a/app/Controller/Comment.php b/app/Controller/Comment.php index 9796ea38..68dc44a7 100644 --- a/app/Controller/Comment.php +++ b/app/Controller/Comment.php @@ -41,6 +41,7 @@ class Comment extends Base public function create(array $values = array(), array $errors = array()) { $task = $this->getTask(); + $ajax = $this->request->isAjax() || $this->request->getIntegerParam('ajax'); if (empty($values)) { $values = array( @@ -49,14 +50,23 @@ class Comment extends Base ); } + if ($ajax) { + $this->response->html($this->template->render('comment/create', array( + 'values' => $values, + 'errors' => $errors, + 'task' => $task, + 'ajax' => $ajax + ))); + } else { $this->response->html($this->taskLayout('comment/create', array( 'values' => $values, 'errors' => $errors, 'task' => $task, 'title' => t('Add a comment') ))); + } } - + /** * Add a comment * @@ -66,7 +76,8 @@ class Comment extends Base { $task = $this->getTask(); $values = $this->request->getValues(); - + $ajax = $this->request->isAjax() || $this->request->getIntegerParam('ajax'); + list($valid, $errors) = $this->comment->validateCreation($values); if ($valid) { @@ -78,7 +89,12 @@ class Comment extends Base $this->session->flashError(t('Unable to create your comment.')); } - $this->response->redirect('?controller=task&action=show&task_id='.$task['id'].'&project_id='.$task['project_id'].'#comments'); + if ($ajax) { + $this->response->redirect('?controller=board&action=show&project_id='.$task['project_id']); + } + else { + $this->response->redirect('?controller=task&action=show&task_id='.$task['id'].'&project_id='.$task['project_id'].'#comments'); + } } $this->create($values, $errors); diff --git a/app/Template/board/task.php b/app/Template/board/task.php index a2a617a6..f3014dfa 100644 --- a/app/Template/board/task.php +++ b/app/Template/board/task.php @@ -20,6 +20,7 @@ <li><i class="fa fa-user"></i> <?= $this->a(t('Change assignee'), 'board', 'changeAssignee', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?></li> <li><i class="fa fa-tag"></i> <?= $this->a(t('Change category'), 'board', 'changeCategory', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?></li> <li><i class="fa fa-align-left"></i> <?= $this->a(t('Change description'), 'task', 'description', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?></li> + <li><i class="fa fa-comment-o"></i> <?= $this->a(t('Add a comment'), 'comment', 'create', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?></li> <li><i class="fa fa-pencil-square-o"></i> <?= $this->a(t('Edit this task'), 'task', 'edit', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?></li> <li><i class="fa fa-close"></i> <?= $this->a(t('Close this task'), 'task', 'close', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'redirect' => 'board'), false, 'task-board-popover') ?></li> </ul> diff --git a/app/Template/comment/create.php b/app/Template/comment/create.php index 1bdbac49..2ef3ee59 100644 --- a/app/Template/comment/create.php +++ b/app/Template/comment/create.php @@ -2,7 +2,7 @@ <h2><?= t('Add a comment') ?></h2> </div> -<form method="post" action="<?= $this->u('comment', 'save', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?>" autocomplete="off"> +<form method="post" action="<?= $this->u('comment', 'save', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'ajax' => $ajax)) ?>" autocomplete="off"> <?= $this->formCsrf() ?> <?= $this->formHidden('task_id', $values) ?> <?= $this->formHidden('user_id', $values) ?> @@ -30,7 +30,11 @@ <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/> <?php if (! isset($skip_cancel)): ?> <?= t('or') ?> - <?= $this->a(t('cancel'), 'task', 'show', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?> + <?php if ($ajax): ?> + <?= $this->a(t('cancel'), 'board', 'show', array('project_id' => $task['project_id'])) ?> + <?php else: ?> + <?= $this->a(t('cancel'), 'task', 'show', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?> + <?php endif ?> <?php endif ?> </div> </form> |