openerp-dev-web team mailing list archive

["Jra \(Open ERP\)" <jra@xxxxxxxxxxx>]

Jra (Open ERP) has proposed merging lp:~openerp-dev/openobject-client-web/Image_widget into lp:openobject-client-web.

Requested reviews:
  OpenERP SA's Web Client R&D (openerp-dev-web)


Minor change with binary field (Image).
For testing refer Bug #675720.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/Image_widget/+merge/41564
Your team OpenERP SA's Web Client R&D is requested to review the proposed merge of lp:~openerp-dev/openobject-client-web/Image_widget into lp:openobject-client-web.

=== modified file 'addons/openerp/widgets/form/templates/image.mako'
--- addons/openerp/widgets/form/templates/image.mako	2010-11-04 14:42:07 +0000
+++ addons/openerp/widgets/form/templates/image.mako	2010-11-23 10:12:37 +0000
@@ -32,7 +32,8 @@
             onclick="openobject.tools.openWindow(openobject.http.getURL('/openerp/image', {model: '${model}', id: '${id}', field : '${field}'}), {width: 500, height: 300});"
         />
     % else:
-        <input type="file" class="${css_class}" id="${name}" ${py.attrs(attrs)} name="${name}"/>
+        <input type="file" class="${css_class}" id="${name}" ${py.attrs(attrs)} value="" />
+        <input type="hidden" id="${name}" ${py.attrs(attrs)} name="${name}" />
     % endif
 % else:
     % if id:
@@ -81,4 +82,7 @@
         
     % endif
 % endif
-<input type="hidden" id="_${field}" name="${name}" value="${src or bin_src or ''}">
+
+% if id or src or bin_src:
+	<input type="hidden" id="_${field}" name="${name}" value="${src or bin_src}" />
+% endif