fix

antoinerg · antoinerg · commit 4ce3c0f8e99b · 2019-05-14T12:03:44.000-04:00
diff --git a/src/plots/cartesian/set_convert.js b/src/plots/cartesian/set_convert.js
@@ -613,6 +613,7 @@ module.exports = function setConvert(ax, fullLayout) {
     };
 
     // sort the axis (and all the matching ones) by _initialCategories
+    // returns the indices of the traces affected by the reordering
     ax.sortByInitialCategories = function() {
         var affectedTraces = [];
         var emptyCategories = function() {
@@ -630,19 +631,14 @@ module.exports = function setConvert(ax, fullLayout) {
 
         affectedTraces = affectedTraces.concat(ax._traceIndices);
 
-        var matchGroups = fullLayout._axisMatchGroups;
-
-        if(matchGroups && matchGroups.length) {
-            for(var i = 0; i < matchGroups.length; i++) {
-                var group = matchGroups[i];
-                for(var axId2 in group) {
-                    if(axId === axId2) continue;
-                    var ax2 = fullLayout[axisIds.id2name(axId2)];
-                    ax2._categories = ax._categories;
-                    ax2._categoriesMap = ax._categoriesMap;
-                    affectedTraces = affectedTraces.concat(ax2._traceIndices);
-                }
-            }
+        // Propagate to matching axes
+        var group = ax._matchGroup;
+        for(var axId2 in group) {
+            if(axId === axId2) continue;
+            var ax2 = fullLayout[axisIds.id2name(axId2)];
+            ax2._categories = ax._categories;
+            ax2._categoriesMap = ax._categoriesMap;
+            affectedTraces = affectedTraces.concat(ax2._traceIndices);
         }
         return affectedTraces;
     };
diff --git a/src/plots/plots.js b/src/plots/plots.js
@@ -3001,7 +3001,7 @@ function sortAxisCategoriesByValue(axList, gd) {
             }
 
             // Sort all matching axes
-            affectedTraces = ax.sortByInitialCategories();
+            affectedTraces = affectedTraces.concat(ax.sortByInitialCategories());
         }
     }
     return affectedTraces;

Original file line number	Diff line number	Diff line change
`@@ -3001,7 +3001,7 @@ function sortAxisCategoriesByValue(axList, gd) {`
`3001`	`3001`	`}`
`3002`	`3002`
`3003`	`3003`	`// Sort all matching axes`
`3004`		`- affectedTraces = ax.sortByInitialCategories();`
	`3004`	`+ affectedTraces = affectedTraces.concat(ax.sortByInitialCategories());`
`3005`	`3005`	`}`
`3006`	`3006`	`}`
`3007`	`3007`	`return affectedTraces;`