From: CMDR furrycat <elite@furrycat.net>
Date: Mon, 25 Jan 2016 13:21:52 +0000 (-0500)
Subject: var var block scope, have syntactic sugar?
X-Git-Url: http://git.furryclan.net/?a=commitdiff_plain;h=3d84784d127490e143edc4e920d89a3ae8f88edc;p=furrycat%2Frace-o-matic.git

var var block scope, have syntactic sugar?
---

diff --git a/entry.js b/entry.js
index 5cff531..568c40a 100644
--- a/entry.js
+++ b/entry.js
@@ -758,7 +758,7 @@ function add_lap(table, date) {
 /* Remove the last lap. */
 function remove_lap(table) {
   var lap = table.rows[0].cells.length - 1;
-  for (row of table.rows) row.removeChild(row.cells[lap]);
+  for (var row of table.rows) row.removeChild(row.cells[lap]);
 }
 
 /* Delete empty laps other than the final one. */
@@ -834,7 +834,7 @@ function loaded() {
     remove_lap(table);
     /* Readonly if retrieving an entry but allow joker toggling. */
     if (entry_id) {
-      for (input of document.getElementsByTagName('INPUT')) {
+      for (var input of document.getElementsByTagName('INPUT')) {
         if (input.name == 'endorsement_j') input.disabled = false;
         else {
           input.disabled = true;
diff --git a/race.js b/race.js
index 4f07761..84917ce 100644
--- a/race.js
+++ b/race.js
@@ -30,7 +30,7 @@ function copy_classification(event) {
 
 /* Switch between HTML and bbcode. */
 function toggle_classification(event) {
-  for (div of event.currentTarget.getElementsByTagName('div')) {
+  for (var div of event.currentTarget.getElementsByTagName('div')) {
     if (window.getComputedStyle(div).getPropertyValue('display') == 'none') {
       div.style.display = 'block';
     }
@@ -44,7 +44,7 @@ function toggle_classification(event) {
 function validate() {
   var flags = 0;
   var stations = [];
-  for (select of document.getElementsByClassName('station')) {
+  for (var select of document.getElementsByClassName('station')) {
     var value = parseInt(select.value);
     var flag = value & mandatory_flags;
     if (value && ! flag) stations.push(select);
@@ -55,7 +55,7 @@ function validate() {
   if (valid) {
     /* Either all stations must be ordered or none. */
     var ordered = {};
-    for (input of document.getElementsByTagName('input')) {
+    for (var input of document.getElementsByTagName('input')) {
       if (! input.name.match(/^station\d+order$/)) continue;
       if (input.value) {
         ordered[input.value] = input;
@@ -68,7 +68,7 @@ function validate() {
     var keys = Object.keys(ordered);
     if (keys.length) {
       /* If one station is ordered they all must be. */
-      for (select of stations) {
+      for (var select of stations) {
         var input = document.getElementById(select.name + "order");
         if (input && ! input.value) input.setCustomValidity("Missing order");
       }
@@ -93,7 +93,7 @@ function validate() {
 
 /* Action of a waypoint was changed. */
 function changed_waypoint(event) {
-  for (input of document.getElementsByClassName('station')) {
+  for (var input of document.getElementsByClassName('station')) {
     if (input == event.target) continue;
     input.value &= ~(event.target.value & mandatory_flags);
   }
@@ -113,7 +113,7 @@ function changed_order(event) {
   if (order) {
     var select = document.getElementById(event.target.name.replace(/order$/, ''));
     if (select) select.value = 1;
-    for (input of document.getElementsByTagName('input')) {
+    for (var input of document.getElementsByTagName('input')) {
       if (input == event.target) continue;
       if (! input.name.match(/^station\d+order$/)) continue;
       if (parseInt(input.value) == order) input.value = '';
@@ -125,19 +125,19 @@ function changed_order(event) {
 
 /* Document onload. */
 function loaded() {
-  for (classification of document.getElementsByClassName('classification')) {
+  for (var classification of document.getElementsByClassName('classification')) {
     classification.addEventListener('dblclick', copy_classification, true);
   }
   var waypoints = document.getElementsByClassName('station');
   if (waypoints.length) {
     var div = document.getElementById('waypoints');
     if (div && div.offsetHeight > window.innerHeight * 0.9) div.style.height = '80%';
-    for (input of waypoints) {
+    for (var input of waypoints) {
       input.id = input.name;
       input.addEventListener('change', changed_waypoint, true);
       input.addEventListener('focusout', changed_waypoint, true);
     }
-    for (input of document.getElementsByTagName('input')) {
+    for (var input of document.getElementsByTagName('input')) {
       var m = input.name.match(/^station\d+order$/);
       if (! m) continue;
       input.autocomplete = 'off';
diff --git a/raikogram.js b/raikogram.js
index 28518b0..cf34789 100644
--- a/raikogram.js
+++ b/raikogram.js
@@ -1,7 +1,7 @@
 /* Validate distances. */
 function validate() {
   var valid = true;
-  for (input of document.getElementsByTagName('input')) {
+  for (var input of document.getElementsByTagName('input')) {
     var m = input.name.match(/^from\d+to\d+$/);
     if (! m) continue;
     input.setCustomValidity("");
@@ -28,7 +28,7 @@ function changed_distance(event) {
 
 /* Document onload. */
 function loaded() {
-  for (input of document.getElementsByTagName('input')) {
+  for (var input of document.getElementsByTagName('input')) {
     var m = input.name.match(/^from\d+to\d+$/);
     if (! m) continue;
     input.autocomplete = 'off';
diff --git a/timeline.js b/timeline.js
index 3bd2766..a3eb00f 100644
--- a/timeline.js
+++ b/timeline.js
@@ -1,8 +1,8 @@
 /* Set the background for a leg element. */
 function leg_background(classes, colour) {
   var changed = false;
-  for (c of classes) {
-    for (element of document.getElementsByClassName(c)) {
+  for (var c of classes) {
+    for (var element of document.getElementsByClassName(c)) {
       if (! element.classList.contains('leg')) continue;
       element.style.backgroundColor = colour;
       changed = true;
@@ -27,7 +27,7 @@ function leave(event) {
 
 /* Document onload. */
 function loaded() {
-  for (span of document.getElementsByTagName('span')) {
+  for (var span of document.getElementsByTagName('span')) {
     if (! span.classList.length) continue;
     span.addEventListener('mouseenter', enter, true);
     span.addEventListener('mouseleave', leave, true);