X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-mydata.php;h=4ed6637853b689e4d3897abd6039ffa1c2eb15fa;hp=e6bd332db9912ea378ae57e7691c64b47980d32d;hb=63f159414369b5ea19a8ca75d8cd8033c45d8341;hpb=9afd6ec5878544a7982c50ed9c0dd7de37606d5b diff --git a/inc/modules/member/what-mydata.php b/inc/modules/member/what-mydata.php index e6bd332db9..4ed6637853 100644 --- a/inc/modules/member/what-mydata.php +++ b/inc/modules/member/what-mydata.php @@ -1,7 +1,7 @@ getMemberId() -); +$content = array(); // Init variable to prevent notices -$URL = ''; +$url = ''; // Detect what the member wants to do $mode = 'show'; // Show his data if (isPostRequestElementSet('save')) $mode = 'save'; // Save entered data -if (isPostRequestElementSet('edit')) $mode = 'edit'; // Edit data +if (isFormSent('edit')) $mode = 'edit'; // Edit data if (isPostRequestElementSet('notify')) $mode = 'notify'; // Switch off notification switch ($mode) { case 'show': // Show his data - // Get user data - if (!fetchUserData(getMemberId())) { - // Something really bad happended - debug_report_bug('No user account ' . getMemberId() . ' found.'); - } // END - if - // Get line $content = merge_array($content, getUserDataArray()); // Translate some things - $content['gender'] = translateGender($content['gender']); $content['last_update'] = generateDateTime($content['last_update'], 0); // How far is last change on his profile away from now? - if ((($content['last_update'] + getConfig('profile_lock')) > time()) && (!isAdmin()) && (getConfig('profile_lock') > 0)) { + if ((($content['last_update'] + getProfileLock()) > time()) && (!isAdmin()) && (getProfileLock() > 0)) { // You cannot change your account - $content['change'] = loadTemplate('admin_settings_saved', true, "
".sprintf(getMessage('MEMBER_PROFILE_LOCKED'), generateDateTime($content['last_update'] + getConfig('profile_lock'), 0))."
"); + $content['change'] = displayMessage('
{%message,MEMBER_PROFILE_LOCKED=' . generateDateTime($content['last_update'] + getProfileLock(), 0) . '%}
', true); } else { // He is allowed to change his profile $content['change'] = loadTemplate('member_mydata_button', true); @@ -108,8 +98,22 @@ switch ($mode) { case 'edit': // Edit data if (isExtensionActive('country', true)) { - // New way 0 1 2 3 4 5 6 7 8 9 10 11 12 13 - $result = SQL_QUERY_ESC("SELECT `surname`, `family`, `street_nr`, `country_code`, `zip`, `city`, `email`, `birth_day`, `birth_month`, `birth_year`, `gender`, `max_mails`, `receive_mails`, `last_update` + // New way + $result = SQL_QUERY_ESC("SELECT + `surname`, + `family`, + `street_nr`, + `country_code`, + `zip`, + `city`, + `email`, + `birth_day`, + `birth_month`, + `birth_year`, + `gender`, + `max_mails`, + `receive_mails`, + `last_update` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE @@ -117,8 +121,22 @@ WHERE LIMIT 1", array(getMemberId()), __FILE__, __LINE__); } else { - // Old way 0 1 2 3 4 5 6 7 8 9 10 11 12 13 - $result = SQL_QUERY_ESC("SELECT `surname`, `family`, `street_nr`, `country`, `zip`, `city`, `email`, `birth_day`, `birth_month`, `birth_year`, `gender`, `max_mails`, `receive_mails`, `last_update` + // Old way + $result = SQL_QUERY_ESC("SELECT + `surname`, + `family`, + `street_nr`, + `country`, + `zip`, + `city`, + `email`, + `birth_day`, + `birth_month`, + `birth_year`, + `gender`, + `max_mails`, + `receive_mails`, + `last_update` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE @@ -128,26 +146,22 @@ LIMIT 1", } // Get line - $content = merge_array($content, SQL_FETCHARRAY($result, 0, false)); + $content = merge_array($content, SQL_FETCHARRAY($result)); // Free result SQL_FREERESULT($result); - $content['update_check'] = $content['last_update'] + getConfig('profile_lock'); + $content['update_check'] = $content['last_update'] + getProfileLock(); // How far is last change on his profile away from now? - if (($content['update_check'] > time()) && (!isAdmin()) && (getConfig('profile_lock') > 0)) { - $content['update_check'] = sprintf(getMessage('MEMBER_PROFILE_LOCKED'), generateDateTime($content['update_check'] + getConfig('profile_lock'), 0)); + if (($content['update_check'] > time()) && (!isAdmin()) && (getProfileLock() > 0)) { + // Profile is locked, so output propper message + $content['update_check'] = '{%message,MEMBER_PROFILE_LOCKED=' . generateDateTime($content['update_check'] + getProfileLock(), '0') . '%}'; // You cannot change your account - loadTemplate('member_mydata_locked', false, $content); + displayMessage($content['update_check']); } else { - // He is allowed to change his profile - foreach (array('gender_m','gender_f','gender_c') as $entry) { - $content[$entry] = ''; - } // END - foreach - - $content['gender_' . strtolower($content['gender'])] = ' selected="selected"'; + // Init elements $content['dob'] = ''; switch (getLanguage()) { @@ -177,16 +191,18 @@ LIMIT 1", $content['max_receive_list'] = addMaxReceiveList('member', $content['max_mails'], true); if (isExtensionActive('country')) { - // Generate selection box - $OUT = ""; + + // Generate selection box + $OUT = ''; $content['country'] = $OUT; } else { // Ouput default input box - $content['country'] = ""; + $content['country'] = ''; } // Load template @@ -195,35 +211,37 @@ LIMIT 1", break; case 'save': // Save entered data - // Load old email / password - if (!fetchUserData(getMemberId())) { - // Something really bad happened - debug_report_bug('No user account ' . getMemberId() . ' found.'); - } // END - if - // Get line $content = merge_array($content, getUserDataArray()); // Calculate time to check - $content['update_check'] = $content['last_update'] + getConfig('profile_lock'); + $content['update_check'] = $content['last_update'] + getProfileLock(); // How far is last change on his profile away from now? - if (($content['update_check'] > time()) && (!isAdmin()) && (getConfig('profile_lock') > 0)) { - $content['update_check'] = generateDateTime($content['update_check'] + getConfig('profile_lock'), 0); + if (($content['update_check'] > time()) && (!isAdmin()) && (getProfileLock() > 0)) { // You cannot change your account - loadTemplate('member_mydata_locked', false, $content); + $content['update_check'] = generateDateTime($content['update_check'] + getProfileLock(), 0); + + displayMessage($content['update_check']); } elseif ((!isEmailValid(postRequestElement('email'))) && (!isAdmin())) { // Invalid email address! - loadTemplate('admin_settings_saved', false, getMessage('INVALID_EMAIL_ADDRESS_ENTERED')); + displayMessage('{--INVALID_EMAIL_ENTERED--}'); + } elseif ((isEmailTaken(postRequestElement('email'))) && (isCheckDoubleEmailEnabled()) && (!isAdmin())) { + // Email address is already registered + displayMessage('{--MEMBER_EMAIL_IS_ALREADY_REGISTERED--}'); } else { // Generate hash $hash = generateHash(postRequestElement('pass1'), substr($content['password'], 0, -40)); if ((($hash == $content['password']) || (postRequestElement('pass1') == postRequestElement('pass2'))) && (isPostRequestElementSet('pass1'))) { // Only on simple changes normal mode is active = no email or password changed - $mode = 'normal'; $AND = ''; + $mode = 'normal'; + $AND = ''; // Did the user changed the password? - if ($hash != $content['password']) { $AND = ", `password`='".$hash."'"; $mode = 'pass'; } + if ($hash != $content['password']) { + $AND = ",`password`='" . $hash . "'"; + $mode = 'password'; + } // END - if // Or did he changed his password? if (postRequestElement('email') != $content['email']) { @@ -233,7 +251,7 @@ LIMIT 1", } else { $mode .= ';email'; } - setRequestPostElement('old_email', $content['email']); + setPostRequestElement('old_email', $content['email']); } // END - if // Update member's profile @@ -242,11 +260,17 @@ LIMIT 1", SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET - `gender`='%s', `surname`='%s', `family`='%s', + `gender`='%s', + `surname`='%s', + `family`='%s', `street_nr`='%s', - `country_code`=%s, `zip`=%s, `city`='%s', + `country_code`=%s, + `zip`=%s, + `city`='%s', `email`='%s', - `birth_day`=%s, `birth_month`=%s, `birth_year`=%s, + `birth_day`=%s, + `birth_month`=%s, + `birth_year`=%s, `max_mails`='%s', `last_update`=UNIX_TIMESTAMP()".$AND.", `notified`='N', @@ -274,11 +298,17 @@ LIMIT 1", SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET - `gender`='%s', `surname`='%s', `family`='%s', + `gender`='%s', + `surname`='%s', + `family`='%s', `street_nr`='%s', - `country`='%s', `zip`=%s, `city`='%s', + `country`='%s', + `zip`=%s, + `city`='%s', `email`='%s', - `birth_day`=%s, `birth_month`=%s, `birth_year`=%s, + `birth_day`=%s, + `birth_month`=%s, + `birth_year`=%s, `max_mails`='%s', `last_update`=UNIX_TIMESTAMP()".$AND.", `notified`='N', @@ -304,7 +334,7 @@ LIMIT 1", } // Did something change? - if (SQL_AFFECTEDROWS() == 1) { + if (!SQL_HASZEROAFFECTED()) { // Get all modes ... $modes = explode(';', $mode); @@ -312,25 +342,25 @@ LIMIT 1", sendModeMails ('mydata', $modes); } else { // Something went wrong - loadTemplate('admin_settings_saved', false, getMessage('MEMBER_UPDATE_FAILED')); + displayMessage('{--MEMBER_UPDATE_FAILED--}'); } } else { // Entered wrong pass for updating profile - loadTemplate('admin_settings_saved', false, getMessage('MEBER_UPDATE_PWD_WRONG')); + displayMessage('{--MEBER_UPDATE_PWD_WRONG--}'); } } break; case 'notify': // Switch off notfication - SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET `notified`='N', `last_update`=UNIX_TIMESTAMP() WHERE `userid`=%s LIMIT 1", + SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET `notified`='N',`last_update`=UNIX_TIMESTAMP() WHERE `userid`=%s LIMIT 1", array(getMemberId()), __FILE__, __LINE__); - $URL = 'modules.php?module=login&what=welcome&code=' . urlencode(getMessage('PROFILE_UPDATED')); + $url = 'modules.php?module=login&what=mydata&code=' . getCode('PROFILE_UPDATED'); break; } // END - switch -if (!empty($URL)) { +if (!empty($url)) { // Load generated URL - redirectToUrl($URL); + redirectToUrl($url); } // END - if // [EOF]