Merge lp:~openerp-commiter/openobject-client/many2many_dbl_click_open_new_tab-5.0 into lp:openobject-client/5.0

[Nicolas DS]

[Nicolas DS (zyphos)]

Double click on a many2many widget open a new tab with the clicked resource in a form. Like Web client does.

[Naresh(OpenERP) (nch-openerp)]

Hello Nicolas DS,

Thank you for your Input.

I had reviewed and made a bit modification like instead of opening in a new tab allow the user to open a specific record (with a dialog as like as the one2many field) for modification.

and has been merge in trunk client with revision info as 1172 <email address hidden>

Thanks

review: Approve

[Nicolas DS (zyphos)]

Hi,

I was hesitating of opening it in new window, but the tab pros is that you have a tree view too.

So in your merge I see that you set the domain, but it is not needed if you open it in new window, because only one record (resource) is shown.

Another thing,
the merge proposal for the "trunk" version is here:
https://code.launchpad.net/~openerp-commiter/openobject-client/many2many_dbl_click_open_new_tab-trunk/+merge/19971

this is the merge proposal for the stable 5.0 version. Could you review merge in 5.0 too ?

thank you for the merge.

Regards,
Nicolas DS

[Naresh(OpenERP) (nch-openerp)]

Hello Nicolas DS ,

Ok I have changed the domain instead of Ids the domain should be from the default domain passed either from xml or .py.

and regarding stable merge we cannot merge this right now in stable branch as this is new improvement and will be included in the next stable version.

Thanks,

[Naresh(OpenERP) (nch-openerp)]

hi,
It will be include in next release for stable version. We included it in Trunk verison.

thanks

review: Disapprove

Unmerged revisions

1049. By Nicolas DS on 2010-02-23

[FIX] Many2many widget action open a new tab with the selected resource like Web Client does

=== modified file 'bin/widget/view/form_gtk/many2many.py'
--- bin/widget/view/form_gtk/many2many.py	2009-07-23 11:37:09 +0000
+++ bin/widget/view/form_gtk/many2many.py	2010-02-23 14:13:15 +0000
@@ -34,6 +34,7 @@
 import interface
 
 import rpc
+import service
 
 from modules.gui.window.win_search import win_search
 
@@ -73,7 +74,8 @@
         scroll.set_shadow_type(gtk.SHADOW_NONE)
 
         self.screen = Screen(attrs['relation'], view_type=['tree'],
-                views_preload=attrs.get('views', {}))
+                views_preload=attrs.get('views', {}),
+                row_activate=self.row_activate)
         scroll.add_with_viewport(self.screen.widget)
         self.widget.pack_start(scroll, expand=True, fill=True)
 
@@ -162,6 +164,12 @@
     def set_value(self, model, model_field):
         model_field.set_client(model, [x.id for x in self.screen.models.models])
 
+    def row_activate(self, screen):
+        """ Open new tab with the resource in it """
+        gui_window = service.LocalService('gui.window')
+        domain = [('id','in', screen.ids_get())]
+        gui_window.create(None, screen.name, screen.id_get(), domain, u'form', 
+                          screen.window, screen.context, u'form,tree')
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: