Fixing various javascript warnings, as well as renaming all //!! comments to //TODO

7 files changed, 26 insertions, 24 deletions

diff --git a/pages/achievements.php b/pages/achievements.php
index 1d81220..9e91366 100644
--- a/pages/achievements.php
+++ b/pages/achievements.php
@@ -143,7 +143,7 @@ function toColor($n)
 return("#".substr("000000".dechex($n),-6));

 }

 

-// !! END TESTING AREA;

+// TODO: END TESTING AREA;

 

 //Self-healing code for unlocks.

 if (!$viewer) {

diff --git a/pages/challenge.php b/pages/challenge.php
index f011d3e..1f0e258 100644
--- a/pages/challenge.php
+++ b/pages/challenge.php
@@ -48,6 +48,7 @@ challenge5.help = "Try placing the walls as highlighted.";
 

 function challengeGo(mapid) {

 	var moves = mapjson[mapid].moves;

+	var tiles;

 	

 	switch (mapid) {

 	case '1':

@@ -85,7 +86,7 @@ function challengeGo(mapid) {
 			tv.unlock(5);

 		} else {

 			updateDsp(1, 'instructions', challenge4.help);

-			var tiles = ["4,1,5","4,2,6","4,3,7","4,4,9"];

+			tiles = ["4,1,5","4,2,6","4,3,7","4,4,9"];

 			markTiles(tiles);

 		}

 		break;

@@ -96,7 +97,7 @@ function challengeGo(mapid) {
 			//Unlock next button.

 		} else {

 			updateDsp(1, 'instructions', challenge5.help);

-			var tiles = ["5,1,5","5,2,6","5,3,3","5,4,2","5,3,1"];

+			tiles = ["5,1,5","5,2,6","5,3,3","5,4,2","5,3,1"];

 			markTiles(tiles);

 		}

 		break;

@@ -113,7 +114,7 @@ function markTiles(tiles) {
 	}

 }

 

-//!!

+//TODO:

 function challengeWall(mapid) {

 	//Disable function

 	return;

@@ -192,7 +193,7 @@ TutorialView.prototype.showTutorial = function(num) {
 		if (elem.className.indexOf('hidden') < 0)

 			elem.className += 'hidden';

 	}

-	var elem = document.getElementById("tut-" + num);

+	elem = document.getElementById("tut-" + num);

 	elem.className = elem.className.replace('hidden', '');

 	//Is the map after this disabled?

 	this.nextbtn.className = this.nextbtn.className.replace(' disabled', '');

diff --git a/pages/gallery.php b/pages/gallery.php
index 2d540b6..098cf74 100644
--- a/pages/gallery.php
+++ b/pages/gallery.php
@@ -30,14 +30,14 @@ include('./includes/mapoftheday.php');
 
 
 $openmaps[] = 'Souxoooooooooooooooooooo';
-$openmaps[] = 'ooxooooooooooooooooooooo';
-$openmaps[] = 'oxooooooooooooooeocooooo';
-$openmaps[] = 'oooooooooooooooooooooooo';
-$openmaps[] = 'ooooooooooaoootooooooooo';
-$openmaps[] = 'oooooooooooooooooooooooo';
-$openmaps[] = 'oooooooooooooooooooooooo';
-$openmaps[] = 'ooooooXXoooooooooooooooo';
-$openmaps[] = 'oooooXbdoooooooooooooooo';
+$openmaps[] = 'ooxoooXooooooooooooooooo';
+$openmaps[] = 'oxooooXoooooooooeocooooo';
+$openmaps[] = 'oxooooXooooooooooooooooo';
+$openmaps[] = 'oxooooXoooaoootooooooooo';
+$openmaps[] = 'oxooooXooooooooooooooooo';
+$openmaps[] = 'oxooooXooooooooooooooooo';
+$openmaps[] = 'oxooooXooooooooooooooooo';
+$openmaps[] = 'oxoooXbdoooooooooooooooo';
 $openmaps[] = 'fooooXooooooooooooooooos';
 
 $myparams['checkpoints'] = 5;
diff --git a/pages/home.php b/pages/home.php
index 2b59bc2..4e46ac6 100644
--- a/pages/home.php
+++ b/pages/home.php
@@ -159,7 +159,7 @@ function displayMaze($motd, $mapType) {
 	if ($accepted) {

 		$sol = getSolution($userID, $mapID);

 		$mysolution = $sol['solution'];

-		//!! implement mymoves

+		//TODO: implement mymoves

 		$mymoves = $sol['moves'];

 	}

 	if (isset($_SESSION[$mapID.'sol']) && $mysolution == '') {

@@ -263,7 +263,7 @@ function showStats(type) {
 		elem = document.getElementById("dl-" + i);

 		elem.className = elem.className.replace('selected', '');

 	}

-	var elem = document.getElementById("yms-" + type);

+	elem = document.getElementById("yms-" + type);

 	elem.className = elem.className.replace('hidden-maps', 'shown-maps');

 	elem = document.getElementById("dl-" + type);

 	if (elem.className.indexOf('selected') < 0)

diff --git a/pages/howtoplay.php b/pages/howtoplay.php
index b600fd8..8c9f2f5 100644
--- a/pages/howtoplay.php
+++ b/pages/howtoplay.php
@@ -116,7 +116,7 @@ function showTutorial(num) {
 		if (elem.className.indexOf('hidden') < 0)
 			elem.className += 'hidden';
 	}
-	var elem = document.getElementById("tut-" + num);
+	elem = document.getElementById("tut-" + num);
 	elem.className = elem.className.replace('hidden', '');
 }
 showTutorial(0);
diff --git a/pages/leaderboard.php b/pages/leaderboard.php
index 440f1e5..a457a8d 100644
--- a/pages/leaderboard.php
+++ b/pages/leaderboard.php
@@ -320,7 +320,7 @@ function showStats(type) {
 		elem = document.getElementById("dl-" + i);

 		elem.className = elem.className.replace('selected', '');

 	}

-	var elem = document.getElementById("yms-" + type);

+	elem = document.getElementById("yms-" + type);

 	elem.className = elem.className.replace('hidden-stats', '');

 	elem = document.getElementById("dl-" + type);

 	if (elem.className.indexOf('selected') < 0)

diff --git a/pages/tutorial.php b/pages/tutorial.php
index 95a5a53..d09480c 100644
--- a/pages/tutorial.php
+++ b/pages/tutorial.php
@@ -11,7 +11,7 @@ else
 topbar($Links);

 

 

-//!! Turn this into a function?

+//TODO: Turn this into a function?

 if ($accepted) {

 	include_once ('./includes/db.inc.php');

 	$userID = $_SESSION['userID'];

@@ -71,6 +71,7 @@ challenge5.help = "Try placing the walls as highlighted.";
 

 function challengeGo(mapid) {

 	var moves = mapjson[mapid].moves;

+	var tiles;

 	

 	switch (mapid) {

 	case '1':

@@ -84,7 +85,7 @@ function challengeGo(mapid) {
 			tv.unlock(3);

 		} else {

 			updateDsp(1, 'instructions', challenge2.help);

-			var tiles = ["2,3,5","2,4,6"];

+			tiles = ["2,3,5","2,4,6"];

 			markTiles(tiles);

 		}

 		break;

@@ -95,7 +96,7 @@ function challengeGo(mapid) {
 			tv.unlock(4);

 		} else {

 			updateDsp(1, 'instructions', challenge3.help);

-			var tiles = ["3,3,6","3,4,7","3,3,2","3,2,1"];

+			tiles = ["3,3,6","3,4,7","3,3,2","3,2,1"];

 			markTiles(tiles);

 		}

 		break;

@@ -106,7 +107,7 @@ function challengeGo(mapid) {
 			tv.unlock(5);

 		} else {

 			updateDsp(1, 'instructions', challenge4.help);

-			var tiles = ["4,1,5","4,2,6","4,3,7","4,4,9"];

+			tiles = ["4,1,5","4,2,6","4,3,7","4,4,9"];

 			markTiles(tiles);

 		}

 		break;

@@ -117,7 +118,7 @@ function challengeGo(mapid) {
 			//Unlock next button.

 		} else {

 			updateDsp(1, 'instructions', challenge5.help);

-			var tiles = ["5,1,5","5,2,6","5,3,3","5,4,2","5,3,1"];

+			tiles = ["5,1,5","5,2,6","5,3,3","5,4,2","5,3,1"];

 			markTiles(tiles);

 		}

 		break;

@@ -134,7 +135,7 @@ function markTiles(tiles) {
 	}

 }

 

-//!!

+//TODO:

 function challengeWall(mapid) {

 	//Required walls:

 	var reqwall = new Array();

@@ -212,7 +213,7 @@ TutorialView.prototype.showTutorial = function(num) {
 		if (elem.className.indexOf('hidden') < 0)

 			elem.className += 'hidden';

 	}

-	var elem = document.getElementById("tut-" + num);

+	elem = document.getElementById("tut-" + num);

 	elem.className = elem.className.replace('hidden', '');

 	//Is the map after this disabled?

 	this.nextbtn.className = this.nextbtn.className.replace(' disabled', '');