On 18-04-11 09:31, Xavier (Open ERP) wrote: > Review: Needs Information > Do we really need to append "action" to the fields sequence? Wouldn't using > ``.get`` on ``item`` (yielding ``not item.get('action')`` instead of ``not > item['action']``) work just as well?
Hi, thanks for the review. Unfortunately, item.get('action') does not work because the ir_ui_menu.action is not retrieved at all when expanding nested menus. Instead, record['action'] is always "/openerp/tree/open?model=ir.ui.menu&id=$ID". Only when the user clicks a menu item, a lookup on the menu's actual ir_action is performed based on this URI which triggers the 'No action defined' on menu's without ir_ui_menu.action. Regards, Stefan. -- Therp - Maatwerk in open ontwikkeling Stefan Rijnhart - Ontwerp en implementatie mail: ste...@therp.nl tel: +31 (0) 614478606 web: http://therp.nl https://code.launchpad.net/~openerp-community/openobject-client-web/stefan-therp_lp759896-6.0/+merge/57552 Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-client-web/stefan-therp_lp759896-6.0. _______________________________________________ Mailing list: https://launchpad.net/~openerp-community Post to : openerp-community@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community More help : https://help.launchpad.net/ListHelp