X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-categories.php;h=13075a351e7460a95849f4b0e5960920c7176cfa;hb=fa605a46036192852dbdebfbd493e93d77da2718;hp=df72cb5d3ddbb9a8ec5d902eebd8984353547311;hpb=9afd6ec5878544a7982c50ed9c0dd7de37606d5b;p=mailer.git diff --git a/inc/modules/member/what-categories.php b/inc/modules/member/what-categories.php index df72cb5d3d..13075a351e 100644 --- a/inc/modules/member/what-categories.php +++ b/inc/modules/member/what-categories.php @@ -1,7 +1,7 @@ 0) { +// Do we have entries? +if (!SQL_HASZERONUMS($result)) { $LEAST = false; if (isFormSent()) { - $cnt = '0'; - foreach (postRequestElement('cat') as $cat => $joined) { - if ($joined != 'Y') $cnt++; + $count = '0'; + foreach (postRequestParameter('cat') as $cat => $joined) { + if ($joined != 'Y') $count++; } // END - foreach - if (($cats - $cnt) < getConfig('least_cats')) { - unsetPostRequestElement('ok'); + if ((SQL_NUMROWS($result) - $count) < getLeastCats()) { + unsetPostRequestParameter('ok'); $LEAST = true; } // END - if } // END - if @@ -70,10 +69,10 @@ if ($cats > 0) { // Is the form sent? if (isFormSent()) { // Start counting all - $cnt = '0'; + $count = '0'; // Go through all entries - foreach (postRequestElement('cat') as $cat => $joined) { + foreach (postRequestParameter('cat') as $cat => $joined) { // Ini sql here $sql = ''; @@ -81,10 +80,13 @@ if ($cats > 0) { switch ($joined) { case 'Y': $result_user = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_user_cats` WHERE `userid`=%s AND `cat_id`=%s LIMIT 1", - array($UID, bigintval($cat)), __FILE__, __LINE__); + array(getMemberId(), bigintval($cat)), __FILE__, __LINE__); - if (SQL_NUMROWS($result_user) == '0') { - $sql = "INSERT INTO `{?_MYSQL_PREFIX?}_user_cats` (userid, cat_id) VALUES ('%s','%s')"; + // Debug message + //* DEBUG: */ logDebugMessage(__FILE__, __LINE__, 'cat='.$cat.',joined='.$joined.',getMemberId()='.getMemberId().',SQL_HASZERONUMS()='.intval(SQL_HASZERONUMS($result_user))); + + if (SQL_HASZERONUMS($result_user)) { + $sql = "INSERT INTO `{?_MYSQL_PREFIX?}_user_cats` (`userid`, `cat_id`) VALUES (%s,%s)"; } // END - if // Free memory @@ -98,74 +100,70 @@ if ($cats > 0) { if (!empty($sql)) { // Run SQL command - SQL_QUERY_ESC(trim($sql), array($UID, bigintval($cat)), __FILE__, __LINE__); + SQL_QUERY_ESC(trim($sql), array(getMemberId(), bigintval($cat)), __FILE__, __LINE__); // Count this row - $cnt += SQL_AFFECTEDROWS(); + $count += SQL_AFFECTEDROWS(); } // END - if } // END - foreach // Categories saved? - if ($cnt > 0) { + if ($count > 0) { // Output message - loadTemplate('admin_settings_saved', false, getMessage('MEMBER_CATS_SAVED')); + loadTemplate('admin_settings_saved', false, '{--MEMBER_CATEGORIES_SAVED--}'); } else { // None save - loadTemplate('admin_settings_saved', false, getMessage('MEMBER_CATS_NOT_SAVED')); + loadTemplate('admin_settings_saved', false, '{--MEMBER_CATEGORIES_NOT_SAVED--}'); } } else { - if ($LEAST) { + if ($LEAST === true) { // Also here we have to secure it... :( - loadTemplate('admin_settings_saved', false, sprintf(getMessage('CATS_LEAST'), getConfig('least_cats'))); + loadTemplate('admin_settings_saved', false, '{--CHOOSE_MORE_CATEGORIES--}'); } // END - if // Put some data into constants for the template - $content['rows'] = ($cats*2+4); + $content['rows'] = (SQL_NUMROWS($result) * 2 + 4); // Load header template loadTemplate('member_cats_header', false, $content); // Start switching colors and load all visible categories // @TODO Rewrite this to use $OUT .= ... - $OUT = ''; $SW = 2; + $OUT = ''; while ($content = SQL_FETCHARRAY($result)) { // Default he has not joined $content['jn'] = ' checked="checked"'; $content['jy'] = ''; // When we found an entry don't read it, just change the jx elements - if (isPostRequestElementSet(('cat'))) { + if ((isFormSent()) && (isPostRequestParameterSet('cat'))) { // Form sent? - if (postRequestElement('cat', $content['id']) =='Y') { + if (postRequestParameter('cat', $content['id']) =='Y') { $content['jy'] = ' checked="checked"'; $content['jn'] = ''; - } + } // END - if } else { // Check if he has an entry - if (countSumTotalData($UID, 'user_cats', 'id', 'userid', true, sprintf(" AND `cat_id`=%s", bigintval($content['id']))) == 1) { + if (countSumTotalData(getMemberId(), 'user_cats', 'id', 'userid', true, sprintf(" AND `cat_id`=%s", bigintval($content['id']))) == 1) { $content['jn'] = ''; $content['jy'] = ' checked="checked"'; - } + } // END - if } - // Add some more - $content['sw'] = $SW; - // Load row template and switch colors loadTemplate('member_cat_row', false, $content); - $SW = 3 - $SW; - } + } // END - while // Load footer template loadTemplate('member_cats_footer'); } } else { // No cateogries are defined yet - loadTemplate('admin_settings_saved', true, getMessage('MEMBER_NO_CATS')); + loadTemplate('admin_settings_saved', false, '{--MEMBER_NO_CATEGORIES--}'); } // Free result SQL_FREERESULT($result); -// +// [EOF] ?>