Refresh the board only when it's necessary and add a link on the task title

assets/css/app.css

@@ -523,6 +523,10 @@ div.task a:hover {
     text-decoration: underline;
 }
 
+div.task-title a {
+    font-weight: normal;
+}
+
 #infos,
 div.task {
     position: relative;

assets/js/board.js

@@ -147,9 +147,10 @@
                 try {
                     var response = JSON.parse(this.responseText);
 
-                    if (response.result == true) {
+                    if (response.result == false) {
-
+                        window.alert('Unable to update the board');
-                        // TODO: don't refresh the whole page!
+                    }
+                    else if (response.refresh == true) {
                         window.location = "?controller=board&action=show&project_id=" + projectId;
                     }
                 }

controllers/base.php

@@ -14,6 +14,7 @@ abstract class Base
         $this->task = $registry->task;
         $this->user = $registry->user;
         $this->comment = $registry->comment;
+        $this->event = $registry->shared('event');
     }
 
     public function beforeAction($controller, $action)

controllers/board.php

@@ -276,7 +276,8 @@ class Board extends Base
         }
 
         $this->response->json(array(
-            'result' => $this->board->saveTasksPosition($this->request->getValues())
+            'result' => $this->board->saveTasksPosition($this->request->getValues()),
+            'refresh' => $this->event->getLastListenerExecuted() !== ''
         ));
     }
 }

core/event.php

@@ -28,6 +28,14 @@ class Event
      */
     private $listeners = array();
 
+    /**
+     * The last listener executed
+     *
+     * @access private
+     * @var string
+     */
+    private $lastListener = '';
+
     /**
      * The last triggered event
      *
@@ -74,11 +82,24 @@ class Event
 
         if (isset($this->listeners[$eventName])) {
             foreach ($this->listeners[$eventName] as $listener) {
-                $listener->execute($data); // TODO: keep an history of executed actions for unit test
+                if ($listener->execute($data)) {
+                    $this->lastListener = get_class($listener);
+                }
             }
         }
     }
 
+    /**
+     * Get the last listener executed
+     *
+     * @access public
+     * @return string Event name
+     */
+    public function getLastListenerExecuted()
+    {
+        return $this->lastListener;
+    }
+
     /**
      * Get the last fired event
      *

templates/board_index.php

@@ -71,7 +71,7 @@
                             <?php endif ?>
 
                             <div class="task-title">
-                                <?= Helper\escape($task['title']) ?>
+                                <a href="?controller=task&amp;action=show&amp;task_id=<?= $task['id'] ?>" title="<?= t('Show this task') ?>"><?= Helper\escape($task['title']) ?></a>
                             </div>
 
                             <div class="task-footer">