Merge pull request #73 from sgratzl/release/v4.3.0

Release v4.3.0

Author: sgratzl

package.json

@@ -1,7 +1,7 @@
 {
   "name": "chartjs-chart-error-bars",
   "description": "Chart.js module for charting error bars",
-  "version": "4.2.4",
+  "version": "4.3.0",
   "author": {
     "name": "Samuel Gratzl",
     "email": "samu@sgratzl.com",

src/controllers/BarWithErrorBarsController.spec.ts

@@ -1,13 +1,14 @@
-import { registry, LinearScale, CategoryScale } from 'chart.js';
+import { registry, LinearScale, CategoryScale, TimeScale } from 'chart.js';
 import createChart from '../__tests__/createChart';
 import { BarWithErrorBarsController } from './BarWithErrorBarsController';
 import { BarWithErrorBar } from '../elements';
+import 'chartjs-adapter-date-fns';
 
 describe('bar', () => {
   beforeAll(() => {
     registry.addControllers(BarWithErrorBarsController);
     registry.addElements(BarWithErrorBar);
-    registry.addScales(LinearScale, CategoryScale);
+    registry.addScales(LinearScale, CategoryScale, TimeScale);
   });
   test('default', () => {
     return createChart({
@@ -43,4 +44,49 @@ describe('bar', () => {
       },
     }).toMatchImageSnapshot();
   });
+  test('time scale', () => {
+    return createChart({
+      type: BarWithErrorBarsController.id,
+      data: {
+        labels: ['A', 'B'],
+        datasets: [
+          {
+            data: [
+              {
+                x: '2017-12-25',
+                y: 4,
+                yMin: 1,
+                yMax: 6,
+              },
+              {
+                x: '2018-12-25',
+                y: 2,
+                yMin: 1,
+                yMax: 4,
+              },
+              {
+                x: '2019-12-25',
+                y: 3,
+                yMin: 2,
+                yMax: 5,
+              },
+            ],
+          },
+        ],
+      },
+      options: {
+        scales: {
+          x: {
+            type: 'time',
+            display: false,
+            min: '2017-01-01',
+            max: '2020-01-01',
+          },
+          y: {
+            display: false,
+          },
+        },
+      },
+    }).toMatchImageSnapshot();
+  });
 });

src/controllers/BarWithErrorBarsController.ts

@@ -16,7 +16,7 @@
   CartesianScaleTypeRegistry,
 } from 'chart.js';
 import { merge } from 'chart.js/helpers';
-import { calculateScale } from './utils';
+import { calculateScale, isNumericScale } from './utils';
 import type { IErrorBarOptions } from '../elements/render';
 import { BarWithErrorBar } from '../elements';
 import { generateBarTooltip } from './tooltip';
@@ -67,7 +67,13 @@ export class BarWithErrorBarsController extends BarController {
     // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     parseErrorNumberData(parsed, meta.vScale!, data, start, count);
     // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-    parseErrorLabelData(parsed, meta.iScale!, start, count);
+    const iScale = meta.iScale as Scale;
+    const hasNumberIScale = isNumericScale(iScale);
+    if (hasNumberIScale) {
+      parseErrorNumberData(parsed, meta.iScale!, data, start, count);
+    } else {
+      parseErrorLabelData(parsed, meta.iScale!, start, count);
+    }
   }
 
   /**

src/controllers/LineWithErrorBarsController.ts

@@ -16,7 +16,7 @@
   CartesianScaleTypeRegistry,
 } from 'chart.js';
 import { merge } from 'chart.js/helpers';
-import { calculateScale } from './utils';
+import { calculateScale, isNumericScale } from './utils';
 import type { IErrorBarOptions } from '../elements/render';
 import { PointWithErrorBar } from '../elements';
 import { generateBarTooltip } from './tooltip';
@@ -31,8 +31,6 @@ import {
 } from './base';
 import patchController from './patchController';
 
-const NUMERIC_SCALE_TYPES = ['linear', 'logarithmic', 'time', 'timeseries'];
-
 export class LineWithErrorBarsController extends LineController {
   /**
    * @hidden
@@ -73,7 +71,7 @@ export class LineWithErrorBarsController extends LineController {
     parseErrorNumberData(parsed, meta.vScale!, data, start, count);
     // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     const iScale = meta.iScale as Scale;
-    const hasNumberIScale = NUMERIC_SCALE_TYPES.includes(iScale.type);
+    const hasNumberIScale = isNumericScale(iScale);
     if (hasNumberIScale) {
       parseErrorNumberData(parsed, meta.iScale!, data, start, count);
     } else {
@@ -111,7 +109,7 @@ export class LineWithErrorBarsController extends LineController {
     );
 
     const iScale = this._cachedMeta.iScale as Scale;
-    const hasNumberIScale = NUMERIC_SCALE_TYPES.includes(iScale.type);
+    const hasNumberIScale = isNumericScale(iScale);
     if (hasNumberIScale) {
       calculateScale(
         properties,

src/controllers/__image_snapshots__/bar-with-error-bars-controller-spec-ts-bar-time-scale-1-snap.png

@@ -1,4 +1,4 @@
-import type { LinearScale, RadialLinearScale } from 'chart.js';
+import type { LinearScale, RadialLinearScale, Scale } from 'chart.js';
 import type { IErrorBarRDataPoint, IErrorBarXYDataPoint } from './base';
 
 export const allModelKeys = ['xMin', 'xMax', 'yMin', 'yMax'];
@@ -63,3 +63,9 @@ export function calculatePolarScale(
     }
   }
 }
+
+const NUMERIC_SCALE_TYPES = ['linear', 'logarithmic', 'time', 'timeseries'];
+
+export function isNumericScale(scale: Scale): boolean {
+  return NUMERIC_SCALE_TYPES.includes(scale.type);
+}