Review: Disapprove

As far as I can see, your change is useless.

The method fields_view_get() in the ORM never uses search() on views.  Instead 
it makes direct sql queries, and uses the method get_inheriting_views_arch() to 
retrieve the inheriting views, and get_inheriting_views_arch() also makes 
direct sql queries.

I have modified the method get_inheriting_views_arch() on views to filter views 
based on groups.  It does the job for inheriting views.

Thanks for your attempt, anyway.

-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-ir_ui_view_access_rights-mdi/+merge/121858
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/trunk-editable_list.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to