From d6763de46a753a7e28cae188ea75dea168abc251 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Roland=20H=C3=A4der?= <roland@mxchange.org>
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.'<br />';
+ //* 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!<br />';
+ //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element=' . $element . ',subElement=' . $subElement . ' - SUB!');
} elseif (!is_array($value)) {
// Escape it here
$value = SQL_ESCAPE($value);
- //* DEBUG: */ print 'no-array!<br />';
+ //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'element=' . $element . ' - REGULAR!');
}
// Set it in cache
- //* DEBUG: */ print $element.'/'.$subElement.'='.$value.'<br />';
- //* DEBUG: */ print('<pre>'.print_r($_POST,true).'</pre>');
+ //* 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