Moderize Database handling

* Auto-convert to utf8mb4_unicode_520_ci
* Use foreach instead of while to loop over results
* Drop useless statement caching
* Keep emulated prepares, as we sometimes loop over nested queries

3 files changed, 7 insertions, 7 deletions

diff --git a/modules-available/locations/pages/details.inc.php b/modules-available/locations/pages/details.inc.php
index 356620d3..86bfebd6 100644
--- a/modules-available/locations/pages/details.inc.php
+++ b/modules-available/locations/pages/details.inc.php
@@ -322,7 +322,7 @@ class SubPage
 		$res = Database::simpleQuery("SELECT subnetid, startaddr, endaddr FROM subnet WHERE locationid = :lid",
 			array('lid' => $locationId));
 		$rows = array();
-		while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($res as $row) {
 			$row['startaddr'] = long2ip($row['startaddr']);
 			$row['endaddr'] = long2ip($row['endaddr']);
 			$rows[] = $row;
@@ -360,7 +360,7 @@ class SubPage
 		if (Module::get('statistics') !== false) {
 			$mres = Database::simpleQuery("SELECT state FROM machine"
 				. " WHERE machine.locationid = :lid", array('lid' => $locationId));
-			while ($row = $mres->fetch(PDO::FETCH_ASSOC)) {
+			foreach ($mres as $row) {
 				$count++;
 				if ($row['state'] === 'IDLE') {
 					$online++;
diff --git a/modules-available/locations/pages/locations.inc.php b/modules-available/locations/pages/locations.inc.php
index 8afb454a..ce08e597 100644
--- a/modules-available/locations/pages/locations.inc.php
+++ b/modules-available/locations/pages/locations.inc.php
@@ -121,7 +121,7 @@ class SubPage
 			$res = Database::simpleQuery("SELECT m.locationid, Count(*) AS cnt,
        		Sum(If(m.state = 'OCCUPIED', 1, 0)) AS used, Sum(If(m.state = 'IDLE', 1, 0)) AS idle
  				 FROM machine m WHERE (locationid IN (:allowedLocationIds) $extra) GROUP BY locationid", compact('allowedLocationIds'));
-			while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+			foreach ($res as $row) {
 				$locId = (int)$row['locationid'];
 				if (isset($locationList[$locId])) {
 					$locationList[$locId]['clientCount'] = $row['cnt'];
@@ -135,7 +135,7 @@ class SubPage
 			}
 			$res = Database::simpleQuery("SELECT m.locationid, Count(DISTINCT sm.machineuuid) AS cnt FROM setting_machine sm
 				INNER JOIN machine m USING (machineuuid) GROUP BY m.locationid");
-			while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+			foreach ($res as $row) {
 				$locId = (int)$row['locationid'];
 				if (isset($locationList[$locId])) {
 					$locationList[$locId]['machineVarsOverrideCount'] = $row['cnt'];
@@ -188,7 +188,7 @@ class SubPage
 		if (Module::get('baseconfig') !== false) {
 			$res = Database::simpleQuery("SELECT locationid, Count(*) AS cnt FROM `setting_location`
  					WHERE locationid IN (:allowedLocationIds) GROUP BY locationid", compact('allowedLocationIds'));
-			while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+			foreach ($res as $row) {
 				$lid = (int)$row['locationid'];
 				if (isset($locationList[$lid])) {
 					$locationList[$lid]['overriddenVars'] = $row['cnt'];
@@ -202,7 +202,7 @@ class SubPage
 			$res = Database::simpleQuery("SELECT ml.locationid, m.title, ml.defaultentryid FROM serversetup_menu m
 				INNER JOIN serversetup_menu_location ml USING (menuid)
 				WHERE locationid IN (:allowedLocationIds) GROUP BY locationid", compact('allowedLocationIds'));
-			while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+			foreach ($res as $row) {
 				$lid = (int)$row['locationid'];
 				if (isset($locationList[$lid])) {
 					if ($row['defaultentryid'] !== null) {
diff --git a/modules-available/locations/pages/subnets.inc.php b/modules-available/locations/pages/subnets.inc.php
index fb1e1e80..e568c655 100644
--- a/modules-available/locations/pages/subnets.inc.php
+++ b/modules-available/locations/pages/subnets.inc.php
@@ -67,7 +67,7 @@ class SubPage
 					FROM subnet
 					ORDER BY startaddr ASC, endaddr DESC");
 			$rows = array();
-			while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+			foreach ($res as $row) {
 				$row['startaddr'] = long2ip($row['startaddr']);
 				$row['endaddr'] = long2ip($row['endaddr']);
 				$row['locations'] = Location::getLocations($row['locationid']);