EC: fix last interval in edit size histogram (x-tools#590)

It currently wasn't initialised, which meant our += operation made (undefined + int) = NaN, which chart.js interpreted as 0, so that last interval was always empty.

Bug: T411310

Author: agoujot

assets/js/editcounter.js

@@ -304,7 +304,7 @@ xtools.editcounter.setupMonthYearChart = function (id, datasets, labels, maxTota
                 createchart(uselog?"logarithmic":"linear");
             });
     });
-                
+
 };
 
 /**
@@ -315,7 +315,7 @@ xtools.editcounter.setupMonthYearChart = function (id, datasets, labels, maxTota
  * @param {Array} barLabels i18n'd bar labels for additions, removals and same-size, in that order.
  */
 xtools.editcounter.setupSizeHistogram = function (data, colors, barLabels) {
-    let bars = 11;
+    let bars = 12; // Counting the >10240 interval!
     // First sanitize input, to get array.
     let total = Object.keys(data).length;
     data.length = total;
@@ -339,14 +339,26 @@ xtools.editcounter.setupSizeHistogram = function (data, colors, barLabels) {
             datasetZero.data[0] += 1;
         } else {
             // That's the slice index
-            let index = Math.ceil(Math.min(11, Math.max(0, Math.log(Math.abs(x)/10)/Math.log(2))));
+            let index = Math.ceil(
+                Math.min(
+                    bars-1,
+                    Math.max(
+                        0,
+                        Math.log(
+                            Math.abs(x)/10
+                        )
+                        /
+                        Math.log(2)
+                    )
+                )
+            );
             ( x < 0 ? datasetNeg : datasetPos ).data[index] += ( x < 0 ? -1 : 1);
         }
     });
     // The labels for intervals
-    let bounds = [0].concat(Array.from(new Array(bars), (_,i) => 10*2**i));
-    let labels = Array.from(new Array(bars), (_,i) => (new Intl.NumberFormat(i18nLang)).formatRange(bounds[i], bounds[i+1]));
-    labels.push(">"+bounds[bars].toLocaleString(i18nLang));
+    let bounds = [0].concat(Array.from(new Array(bars-1), (_,i) => 10*2**i));
+    let labels = Array.from(new Array(bars-1), (_,i) => (new Intl.NumberFormat(i18nLang)).formatRange(bounds[i], bounds[i+1]));
+    labels.push(">"+bounds[bars-1].toLocaleString(i18nLang));
 
     window['sizeHistogramChart'] = new Chart($("#sizechart-canvas"), {
         type: 'bar',

public/build/app.93b4ef2d.js

@@ -4,7 +4,7 @@
       "js": [
         "/build/runtime.c217f8c4.js",
         "/build/852.96913092.js",
-        "/build/app.93b4ef2d.js"
+        "/build/app.a7ec0e72.js"
       ],
       "css": [
         "/build/app.7692d209.css"

public/build/app.a7ec0e72.js

@@ -1,6 +1,6 @@
 {
   "build/app.css": "/build/app.7692d209.css",
-  "build/app.js": "/build/app.93b4ef2d.js",
+  "build/app.js": "/build/app.a7ec0e72.js",
   "build/runtime.js": "/build/runtime.c217f8c4.js",
   "build/852.96913092.js": "/build/852.96913092.js",
   "build/images/VPS-badge.svg": "/build/images/VPS-badge.svg",