From 32cd7c15085023b8526e8fba22d5fbad7d6b8679 Mon Sep 17 00:00:00 2001 From: ashbike Date: Mon, 16 Feb 2015 23:53:09 +0530 Subject: This adds a quick comment creation link in the task menu on board. --- app/Controller/Comment.php | 22 +++++++++++++++++++--- app/Template/board/task.php | 1 + 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 @@
  • a(t('Change assignee'), 'board', 'changeAssignee', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?>
  • a(t('Change category'), 'board', 'changeCategory', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?>
  • a(t('Change description'), 'task', 'description', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?>
  • +
  • a(t('Add a comment'), 'comment', 'create', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?>
  • a(t('Edit this task'), 'task', 'edit', array('task_id' => $task['id'], 'project_id' => $task['project_id']), false, 'task-board-popover') ?>
  • a(t('Close this task'), 'task', 'close', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'redirect' => 'board'), false, 'task-board-popover') ?>
  • 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 @@

    -
    + formCsrf() ?> formHidden('task_id', $values) ?> formHidden('user_id', $values) ?> @@ -30,7 +30,11 @@ - a(t('cancel'), 'task', 'show', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?> + + a(t('cancel'), 'board', 'show', array('project_id' => $task['project_id'])) ?> + + a(t('cancel'), 'task', 'show', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?> +
    -- cgit v1.2.3