only return label from the helper - set the label of dummy root to blank

Author: archmoj

src/traces/sunburst/calc.js

@@ -128,7 +128,8 @@ exports.calc = function(gd, trace) {
         cd.unshift({
             hasMultipleRoots: true,
             id: dummyId,
-            pid: ''
+            pid: '',
+            label: ''
         });
     }
 

src/traces/sunburst/helpers.js

@@ -52,8 +52,7 @@ exports.getPtId = function(pt) {
 };
 
 exports.getPtLabel = function(pt) {
-    var label = pt.data.data.label;
-    return label === undefined ? pt.data.data.pid : label;
+    return pt.data.data.label;
 };
 
 exports.getValue = function(d) {

test/jasmine/tests/sunburst_test.js

@@ -235,7 +235,7 @@ describe('Test sunburst calc:', function() {
 
         expect(extract('id')).toEqual(['dummy', 'A', 'B', 'b']);
         expect(extract('pid')).toEqual(['', 'dummy', 'dummy', 'B']);
-        expect(extract('label')).toEqual([undefined, 'A', 'B', 'b']);
+        expect(extract('label')).toEqual(['', 'A', 'B', 'b']);
     });
 
     it('should compute hierarchy values', function() {

test/jasmine/tests/treemap_test.js

@@ -346,7 +346,7 @@ describe('Test treemap calc:', function() {
 
         expect(extract('id')).toEqual(['dummy', 'A', 'B', 'b']);
         expect(extract('pid')).toEqual(['', 'dummy', 'dummy', 'B']);
-        expect(extract('label')).toEqual([undefined, 'A', 'B', 'b']);
+        expect(extract('label')).toEqual(['', 'A', 'B', 'b']);
     });
 
     it('should compute hierarchy values', function() {