fixups

Author: archmoj

src/traces/parcoords/parcoords.js

@@ -457,15 +457,14 @@ module.exports = function(root, svg, parcoordsLineLayers, styledData, layout, ca
     parcoordsLineLayer
         .each(function(d) {
             if(d.viewModel) {
-                if((!d.lineLayer) ||
-                    (callbacks)) { // recreate in case of having callbacks e.g. restyle. Should we test for callback to be a restyle?
+                if(!d.lineLayer || callbacks) { // recreate in case of having callbacks e.g. restyle. Should we test for callback to be a restyle?
                     d.lineLayer = lineLayerMaker(this, d);
                 } else d.lineLayer.update(d);
 
-                if(d.key) d.viewModel[d.key] = d.lineLayer;
+                if(d.key || d.key === 0) d.viewModel[d.key] = d.lineLayer;
 
-                var setChanged = ((!d.context) || // don't update background
-                                    (callbacks));   // unless there is a callback on the context layer. Should we test the callback?
+                var setChanged = (!d.context || // don't update background
+                                  callbacks);   // unless there is a callback on the context layer. Should we test the callback?
 
                 d.lineLayer.render(d.viewModel.panels, setChanged);
             }

test/jasmine/tests/parcoords_test.js

@@ -913,7 +913,7 @@ describe('parcoords basic use', function() {
 
     });
 
-    it('@gl Calling `Plotly.restyle` with a string path should amend the preexisting parcoords', function(done) {
+    it('@gl Calling `Plotly.restyle` with a string path to colorscale should amend the preexisting parcoords', function(done) {
 
         expect(gd.data.length).toEqual(1);