X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-categories.php;h=dad195c7088d797efc1e84baa55b245430aca217;hb=4b93525eb57c07b89c127494a45ec3d5666fa71a;hp=1117465b0d19fed15510dee85f5e03a2118e865c;hpb=241acfbd7e38ff9916100dac9d88892713f85c7f;p=mailer.git diff --git a/inc/modules/member/what-categories.php b/inc/modules/member/what-categories.php index 1117465b0d..dad195c708 100644 --- a/inc/modules/member/what-categories.php +++ b/inc/modules/member/what-categories.php @@ -14,11 +14,9 @@ * $Date:: $ * * $Tag:: 0.2.1-FINAL $ * * $Author:: $ * - * Needs to be in all Files and every File needs "svn propset * - * svn:keywords Date Revision" (autoprobset!) at least!!!!!! * * -------------------------------------------------------------------- * * Copyright (c) 2003 - 2009 by Roland Haeder * - * Copyright (c) 2009, 2010 by Mailer Developer Team * + * Copyright (c) 2009 - 2011 by Mailer Developer Team * * For more information visit: http://www.mxchange.org * * * * This program is free software; you can redistribute it and/or modify * @@ -45,7 +43,7 @@ if (!defined('__SECURITY')) { } // Add description as navigation point -addMenuDescription('member', __FILE__); +addYouAreHereLink('member', __FILE__); $whereStatement = " WHERE `visible`='Y'"; if (isAdmin()) $whereStatement = ''; @@ -53,18 +51,16 @@ if (isAdmin()) $whereStatement = ''; // Get all categories $result = SQL_QUERY("SELECT `id`, `cat` FROM `{?_MYSQL_PREFIX?}_cats`".$whereStatement." ORDER BY `sort` ASC", __FILE__, __LINE__); -// Get num rows -$numCategories = SQL_NUMROWS($result); - -if ($numCategories > 0) { +// Do we have entries? +if (!SQL_HASZERONUMS($result)) { $LEAST = false; if (isFormSent()) { - $cnt = '0'; - foreach (postRequestParameter('cat') as $cat => $joined) { - if ($joined != 'Y') $cnt++; + $count = '0'; + foreach (postRequestParameter('cat') as $categoryId => $joined) { + if ($joined != 'Y') $count++; } // END - foreach - if (($numCategories - $cnt) < getConfig('least_cats')) { + if ((SQL_NUMROWS($result) - $count) < getLeastCats()) { unsetPostRequestParameter('ok'); $LEAST = true; } // END - if @@ -73,10 +69,10 @@ if ($numCategories > 0) { // Is the form sent? if (isFormSent()) { // Start counting all - $cnt = '0'; + $count = '0'; // Go through all entries - foreach (postRequestParameter('cat') as $cat => $joined) { + foreach (postRequestParameter('cat') as $categoryId => $joined) { // Ini sql here $sql = ''; @@ -84,10 +80,13 @@ if ($numCategories > 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(getMemberId(), bigintval($cat)), __FILE__, __LINE__); + array(getMemberId(), bigintval($categoryId)), __FILE__, __LINE__); + + // Debug message + //* DEBUG: */ logDebugMessage(__FILE__, __LINE__, 'categoryId='.$categoryId.',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')"; + if (SQL_HASZERONUMS($result_user)) { + $sql = "INSERT INTO `{?_MYSQL_PREFIX?}_user_cats` (`userid`, `cat_id`) VALUES (%s,%s)"; } // END - if // Free memory @@ -101,29 +100,29 @@ if ($numCategories > 0) { if (!empty($sql)) { // Run SQL command - SQL_QUERY_ESC(trim($sql), array(getMemberId(), bigintval($cat)), __FILE__, __LINE__); + SQL_QUERY_ESC(trim($sql), array(getMemberId(), bigintval($categoryId)), __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, '{--MEMBER_CATEGORIES_SAVED--}'); + displayMessage('{--MEMBER_CATEGORIES_SAVED--}'); } else { // None save - loadTemplate('admin_settings_saved', false, '{--MEMBER_CATEGORIES_NOT_SAVED--}'); + displayMessage('{--MEMBER_CATEGORIES_NOT_SAVED--}'); } } else { if ($LEAST === true) { // Also here we have to secure it... :( - loadTemplate('admin_settings_saved', false, '{--CATS_LEAST--}'); + displayMessage('{--CHOOSE_MORE_CATEGORIES--}'); } // END - if // Put some data into constants for the template - $content['rows'] = ($numCategories * 2 + 4); + $content['rows'] = (SQL_NUMROWS($result) * 2 + 4); // Load header template loadTemplate('member_cats_header', false, $content); @@ -137,7 +136,7 @@ if ($numCategories > 0) { $content['jy'] = ''; // When we found an entry don't read it, just change the jx elements - if (isPostRequestParameterSet('cat')) { + if ((isFormSent()) && (isPostRequestParameterSet('cat'))) { // Form sent? if (postRequestParameter('cat', $content['id']) =='Y') { $content['jy'] = ' checked="checked"'; @@ -148,7 +147,7 @@ if ($numCategories > 0) { 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 } // Load row template and switch colors @@ -160,7 +159,7 @@ if ($numCategories > 0) { } } else { // No cateogries are defined yet - loadTemplate('admin_settings_saved', false, '{--MEMBER_NO_CATEGORIES--}'); + displayMessage('{--MEMBER_NO_CATEGORIES--}'); } // Free result