Merge lp:~openerp-dev/openerp-web/6.1-opw-581888-msh into lp:openerp-web/6.1

[Mohammed Shekha(Open ERP)]

Hello,

Fixed the issue of image not displayed newly created record when it is return from on_change result or from default values because still datarecord will not have id so it can not read image from database, hence set the value in src directly with content-type image/png.

Demo :- To generate this create an on_change method which returns binary image as a result, now call that on_change method will not display image in image widget, I have created an on_change method on hr.employee form for demo, added a binary field in res.users for instance.

Expected :- when on_change returns binary data it should be displayed on image widget.

Thanks.

Unmerged revisions

2482. By Mohammed Shekha(Open ERP) on 2012-11-16

[FIX]Fixed the issue of image not displayed newly created record when it is return from on_change result or from default values because still datarecord will not have id so it can not read image from database, hence set the value in src directly with content-type image/png

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2012-10-23 07:30:10 +0000
+++ addons/web/static/src/js/view_form.js	2012-11-16 07:41:22 +0000
@@ -3562,9 +3562,13 @@
     set_value: function(value) {
         this._super.apply(this, arguments);
         this.set_image_maxwidth();
-        var url = '/web/binary/image?session_id=' + this.session.session_id + '&model=' +
-            this.view.dataset.model +'&id=' + (this.view.datarecord.id || '') + '&field=' + this.name + '&t=' + (new Date().getTime());
-        this.$image.attr('src', url);
+        if(this.view.datarecord.id) {
+            var url = '/web/binary/image?session_id=' + this.session.session_id + '&model=' +
+                this.view.dataset.model +'&id=' + (this.view.datarecord.id || '') + '&field=' + this.name + '&t=' + (new Date().getTime());
+            this.$image.attr('src', url);
+        }else {
+            this.$image.attr('src', 'data:image/png;base64,' + value);
+        }
     },
     set_image_maxwidth: function() {
         this.$image.css('max-width', this.$element.width());