[FIX] web: edit x2many editable list view after resequence

In an embedded editable list view with the widget `handle`, resequencing a
record and try to edit it afterwards won't work. The selected record isn't the
correct one, as if the resequencing had no impact.

To make this work, the list datapoint needs to be ordered by the field with the
widget `handle` ; if it isn't, resequencing the records won't change the x2many
data order but the list view expects the same order.

1 files changed, 13 insertions, 3 deletions

diff --git a/addons/web/static/src/js/services/data_manager.js b/addons/web/static/src/js/services/data_manager.js
index 105d7f30d69..4bf1748fbab 100644
--- a/addons/web/static/src/js/services/data_manager.js
+++ b/addons/web/static/src/js/services/data_manager.js
@@ -336,20 +336,30 @@ return core.Class.extend({
                 attrs.mode = mode;
                 if (mode in attrs.views) {
                     var view = attrs.views[mode];
-                    // process the default_order, which is like 'name,id desc'
-                    // but we need it like [{name: 'name', asc: true}, {name: 'id', asc: false}]
                     var defaultOrder = view.arch.attrs.default_order;
                     if (defaultOrder) {
+                        // process the default_order, which is like 'name,id desc'
+                        // but we need it like [{name: 'name', asc: true}, {name: 'id', asc: false}]
                         attrs.orderedBy = _.map(defaultOrder.split(','), function (order) {
                             order = order.trim().split(' ');
                             return {name: order[0], asc: order[1] !== 'desc'};
                         });
+                    } else {
+                        // if there is a field with widget `handle`, the x2many
+                        // needs to be ordered by this field to correctly display
+                        // the records
+                        var handleField = _.find(view.arch.children, function (child) {
+                            return child.attrs && child.attrs.widget === 'handle';
+                        });
+                        if (handleField) {
+                            attrs.orderedBy = [{name: handleField.attrs.name, asc: true}];
+                        }
                     }
                     // detect editables list has they behave differently with respect
                     // to the sorting (changes are not sorted directly)
                     if (mode === 'list' && view.arch.attrs.editable) {
                          attrs.keepChangesUnsorted = true;
-                     }
+                    }
                 }
             }
             if (attrs.Widget.prototype.fieldsToFetch) {