Add bounds checks and fix 12-hour format in CompareDashboard chart functions
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -467,6 +467,7 @@ function getHorizontalBarHeight(chartData) {
|
|||||||
* @returns {Array} [chartData, chartOptions] tuple.
|
* @returns {Array} [chartData, chartOptions] tuple.
|
||||||
*/
|
*/
|
||||||
function getLineChart(chartData, content, yUnit) {
|
function getLineChart(chartData, content, yUnit) {
|
||||||
|
if (!chartData?.data || chartData.data.length < 2) return [];
|
||||||
let datasetsPrimary;
|
let datasetsPrimary;
|
||||||
let datasetsSecondary;
|
let datasetsSecondary;
|
||||||
const minX = chartData.x_axis.min;
|
const minX = chartData.x_axis.min;
|
||||||
@@ -477,7 +478,7 @@ function getLineChart(chartData, content, yUnit) {
|
|||||||
let primeVueSetData = {};
|
let primeVueSetData = {};
|
||||||
let primeVueSetOption = {};
|
let primeVueSetOption = {};
|
||||||
const getMoment = (time) => {
|
const getMoment = (time) => {
|
||||||
return moment(time).format("YYYY/M/D hh:mm:ss");
|
return moment(time).format("YYYY/M/D HH:mm:ss");
|
||||||
};
|
};
|
||||||
const getSimpleTimeLabel = simpleTimeLabel;
|
const getSimpleTimeLabel = simpleTimeLabel;
|
||||||
|
|
||||||
@@ -619,6 +620,7 @@ function getLineChart(chartData, content, yUnit) {
|
|||||||
* @returns {Array} [chartData, chartOptions] tuple.
|
* @returns {Array} [chartData, chartOptions] tuple.
|
||||||
*/
|
*/
|
||||||
function getBarChart(chartData, content, caller) {
|
function getBarChart(chartData, content, caller) {
|
||||||
|
if (!chartData?.data || chartData.data.length < 2) return [];
|
||||||
const getMoment = (time) => moment(time).format("YYYY/MM/DD");
|
const getMoment = (time) => moment(time).format("YYYY/MM/DD");
|
||||||
const labelPrimary = chartData.data[0].label;
|
const labelPrimary = chartData.data[0].label;
|
||||||
const labelSecondary = chartData.data[1].label;
|
const labelSecondary = chartData.data[1].label;
|
||||||
@@ -938,6 +940,7 @@ function customizeScaleChartOptionTicks(scaleObjectToAlter, ticksOfXAxis) {
|
|||||||
* @returns {Array} [chartData, chartOptions] tuple.
|
* @returns {Array} [chartData, chartOptions] tuple.
|
||||||
*/
|
*/
|
||||||
function getLineChart0(chartData, content, yUnit) {
|
function getLineChart0(chartData, content, yUnit) {
|
||||||
|
if (!chartData?.data || chartData.data.length < 2) return [];
|
||||||
let datasetsPrimary;
|
let datasetsPrimary;
|
||||||
let datasetsSecondary;
|
let datasetsSecondary;
|
||||||
const minX = chartData.x_axis.min;
|
const minX = chartData.x_axis.min;
|
||||||
@@ -948,7 +951,7 @@ function getLineChart0(chartData, content, yUnit) {
|
|||||||
let primeVueSetData = {};
|
let primeVueSetData = {};
|
||||||
let primeVueSetOption = {};
|
let primeVueSetOption = {};
|
||||||
const getMoment = (time) => {
|
const getMoment = (time) => {
|
||||||
return moment(time).format("YYYY/M/D hh:mm:ss");
|
return moment(time).format("YYYY/M/D HH:mm:ss");
|
||||||
};
|
};
|
||||||
const getSimpleTimeLabel = simpleTimeLabel;
|
const getSimpleTimeLabel = simpleTimeLabel;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user