From: Jukka Kurkela <jukka.kurkela@gmail.com>
Date: Tue, 2 Mar 2021 22:00:03 +0000 (+0200)
Subject: Time: limit offset to sane values, gen >= 2 ticks (#8560)
X-Git-Tag: v3.0.0-beta.13~9
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=08b76b0efd6088d865e14b8a7edee07a4332c97a;p=thirdparty%2FChart.js.git

Time: limit offset to sane values, gen >= 2 ticks (#8560)
---

diff --git a/src/scales/scale.time.js b/src/scales/scale.time.js
index 04df0ae1d..8e400db3f 100644
--- a/src/scales/scale.time.js
+++ b/src/scales/scale.time.js
@@ -1,6 +1,6 @@
 import adapters from '../core/core.adapters';
 import {isFinite, isNullOrUndef, mergeIf, valueOrDefault} from '../helpers/helpers.core';
-import {toRadians, isNumber} from '../helpers/helpers.math';
+import {toRadians, isNumber, _limitValue} from '../helpers/helpers.math';
 import Scale from '../core/core.scale';
 import {_arrayUnique, _filterBetween, _lookup} from '../helpers/helpers.collection';
 
@@ -379,6 +379,8 @@ export default class TimeScale extends Scale {
         end = (last - me.getDecimalForValue(timestamps[timestamps.length - 2])) / 2;
       }
     }
+    start = _limitValue(start, 0, 0.25);
+    end = _limitValue(end, 0, 0.25);
 
     me._offsets = {start, end, factor: 1 / (start + 1 + end)};
   }
@@ -404,7 +406,7 @@ export default class TimeScale extends Scale {
     const hasWeekday = isNumber(weekday) || weekday === true;
     const ticks = {};
     let first = min;
-    let time;
+    let time, count;
 
     // For 'week' unit, handle the first day of week option
     if (hasWeekday) {
@@ -420,11 +422,11 @@ export default class TimeScale extends Scale {
     }
 
     const timestamps = options.ticks.source === 'data' && me.getDataTimestamps();
-    for (time = first; time < max; time = +adapter.add(time, stepSize, minor)) {
+    for (time = first, count = 0; time < max; time = +adapter.add(time, stepSize, minor), count++) {
       addTick(ticks, time, timestamps);
     }
 
-    if (time === max || options.bounds === 'ticks') {
+    if (time === max || options.bounds === 'ticks' || count === 1) {
       addTick(ticks, time, timestamps);
     }
 
diff --git a/test/fixtures/controller.bar/bar-thickness-flex-offset.png b/test/fixtures/controller.bar/bar-thickness-flex-offset.png
index e20cc4eb4..59171e08c 100644
Binary files a/test/fixtures/controller.bar/bar-thickness-flex-offset.png and b/test/fixtures/controller.bar/bar-thickness-flex-offset.png differ
diff --git a/test/fixtures/scale.time/offset-with-no-ticks.js b/test/fixtures/scale.time/offset-with-no-ticks.js
new file mode 100644
index 000000000..010dd519b
--- /dev/null
+++ b/test/fixtures/scale.time/offset-with-no-ticks.js
@@ -0,0 +1,81 @@
+const data = {
+  datasets: [
+    {
+      data: [
+        {
+          x: moment('15/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 55
+        },
+        {
+          x: moment('18/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 10
+        },
+        {
+          x: moment('19/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 15
+        }
+      ],
+      backgroundColor: 'blue'
+    },
+    {
+      data: [
+        {
+          x: moment('15/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 6
+        },
+        {
+          x: moment('18/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 11
+        },
+        {
+          x: moment('19/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 16
+        }
+      ],
+      backgroundColor: 'green',
+    },
+    {
+      data: [
+        {
+          x: moment('15/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 7
+        },
+        {
+          x: moment('18/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 12
+        },
+        {
+          x: moment('19/10/2020', 'DD/MM/YYYY').valueOf(),
+          y: 17
+        }
+      ],
+      backgroundColor: 'red',
+    }
+  ]
+};
+
+module.exports = {
+  description: 'https://github.com/chartjs/Chart.js/issues/7991',
+  config: {
+    type: 'bar',
+    data,
+    options: {
+      scales: {
+        x: {
+          type: 'time',
+          // offset: false,
+          time: {
+            unit: 'month',
+          },
+        },
+        y: {
+          display: false
+        }
+      }
+    }
+  },
+  options: {
+    spriteText: true,
+    canvas: {width: 256, height: 128}
+  }
+};
diff --git a/test/fixtures/scale.time/offset-with-no-ticks.png b/test/fixtures/scale.time/offset-with-no-ticks.png
new file mode 100644
index 000000000..9bea4ec59
Binary files /dev/null and b/test/fixtures/scale.time/offset-with-no-ticks.png differ