Review: Needs Fixing

Hello Amit,

The issue is due to lack of setting the fields value in the model field while 
adding new image ! your fix will just bypass the root cause. It should be 
something like:
nch@Naresh:~/workspace/OpenERP2012/6.1/client/bin$ bzr diff
=== modified file 'bin/widget/view/form_gtk/image.py'
--- bin/widget/view/form_gtk/image.py   2011-09-26 12:53:49 +0000
+++ bin/widget/view/form_gtk/image.py   2012-07-17 11:06:16 +0000
@@ -114,9 +114,11 @@
         if filename:
             self._value = encodestring(file(filename, 'rb').read())
             self.update_img()
+            self.set_value(self._view.model, 
self._view.model.mgroup.mfields[self.attrs['name']])
             if self.has_filename and self.has_filename in 
self._view.model.mgroup.mfields:
                 self._view.model.set({self.has_filename: 
os.path.basename(filename)}, modified=True)

can you please test this and update your MP accordingly ?

Thanks,
Naresh

-- 
https://code.launchpad.net/~openerp-dev/openobject-client/6.1-opw-573333-ado/+merge/100075
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client/6.1-opw-573333-ado.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to