Code review comment for lp:~openerp-dev/openerp-web/7.0-timesheet-vme

Revision history for this message
Vidhin Mehta (OpenERP) (vme-openerp) wrote :

> Could you elaborate your first sentence ? I don't understand what this branch
> is supposed to fix.

>> In class *instance.web.ViewManager* in that method *do_create_view* for *container* variable.

>> container holds list of element which is find by (".oe_view_manager_view_" + view_type).

>> when appending controller to container it will select first element of *container*,not accurate element.

>> So, in solution when creating hook given unique_id,so it will helpfull while finding accurate container.

>> From menu when form view load first (and contain one2manylist) than switching to list,it shows blank.because
 list view rendered on wrong container.

« Back to merge proposal