copy axis type into missing axes

... so that their references are valid during
    handleConstraintDefaults

Author: etpinard

src/plots/cartesian/layout_defaults.js

@@ -189,15 +189,19 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
     var counterAxes = {x: getCounterAxes('x'), y: getCounterAxes('y')};
     // list of all x AND y axis ids
     var allAxisIds = counterAxes.x.concat(counterAxes.y);
-    // list of axis ids that axes in axNames have a reference to,
+    // lookup and list of axis ids that axes in axNames have a reference to,
     // even though they are missing from allAxisIds
+    var missingMatchedAxisIdsLookup = {};
     var missingMatchedAxisIds = [];
 
-    // fill in 'missing' axis list when an axis is set to match an axis
-    // not part of the allAxisIds list
-    function addMissingMatchedAxis(matchesIn) {
+    // fill in 'missing' axis lookup when an axis is set to match an axis
+    // not part of the allAxisIds list, save axis type so that we can propagate
+    // it to the missing axes
+    function addMissingMatchedAxis() {
+        var matchesIn = axLayoutIn.matches;
         if(AX_ID_PATTERN.test(matchesIn) && allAxisIds.indexOf(matchesIn) === -1) {
-            Lib.pushUnique(missingMatchedAxisIds, matchesIn);
+            missingMatchedAxisIdsLookup[matchesIn] = axLayoutIn.type;
+            missingMatchedAxisIds = Object.keys(missingMatchedAxisIdsLookup);
         }
     }
 
@@ -269,7 +273,7 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
 
         coerce('title.standoff');
 
-        addMissingMatchedAxis(axLayoutIn.matches);
+        addMissingMatchedAxis();
 
         axLayoutOut._input = axLayoutIn;
     }
@@ -305,7 +309,8 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
 
         coerce('uirevision', layoutOut.uirevision);
 
-        handleTypeDefaults(axLayoutIn, axLayoutOut, coerce, defaultOptions2);
+        axLayoutOut.type = missingMatchedAxisIdsLookup[axId] || 'linear';
+
         handleAxisDefaults(axLayoutIn, axLayoutOut, coerce, defaultOptions2, layoutOut);
 
         handlePositionDefaults(axLayoutIn, axLayoutOut, coerce, {
@@ -317,7 +322,7 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
 
         coerce('fixedrange');
 
-        addMissingMatchedAxis(axLayoutIn.matches);
+        addMissingMatchedAxis();
 
         axLayoutOut._input = axLayoutIn;
     }

test/jasmine/tests/axes_test.js

@@ -978,6 +978,27 @@ describe('Test axes', function() {
             expect(layoutOut.xaxis4.range).withContext('xaxis4.range').toEqual([0, 1]);
         });
 
+        it('should propagate axis type into *missing* axes', function() {
+            layoutIn = {
+                xaxis2: {type: 'date', matches: 'x'},
+                yaxis: {type: 'category', matches: 'y2'}
+            };
+            layoutOut._subplots.cartesian = ['x2y'];
+            layoutOut._subplots.xaxis = ['x2'];
+            layoutOut._subplots.yaxis = ['y'];
+
+            supplyLayoutDefaults(layoutIn, layoutOut, fullData);
+
+            expect(layoutOut._axisMatchGroups.length).toBe(2);
+            expect(layoutOut._axisMatchGroups).toContain({x: 1, x2: 1});
+            expect(layoutOut._axisMatchGroups).toContain({y: 1, y2: 1});
+
+            expect(layoutOut.xaxis.type).withContext('xaxis.type').toBe('date');
+            expect(layoutOut.xaxis2.type).withContext('xaxis2.type').toBe('date');
+            expect(layoutOut.yaxis.type).withContext('yaxis.type').toBe('category');
+            expect(layoutOut.yaxis2.type).withContext('yaxis2.type').toBe('category');
+        });
+
         it('should adapt default axis ranges to *rangemode*', function() {
             layoutIn = {
                 xaxis: {rangemode: 'tozero'},