centralize autorange options defaults

Author: archmoj

src/plots/cartesian/autorange_options_defaults.js

@@ -0,0 +1,11 @@
+'use strict';
+
+module.exports = function handleAutorangeOptionsDefaults(coerce) {
+    var autorangemin = coerce('autorangemin');
+    var autorangemax = coerce('autorangemax');
+
+    if(autorangemin === undefined) coerce('autorangeclipmin');
+    if(autorangemax === undefined) coerce('autorangeclipmax');
+
+    coerce('autorangeinclude');
+};

src/plots/cartesian/axis_defaults.js

@@ -15,6 +15,7 @@ var handleTickLabelDefaults = require('./tick_label_defaults');
 var handlePrefixSuffixDefaults = require('./prefix_suffix_defaults');
 var handleCategoryOrderDefaults = require('./category_order_defaults');
 var handleLineGridDefaults = require('./line_grid_defaults');
+var handleAutorangeOptionsDefaults = require('./autorange_options_defaults');
 var setConvert = require('./set_convert');
 
 var DAY_OF_WEEK = require('./constants').WEEKDAY_PATTERN;
@@ -95,13 +96,7 @@ module.exports = function handleAxisDefaults(containerIn, containerOut, coerce,
     if(autorangeDflt && options.reverseDflt) autorangeDflt = 'reversed';
     var autoRange = coerce('autorange', autorangeDflt);
     if(autoRange) {
-        var autorangemin = coerce('autorangemin');
-        var autorangemax = coerce('autorangemax');
-
-        if(autorangemin === undefined) coerce('autorangeclipmin');
-        if(autorangemax === undefined) coerce('autorangeclipmax');
-
-        coerce('autorangeinclude');
+        handleAutorangeOptionsDefaults(coerce);
 
         if(axType === 'linear' || axType === '-') coerce('rangemode');
     }

src/plots/polar/layout_defaults.js

@@ -13,6 +13,7 @@ var handleTickLabelDefaults = require('../cartesian/tick_label_defaults');
 var handlePrefixSuffixDefaults = require('../cartesian/prefix_suffix_defaults');
 var handleCategoryOrderDefaults = require('../cartesian/category_order_defaults');
 var handleLineGridDefaults = require('../cartesian/line_grid_defaults');
+var handleAutorangeOptionsDefaults = require('../cartesian/autorange_options_defaults');
 var autoType = require('../cartesian/axis_autotype');
 
 var layoutAttributes = require('./layout_attributes');
@@ -79,13 +80,7 @@ function handleDefaults(contIn, contOut, coerce, opts) {
                 var autoRange = coerceAxis('autorange', !axOut.isValidRange(axIn.range));
                 axIn.autorange = autoRange;
                 if(autoRange) {
-                    var autorangemin = coerceAxis('autorangemin');
-                    var autorangemax = coerceAxis('autorangemax');
-
-                    if(autorangemin === undefined) coerceAxis('autorangeclipmin');
-                    if(autorangemax === undefined) coerceAxis('autorangeclipmax');
-
-                    coerceAxis('autorangeinclude');
+                    handleAutorangeOptionsDefaults(coerceAxis);
 
                     if(axType === 'linear' || axType === '-') coerceAxis('rangemode');
                     if(autoRange === 'reversed') axOut._m = -1;