openerp-dev-web team mailing list archive

["Ravi Gadhia \(OpenERP\)" <rga@xxxxxxxxxxx>]

Ravi Gadhia (OpenERP) has proposed merging lp:~openerp-dev/openobject-client/trunk-bug-771220-rga into lp:openobject-client.

Requested reviews:
  Naresh(OpenERP) (nch-openerp)
Related bugs:
  Bug #771220 in OpenERP GTK Client: "Incorrect active_id  editable lists"
  https://bugs.launchpad.net/openobject-client/+bug/771220

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client/trunk-bug-771220-rga/+merge/61543
-- 
https://code.launchpad.net/~openerp-dev/openobject-client/trunk-bug-771220-rga/+merge/61543
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client/trunk-bug-771220-rga.

=== modified file 'bin/widget/view/list.py'
--- bin/widget/view/list.py	2011-05-17 12:21:48 +0000
+++ bin/widget/view/list.py	2011-05-19 11:45:58 +0000
@@ -576,14 +576,13 @@
                              current_active_model['state'].get(current_active_model) in states):
                         if self.widget_tree.editable:
                             if current_active_model.validate():
-                                id = self.screen.save_current()
+                                self.screen.save_current()
                             else:
                                common.warning(_('Invalid form, correct red fields !'), _('Error !'),parent=self.window)
                                self.widget_tree.warn('misc-message', _('Invalid form, correct red fields !'), "red")
                                self.screen.display()
                                return False
-                        else:
-                            id = current_active_model.id
+                        id = current_active_model.id
                         current_active_model.get_button_action(self.screen, id, path[1].attrs)
                         self.screen.current_model = None
                         if self.screen.parent and isinstance(self.screen.parent, ModelRecord):