move code to put functions on the buttons into updateClasses() function,

so that the buttons work more than once without a page refresh...

1 files changed, 65 insertions, 64 deletions

diff --git a/js/main.js b/js/main.js
index 4dbb0dc..cb84050 100644
--- a/js/main.js
+++ b/js/main.js
@@ -59,6 +59,7 @@ function sortClasses()
 // function to update the HTML display of classes
 function updateClasses ()
 {
+    // first update the HTML displayed
     sortClasses();
     var i = 0;
     $('#classes').html("");
@@ -87,6 +88,70 @@ function updateClasses ()
           </p> \
         </div>');
     }
+
+    // now put functions on the buttons
+    var i = 0;
+    for (i = 0; i < classes.length; i++)
+    {
+        var classstring = classes[i].grade + '-' + classes[i].clas;
+
+        // put a function on the each class's points button
+        $('#' + classstring + 'points').button();
+        $('#' + classstring + 'points').click(function (){
+            var classToUpdateGrade = parseInt(this.id.substring(0, 1));
+            var classToUpdateClas = parseInt(this.id.substring(2, 3));
+            var change = parseInt(prompt("Change points by how much?", 0));
+
+            // look up the class to modify
+            var j = 0;
+            for (j = 0; j < classes.length; j++)
+            {
+                if (classes[j].grade == classToUpdateGrade && classes[j].clas == classToUpdateClas)
+                    break
+            }
+
+            // modify it
+            classes[j].changePoints(change);
+        });
+
+        // put a function on the each class's time button
+        $('#' + classstring + 'time').button();
+        $('#' + classstring + 'time').click(function (){
+            var classToUpdateGrade = parseInt(this.id.substring(0, 1));
+            var classToUpdateClas = parseInt(this.id.substring(2, 3));
+            var change = parseInt(prompt("Change time by how many seconds?", timeWastingClock.getTime().time - 1));
+
+            // look up the class to modify
+            var j = 0;
+            for (j = 0; j < classes.length; j++)
+            {
+                if (classes[j].grade == classToUpdateGrade && classes[j].clas == classToUpdateClas)
+                    break
+            }
+
+            // modify it
+            classes[j].changeTime(change);
+        });
+
+        // put a function on the each class's lucky number button
+        $('#' + classstring + 'random').button();
+        $('#' + classstring + 'random').click(function (){
+            var classToUseGrade = parseInt(this.id.substring(0, 1));
+            var classToUseClas = parseInt(this.id.substring(2, 3));
+
+            // look up the class to use
+            var j = 0;
+            for (j = 0; j < classes.length; j++)
+            {
+                if (classes[j].grade == classToUseGrade && classes[j].clas == classToUseClas)
+                    break
+            }
+
+            // modify it
+            var max = classes[j].total;
+            alert(getRandomInt(1, max));
+        });
+    }
 }
 
 $.ionSound({
@@ -313,69 +378,5 @@ $(document).ready(function(){
     $('#date-toggle').click(function (){toggleDateStyle();});
 
     updateClasses();
-
-    var i = 0;
-    for (i = 0; i < classes.length; i++)
-    {
-        var classstring = classes[i].grade + '-' + classes[i].clas;
-
-        // put a function on the each class's points button
-        $('#' + classstring + 'points').button();
-        $('#' + classstring + 'points').click(function (){
-            var classToUpdateGrade = parseInt(this.id.substring(0, 1));
-            var classToUpdateClas = parseInt(this.id.substring(2, 3));
-            var change = parseInt(prompt("Change points by how much?", 0));
-
-            // look up the class to modify
-            var j = 0;
-            for (j = 0; j < classes.length; j++)
-            {
-                if (classes[j].grade == classToUpdateGrade && classes[j].clas == classToUpdateClas)
-                    break
-            }
-
-            // modify it
-            classes[j].changePoints(change);
-        });
-
-        // put a function on the each class's time button
-        $('#' + classstring + 'time').button();
-        $('#' + classstring + 'time').click(function (){
-            var classToUpdateGrade = parseInt(this.id.substring(0, 1));
-            var classToUpdateClas = parseInt(this.id.substring(2, 3));
-            var change = parseInt(prompt("Change time by how many seconds?", timeWastingClock.getTime().time - 1));
-
-            // look up the class to modify
-            var j = 0;
-            for (j = 0; j < classes.length; j++)
-            {
-                if (classes[j].grade == classToUpdateGrade && classes[j].clas == classToUpdateClas)
-                    break
-            }
-
-            // modify it
-            classes[j].changeTime(change);
-        });
-
-        // put a function on the each class's lucky number button
-        $('#' + classstring + 'random').button();
-        $('#' + classstring + 'random').click(function (){
-            var classToUseGrade = parseInt(this.id.substring(0, 1));
-            var classToUseClas = parseInt(this.id.substring(2, 3));
-
-            // look up the class to use
-            var j = 0;
-            for (j = 0; j < classes.length; j++)
-            {
-                if (classes[j].grade == classToUseGrade && classes[j].clas == classToUseClas)
-                    break
-            }
-
-            // modify it
-            var max = classes[j].total;
-            alert(getRandomInt(1, max));
-        });
-
-    }
 });