]> git.ipfire.org Git - thirdparty/Chart.js.git/commitdiff
Minor cleanup (#6362)
authorBen McCann <322311+benmccann@users.noreply.github.com>
Tue, 13 Aug 2019 00:36:46 +0000 (17:36 -0700)
committerEvert Timberg <evert.timberg+github@gmail.com>
Tue, 13 Aug 2019 00:36:46 +0000 (20:36 -0400)
src/controllers/controller.bar.js
src/core/core.scale.js
src/scales/scale.category.js

index 4c36c74a8200471b97301b80e60e897ade2e6825..cd07fdbc1deb8e486ecf952b6e2289f468aa5472 100644 (file)
@@ -33,14 +33,13 @@ defaults._set('bar', {
  */
 function computeMinSampleSize(scale, pixels) {
        var min = scale._length;
-       var ticks = scale.getTicks();
        var prev, curr, i, ilen;
 
        for (i = 1, ilen = pixels.length; i < ilen; ++i) {
                min = Math.min(min, Math.abs(pixels[i] - pixels[i - 1]));
        }
 
-       for (i = 0, ilen = ticks.length; i < ilen; ++i) {
+       for (i = 0, ilen = scale.getTicks().length; i < ilen; ++i) {
                curr = scale.getPixelForTick(i);
                min = i > 0 ? Math.min(min, Math.abs(curr - prev)) : min;
                prev = curr;
@@ -260,13 +259,11 @@ module.exports = DatasetController.extend({
        getRuler: function() {
                var me = this;
                var scale = me._getIndexScale();
-               var stackCount = me.getStackCount();
-               var datasetIndex = me.index;
                var pixels = [];
                var i, ilen, min;
 
                for (i = 0, ilen = me.getMeta().data.length; i < ilen; ++i) {
-                       pixels.push(scale.getPixelForValue(null, i, datasetIndex));
+                       pixels.push(scale.getPixelForValue(null, i, me.index));
                }
 
                min = helpers.isNullOrUndef(scale.options.barThickness)
@@ -278,7 +275,7 @@ module.exports = DatasetController.extend({
                        pixels: pixels,
                        start: scale._startPixel,
                        end: scale._endPixel,
-                       stackCount: stackCount,
+                       stackCount: me.getStackCount(),
                        scale: scale
                };
        },
@@ -290,14 +287,13 @@ module.exports = DatasetController.extend({
        calculateBarValuePixels: function(datasetIndex, index) {
                var me = this;
                var chart = me.chart;
-               var meta = me.getMeta();
                var scale = me._getValueScale();
                var isHorizontal = scale.isHorizontal();
                var datasets = chart.data.datasets;
                var value = scale._parseValue(datasets[datasetIndex].data[index]);
                var minBarLength = scale.options.minBarLength;
                var stacked = scale.options.stacked;
-               var stack = meta.stack;
+               var stack = me.getMeta().stack;
                var start = value.start === undefined ? 0 : value.max >= 0 && value.min >= 0 ? value.min : value.max;
                var length = value.start === undefined ? value.end : value.max >= 0 && value.min >= 0 ? value.max - value.min : value.min - value.max;
                var i, imeta, ivalue, base, head, size, stackLength;
index c0040382c1efb0cf3531c97446218f7dd6617c03..4dcd62680e47c9295ba856f269f17f5a522316d0 100644 (file)
@@ -456,13 +456,13 @@ var Scale = Element.extend({
                var me = this;
                var options = me.options;
                var tickOpts = options.ticks;
-               var ticks = me.getTicks();
+               var numTicks = me.getTicks().length;
                var minRotation = tickOpts.minRotation || 0;
                var maxRotation = tickOpts.maxRotation;
                var labelRotation = minRotation;
                var labelSizes, maxLabelWidth, maxLabelHeight, maxWidth, tickWidth, maxHeight, maxLabelDiagonal;
 
-               if (!me._isVisible() || !tickOpts.display || minRotation >= maxRotation || ticks.length <= 1 || !me.isHorizontal()) {
+               if (!me._isVisible() || !tickOpts.display || minRotation >= maxRotation || numTicks <= 1 || !me.isHorizontal()) {
                        me.labelRotation = minRotation;
                        return;
                }
@@ -474,11 +474,11 @@ var Scale = Element.extend({
                // Estimate the width of each grid based on the canvas width, the maximum
                // label width and the number of tick intervals
                maxWidth = Math.min(me.maxWidth, me.chart.width - maxLabelWidth);
-               tickWidth = options.offset ? me.maxWidth / ticks.length : maxWidth / (ticks.length - 1);
+               tickWidth = options.offset ? me.maxWidth / numTicks : maxWidth / (numTicks - 1);
 
                // Allow 3 pixels x2 padding either side for label readability
                if (maxLabelWidth + 6 > tickWidth) {
-                       tickWidth = maxWidth / (ticks.length - (options.offset ? 0.5 : 1));
+                       tickWidth = maxWidth / (numTicks - (options.offset ? 0.5 : 1));
                        maxHeight = me.maxHeight - getTickMarkLength(options.gridLines)
                                - tickOpts.padding - getScaleLabelHeight(options.scaleLabel);
                        maxLabelDiagonal = Math.sqrt(maxLabelWidth * maxLabelWidth + maxLabelHeight * maxLabelHeight);
@@ -508,13 +508,12 @@ var Scale = Element.extend({
                        height: 0
                };
 
-               var ticks = me.getTicks();
                var opts = me.options;
                var tickOpts = opts.ticks;
                var scaleLabelOpts = opts.scaleLabel;
                var gridLineOpts = opts.gridLines;
                var display = me._isVisible();
-               var position = opts.position;
+               var isBottom = opts.position === 'bottom';
                var isHorizontal = me.isHorizontal();
 
                // Width
@@ -556,16 +555,16 @@ var Scale = Element.extend({
                                minSize.height = Math.min(me.maxHeight, minSize.height + labelHeight + tickPadding);
 
                                var offsetLeft = me.getPixelForTick(0) - me.left;
-                               var offsetRight = me.right - me.getPixelForTick(ticks.length - 1);
+                               var offsetRight = me.right - me.getPixelForTick(me.getTicks().length - 1);
                                var paddingLeft, paddingRight;
 
                                // Ensure that our ticks are always inside the canvas. When rotated, ticks are right aligned
                                // which means that the right padding is dominated by the font height
                                if (isRotated) {
-                                       paddingLeft = position === 'bottom' ?
+                                       paddingLeft = isBottom ?
                                                cosRotation * firstLabelSize.width + sinRotation * firstLabelSize.offset :
                                                sinRotation * (firstLabelSize.height - firstLabelSize.offset);
-                                       paddingRight = position === 'bottom' ?
+                                       paddingRight = isBottom ?
                                                sinRotation * (lastLabelSize.height - lastLabelSize.offset) :
                                                cosRotation * lastLabelSize.width + sinRotation * lastLabelSize.offset;
                                } else {
index bb74d40db3ae3c56a191984456bd09d98e9c36ff..29dbf64931432da48dc4a950e2c2c9af914509fb 100644 (file)
@@ -60,7 +60,7 @@ module.exports = Scale.extend({
                        return me.getRightValue(chart.data.datasets[datasetIndex].data[index]);
                }
 
-               return me.ticks[index - me.minIndex];
+               return me._getLabels()[index];
        },
 
        _configure: function() {