From 9442be5c230e286073244451189ffc05ae6c8083 Mon Sep 17 00:00:00 2001
From: Jefferson Stachelski <jstachel@thoughtworks.com>
Date: Wed, 3 Feb 2016 14:31:53 -0200
Subject: Issue #549 - Implemented remove attachment

---
 web-ui/app/js/mail_view/ui/attachment_list.js | 20 ++++++++++++++++++--
 web-ui/app/js/mail_view/ui/forward_box.js     |  5 +++--
 2 files changed, 21 insertions(+), 4 deletions(-)

(limited to 'web-ui/app/js')

diff --git a/web-ui/app/js/mail_view/ui/attachment_list.js b/web-ui/app/js/mail_view/ui/attachment_list.js
index b3b26710..3d57ede5 100644
--- a/web-ui/app/js/mail_view/ui/attachment_list.js
+++ b/web-ui/app/js/mail_view/ui/attachment_list.js
@@ -19,10 +19,11 @@ define(
     [
         'views/templates',
         'page/events',
-        'helpers/view_helper'
+        'helpers/view_helper',
+        'helpers/monitored_ajax'
     ],
 
-    function (templates, events, viewHelper) {
+    function (templates, events, viewHelper, monitoredAjax) {
         'use strict';
 
         function attachmentList() {
@@ -180,11 +181,26 @@ define(
                 this.select('inputFileUpload').click();
             };
 
+            this.removeAttachments = function(event, data) {
+              var success = function() {
+                console.log('Success');
+              };
+
+              var failure = function() {
+                console.log('Fail!!!!!');
+              };
+
+              monitoredAjax(this, '/attachment/' + data.ident, {
+                type: 'DELETE'
+              }).done(success.bind(this)).fail(failure.bind(this));
+            };
+
             this.after('initialize', function () {
                 this.addJqueryFileUploadConfig();
                 this.on(document, events.mail.uploadedAttachment, this.showAttachment);
                 this.on(document, events.mail.startUploadAttachment, this.startUpload);
                 this.on(document, events.mail.appendAttachment, this.addAttachment);
+                this.on(document, events.mail.removeAttachment, this.removeAttachments);
             });
         }
 
diff --git a/web-ui/app/js/mail_view/ui/forward_box.js b/web-ui/app/js/mail_view/ui/forward_box.js
index 3d643b2f..5acef337 100644
--- a/web-ui/app/js/mail_view/ui/forward_box.js
+++ b/web-ui/app/js/mail_view/ui/forward_box.js
@@ -56,7 +56,7 @@ define(
           self.trigger(document, events.mail.removeAttachment, {ident: ident});
           event.preventDefault();
         });
-        
+
         this.on(this.select('subjectDisplay'), 'click', this.showSubjectInput);
         this.select('recipientsDisplay').hide();
         this.select('recipientsFields').show();
@@ -64,7 +64,8 @@ define(
 
       this.convertToRemovableAttachments = function(attachments) {
         return attachments.map(function(attachment) {
-          attachment.removable = true;
+          attachment.removable = false; // don't show this button until the bug is fixed
+          attachment.received = true;
           return attachment;
         });
       };
-- 
cgit v1.2.3