fix: exception thrown for charts without a x-axis (#36)

* fix: pie chart exception * fix: incoporate krist's feedbacl
apache-superset · Apr 1, 2019 · 04cae86 · 04cae86
1 parent 2dfa077
commit 04cae86
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/packages/superset-ui-legacy-preset-chart-nvd3/src/NVD3Vis.js b/packages/superset-ui-legacy-preset-chart-nvd3/src/NVD3Vis.js
@@ -495,15 +495,17 @@ function nvd3Vis(element, props) {
     if (chart.x2Axis && chart.x2Axis.tickFormat) {
       chart.x2Axis.tickFormat(xAxisFormatter);
     }
-    const isXAxisString = isVizTypes(['dist_bar', 'box_plot']);
-    if (!isXAxisString && chart.xAxis && chart.xAxis.tickFormat) {
-      chart.xAxis.tickFormat(xAxisFormatter);
-    } else {
-      chart.xAxis.tickFormat(d =>
-        d.length > MAX_NO_CHARACTERS_IN_LABEL
-          ? `${d.substring(0, MAX_NO_CHARACTERS_IN_LABEL)}…`
-          : d,
-      );
+    if (chart.xAxis && chart.xAxis.tickFormat) {
+      const isXAxisString = isVizTypes(['dist_bar', 'box_plot']);
+      if (isXAxisString) {
+        chart.xAxis.tickFormat(d =>
+          d.length > MAX_NO_CHARACTERS_IN_LABEL
+            ? `${d.substring(0, MAX_NO_CHARACTERS_IN_LABEL)}…`
+            : d,
+        );
+      } else {
+        chart.xAxis.tickFormat(xAxisFormatter);
+      }
     }
 
     let yAxisFormatter = getTimeOrNumberFormatter(yAxisFormat);