> I don't understand the purpose of this fix, could you explain?
Yes sure.
> Also why you had to rely on ids?
As you can see code current code *var container =
this.$el.find(".oe_view_manager_view_" + view_type);*
No type of logic implement to find container to render view.It just find
*oe_view_manager_view_list* and render there.But if there are many list view
available than it also render to first list view element(not right one).
That is an issue, to reproduce description is given above.
>From container after finding we can work on *this.$el* but to find container
>we must give unique identity to it.
In most of cases we have first list/kanaban view than other views.But in this
issue we render first form view from that we switch to list view,but in form
view we have many one2many list view.As above code its find list view of
hr_timesheet_sheet.sheet (under detail tab) and render there main
hr_timesheet_sheet's list view.
In solution given ids to find out accurate container.As in future if there are
number of list/form/kanban views in that is also works.
To see video : https://docs.google.com/open?id=0By4GWTWr6HDNckhiQ2lYbDVTbnM
--
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-timesheet-vme/+merge/134853
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/trunk-timesheet-vme.
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : openerp-dev-gtk@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
