Merge lp:~openerp-dev/openerp-web/trunk-bug-849020-vja into lp:openerp-web

[Vishmita Jadeja (openerp)]

 Need to Update Pager after delete the record.

[Xavier (Open ERP) (xmo-deactivatedaccount)]

Calling self.reload_content means the whole list is going to be removed an reinserted, not just that the pager is going to be updated.

This has the following issues:

* If the list is long, the user will lose his position
* State unsaved (e.g. a line being edited) will be lost
* The list will flicker (all records will disappear and reappear)

Not to mention it is logically nonsensical to do it that way: `reload_content` empties the whole list and reloads it fully, so there is no need to recompute the aggregate results or to remove records one by one from the internal records collection.

review: Disapprove

Unmerged revisions

1055. By vishmita <vja@vja-desktop> on 2011-09-19

[FIX]Counter of History items shows correct number.

=== modified file 'addons/web/static/src/js/view_list.js'
--- addons/web/static/src/js/view_list.js	2011-09-15 09:42:46 +0000
+++ addons/web/static/src/js/view_list.js	2011-09-21 06:03:24 +0000
@@ -455,6 +455,7 @@
             _(ids).each(function (id) {
                 self.records.remove(self.records.get(id));
             });
+            self.reload_content();
             self.compute_aggregates();
         });
     },