Cleanup line defaults logic

Author: Jon M. Mease

src/traces/parcats/calc.js

@@ -13,7 +13,6 @@
 var wrap = require('../../lib/gup').wrap;
 var hasColorscale = require('../../components/colorscale/has_colorscale');
 var colorscaleCalc = require('../../components/colorscale/calc');
-var parcatConstants = require('./constants');
 var filterUnique = require('../../lib/filter_unique.js');
 var Drawing = require('../../components/drawing');
 var Lib = require('../../lib');
@@ -101,9 +100,7 @@ module.exports = function calc(gd, trace) {
     // Build color generation function
     function getMarkerColorInfo(index) {
         var value;
-        if(!line) {
-            value = parcatConstants.defaultColor;
-        } else if(Lib.isArrayOrTypedArray(line.color)) {
+        if(Lib.isArrayOrTypedArray(line.color)) {
             value = line.color[index % line.color.length];
         } else {
             value = line.color;

src/traces/parcats/constants.js

@@ -19,10 +19,7 @@ var mergeLength = require('../parcoords/merge_length');
 
 function handleLineDefaults(traceIn, traceOut, defaultColor, layout, coerce) {
 
-    if(traceIn.line) {
-        coerce('line.shape');
-    }
-
+    coerce('line.shape');
     var lineColor = coerce('line.color', defaultColor);
     if(hasColorscale(traceIn, 'line') && Lib.isArrayOrTypedArray(lineColor)) {
         if(lineColor.length) {

src/traces/parcats/defaults.js

@@ -1465,12 +1465,7 @@ function createParcatsViewModel(graphDiv, layout, wrappedParcatsModel) {
 
     // Handle path shape
     // -----------------
-    var pathShape;
-    if(trace.line && trace.line.shape) {
-        pathShape = trace.line.shape;
-    } else {
-        pathShape = 'linear';
-    }
+    var pathShape = trace.line.shape;
 
     // Handle hover info
     // -----------------