From b5e2e8aa1daeaf304cf9f7b3d67a32e6df130037 Mon Sep 17 00:00:00 2001
From: James Cole <thegrumpydictator@gmail.com>
Date: Sat, 18 Jul 2015 23:51:51 +0200
Subject: [PATCH] Edit attachment page.

---
 app/Http/Controllers/AttachmentController.php | 28 ++++++++-
 app/Support/ExpandedForm.php                  | 19 ++++++
 config/twigbridge.php                         |  2 +-
 resources/lang/en/firefly.php                 |  4 ++
 resources/lang/en/form.php                    |  5 ++
 resources/twig/attachments/edit.twig          | 60 +++++++++++++++++++
 resources/twig/form/static.twig               |  7 +++
 7 files changed, 123 insertions(+), 2 deletions(-)
 create mode 100644 resources/twig/attachments/edit.twig
 create mode 100644 resources/twig/form/static.twig

diff --git a/app/Http/Controllers/AttachmentController.php b/app/Http/Controllers/AttachmentController.php
index 8ff0106b3d..56a8ccfe79 100644
--- a/app/Http/Controllers/AttachmentController.php
+++ b/app/Http/Controllers/AttachmentController.php
@@ -3,9 +3,11 @@
 namespace FireflyIII\Http\Controllers;
 
 
+use Crypt;
 use FireflyIII\Helpers\Attachments\AttachmentHelperInterface;
 use FireflyIII\Models\Attachment;
-use Crypt;
+use View;
+
 /**
  * Class AttachmentController
  *
@@ -14,6 +16,30 @@ use Crypt;
 class AttachmentController extends Controller
 {
 
+    /**
+     * @codeCoverageIgnore
+     */
+    public function __construct()
+    {
+        parent::__construct();
+        View::share('mainTitleIcon', 'fa-paperclip');
+        View::share('title', trans('firefly.attachments'));
+    }
+
+
+    /**
+     * @param Attachment $attachment
+     *
+     * @return \Illuminate\View\View
+     */
+    public function edit(Attachment $attachment)
+    {
+        $subTitleIcon = 'fa-pencil';
+        $subTitle     = trans('firefly.edit_attachment', ['name' => $attachment->filename]);
+
+        return view('attachments.edit', compact('attachment', 'subTitleIcon', 'subTitle'));
+    }
+
     /**
      * @param Attachment $attachment
      */
diff --git a/app/Support/ExpandedForm.php b/app/Support/ExpandedForm.php
index dd68cfd198..59e3db0fb5 100644
--- a/app/Support/ExpandedForm.php
+++ b/app/Support/ExpandedForm.php
@@ -20,6 +20,25 @@ use Session;
 class ExpandedForm
 {
 
+    /**
+     * @param       $name
+     * @param null  $value
+     * @param array $options
+     *
+     * @return string
+     */
+    public function staticText($name, $value, array $options = [])
+    {
+        $label   = $this->label($name, $options);
+        $options = $this->expandOptionArray($name, $label, $options);
+        $classes = $this->getHolderClasses($name);
+        $value   = $this->fillFieldValue($name, $value);
+        $html    = view('form.static', compact('classes', 'name', 'label', 'value', 'options'))->render();
+
+        return $html;
+
+    }
+
     /**
      * @param       $name
      * @param null  $value
diff --git a/config/twigbridge.php b/config/twigbridge.php
index dd60091875..ee32836a9e 100644
--- a/config/twigbridge.php
+++ b/config/twigbridge.php
@@ -145,7 +145,7 @@ return [
             'ExpandedForm' => [
                 'is_safe' => [
                     'date', 'text', 'select', 'balance', 'optionsList', 'checkbox', 'amount', 'tags', 'integer', 'textarea', 'location',
-                    'multiRadio','file','multiCheckbox'
+                    'multiRadio','file','multiCheckbox','staticText'
                 ]
             ],
             'Form'         => [
diff --git a/resources/lang/en/firefly.php b/resources/lang/en/firefly.php
index 35b9f7599d..bfd130b1ab 100644
--- a/resources/lang/en/firefly.php
+++ b/resources/lang/en/firefly.php
@@ -18,8 +18,12 @@ return [
     'showEverything'                   => 'Show everything',
     'never'                            => 'Never',
     'search_results_for'               => 'Search results for ":query"',
+
+    // attachments
     'nr_of_attachments'                => 'One attachment|:count attachments',
     'attachments'                      => 'Attachments',
+    'edit_attachment'                  => 'Edit attachment ":name"',
+    'update_attachment'                => 'Update attachment',
 
     // tour:
     'prev'                             => 'Prev',
diff --git a/resources/lang/en/form.php b/resources/lang/en/form.php
index 4a962baa67..1822182400 100644
--- a/resources/lang/en/form.php
+++ b/resources/lang/en/form.php
@@ -61,6 +61,11 @@ return [
     'add_new_transfer'            => 'Add a new transfer',
     'noPiggybank'                 => '(no piggy bank)',
     'noBudget'                    => '(no budget)',
+    'title'                       => 'Title',
+    'notes'                       => 'Notes',
+    'filename'                    => 'File name',
+    'mime'                        => 'Mime type',
+    'size'                        => 'Size',
 
     'delete_account'              => 'Delete account ":name"',
     'delete_bill'                 => 'Delete bill ":name"',
diff --git a/resources/twig/attachments/edit.twig b/resources/twig/attachments/edit.twig
new file mode 100644
index 0000000000..9aaa0a84d1
--- /dev/null
+++ b/resources/twig/attachments/edit.twig
@@ -0,0 +1,60 @@
+{% extends "./layout/default.twig" %}
+
+{% block breadcrumbs %}
+    {{ Breadcrumbs.renderIfExists(Route.getCurrentRoute().getName(), attachment) }}
+{% endblock %}
+
+{% block content %}
+    <form method="POST" action="{{ route('attachment.update', attachment.id) }}" accept-charset="UTF-8" class="form-horizontal" id="update">
+
+        <input type="hidden" name="_token" value="{{ csrf_token() }}"/>
+        <input type="hidden" name="id" value="{{ attachment.id }}"/>
+
+
+        <div class="row">
+            <div class="col-lg-6 col-md-6 col-sm-12">
+                <div class="box box-primary">
+                    <div class="box-header with-border">
+                        <h3 class="box-title">{{ 'mandatoryFields'|_ }}</h3>
+                    </div>
+                    <div class="box-body">
+                        {{ ExpandedForm.staticText('filename',attachment.filename) }}
+                        {{ ExpandedForm.staticText('mime',attachment.mime) }}
+                        {{ ExpandedForm.staticText('size',attachment.size|filesize) }}
+                    </div>
+                </div>
+
+            </div>
+            <div class="col-lg-6 col-md-6 col-sm-12">
+                <div class="box">
+                    <div class="box-header with-border">
+                        <h3 class="box-title">{{ 'optionalFields'|_ }}</h3>
+                    </div>
+                    <div class="box-body">
+                        {{ ExpandedForm.text('title') }}
+                        {{ ExpandedForm.textarea('description') }}
+                        {{ ExpandedForm.textarea('notes') }}
+                    </div>
+                </div>
+
+                <!-- panel for options  -->
+                <div class="box">
+                    <div class="box-header with-border">
+                        <h3 class="box-title">{{ 'options'|_ }}</h3>
+                    </div>
+                    <div class="box-body">
+                        {{ ExpandedForm.optionsList('update','attachment') }}
+                    </div>
+                    <div class="box-footer">
+                        <button type="submit" class="btn pull-right btn-success">
+                            {{ ('update_attachment')|_ }}
+                        </button>
+                    </div>
+                </div>
+            </div>
+        </div>
+
+
+
+    </form>
+{% endblock %}
diff --git a/resources/twig/form/static.twig b/resources/twig/form/static.twig
new file mode 100644
index 0000000000..e70ba496ba
--- /dev/null
+++ b/resources/twig/form/static.twig
@@ -0,0 +1,7 @@
+<div class="{{ classes }}" id="{{ name }}_holder">
+    <label for="{{ options.id }}" class="col-sm-4 control-label">{{ label }}</label>
+
+    <div class="col-sm-8">
+        <p class="form-control-static">{{ value }}</p>
+    </div>
+</div>