migrate updateFunctions to PDO and fix version_compare2

Signed-off-by: Michael Kaufmann (d00p) <d00p@froxlor.org>

lib/functions/froxlor/function.updateFunctions.php

@@ -15,7 +15,7 @@
  *
  */
 
-/*
+/**
  * Function updateToVersion
  *
  * updates the panel.version field
@@ -25,40 +25,41 @@
  *
  * @return	bool		true on success, else false
  */
-function updateToVersion($new_version = null)
+function updateToVersion($new_version = null) {
-{
-	global $db, $settings, $theme;
 
-	if($new_version !== null && $new_version != '')
+	global $settings;
-	{
+
-		$query = "UPDATE `".TABLE_PANEL_SETTINGS."` SET `value` = '" . $new_version . "' WHERE `settinggroup` = 'panel' AND `varname` = 'version'";
+	if ($new_version !== null && $new_version != '') {
-		$db->query($query);
+		$upd_stmt = Database::prepare("
+				UPDATE `".TABLE_PANEL_SETTINGS."` SET `value` = :newversion
+				WHERE `settinggroup` = 'panel' AND `varname` = 'version'"
+		);
+		Database::pexecute($upd_stmt, array('newversion' => $new_version));
 		$settings['panel']['version'] = $new_version;
 		return true;
 	}
 	return false;
 }
 
-/*
+/**
  * Function isFroxlor
  *
  * checks if the panel is froxlor
  *
  * @return	bool		true if panel is froxlor, else false
  */
-function isFroxlor()
+function isFroxlor() {
-{
+	global $settings;
-	global $settings, $theme;
 
 	if (isset($settings['panel']['frontend'])
-	&& $settings['panel']['frontend'] == 'froxlor')
+			&& $settings['panel']['frontend'] == 'froxlor'
-	{
+	) {
 		return true;
 	}
 	return false;
 }
 
-/*
+/**
  * Function isFroxlorVersion
  *
  * checks if a given version is the
@@ -68,19 +69,19 @@ function isFroxlor()
  *
  * @return	bool		true if version to check matches, else false
  */
-function isFroxlorVersion($to_check = null)
+function isFroxlorVersion($to_check = null) {
-{
+
-	global $settings, $theme;
+	global $settings;
 
 	if ($settings['panel']['frontend'] == 'froxlor'
-	&& $settings['panel']['version'] == $to_check)
+			&& $settings['panel']['version'] == $to_check
-	{
+	) {
 		return true;
 	}
 	return false;
 }
 
-/*
+/**
  * Function isFroxlorVersion
  *
  * checks if a given version is the
@@ -90,19 +91,19 @@ function isFroxlorVersion($to_check = null)
  *
  * @return	bool		true if version to check matches, else false
  */
-function hasUpdates($to_check = null)
+function hasUpdates($to_check = null) {
-{
+
-	global $settings, $theme;
+	global $settings;
 
 	if (!isset($settings['panel']['version'])
-	|| $settings['panel']['version'] != $to_check)
+			|| $settings['panel']['version'] != $to_check
-	{
+	) {
 		return true;
 	}
 	return false;
 }
 
-/*
+/**
  * Function showUpdateStep
  *
  * outputs and logs the current
@@ -113,15 +114,14 @@ function hasUpdates($to_check = null)
  *
  * @return	string		formatted output and log-entry
  */
-function showUpdateStep($task = null, $needs_status = true)
+function showUpdateStep($task = null, $needs_status = true) {
-{
+
-	global $updatelog, $filelog, $theme;
+	global $updatelog, $filelog;
 
 	// output
 	echo $task;
 
-	if(!$needs_status)
+	if (!$needs_status) {
-	{
 		echo "<br />";
 	}
 
@@ -129,7 +129,7 @@ function showUpdateStep($task = null, $needs_status = true)
 	$filelog->logAction(ADM_ACTION, LOG_WARNING, $task);
 }
 
-/*
+/**
  * Function lastStepStatus
  *
  * outputs [OK] (success), [??] (warning) or [!!] (failure)
@@ -139,12 +139,12 @@ function showUpdateStep($task = null, $needs_status = true)
  *
  * @return	string		formatted output and log-entry
  */
-function lastStepStatus($status = -1, $message = '')
+function lastStepStatus($status = -1, $message = '') {
-{
+
-	global $updatelog, $filelog, $theme;
+	global $updatelog, $filelog;
+
+	switch($status) {
 
-	switch($status)
-	{
 		case 0:
 			$status_sign = ($message != '') ? '['.$message.']' : '[OK]';
 			$status_color = '1dcd00';
@@ -162,16 +162,15 @@ function lastStepStatus($status = -1, $message = '')
 			$status_color = '000000';
 			break;
 	}
+
 	// output
 	echo "<span style=\"margin-left: 5em; font-weight: bold; color: #".$status_color."\">".$status_sign."</span><br />";
 
-	if($status == -1 || $status == 2)
+	if ($status == -1 || $status == 2) {
-	{
 		$updatelog->logAction(ADM_ACTION, LOG_WARNING, 'Attention - last update task failed!!!');
 		$filelog->logAction(ADM_ACTION, LOG_WARNING, 'Attention - last update task failed!!!');
-	}
+
-	elseif($status == 0 || $status == 1)
+	} elseif($status == 0 || $status == 1) {
-	{
 		$filelog->logAction(ADM_ACTION, LOG_WARNING, 'Success');
 	}
 }
@@ -184,13 +183,11 @@ function lastStepStatus($status = -1, $message = '')
  *
  * @return string the full path with filename (can differ if not writeable => /tmp)
  */
-function validateUpdateLogFile($filename)
+function validateUpdateLogFile($filename) {
-{
+
-	if(!is_dir($filename))
+	if (!is_dir($filename)) {
-	{
 		$fh = @fopen($filename, 'a');
-		if($fh)
+		if ($fh) {
-		{
 			fclose($fh);
 			return $filename;
 		}

lib/functions/froxlor/function.version_compare2.php

@@ -26,8 +26,8 @@
 function version_compare2($a, $b) {
 
 	// split version into pieces and remove trailing .0
-	$a = explode(".", rtrim($a, ".0"));
+	$a = explode(".", $a);
-	$b = explode(".", rtrim($b, ".0"));
+	$b = explode(".", $b);
 
 	_parseVersionArray($a);
 	_parseVersionArray($b);