early return from sendDataToCloud when using blank plotlyServerURL default

Author: archmoj

src/plot_api/plot_config.js

@@ -34,8 +34,9 @@ var configAttributes = {
         valType: 'string',
         dflt: '',
         description: [
-            'Sets base URL for the \'Edit in Chart Studio\' (aka sendDataToCloud) mode bar button',
-            'and the showLink/sendData on-graph link'
+            'When set it determines base URL for',
+            'the \'Edit in Chart Studio\' (aka sendDataToCloud) mode bar button',
+            'and the showLink/sendData on-graph link.'
         ].join(' ')
     },
 

src/plots/plots.js

@@ -215,9 +215,10 @@ function positionPlayWithData(gd, container) {
 }
 
 plots.sendDataToCloud = function(gd) {
-    gd.emit('plotly_beforeexport');
-
     var baseUrl = (window.PLOTLYENV || {}).BASE_URL || gd._context.plotlyServerURL;
+    if(!baseUrl) return;
+
+    gd.emit('plotly_beforeexport');
 
     var hiddenformDiv = d3.select(gd)
         .append('div')

test/jasmine/tests/config_test.js

@@ -509,8 +509,7 @@ describe('config argument', function() {
                 expect(gd._context.plotlyServerURL).toBe('');
 
                 Plotly.Plots.sendDataToCloud(gd);
-                expect(form.action.substring(0, 17)).toBe('http://localhost:');
-                expect(form.method).toBe('post');
+                expect(form).toBe(undefined);
             })
             .catch(failTest)
             .then(done);