X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Fwhat-config_mods.php;h=dee6e47d26f7ad2e2efc5993a7f576e325d99972;hb=00974432ffe0ed217b993e1ea5aaaac7dc31e82b;hp=20488564182465e58a60c31c4e252ca9392a8832;hpb=c85ddc06cc16f2cd38ddd7d4c142a3f578bbab69;p=mailer.git

diff --git a/inc/modules/admin/what-config_mods.php b/inc/modules/admin/what-config_mods.php
index 2048856418..dee6e47d26 100644
--- a/inc/modules/admin/what-config_mods.php
+++ b/inc/modules/admin/what-config_mods.php
@@ -45,7 +45,7 @@ if ((!defined('__SECURITY')) || (!isAdmin())) {
 // Add description as navigation point
 addMenuDescription('admin', __FILE__);
 
-if (isPostRequestParameterSet('edit')) {
+if (isFormSent('edit')) {
 	// Check if at least one module is selected
 	if (countPostSelection() > 0) {
 		// Output header
@@ -90,7 +90,7 @@ if (isPostRequestParameterSet('edit')) {
 		unsetPostRequestParameter('edit');
 		unsetPostRequestParameter('change');
 	}
-} elseif (isPostRequestParameterSet('change')) {
+} elseif (isFormSent('change')) {
 	// Init SQLs
 	initSqls();
 
@@ -115,7 +115,7 @@ if (isPostRequestParameterSet('edit')) {
 	runFilterChain('run_sqls');
 
 	// Remove cache file if version matches
-	if (getExtensionVersion('cache') >= '0.1.2') {
+	if (isExtensionInstalledAndNewer('cache', '0.1.2')) {
 		if ($GLOBALS['cache_instance']->loadCacheFile('modules')) $GLOBALS['cache_instance']->removeCacheFile();
 	} // END - if
 
@@ -125,44 +125,38 @@ if (isPostRequestParameterSet('edit')) {
 
 if (countPostSelection() == 0) {
 	// Load module data (We do not need to check if there's at least one...)
-	$result = SQL_QUERY('SELECT id, module, locked, hidden, admin_only, title, mem_only
-FROM `{?_MYSQL_PREFIX?}_mod_reg`
-ORDER BY module', __FILE__, __LINE__);
+	$result = SQL_QUERY('SELECT
+	`id`, `module`, `locked`, `hidden`, `admin_only`, `title`, `mem_only`
+FROM
+	`{?_MYSQL_PREFIX?}_mod_reg`
+ORDER BY
+	`module` ASC', __FILE__, __LINE__);
+
 	$OUT = ''; $SW = 2;
 	while ($content = SQL_FETCHARRAY($result)) {
 		// Reset title to --- if it is NULL
 		if (empty($content['title'])) $content['title'] = '---';
 		if (($content['module'] == 'index') || ($content['module'] == 'login')) {
 			// Add link to detail statistics
-			$content['module'] = "<strong><a href=\"{%url=modules.php?module=admin&amp;what=stats_mods&amp;mod=" . $content['module'] . "%}\">" . $content['module'] . "</a></strong>";
+			$content['module'] = '<strong><a href="{%url=modules.php?module=admin&amp;what=stats_mods&amp;mod=' . $content['module'] . '%}">' . $content['module'] . '</a></strong>';
 		} // END - if
 
 		// Prepare array for the template
-		// @TODO Rewritings: mod->module in template
-		$content = array(
-			'sw'     => $SW,
-			'id'     => $content['id'],
-			'mod'    => $content['module'],
-			'title'  => $content['title'],
-			'locked' => translateYesNo($content['locked']),
-			'hidden' => translateYesNo($content['hidden']),
-			'admin'  => translateYesNo($content['admin_only']),
-			'mem'    => translateYesNo($content['mem_only']),
-		);
+		$content['sw'] = $SW;
 
 		// Load row template
 		$OUT .= loadTemplate('admin_list_mods_row', true, $content);
 
 		// Switch colors
 		$SW = 3 - $SW;
-	}
+	} // END - while
 
 	// Free memory
 	SQL_FREERESULT($result);
 
 	// Load main template
 	loadTemplate('admin_list_mods', false, $OUT);
-}
+} // END - if
 
 // [EOF]
 ?>