From 11b6381cc0f0583a11287202463d20a0ac1ebf00 Mon Sep 17 00:00:00 2001
From: Frédéric Guillot <fred@kanboard.net>
Date: Thu, 20 Nov 2014 20:26:21 -0500
Subject: Move and clean some templates to a subfolder

---
 app/Controller/Config.php           | 12 +++++------
 app/Controller/File.php             |  6 +++---
 app/Template/config/about.php       | 41 +++++++++++++++++++++++++++++++++++++
 app/Template/config/api.php         | 18 ++++++++++++++++
 app/Template/config/application.php | 27 ++++++++++++++++++++++++
 app/Template/config/board.php       | 29 ++++++++++++++++++++++++++
 app/Template/config/layout.php      | 10 +++++++++
 app/Template/config/sidebar.php     | 20 ++++++++++++++++++
 app/Template/config/webhook.php     | 38 ++++++++++++++++++++++++++++++++++
 app/Template/config_about.php       | 41 -------------------------------------
 app/Template/config_api.php         | 18 ----------------
 app/Template/config_application.php | 27 ------------------------
 app/Template/config_board.php       | 29 --------------------------
 app/Template/config_layout.php      | 10 ---------
 app/Template/config_sidebar.php     | 20 ------------------
 app/Template/config_webhook.php     | 38 ----------------------------------
 app/Template/file/new.php           | 14 +++++++++++++
 app/Template/file/open.php          |  6 ++++++
 app/Template/file/remove.php        | 15 ++++++++++++++
 app/Template/file/show.php          | 23 +++++++++++++++++++++
 app/Template/file_new.php           | 14 -------------
 app/Template/file_open.php          |  6 ------
 app/Template/file_remove.php        | 14 -------------
 app/Template/file_show.php          | 23 ---------------------
 app/Template/task_show.php          |  2 +-
 assets/css/app.css                  |  2 +-
 assets/css/header.css               |  2 +-
 27 files changed, 253 insertions(+), 252 deletions(-)
 create mode 100644 app/Template/config/about.php
 create mode 100644 app/Template/config/api.php
 create mode 100644 app/Template/config/application.php
 create mode 100644 app/Template/config/board.php
 create mode 100644 app/Template/config/layout.php
 create mode 100644 app/Template/config/sidebar.php
 create mode 100644 app/Template/config/webhook.php
 delete mode 100644 app/Template/config_about.php
 delete mode 100644 app/Template/config_api.php
 delete mode 100644 app/Template/config_application.php
 delete mode 100644 app/Template/config_board.php
 delete mode 100644 app/Template/config_layout.php
 delete mode 100644 app/Template/config_sidebar.php
 delete mode 100644 app/Template/config_webhook.php
 create mode 100644 app/Template/file/new.php
 create mode 100644 app/Template/file/open.php
 create mode 100644 app/Template/file/remove.php
 create mode 100644 app/Template/file/show.php
 delete mode 100644 app/Template/file_new.php
 delete mode 100644 app/Template/file_open.php
 delete mode 100644 app/Template/file_remove.php
 delete mode 100644 app/Template/file_show.php

diff --git a/app/Controller/Config.php b/app/Controller/Config.php
index 3e0b290b..199259d7 100644
--- a/app/Controller/Config.php
+++ b/app/Controller/Config.php
@@ -25,7 +25,7 @@ class Config extends Base
         $params['errors'] = array();
         $params['config_content_for_layout'] = $this->template->load($template, $params);
 
-        return $this->template->layout('config_layout', $params);
+        return $this->template->layout('config/layout', $params);
     }
 
     /**
@@ -59,7 +59,7 @@ class Config extends Base
      */
     public function index()
     {
-        $this->response->html($this->layout('config_about', array(
+        $this->response->html($this->layout('config/about', array(
             'db_size' => $this->config->getDatabaseSize(),
             'title' => t('Settings').' &gt; '.t('About'),
         )));
@@ -74,7 +74,7 @@ class Config extends Base
     {
         $this->common('application');
 
-        $this->response->html($this->layout('config_application', array(
+        $this->response->html($this->layout('config/application', array(
             'languages' => $this->config->getLanguages(),
             'timezones' => $this->config->getTimezones(),
             'date_formats' => $this->dateParser->getAvailableFormats(),
@@ -91,7 +91,7 @@ class Config extends Base
     {
         $this->common('board');
 
-        $this->response->html($this->layout('config_board', array(
+        $this->response->html($this->layout('config/board', array(
             'default_columns' => implode(', ', $this->board->getDefaultColumns()),
             'title' => t('Settings').' &gt; '.t('Board settings'),
         )));
@@ -106,7 +106,7 @@ class Config extends Base
     {
         $this->common('webhook');
 
-        $this->response->html($this->layout('config_webhook', array(
+        $this->response->html($this->layout('config/webhook', array(
             'title' => t('Settings').' &gt; '.t('Webhook settings'),
         )));
     }
@@ -118,7 +118,7 @@ class Config extends Base
      */
     public function api()
     {
-        $this->response->html($this->layout('config_api', array(
+        $this->response->html($this->layout('config/api', array(
             'title' => t('Settings').' &gt; '.t('API'),
         )));
     }
diff --git a/app/Controller/File.php b/app/Controller/File.php
index 24db7565..ae44cac0 100644
--- a/app/Controller/File.php
+++ b/app/Controller/File.php
@@ -21,7 +21,7 @@ class File extends Base
     {
         $task = $this->getTask();
 
-        $this->response->html($this->taskLayout('file_new', array(
+        $this->response->html($this->taskLayout('file/new', array(
             'task' => $task,
             'max_size' => ini_get('upload_max_filesize'),
         )));
@@ -75,7 +75,7 @@ class File extends Base
         $file = $this->file->getById($this->request->getIntegerParam('file_id'));
 
         if ($file['task_id'] == $task['id']) {
-            $this->response->html($this->template->load('file_open', array(
+            $this->response->html($this->template->load('file/open', array(
                 'file' => $file
             )));
         }
@@ -132,7 +132,7 @@ class File extends Base
         $task = $this->getTask();
         $file = $this->file->getById($this->request->getIntegerParam('file_id'));
 
-        $this->response->html($this->taskLayout('file_remove', array(
+        $this->response->html($this->taskLayout('file/remove', array(
             'task' => $task,
             'file' => $file,
         )));
diff --git a/app/Template/config/about.php b/app/Template/config/about.php
new file mode 100644
index 00000000..3f34f802
--- /dev/null
+++ b/app/Template/config/about.php
@@ -0,0 +1,41 @@
+<div class="page-header">
+    <h2><?= t('About') ?></h2>
+</div>
+<section class="listing">
+    <ul>
+        <li>
+            <?= t('Official website:') ?>
+            <a href="http://kanboard.net/" target="_blank" rel="noreferer">http://kanboard.net/</a>
+        </li>
+        <li>
+            <?= t('Application version:') ?>
+            <strong><?= APP_VERSION ?></strong>
+        </li>
+    </ul>
+</section>
+
+<div class="page-header">
+    <h2><?= t('Database') ?></h2>
+</div>
+<section class="listing">
+    <ul>
+        <li>
+            <?= t('Database driver:') ?>
+            <strong><?= Helper\escape(DB_DRIVER) ?></strong>
+        </li>
+        <?php if (DB_DRIVER === 'sqlite'): ?>
+            <li>
+                <?= t('Database size:') ?>
+                <strong><?= Helper\format_bytes($db_size) ?></strong>
+            </li>
+            <li>
+                <?= Helper\a(t('Download the database'), 'config', 'downloadDb', array(), true) ?>&nbsp;
+                <?= t('(Gzip compressed Sqlite file)') ?>
+            </li>
+            <li>
+                <?= Helper\a(t('Optimize the database'), 'config', 'optimizeDb', array(), true) ?>&nbsp;
+                <?= t('(VACUUM command)') ?>
+            </li>
+        <?php endif ?>
+    </ul>
+</section>
\ No newline at end of file
diff --git a/app/Template/config/api.php b/app/Template/config/api.php
new file mode 100644
index 00000000..037ea08d
--- /dev/null
+++ b/app/Template/config/api.php
@@ -0,0 +1,18 @@
+<div class="page-header">
+    <h2><?= t('API') ?></h2>
+</div>
+<section class="listing">
+    <ul>
+        <li>
+            <?= t('API token:') ?>
+            <strong><?= Helper\escape($values['api_token']) ?></strong>
+        </li>
+        <li>
+            <?= t('API endpoint:') ?>
+            <input type="text" readonly="readonly" value="<?= Helper\get_current_base_url().'jsonrpc.php' ?>">
+        </li>
+        <li>
+            <?= Helper\a(t('Reset token'), 'config', 'token', array('type' => 'api'), true) ?>
+        </li>
+    </ul>
+</section>
\ No newline at end of file
diff --git a/app/Template/config/application.php b/app/Template/config/application.php
new file mode 100644
index 00000000..c92975bc
--- /dev/null
+++ b/app/Template/config/application.php
@@ -0,0 +1,27 @@
+<div class="page-header">
+    <h2><?= t('Application settings') ?></h2>
+</div>
+<section>
+<form method="post" action="<?= Helper\u('config', 'application') ?>" autocomplete="off">
+
+    <?= Helper\form_csrf() ?>
+
+    <?= Helper\form_label(t('Application URL'), 'application_url') ?>
+    <?= Helper\form_text('application_url', $values, $errors, array('placeholder="http://example.kanboard.net/"')) ?><br/>
+    <p class="form-help"><?= t('Example: http://example.kanboard.net/ (used by email notifications)') ?></p>
+
+    <?= Helper\form_label(t('Language'), 'application_language') ?>
+    <?= Helper\form_select('application_language', $languages, $values, $errors) ?><br/>
+
+    <?= Helper\form_label(t('Timezone'), 'application_timezone') ?>
+    <?= Helper\form_select('application_timezone', $timezones, $values, $errors) ?><br/>
+
+    <?= Helper\form_label(t('Date format'), 'application_date_format') ?>
+    <?= Helper\form_select('application_date_format', $date_formats, $values, $errors) ?><br/>
+    <p class="form-help"><?= t('ISO format is always accepted, example: "%s" and "%s"', date('Y-m-d'), date('Y_m_d')) ?></p>
+
+    <div class="form-actions">
+        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
+    </div>
+</form>
+</section>
\ No newline at end of file
diff --git a/app/Template/config/board.php b/app/Template/config/board.php
new file mode 100644
index 00000000..f260d084
--- /dev/null
+++ b/app/Template/config/board.php
@@ -0,0 +1,29 @@
+<div class="page-header">
+    <h2><?= t('Board settings') ?></h2>
+</div>
+<section>
+<form method="post" action="<?= Helper\u('config', 'board') ?>" autocomplete="off">
+
+    <?= Helper\form_csrf() ?>
+
+    <?= Helper\form_label(t('Task highlight period'), 'board_highlight_period') ?>
+    <?= Helper\form_number('board_highlight_period', $values, $errors) ?><br/>
+    <p class="form-help"><?= t('Period (in second) to consider a task was modified recently (0 to disable, 2 days by default)') ?></p>
+
+    <?= Helper\form_label(t('Refresh interval for public board'), 'board_public_refresh_interval') ?>
+    <?= Helper\form_number('board_public_refresh_interval', $values, $errors) ?><br/>
+    <p class="form-help"><?= t('Frequency in second (60 seconds by default)') ?></p>
+
+    <?= Helper\form_label(t('Refresh interval for private board'), 'board_private_refresh_interval') ?>
+    <?= Helper\form_number('board_private_refresh_interval', $values, $errors) ?><br/>
+    <p class="form-help"><?= t('Frequency in second (0 to disable this feature, 10 seconds by default)') ?></p>
+
+    <?= Helper\form_label(t('Default columns for new projects (Comma-separated)'), 'board_columns') ?>
+    <?= Helper\form_text('board_columns', $values, $errors) ?><br/>
+    <p class="form-help"><?= t('Default values are "%s"', $default_columns) ?></p>
+
+    <div class="form-actions">
+        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
+    </div>
+</form>
+</section>
\ No newline at end of file
diff --git a/app/Template/config/layout.php b/app/Template/config/layout.php
new file mode 100644
index 00000000..e0ac2fa8
--- /dev/null
+++ b/app/Template/config/layout.php
@@ -0,0 +1,10 @@
+<section id="main">
+    <section class="sidebar-container" id="config-section">
+
+        <?= Helper\template('config/sidebar') ?>
+
+        <div class="sidebar-content">
+            <?= $config_content_for_layout ?>
+        </div>
+    </section>
+</section>
\ No newline at end of file
diff --git a/app/Template/config/sidebar.php b/app/Template/config/sidebar.php
new file mode 100644
index 00000000..5870ca81
--- /dev/null
+++ b/app/Template/config/sidebar.php
@@ -0,0 +1,20 @@
+<div class="sidebar">
+    <h2><?= t('Actions') ?></h2>
+    <ul>
+        <li>
+            <?= Helper\a(t('About'), 'config', 'index') ?>
+        </li>
+        <li>
+            <?= Helper\a(t('Application settings'), 'config', 'application') ?>
+        </li>
+        <li>
+            <?= Helper\a(t('Board settings'), 'config', 'board') ?>
+        </li>
+        <li>
+            <?= Helper\a(t('Webhooks'), 'config', 'webhook') ?>
+        </li>
+        <li>
+            <?= Helper\a(t('API'), 'config', 'api') ?>
+        </li>
+    </ul>
+</div>
\ No newline at end of file
diff --git a/app/Template/config/webhook.php b/app/Template/config/webhook.php
new file mode 100644
index 00000000..052a2a99
--- /dev/null
+++ b/app/Template/config/webhook.php
@@ -0,0 +1,38 @@
+<div class="page-header">
+    <h2><?= t('Webhook settings') ?></h2>
+</div>
+<section>
+<form method="post" action="<?= Helper\u('config', 'webhook') ?>" autocomplete="off">
+
+    <?= Helper\form_csrf() ?>
+
+    <?= Helper\form_label(t('Webhook URL for task creation'), 'webhook_url_task_creation') ?>
+    <?= Helper\form_text('webhook_url_task_creation', $values, $errors) ?><br/>
+
+    <?= Helper\form_label(t('Webhook URL for task modification'), 'webhook_url_task_modification') ?>
+    <?= Helper\form_text('webhook_url_task_modification', $values, $errors) ?><br/>
+
+    <div class="form-actions">
+        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
+    </div>
+</form>
+</section>
+
+<div class="page-header">
+    <h2><?= t('URL and token') ?></h2>
+</div>
+<section class="listing">
+    <ul>
+        <li>
+            <?= t('Webhook token:') ?>
+            <strong><?= Helper\escape($values['webhook_token']) ?></strong>
+        </li>
+        <li>
+            <?= t('URL for task creation:') ?>
+            <input type="text" readonly="readonly" value="<?= Helper\get_current_base_url().Helper\u('webhook', 'task', array('token' => $values['webhook_token'])) ?>">
+        </li>
+        <li>
+            <?= Helper\a(t('Reset token'), 'config', 'token', array('type' => 'webhook'), true) ?>
+        </li>
+    </ul>
+</section>
\ No newline at end of file
diff --git a/app/Template/config_about.php b/app/Template/config_about.php
deleted file mode 100644
index 3f34f802..00000000
--- a/app/Template/config_about.php
+++ /dev/null
@@ -1,41 +0,0 @@
-<div class="page-header">
-    <h2><?= t('About') ?></h2>
-</div>
-<section class="listing">
-    <ul>
-        <li>
-            <?= t('Official website:') ?>
-            <a href="http://kanboard.net/" target="_blank" rel="noreferer">http://kanboard.net/</a>
-        </li>
-        <li>
-            <?= t('Application version:') ?>
-            <strong><?= APP_VERSION ?></strong>
-        </li>
-    </ul>
-</section>
-
-<div class="page-header">
-    <h2><?= t('Database') ?></h2>
-</div>
-<section class="listing">
-    <ul>
-        <li>
-            <?= t('Database driver:') ?>
-            <strong><?= Helper\escape(DB_DRIVER) ?></strong>
-        </li>
-        <?php if (DB_DRIVER === 'sqlite'): ?>
-            <li>
-                <?= t('Database size:') ?>
-                <strong><?= Helper\format_bytes($db_size) ?></strong>
-            </li>
-            <li>
-                <?= Helper\a(t('Download the database'), 'config', 'downloadDb', array(), true) ?>&nbsp;
-                <?= t('(Gzip compressed Sqlite file)') ?>
-            </li>
-            <li>
-                <?= Helper\a(t('Optimize the database'), 'config', 'optimizeDb', array(), true) ?>&nbsp;
-                <?= t('(VACUUM command)') ?>
-            </li>
-        <?php endif ?>
-    </ul>
-</section>
\ No newline at end of file
diff --git a/app/Template/config_api.php b/app/Template/config_api.php
deleted file mode 100644
index 037ea08d..00000000
--- a/app/Template/config_api.php
+++ /dev/null
@@ -1,18 +0,0 @@
-<div class="page-header">
-    <h2><?= t('API') ?></h2>
-</div>
-<section class="listing">
-    <ul>
-        <li>
-            <?= t('API token:') ?>
-            <strong><?= Helper\escape($values['api_token']) ?></strong>
-        </li>
-        <li>
-            <?= t('API endpoint:') ?>
-            <input type="text" readonly="readonly" value="<?= Helper\get_current_base_url().'jsonrpc.php' ?>">
-        </li>
-        <li>
-            <?= Helper\a(t('Reset token'), 'config', 'token', array('type' => 'api'), true) ?>
-        </li>
-    </ul>
-</section>
\ No newline at end of file
diff --git a/app/Template/config_application.php b/app/Template/config_application.php
deleted file mode 100644
index c92975bc..00000000
--- a/app/Template/config_application.php
+++ /dev/null
@@ -1,27 +0,0 @@
-<div class="page-header">
-    <h2><?= t('Application settings') ?></h2>
-</div>
-<section>
-<form method="post" action="<?= Helper\u('config', 'application') ?>" autocomplete="off">
-
-    <?= Helper\form_csrf() ?>
-
-    <?= Helper\form_label(t('Application URL'), 'application_url') ?>
-    <?= Helper\form_text('application_url', $values, $errors, array('placeholder="http://example.kanboard.net/"')) ?><br/>
-    <p class="form-help"><?= t('Example: http://example.kanboard.net/ (used by email notifications)') ?></p>
-
-    <?= Helper\form_label(t('Language'), 'application_language') ?>
-    <?= Helper\form_select('application_language', $languages, $values, $errors) ?><br/>
-
-    <?= Helper\form_label(t('Timezone'), 'application_timezone') ?>
-    <?= Helper\form_select('application_timezone', $timezones, $values, $errors) ?><br/>
-
-    <?= Helper\form_label(t('Date format'), 'application_date_format') ?>
-    <?= Helper\form_select('application_date_format', $date_formats, $values, $errors) ?><br/>
-    <p class="form-help"><?= t('ISO format is always accepted, example: "%s" and "%s"', date('Y-m-d'), date('Y_m_d')) ?></p>
-
-    <div class="form-actions">
-        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
-    </div>
-</form>
-</section>
\ No newline at end of file
diff --git a/app/Template/config_board.php b/app/Template/config_board.php
deleted file mode 100644
index f260d084..00000000
--- a/app/Template/config_board.php
+++ /dev/null
@@ -1,29 +0,0 @@
-<div class="page-header">
-    <h2><?= t('Board settings') ?></h2>
-</div>
-<section>
-<form method="post" action="<?= Helper\u('config', 'board') ?>" autocomplete="off">
-
-    <?= Helper\form_csrf() ?>
-
-    <?= Helper\form_label(t('Task highlight period'), 'board_highlight_period') ?>
-    <?= Helper\form_number('board_highlight_period', $values, $errors) ?><br/>
-    <p class="form-help"><?= t('Period (in second) to consider a task was modified recently (0 to disable, 2 days by default)') ?></p>
-
-    <?= Helper\form_label(t('Refresh interval for public board'), 'board_public_refresh_interval') ?>
-    <?= Helper\form_number('board_public_refresh_interval', $values, $errors) ?><br/>
-    <p class="form-help"><?= t('Frequency in second (60 seconds by default)') ?></p>
-
-    <?= Helper\form_label(t('Refresh interval for private board'), 'board_private_refresh_interval') ?>
-    <?= Helper\form_number('board_private_refresh_interval', $values, $errors) ?><br/>
-    <p class="form-help"><?= t('Frequency in second (0 to disable this feature, 10 seconds by default)') ?></p>
-
-    <?= Helper\form_label(t('Default columns for new projects (Comma-separated)'), 'board_columns') ?>
-    <?= Helper\form_text('board_columns', $values, $errors) ?><br/>
-    <p class="form-help"><?= t('Default values are "%s"', $default_columns) ?></p>
-
-    <div class="form-actions">
-        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
-    </div>
-</form>
-</section>
\ No newline at end of file
diff --git a/app/Template/config_layout.php b/app/Template/config_layout.php
deleted file mode 100644
index bafad6bb..00000000
--- a/app/Template/config_layout.php
+++ /dev/null
@@ -1,10 +0,0 @@
-<section id="main">
-    <section class="sidebar-container" id="config-section">
-
-        <?= Helper\template('config_sidebar') ?>
-
-        <div class="sidebar-content">
-            <?= $config_content_for_layout ?>
-        </div>
-    </section>
-</section>
\ No newline at end of file
diff --git a/app/Template/config_sidebar.php b/app/Template/config_sidebar.php
deleted file mode 100644
index 5870ca81..00000000
--- a/app/Template/config_sidebar.php
+++ /dev/null
@@ -1,20 +0,0 @@
-<div class="sidebar">
-    <h2><?= t('Actions') ?></h2>
-    <ul>
-        <li>
-            <?= Helper\a(t('About'), 'config', 'index') ?>
-        </li>
-        <li>
-            <?= Helper\a(t('Application settings'), 'config', 'application') ?>
-        </li>
-        <li>
-            <?= Helper\a(t('Board settings'), 'config', 'board') ?>
-        </li>
-        <li>
-            <?= Helper\a(t('Webhooks'), 'config', 'webhook') ?>
-        </li>
-        <li>
-            <?= Helper\a(t('API'), 'config', 'api') ?>
-        </li>
-    </ul>
-</div>
\ No newline at end of file
diff --git a/app/Template/config_webhook.php b/app/Template/config_webhook.php
deleted file mode 100644
index 052a2a99..00000000
--- a/app/Template/config_webhook.php
+++ /dev/null
@@ -1,38 +0,0 @@
-<div class="page-header">
-    <h2><?= t('Webhook settings') ?></h2>
-</div>
-<section>
-<form method="post" action="<?= Helper\u('config', 'webhook') ?>" autocomplete="off">
-
-    <?= Helper\form_csrf() ?>
-
-    <?= Helper\form_label(t('Webhook URL for task creation'), 'webhook_url_task_creation') ?>
-    <?= Helper\form_text('webhook_url_task_creation', $values, $errors) ?><br/>
-
-    <?= Helper\form_label(t('Webhook URL for task modification'), 'webhook_url_task_modification') ?>
-    <?= Helper\form_text('webhook_url_task_modification', $values, $errors) ?><br/>
-
-    <div class="form-actions">
-        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
-    </div>
-</form>
-</section>
-
-<div class="page-header">
-    <h2><?= t('URL and token') ?></h2>
-</div>
-<section class="listing">
-    <ul>
-        <li>
-            <?= t('Webhook token:') ?>
-            <strong><?= Helper\escape($values['webhook_token']) ?></strong>
-        </li>
-        <li>
-            <?= t('URL for task creation:') ?>
-            <input type="text" readonly="readonly" value="<?= Helper\get_current_base_url().Helper\u('webhook', 'task', array('token' => $values['webhook_token'])) ?>">
-        </li>
-        <li>
-            <?= Helper\a(t('Reset token'), 'config', 'token', array('type' => 'webhook'), true) ?>
-        </li>
-    </ul>
-</section>
\ No newline at end of file
diff --git a/app/Template/file/new.php b/app/Template/file/new.php
new file mode 100644
index 00000000..63c29f1d
--- /dev/null
+++ b/app/Template/file/new.php
@@ -0,0 +1,14 @@
+<div class="page-header">
+    <h2><?= t('Attach a document') ?></h2>
+</div>
+
+<form action="<?= Helper\u('file', 'save', array('task_id' => $task['id'])) ?>" method="post" enctype="multipart/form-data">
+    <?= Helper\form_csrf() ?>
+    <input type="file" name="files[]" multiple />
+    <div class="form-help"><?= t('Maximum size: ') ?><?= is_integer($max_size) ? Helper\format_bytes($max_size) : $max_size ?></div>
+    <div class="form-actions">
+        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
+        <?= t('or') ?>
+        <?= Helper\a(t('cancel'), 'task', 'show', array('task_id' => $task['id'])) ?>
+    </div>
+</form>
\ No newline at end of file
diff --git a/app/Template/file/open.php b/app/Template/file/open.php
new file mode 100644
index 00000000..6106e6f8
--- /dev/null
+++ b/app/Template/file/open.php
@@ -0,0 +1,6 @@
+<div class="page-header">
+    <h2><?= Helper\escape($file['name']) ?></h2>
+    <div class="task-file-viewer">
+        <img src="<?= Helper\u('file', 'image', array('file_id' => $file['id'], 'task_id' => $file['task_id'])) ?>" alt="<?= Helper\escape($file['name']) ?>"/>
+    </div>
+</div>
\ No newline at end of file
diff --git a/app/Template/file/remove.php b/app/Template/file/remove.php
new file mode 100644
index 00000000..4f655e5a
--- /dev/null
+++ b/app/Template/file/remove.php
@@ -0,0 +1,15 @@
+<div class="page-header">
+    <h2><?= t('Remove a file') ?></h2>
+</div>
+
+<div class="confirm">
+    <p class="alert alert-info">
+        <?= t('Do you really want to remove this file: "%s"?', Helper\escape($file['name'])) ?>
+    </p>
+
+    <div class="form-actions">
+        <?= Helper\a(t('Yes'), 'file', 'remove', array('task_id' => $task['id'], 'file_id' => $file['id']), true, 'btn btn-red') ?>
+        <?= t('or') ?>
+        <?= Helper\a(t('cancel'), 'task', 'show', array('task_id' => $task['id'])) ?>
+    </div>
+</div>
\ No newline at end of file
diff --git a/app/Template/file/show.php b/app/Template/file/show.php
new file mode 100644
index 00000000..92a7c100
--- /dev/null
+++ b/app/Template/file/show.php
@@ -0,0 +1,23 @@
+<?php if (! empty($files)): ?>
+<div id="attachments" class="task-show-section">
+
+    <div class="page-header">
+        <h2><?= t('Attachments') ?></h2>
+    </div>
+
+    <ul class="task-show-files">
+    <?php foreach ($files as $file): ?>
+        <li>
+            <?= Helper\a(Helper\escape($file['name']), 'file', 'download', array('task_id' => $task['id'], 'file_id' => $file['id'])) ?>
+            <span class="task-show-file-actions">
+                <?php if ($file['is_image']): ?>
+                    <?= Helper\a(t('open'), 'file', 'open', array('task_id' => $task['id'], 'file_id' => $file['id']), false, 'file-popover') ?>,
+                <?php endif ?>
+                <?= Helper\a(t('remove'), 'file', 'confirm', array('task_id' => $task['id'], 'file_id' => $file['id'])) ?>
+            </span>
+        </li>
+    <?php endforeach ?>
+    </ul>
+
+</div>
+<?php endif ?>
\ No newline at end of file
diff --git a/app/Template/file_new.php b/app/Template/file_new.php
deleted file mode 100644
index 7f7f1d1c..00000000
--- a/app/Template/file_new.php
+++ /dev/null
@@ -1,14 +0,0 @@
-<div class="page-header">
-    <h2><?= t('Attach a document') ?></h2>
-</div>
-
-<form action="?controller=file&amp;action=save&amp;task_id=<?= $task['id'] ?>" method="post" enctype="multipart/form-data">
-    <?= Helper\form_csrf() ?>
-    <input type="file" name="files[]" multiple />
-    <div class="form-help"><?= t('Maximum size: ') ?><?= is_integer($max_size) ? Helper\format_bytes($max_size) : $max_size ?></div>
-    <div class="form-actions">
-        <input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>
-        <?= t('or') ?>
-        <a href="?controller=task&amp;action=show&amp;task_id=<?= $task['id'] ?>"><?= t('cancel') ?></a>
-    </div>
-</form>
\ No newline at end of file
diff --git a/app/Template/file_open.php b/app/Template/file_open.php
deleted file mode 100644
index aa181d64..00000000
--- a/app/Template/file_open.php
+++ /dev/null
@@ -1,6 +0,0 @@
-<div class="page-header">
-    <h2><?= Helper\escape($file['name']) ?></h2>
-    <div class="task-file-viewer">
-        <img src="?controller=file&amp;action=image&amp;file_id=<?= $file['id'] ?>&amp;task_id=<?= $file['task_id'] ?>" alt="<?= Helper\escape($file['name']) ?>"/>
-    </div>
-</div>
\ No newline at end of file
diff --git a/app/Template/file_remove.php b/app/Template/file_remove.php
deleted file mode 100644
index af77591c..00000000
--- a/app/Template/file_remove.php
+++ /dev/null
@@ -1,14 +0,0 @@
-<div class="page-header">
-    <h2><?= t('Remove a file') ?></h2>
-</div>
-
-<div class="confirm">
-    <p class="alert alert-info">
-        <?= t('Do you really want to remove this file: "%s"?', Helper\escape($file['name'])) ?>
-    </p>
-
-    <div class="form-actions">
-        <a href="?controller=file&amp;action=remove&amp;task_id=<?= $task['id'] ?>&amp;file_id=<?= $file['id'].Helper\param_csrf() ?>" class="btn btn-red"><?= t('Yes') ?></a>
-        <?= t('or') ?> <a href="?controller=task&amp;action=show&amp;task_id=<?= $task['id'] ?>"><?= t('cancel') ?></a>
-    </div>
-</div>
\ No newline at end of file
diff --git a/app/Template/file_show.php b/app/Template/file_show.php
deleted file mode 100644
index 3832a0f5..00000000
--- a/app/Template/file_show.php
+++ /dev/null
@@ -1,23 +0,0 @@
-<?php if (! empty($files)): ?>
-<div id="attachments" class="task-show-section">
-
-    <div class="page-header">
-        <h2><?= t('Attachments') ?></h2>
-    </div>
-
-    <ul class="task-show-files">
-    <?php foreach ($files as $file): ?>
-        <li>
-            <a href="?controller=file&amp;action=download&amp;file_id=<?= $file['id'] ?>&amp;task_id=<?= $task['id'] ?>"><?= Helper\escape($file['name']) ?></a>
-            <span class="task-show-file-actions">
-                <?php if ($file['is_image']): ?>
-                    <a href="?controller=file&amp;action=open&amp;file_id=<?= $file['id'] ?>&amp;task_id=<?= $task['id'] ?>" class="file-popover"><?= t('open') ?></a>,
-                <?php endif ?>
-                <a href="?controller=file&amp;action=confirm&amp;file_id=<?= $file['id'] ?>&amp;task_id=<?= $task['id'] ?>"><?= t('remove') ?></a>
-            </span>
-        </li>
-    <?php endforeach ?>
-    </ul>
-
-</div>
-<?php endif ?>
\ No newline at end of file
diff --git a/app/Template/task_show.php b/app/Template/task_show.php
index ec5d5da4..a4b49cef 100644
--- a/app/Template/task_show.php
+++ b/app/Template/task_show.php
@@ -3,5 +3,5 @@
 <?= Helper\template('task_show_description', array('task' => $task)) ?>
 <?= Helper\template('subtask_show', array('task' => $task, 'subtasks' => $subtasks)) ?>
 <?= Helper\template('task_timesheet', array('timesheet' => $timesheet)) ?>
-<?= Helper\template('file_show', array('task' => $task, 'files' => $files)) ?>
+<?= Helper\template('file/show', array('task' => $task, 'files' => $files)) ?>
 <?= Helper\template('task_comments', array('task' => $task, 'comments' => $comments, 'project' => $project)) ?>
\ No newline at end of file
diff --git a/assets/css/app.css b/assets/css/app.css
index 0ce83aa4..2894b9ee 100644
--- a/assets/css/app.css
+++ b/assets/css/app.css
@@ -527,7 +527,7 @@ nav .active a {
     margin: 0;
     padding: 0;
     font-size: 140%;
-    border-bottom: 1px dotted #DF5353;
+    border-bottom: 1px dotted #ccc;
 }
 
 .page-header h2 a {
diff --git a/assets/css/header.css b/assets/css/header.css
index 4fc268cc..c8b820d1 100644
--- a/assets/css/header.css
+++ b/assets/css/header.css
@@ -61,7 +61,7 @@ nav .active a {
     margin: 0;
     padding: 0;
     font-size: 140%;
-    border-bottom: 1px dotted #DF5353;
+    border-bottom: 1px dotted #ccc;
 }
 
 .page-header h2 a {
-- 
cgit v1.2.3