From f2ea8950a289899370dc9ccc97497e98d3ef3ffe Mon Sep 17 00:00:00 2001
From: Christophe Matthieu <chm@openerp.com>
Date: Thu, 28 Feb 2013 12:11:35 +0100
Subject: [PATCH] [IMP] view_form: FieldMany2ManyBinaryMultiFiles: get/set
 value

bzr revid: chm@openerp.com-20130228111135-8lhxzc9vya6ltw7p
---
 addons/web/static/src/js/view_form.js | 41 ++++++---------------------
 addons/web/static/src/xml/base.xml    | 16 ++++++-----
 2 files changed, 17 insertions(+), 40 deletions(-)

diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js
index eb102eb6c07..2b0af459259 100644
--- a/addons/web/static/src/js/view_form.js
+++ b/addons/web/static/src/js/view_form.js
@@ -5021,7 +5021,7 @@ instance.web.form.FieldBinaryImage = instance.web.form.FieldBinary.extend({
 });
 
 /**
- * Widget for (one2many field) to upload one or more file in same time and display in list.
+ * Widget for (many2many field) to upload one or more file in same time and display in list.
  * The user can delete his files.
  * Options on attribute ; "blockui" {Boolean} block the UI or not
  * during the file is uploading
@@ -5046,40 +5046,15 @@ instance.web.form.FieldMany2ManyBinaryMultiFiles = instance.web.form.AbstractFie
         this.$el.on('change', 'input.oe_form_binary_file', this.on_file_change );
     },
     set_value: function(value_) {
-        var value_ = value_ || [];
-        var ids = [];
-        if(value_ instanceof Array) {
-            _.each(value_, function(command) {
-                if (command instanceof Array) {
-                    switch (command[0]) {
-                        case commands.CREATE:
-                            ids = ids.concat(command[2]);
-                            return;
-                        case commands.REPLACE_WITH:
-                            ids = ids.concat(command[2]);
-                            return;
-                        case commands.UPDATE:
-                            ids = ids.concat(command[2]);
-                            return;
-                        case commands.LINK_TO:
-                            ids = ids.concat(command[1]);
-                            return;
-                        case commands.DELETE:
-                            ids = _.filter(ids, function (id) { return id != command[1];});
-                            return;
-                        case commands.DELETE_ALL:
-                            ids = [];
-                            return;
-                    }
-                } else if (typeof command == 'number') {
-                    ids.push(command);
-                }
-            });
+        value_ = value_ || [];
+        if (value_.length >= 1 && value_[0] instanceof Array) {
+            value_ = value_[0][2];
         }
-        this._super( ids );
+        this._super(value_);
     },
     get_value: function() {
-        return _.map(this.get('value'), function (id) { return commands.link_to( id ); });
+        var tmp = [commands.replace_with(this.get("value"))];
+        return tmp;
     },
     get_file_url: function (attachment) {
         return this.session.url('/web/binary/saveas', {model: 'ir.attachment', field: 'datas', filename_field: 'datas_fname', id: attachment['id']});
@@ -5181,7 +5156,7 @@ instance.web.form.FieldMany2ManyBinaryMultiFiles = instance.web.form.AbstractFie
                     'url': this.get_file_url(result)
                 };
             }
-            var values = this.get('value');
+            var values = _.clone(this.get('value'));
             values.push(result.id);
             this.set({'value': values});
         }
diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml
index b339497409c..36695d22ca5 100644
--- a/addons/web/static/src/xml/base.xml
+++ b/addons/web/static/src/xml/base.xml
@@ -1278,13 +1278,15 @@
                 </t>
             </div>
         </t>
-        <t t-if="widget.get('effective_readonly')" t-foreach="widget.get('value')" t-as="id">
-            <t t-set="file" t-value="widget.data[id]"/>
-            <div>
-                <a t-att-href="file.url" t-attf-title="{(file.name || file.filename) + (file.date?' \n('+file.date+')':'' )}">
-                    <t t-raw="file.name || file.filename"/>
-                </a>
-            </div>
+        <t t-if="widget.get('effective_readonly')">
+            <t t-foreach="widget.get('value')" t-as="id">
+                <t t-set="file" t-value="widget.data[id]"/>
+                <div>
+                    <a t-att-href="file.url" t-attf-title="{(file.name || file.filename) + (file.date?' \n('+file.date+')':'' )}">
+                        <t t-raw="file.name || file.filename"/>
+                    </a>
+                </div>
+            </t>
         </t>
     </div>
 </t>