From d6763de46a753a7e28cae188ea75dea168abc251 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Roland=20H=C3=A4der?= Date: Tue, 13 Sep 2011 15:03:35 +0000 Subject: [PATCH] Misc cleanups --- inc/request-functions.php | 11 ++++++----- inc/template-functions.php | 7 +++++-- templates/de/emails/newsletter.tpl | 4 ++-- templates/de/emails/newsletter_html.tpl | 4 ++-- 4 files changed, 15 insertions(+), 11 deletions(-) diff --git a/inc/request-functions.php b/inc/request-functions.php index 4f48971f94..6a17a3c324 100644 --- a/inc/request-functions.php +++ b/inc/request-functions.php @@ -119,6 +119,7 @@ function setGetRequestElement ($element, $value) { // Wrapper for elements in $_POST function postRequestElement ($element, $subElement = NULL) { + //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element[' . gettype($element) . ']=' . $element . ',subElement[' . gettype($subElement) . ']=' . $subElement . ' - ENTERED!'); // By default no element is there $value = NULL; @@ -126,7 +127,7 @@ function postRequestElement ($element, $subElement = NULL) { if (isset($GLOBALS['cache_request']['post'][$element][$subElement])) { // Then use it $value = $GLOBALS['cache_request']['post'][$element][$subElement]; - //* DEBUG: */ print $element.'/'.$subElement.'='.$value.'
'; + //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element[' . gettype($element) . ']=' . $element . ',subElement[' . gettype($subElement) . ']=' . $subElement . ',value[' . gettype($value) . ']=' . $value . ' - CACHE!'); } elseif (isPostRequestElementSet($element)) { // Then use it $value = $GLOBALS['raw_request']['post'][$element]; @@ -135,20 +136,20 @@ function postRequestElement ($element, $subElement = NULL) { if ((!is_null($subElement)) && (isPostRequestElementSet($element, $subElement))) { // Then use this $value = SQL_ESCAPE($value[$subElement]); - //* DEBUG: */ print 'sub!
'; + //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element=' . $element . ',subElement=' . $subElement . ' - SUB!'); } elseif (!is_array($value)) { // Escape it here $value = SQL_ESCAPE($value); - //* DEBUG: */ print 'no-array!
'; + //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element=' . $element . ' - REGULAR!'); } // Set it in cache - //* DEBUG: */ print $element.'/'.$subElement.'='.$value.'
'; - //* DEBUG: */ print('
'.print_r($_POST,true).'
'); + //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element=' . $element . ',subElement=' . $subElement . ',value=' . $value.' - ADDED!'); $GLOBALS['cache_request']['post'][$element][$subElement] = $value; } // END - if // Return value + //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element[' . gettype($element) . ']=' . $element . ',subElement[' . gettype($subElement) . ']=' . $subElement . ',value[' . gettype($value) . ']=' . $value . ' - EXIT!'); return $value; } diff --git a/inc/template-functions.php b/inc/template-functions.php index fca5522892..2259ffc2e5 100644 --- a/inc/template-functions.php +++ b/inc/template-functions.php @@ -479,7 +479,7 @@ function loadEmailTemplate ($template, $content = array(), $userid = NULL, $load } // Merge data if valid - //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'content()=' . count($content) . ' - PRE!'); + //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'content()=' . count($content) . ' - BEFORE!'); if ((isUserDataValid()) && ($loadUserData === true)) { // It is valid $content = merge_array($content, getUserDataArray()); @@ -541,7 +541,10 @@ function loadEmailTemplate ($template, $content = array(), $userid = NULL, $load $newContent = "Compiler error for template " . $template . " !\nUncompiled content:\n" . $GLOBALS['tpl_content'][$template]; // Add last error if the required function exists - if (function_exists('error_get_last')) $newContent .= "\n--------------------------------------\nDebug:\n".print_r(error_get_last(), true)."--------------------------------------\nPlease don't alter these informations!\nThanx."; + if (function_exists('error_get_last')) { + // Add last error and some lines for better overview + $newContent .= "\n--------------------------------------\nDebug:\n".print_r(error_get_last(), true)."--------------------------------------\nPlease don't alter these informations!\nThanx."; + } // END - if } // END - if // Remove content and data diff --git a/templates/de/emails/newsletter.tpl b/templates/de/emails/newsletter.tpl index d657202203..d7134873c5 100644 --- a/templates/de/emails/newsletter.tpl +++ b/templates/de/emails/newsletter.tpl @@ -6,9 +6,9 @@ Ihre Mitglied-Id bei uns: $userid Sie erhalten diesen Newsletter von uns, weil Sie Mitglied bei unserem {?mt_word?}-Dienst {?MAIN_TITLE?} ({?URL?}) sind. Dies ist somit kein Spam! Eine Abmeldemöglichkeit finden Sie am Ende dieses Newsletters. -============================================================ +------------------------------------------------------------ $content[text] -============================================================ +------------------------------------------------------------ Möchten Sie keine weiteren Newsletter mehr von uns erhalten, melden Sie sich bitte bei uns ({?WEBMASTER?}) und wir löschen dann Ihren Account bei uns. diff --git a/templates/de/emails/newsletter_html.tpl b/templates/de/emails/newsletter_html.tpl index d657202203..d7134873c5 100644 --- a/templates/de/emails/newsletter_html.tpl +++ b/templates/de/emails/newsletter_html.tpl @@ -6,9 +6,9 @@ Ihre Mitglied-Id bei uns: $userid Sie erhalten diesen Newsletter von uns, weil Sie Mitglied bei unserem {?mt_word?}-Dienst {?MAIN_TITLE?} ({?URL?}) sind. Dies ist somit kein Spam! Eine Abmeldemöglichkeit finden Sie am Ende dieses Newsletters. -============================================================ +------------------------------------------------------------ $content[text] -============================================================ +------------------------------------------------------------ Möchten Sie keine weiteren Newsletter mehr von uns erhalten, melden Sie sich bitte bei uns ({?WEBMASTER?}) und wir löschen dann Ihren Account bei uns. -- 2.39.5