BUGFIX Newsletter-Modul v3.0.3

[Akali]

BUGREPORT Newsletter-Modul (v3.0.3),

im Newsletter-Modul werden die von Besuchern eingegebenen Email-Adressen
NICHT auf ein g?ltiges Email-Adress-Format ?berpr?ft! Das ist kein Witz!
Aber es f?hrt dazu, dass die Datenbank mit der Zeit mit ung?ltigen Adress-Eintr?gen
?berflutet wird. Ein weiterer Grund, die 98,- Euro pro Jahr 'Support' nicht zu zahlen...

Und damit ich nicht als 'Henne, die gackert wenn andere Eier legen' bezichtigt werde,
hier der BUGFIX f?r alle:
(OHNE GARANTIE, aber nach bestem Wissen; VORHER Backup ziehen nicht vergessen!)

Einzig zu ?ndernde Datei:

/newsletter.php

Finde (~Zeile 32):

 require_once(DIR_FS_INC . 'xtc_validate_password.inc.php');

if (isset($_GET['action']) && ($_GET['action'] == 'process')) {

?ndern in:

 require_once(DIR_FS_INC . 'xtc_validate_password.inc.php');
 require_once(DIR_FS_INC . 'xtc_validate_email.inc.php');

if (isset($_GET['action']) && ($_GET['action'] == 'process') && xtc_validate_email($_POST['email'])) {

Finde (~Zeile 132):

}


// Accountaktivierung per Emaillink

?ndern in:

}else{
  $info_message = ERROR_MAIL;
}


// Accountaktivierung per Emaillink

Das war's auch schon.

Viele Gr??e
Akali

P.S. Falls jemand Interesse an einem Nesletter-Adress-Manager (Adressen
importieren/l?schen) hat, findet in folgendem Thread mehr Infos:


P.P.S. Evtl. kann ja jemand diese Info auch ins Forum der Version 3.x einstellen.
Ich habe da keinen Zugriff mehr, da ich momentan nicht bereit bin, j?hrlich
98,- Euro auszugeben, z.B. f?r einen Support, der zum grossen Teil aus der
Nutzer-Gemeinde selbst kommt.

[Akali]

Da man in diesem Forum seine eigenen Beitr?ge leider nicht editieren kann,
hier noch einmal das vollst?ndige P.S. von oben:

P.S. Falls jemand Interesse an einem Nesletter-Adress-Manager (Adressen
importieren/l?schen usw.) hat, findet in folgendem Thread mehr Infos:
http://www.xt-commerce.com/forums/in...howtopic=30031

Viele Gr??e
Akali

[stefanwie]

Ein weiterer Grund, die 98,- Euro pro Jahr 'Support' nicht zu zahlen...

--> Junge du bist ein Vogel. So ein kleiner Fehler. So ein kleiner Fehler kommt immer mal vor. Schreib uns doch bitte ein Fehlerfreies und ebenfalls professionelles Shopsystem, dann ?berweis ich dir gerne 98 EURO.

[techway]

Hi,

das ganze gilt auch f?r die 3.0.4
weiter so Akali

Gru?
TechWay

[Akali]

Noch eine Kleinigkeit. F?r stefanwie sicher Popelkram, f?r mich als Programmierer aber eben l?stig und verbesserungsw?rdig:

Bugreport:
Da die Zeichenkodierung in der Session nicht in JEDEM Fall gesetzt wird, fehlt diese dann leider in einigen vom System generierten Mails - zum Beispiel in der Mail mit dem Newsletter-Aktivierungs-Link. Dies f?hrt zu Anzeigeproblemen in den Email-Programmen.

Mein Bugfix:
Zeichenkodierung standartm??ig auf charset="iso-8859-1" setzen, falls in der Session keine festgelegt wurde.

Zu ?ndernde Datei:
/inc/xtc_php_mail.inc.php

Finde (~ Zeile 22):

$mail->CharSet=$_SESSION['language_charset'];

?ndern in:

if ($_SESSION['language_charset']=='') {
$mail->CharSet="iso-8859-1"; 
} else {
$mail->CharSet=$_SESSION['language_charset']; 
}

Das ist alles. Backup nicht vergessen!
Viele Gr??e
Akali