X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Flibs%2Fregister_functions.php;h=5ffff9c02f29f97cdae09c60e4d23c6f7aaf7910;hp=8ed526a227bdc66e81344fd1b042fa6d70ae22c1;hb=f928ad2bed60fa256d0641eaf6d2c027a2944688;hpb=7b0f17cd637e388049d2167811e4332cec1e979b

diff --git a/inc/libs/register_functions.php b/inc/libs/register_functions.php
index 8ed526a227..5ffff9c02f 100644
--- a/inc/libs/register_functions.php
+++ b/inc/libs/register_functions.php
@@ -44,12 +44,12 @@ if (!defined('__SECURITY')) {
 
 //
 function registerMustFillout () {
-	$result = SQL_QUERY("SELECT field_name, field_required FROM `{!_MYSQL_PREFIX!}_must_register` ORDER BY `id`",
+	$result = SQL_QUERY("SELECT `field_name`, `field_required` FROM `{!_MYSQL_PREFIX!}_must_register` ORDER BY `id` ASC",
 		__FUNCTION__, __LINE__);
 
 	while ($content = SQL_FETCHARRAY($result)) {
 		$value = '';
-		if ($content['field_required'] == 'Y') $value = "<div class=\\\"guest_failed\\\">&nbsp;(*)</div>";
+		if ($content['field_required'] == 'Y') $value = "<span class=\\\"guest_failed\\\">&nbsp;(*)</span>";
 		// @TODO Rewrite these constants
 		$eval = "define('MUST_".strtoupper($content['field_name'])."', \"".$value."\");";
 		eval($eval);
@@ -60,43 +60,46 @@ function registerMustFillout () {
 
 	// Also fill other constants
 	// @TODO Rewrite these constants, too
-	define('MUST_GENDER', "<div class=\"guest_failed\">&nbsp;(*)</div>");
-	define('MUST_ADDY'  , "<div class=\"guest_failed\">&nbsp;(*)</div>");
-	define('MUST_BIRTH' , "<div class=\"guest_failed\">&nbsp;(*)</div>");
-	define('MUST_MARKER', "<div class=\"guest_failed\">&nbsp;(*)</div>");
+	define('MUST_GENDER', "<span class=\"guest_failed\">&nbsp;(*)</span>");
+	define('MUST_ADDY'  , "<span class=\"guest_failed\">&nbsp;(*)</span>");
+	define('MUST_BIRTH' , "<span class=\"guest_failed\">&nbsp;(*)</span>");
+	define('MUST_MARKER', "<span class=\"guest_failed\">&nbsp;(*)</span>");
 }
 
 //
-function REGISTER_CHECK_REQUIRED_FIELDS (&$array) {
-	$ret = false;
+function ifRequiredRegisterFieldsAreSet (&$array) {
+	// By default all is fine
+	$ret = true;
 	foreach ($array as $key => $value) {
-		$result = SQL_QUERY("SELECT field_required FROM `{!_MYSQL_PREFIX!}_must_register` WHERE field_name='".$key."' LIMIT 1",
+		// Check all fields that must register
+		$result = SQL_QUERY("SELECT `id` FROM `{!_MYSQL_PREFIX!}_must_register` WHERE `field_name`='".$key."' AND `field_required`='Y' LIMIT 1",
 			__FUNCTION__, __LINE__);
 		if (SQL_NUMROWS($result) == 1) {
-			// "Must-line" found
-			list($chk) = SQL_FETCHROW($result);
-
 			// Check if extension country is not found (you have to enter the 2-chars long country code) or
 			// if extensions is present check if country code was selected
 			//         01              2         21    12             3         32    234     5      54    4               43    34                      4    4      5      5432    2      3                      3210
-			$country = ((!EXT_IS_ACTIVE('country')) || ((EXT_IS_ACTIVE('country')) && (((empty($value)) && ($key == "cntry")) || (($key == "country_code") && (!empty($value)))) && (!empty($array['country_code']))));
-			if ((empty($value)) && ($chk == 'Y') && (!$country))
-			{
+			$country = ((!EXT_IS_ACTIVE('country')) || ((EXT_IS_ACTIVE('country')) && (((empty($value)) && ($key == 'cntry')) || (($key == 'country_code') && (!empty($value)))) && (!empty($array['country_code']))));
+			if ((empty($value)) && (!$country)) {
 				// Required field not set
 				$array[$key] = '!';
-				$ret = true;
-			}
-		}
+				$ret = false;
+			} // END - if
+		} // END - if
 
 		// Free result
 		SQL_FREERESULT($result);
-	}
+	} // END - foreach
+
+	// Return result
 	return $ret;
 }
 
-//
+// @TODO This function is no longer used???
 function REGISTER_OUTPUT_REQUIRE_CHECK (&$array) {
+	// Get all fields for output
 	$result = SQL_QUERY("SELECT `field_name`, `field_required` FROM `{!_MYSQL_PREFIX!}_must_register` ORDER BY `id`", __FUNCTION__, __LINE__);
+
+	// Loop through them
 	while ($content = SQL_FETCHARRAY($result)) {
 		if (($array[$content['field_name']] == '!') && ($content['field_required'] == 'Y')) {
 			// Empty entry found
@@ -133,7 +136,7 @@ function registerGenerateCategoryTable ($mode, $return=false) {
 			// Is the array element not set?
 			if (!REQUEST_ISSET_POST('cat', $content['id'])) {
 				// Then set it
-				REQUEST_SET_POST(array('cat', $content['id']), '');
+				REQUEST_SET_POST('cat', array($content['id'] => ''));
 			} // END - if
 
 			// Prepare array for the template
@@ -146,9 +149,9 @@ function registerGenerateCategoryTable ($mode, $return=false) {
 			);
 
 			if ((REQUEST_POST('cat', $content['id']) == 'Y') || ((getConfig('register_default') == 'Y') && (!REQUEST_ISSET_POST('cat', $content['id'])))) {
-				$content['def_y'] = ' chkecked="checked"';
+				$content['def_y'] = ' checked="checked"';
 			} else {
-				$content['def_n'] = ' chkecked="checked"';
+				$content['def_n'] = ' checked="checked"';
 			}
 
 			// Load template and switch color