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

Reply via email to