Merge pull request #1133 from plotly/fix-relayout-array

quick fix - allow relayout array elements

Author: alexcjohnson

src/plot_api/plot_api.js

@@ -1959,6 +1959,7 @@ function _relayout(gd, aobj) {
         }
         // alter gd.layout
         else {
+            var pp1 = String(p.parts[1] || '');
             // check whether we can short-circuit a full redraw
             // 3d or geo at this point just needs to redraw.
             if(p.parts[0].indexOf('scene') === 0) flags.doplot = true;
@@ -1973,17 +1974,17 @@ function _relayout(gd, aobj) {
             else if(ai.indexOf('title') !== -1) flags.doticks = true;
             else if(p.parts[0].indexOf('bgcolor') !== -1) flags.dolayoutstyle = true;
             else if(p.parts.length > 1 &&
-                    Lib.containsAny(p.parts[1], ['tick', 'exponent', 'grid', 'zeroline'])) {
+                    Lib.containsAny(pp1, ['tick', 'exponent', 'grid', 'zeroline'])) {
                 flags.doticks = true;
             }
             else if(ai.indexOf('.linewidth') !== -1 &&
                     ai.indexOf('axis') !== -1) {
                 flags.doticks = flags.dolayoutstyle = true;
             }
-            else if(p.parts.length > 1 && p.parts[1].indexOf('line') !== -1) {
+            else if(p.parts.length > 1 && pp1.indexOf('line') !== -1) {
                 flags.dolayoutstyle = true;
             }
-            else if(p.parts.length > 1 && p.parts[1] === 'mirror') {
+            else if(p.parts.length > 1 && pp1 === 'mirror') {
                 flags.doticks = flags.dolayoutstyle = true;
             }
             else if(ai === 'margin.pad') {

test/jasmine/tests/plot_api_test.js

@@ -148,6 +148,21 @@ describe('Test plot api', function() {
                 })
                 .then(done);
         });
+
+        it('can set items in array objects', function(done) {
+            Plotly.plot(gd, [{ x: [1, 2, 3], y: [1, 2, 3] }])
+                .then(function() {
+                    return Plotly.relayout(gd, {rando: [1, 2, 3]});
+                })
+                .then(function() {
+                    expect(gd.layout.rando).toEqual([1, 2, 3]);
+                    return Plotly.relayout(gd, {'rando[1]': 45});
+                })
+                .then(function() {
+                    expect(gd.layout.rando).toEqual([1, 45, 3]);
+                })
+                .then(done);
+        });
     });
 
     describe('Plotly.restyle', function() {