Foram Katharotiya (OpenERP) has proposed merging
lp:~openerp-dev/openerp-web/trunk-bug-1098508-fka into lp:openerp-web.
Requested reviews:
Xavier (Open ERP) (xmo)
Related bugs:
Bug #1098508 in OpenERP Web: "[Project] Issues issue: "2" number in pager
arrows while there is only one record"
https://bugs.launchpad.net/openerp-web/+bug/1098508
For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-1098508-fka/+merge/144489
Hello,
- improve code as per your suggestion.
Thanks,
FKA
--
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-1098508-fka/+merge/144489
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/trunk-bug-1098508-fka.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js 2013-01-07 16:18:07 +0000
+++ addons/web/static/src/js/view_form.js 2013-01-23 13:06:23 +0000
@@ -248,7 +248,10 @@
do_load_state: function(state, warm) {
if (state.id && this.datarecord.id != state.id) {
if (!this.dataset.get_id_index(state.id)) {
- this.dataset.ids.push(state.id);
+ if (this.dataset.ids.indexOf(state.id) == -1)
+ {
+ this.dataset.ids.push(state.id);
+ }
}
this.dataset.select_id(state.id);
if (warm) {
_______________________________________________
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
