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, 9 insertions, 9 deletions

diff --git a/modules-available/sysconfig/inc/configmodule.inc.php b/modules-available/sysconfig/inc/configmodule.inc.php
index 580c15a0..fadc649a 100644
--- a/modules-available/sysconfig/inc/configmodule.inc.php
+++ b/modules-available/sysconfig/inc/configmodule.inc.php
@@ -153,7 +153,7 @@ abstract class ConfigModule
 		if ($ret === false)
 			return false;
 		$list = array();
-		while ($row = $ret->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($ret as $row) {
 			$instance = self::instanceFromDbRow($row);
 			if ($instance === false)
 				continue;
diff --git a/modules-available/sysconfig/inc/configtgz.inc.php b/modules-available/sysconfig/inc/configtgz.inc.php
index 98f29753..015b5ccf 100644
--- a/modules-available/sysconfig/inc/configtgz.inc.php
+++ b/modules-available/sysconfig/inc/configtgz.inc.php
@@ -69,7 +69,7 @@ class ConfigTgz
 		// Delete old connections
 		Database::exec("DELETE FROM configtgz_x_module WHERE configid = :configid", array('configid' => $this->configId));
 		// Make connection
-		while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($res as $row) {
 			Database::exec("INSERT INTO configtgz_x_module (configid, moduleid) VALUES (:configid, :moduleid)", array(
 				'configid' => $this->configId,
 				'moduleid' => $row['moduleid']
@@ -170,7 +170,7 @@ class ConfigTgz
 				// Get mapping of moduleid to module name for prettier log display
 				$res = Database::simpleQuery('SELECT moduleid, title FROM configtgz_module');
 				$mods = [];
-				while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+				foreach ($res as $row) {
 					$mods[$row['moduleid']] = $row['title'];
 				}
 				// Now extract module id from filename and if applicable, replace filename by module name
@@ -252,7 +252,7 @@ class ConfigTgz
 			'status' => 'OUTDATED'
 		));
 		$res = Database::simpleQuery("SELECT configid FROM configtgz");
-		while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($res as $row) {
 			$configTgz = self::get($row['configid']);
 			if ($configTgz !== false) {
 				$configTgz->generate();
@@ -290,7 +290,7 @@ class ConfigTgz
 		}
 		$res = Database::simpleQuery("SELECT moduleid, moduletype, filepath, status FROM configtgz_module WHERE moduleid IN ($idstr)");
 		// Make connection
-		while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($res as $row) {
 			Database::exec("INSERT INTO configtgz_x_module (configid, moduleid) VALUES (:configid, :moduleid)", array(
 				'configid' => $instance->configId,
 				'moduleid' => $row['moduleid']
@@ -315,7 +315,7 @@ class ConfigTgz
 			. " INNER JOIN configtgz_module USING (moduleid) "
 			. " WHERE configid = :configid", array('configid' => $instance->configId));
 		$instance->modules = array();
-		while ($row = $ret->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($ret as $row) {
 			$instance->modules[] = $row;
 		}
 		return $instance;
@@ -335,7 +335,7 @@ class ConfigTgz
 		if ($res === false)
 			return false;
 		$list = array();
-		while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($res as $row) {
 			$instance = new ConfigTgz;
 			$instance->configId = $row['configid'];
 			$instance->configTitle = $row['title'];
@@ -344,7 +344,7 @@ class ConfigTgz
 				. " INNER JOIN configtgz_module USING (moduleid) "
 				. " WHERE configid = :configid", array('configid' => $instance->configId));
 			$instance->modules = array();
-			while ($innerRow = $innerRes->fetch(PDO::FETCH_ASSOC)) {
+			foreach ($innerRes as $innerRow) {
 				$instance->modules[] = $innerRow;
 			}
 			$list[] = $instance;
diff --git a/modules-available/sysconfig/inc/sysconfig.inc.php b/modules-available/sysconfig/inc/sysconfig.inc.php
index 9ad3a36f..02131366 100644
--- a/modules-available/sysconfig/inc/sysconfig.inc.php
+++ b/modules-available/sysconfig/inc/sysconfig.inc.php
@@ -8,7 +8,7 @@ class SysConfig
 		$res = Database::simpleQuery("SELECT c.configid, c.title, c.filepath, c.status, Group_Concat(cl.locationid) AS locs FROM configtgz c"
 			. " LEFT JOIN configtgz_location cl USING (configid) GROUP BY c.configid");
 		$ret = array();
-		while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+		foreach ($res as $row) {
 			$ret[] = $row;
 		}
 		return $ret;