Merge lp:~openerp-dev/openerp-web/7.0-opw-589888-ado into lp:openerp-web/7.0

[Amit Dodiya (OpenERP)]

Hello,

[FIX] view_list: when we have more than 80 record in list and we create new record and switch the views like form->list->form gives traceback

Steps:
1). Goto Sale--> Address Book--> Localization--> Countries
2). Create new country, now switch to list view and again switch to form view
This gives you blank page
Same issue happens in account creation, in account creation this gives traceback.

Regards,
Amit Dodiya

[Christophe Matthieu (OpenERP) (chm-openerp)]

Hi,
It's a wrong fix.
Forgot index value in all case.
I fixed this in 7.0

review: Disapprove

Unmerged revisions

3868. By Amit Dodiya (OpenERP) on 2013-03-26

[FIX] view_list: when we have more than 80 record in list and we create new record and switch the view to form it gives traceback due to index is set to 81 instead of 0

=== modified file 'addons/web/static/src/js/view_list.js'
--- addons/web/static/src/js/view_list.js	2013-03-22 16:23:10 +0000
+++ addons/web/static/src/js/view_list.js	2013-03-26 17:13:30 +0000
@@ -503,13 +503,7 @@
         var reloaded = $.Deferred();
         this.$el.find('.oe_list_content').append(
             this.groups.render(function () {
-                if (self.dataset.index == null) {
-                    var has_one = false;
-                    self.records.each(function () { has_one = true; });
-                    if (has_one) {
-                        self.dataset.index = 0;
-                    }
-                }
+                self.dataset.index = self.records.length ? 0 : null;
                 self.compute_aggregates();
                 reloaded.resolve();
             }));