More rewrites to make use of (cached) wrapper functions

[mailer.git] / inc / modules / admin / what-config_rallye_prices.php

diff --git a/inc/modules/admin/what-config_rallye_prices.php b/inc/modules/admin/what-config_rallye_prices.php
index c63c04e3b30d5e40029c027a0f9e634d4ad1d8c4..1d211e3d1081d2da97e24d9b0b46b1e748125069 100644 (file)
--- a/inc/modules/admin/what-config_rallye_prices.php
+++ b/inc/modules/admin/what-config_rallye_prices.php
@@ -77,7 +77,7 @@ if (isGetRequestParameterSet('rallye')) {
                }
        } elseif (isFormSent('remove')) {
                // Check if at last one line is selected
-               if (countPostSelection() > 0) {
+               if (ifPostContainsSelections()) {
                        // Delete selected entries
                        foreach (postRequestParameter('sel') as $id => $sel) {
                                SQL_QUERY_ESC("DELETE LOW_PRIORITY FROM `{?_MYSQL_PREFIX?}_rallye_prices` WHERE `id`=%s LIMIT 1",
@@ -121,9 +121,9 @@ LIMIT 1",
 
        if (isFormSent('edit')) {
                // Check if at last one line is selected
-               if (countPostSelection() > 0) {
+               if (ifPostContainsSelections()) {
                        // Make selected editable
-                       $OUT = ''; $SW = 2;
+                       $OUT = '';
                        foreach (postRequestParameter('sel') as $id => $sel) {
                                // Load data to selected rallye
                                $result = SQL_QUERY_ESC("SELECT `rallye_id`, `price_level`, `points`, `info` FROM `{?_MYSQL_PREFIX?}_rallye_prices` WHERE `id`=%s LIMIT 1",
@@ -132,12 +132,10 @@ LIMIT 1",
                                SQL_FREERESULT($result);
 
                                // Add more content
-                               $content['sw']             = $SW;
                                $content['rallye_content'] = generateOptionList('rallye_data', 'id', 'title', $content['rallye_id']);
 
                                // Load row template and switch color
                                $OUT .= loadTemplate('admin_config_rallye_edit_row', true, $content);
-                               $SW = 3 - $SW;
                        } // END - foreach
 
                        // Remember row content
@@ -155,9 +153,9 @@ LIMIT 1",
                }
        } elseif (isFormSent('del')) {
                // Check if at last one line is selected
-               if (countPostSelection() > 0) {
+               if (ifPostContainsSelections()) {
                        // List all prices
-                       $OUT = ''; $SW = 2;
+                       $OUT = '';
                        foreach (postRequestParameter('sel') as $id => $sel) {
                                // Load data to selected rallye
                                $result = SQL_QUERY_ESC("SELECT `rallye_id`, `price_level`, `points`, `info` FROM `{?_MYSQL_PREFIX?}_rallye_prices` WHERE `id`=%s LIMIT 1",
@@ -165,12 +163,8 @@ LIMIT 1",
                                $content = SQL_FETCHARRAY($result);
                                SQL_FREERESULT($result);
 
-                               // Add color switch
-                               $content['sw'] = $SW;
-
                                // Load row template and switch color
                                $OUT .= loadTemplate('admin_config_rallye_del_row', true, $content);
-                               $SW = 3 - $SW;
                        } // END - foreach
                        $content['rows'] = $OUT;
 
@@ -189,16 +183,12 @@ LIMIT 1",
                $result = SQL_QUERY_ESC("SELECT `id`, `price_level`, `points`, `info` FROM `{?_MYSQL_PREFIX?}_rallye_prices` WHERE `rallye_id`=%s ORDER BY `price_level` ASC",
                        array(bigintval(getRequestParameter('rallye'))), __FILE__, __LINE__);
 
-               if (SQL_NUMROWS($result) > 0) {
+               if (!SQL_HASZERONUMS($result)) {
                        // Load all prices for the selected rallye
-                       $OUT = ''; $SW = 2;
+                       $OUT = '';
                        while ($content = SQL_FETCHARRAY($result)) {
-                               // Add color switch
-                               $content['sw'] = $SW;
-
                                // Load row template and switch color
                                $OUT .= loadTemplate('admin_config_rallye_prices_row', true, $content);
-                               $SW = 3 - $SW;
                        } // END - while
 
                        // Free memory
@@ -231,17 +221,15 @@ ON
        d.admin_id=a.id
 ORDER BY
        d.start_time DESC", __FILE__, __LINE__);
-       if (SQL_NUMROWS($result) > 0) {
+       if (!SQL_HASZERONUMS($result)) {
                // List found rallyes
-               $OUT = ''; $SW = 2;
+               $OUT = '';
                while ($content = SQL_FETCHARRAY($result)) {
                        // Prepare data for the row template
                        $content = array(
-                               'sw'         => $SW,
                                'id'         => $content['id'],
                                'title'      => $content['title'],
                                'admin_id'   => $content['admin_id'],
-                               'admin_link' => generateAdminLink($content['admin_id']),
                                'login'      => $content['login'],
                                'start'      => generateDateTime($content['start_time'], 3),
                                'end'        => generateDateTime($content['end_time'], 3),
@@ -249,7 +237,6 @@ ORDER BY
 
                        // Load row template and switch color
                        $OUT .= loadTemplate('admin_list_rallye_prices_row', true, $content);
-                       $SW = 3 - $SW;
                } // END - while
 
                // Free memory