openerp-dev-web team mailing list archive

["Naresh\(OpenERP\)" <nch@xxxxxxxxxxx>]

Naresh(OpenERP) has proposed merging lp:~openerp-dev/openobject-server/trunk-bug-shortcuts_not_updating-nch into lp:~openerp-dev/openobject-server/trunk-bugfixes-Ind.

Requested reviews:
  OpenERP R&D Team (openerp-dev)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-shortcuts_not_updating-nch/+merge/58440

Hello,

The shortcuts created by the user through client and when he modifies the shortcut for eg: changes name to "XYZ" then the modification were not shown to the user when he reloads. This proposal contains the fix for this.


Thanks !
nch(OpenERP)
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-shortcuts_not_updating-nch/+merge/58440
Your team OpenERP R&D Team is requested to review the proposed merge of lp:~openerp-dev/openobject-server/trunk-bug-shortcuts_not_updating-nch into lp:~openerp-dev/openobject-server/trunk-bugfixes-Ind.

=== modified file 'openerp/addons/base/ir/ir_ui_view.py'
--- openerp/addons/base/ir/ir_ui_view.py	2011-04-06 10:05:07 +0000
+++ openerp/addons/base/ir/ir_ui_view.py	2011-04-20 06:23:29 +0000
@@ -196,13 +196,10 @@
 
     def get_sc(self, cr, uid, user_id, model='ir.ui.menu', context=None):
         ids = self.search(cr, uid, [('user_id','=',user_id),('resource','=',model)], context=context)
-        results = self.read(cr, uid, ids, ['res_id'], context=context)
-        name_map = dict(self.pool.get(model).name_get(cr, uid, [x['res_id'] for x in results], context=context))
+        results = self.read(cr, uid, ids, ['res_id','name'], context=context)
+        available_menus = self.pool.get(model).search(cr, uid, [], context=context)
         # Make sure to return only shortcuts pointing to exisintg menu items.
-        filtered_results = filter(lambda result: result['res_id'] in name_map, results)
-        for result in filtered_results:
-            result.update(name=name_map[result['res_id']])
-        return filtered_results
+        return filter(lambda result: result['res_id'] in available_menus, results)
 
     _order = 'sequence,name'
     _defaults = {