» Forum » Clanscript » Kritik und Verbesserungen » Fehler in icmail()
  Mitglieder   |   Gallery   |   FAQ's   |   Regeln

[ Anmelden zum schreiben ]

Fehler in icmail()
SFX
Mitglied
0 Beitragspunkte
Posts: 52

User Pic

0 Mitglieder finden den Beitrag gut.

Merkliste 11.07.2010 - 01:57:53     Zitieren
Es geht um die Funktion

icmail()

in include/includes/func/allg.php

if ($mail[ 0 ] != 'bcc') {
  array_shift($mail);
  foreach ($mail as $m) {
    $mailer->AddBCC(escape_for_email($m));
  }
  $mailer->AddAddress($mailer->From);
} else {
  foreach ($mail as $m) {
    $mailer->AddAddress(escape_for_email($mail));
  }
}


Wenn ich da nicht komplett falsch liege müsste es so aussehen

if ($mail[ 0 ] == 'bcc') {
  array_shift($mail);
  foreach ($mail as $m) {
    $mailer->AddBCC(escape_for_email($m));
  }
  $mailer->AddAddress($mailer->From);
} else {
  foreach ($mail as $m) {
    $mailer->AddAddress(escape_for_email($mail));
  }
}


also $mail[0] == 'bcc'

Der Fehler ist sowohl in in 1.1 O und 1.2 Revision 172 vorhanden.. Zwinker


devteam.ps-evolution.in
Rock@wulf
Ilch 2.0 Entwickler
13 Beitragspunkte
Posts: 1331

User Pic

0 Mitglieder finden den Beitrag gut.

Merkliste 11.07.2010 - 10:38:32     Zitieren
Nein die Version von ilch ist richtig.

Wenn Array mail[0] dann ist dies nicht (!=) BCC

Also mail als array enthält z.b.

yxz@bce.de
cdf@bce.de
...

beim Ilch Code wird
yxz@bce.de nicht mit in die BCC zeile eingetragen, das macht auch sinn denn das ist der empfänger..

Bei deinem Code
wird cdf@bce.de garnicht mehr beachtet


MFG Rock@wulf
SFX
Mitglied
0 Beitragspunkte
Posts: 52

User Pic

0 Mitglieder finden den Beitrag gut.

Merkliste 11.07.2010 - 12:21:03     Zitieren
[QUOTE]Das BCC-Feld enthält eine oder mehrere durch Kommata getrennte E-Mail-Adressen, an die eine Kopie der E-Mail gesendet wird, ohne dass dies jedoch für die anderen angegebenen Empfänger sichtbar ist („Blindkopie“). Durch eine Blindkopie sind die Empfänger von Rund-Mails vor der Adressen-Sammlung von bösartigen Diensten wie z. B. Spambots gesichert.[/QUOTE]

deswegen wäre es ja besser bei einem Newsletter alle Empfänger bei BCC einzutragen und als eigendlichen Empfänger dann nur die Adminmail anzugeben, so wie eben bei dem Beispiel von mir ^^

Es gab zB mal eine Organisation in der Schule, die hat auch Rundmails verschickt, allerdings hat die nicht mit bcc gearbeitet sondern hat alle beim Empfänger eingetragen, und so hatte ich dann sehr sehr viele Email adresse, einschließlich der der Schulleitung..

Und sowas dürfte eigendlich nicht sein.. ^^


Bei dem Code den ich gepostet habe wird jede Email Adresse beachtet, und zwar:

$mail = array("bcc", "abc@email.de", "def@email.de");

if ($mail[ 0 ] != 'bcc') {
  // Trifft zu
  array_shift($mail); // Hier wird der "bcc" Eintrag entfernt
  foreach ($mail as $m) {
    $mailer->AddBCC(escape_for_email($m)); // Hier läuft der Rest des arrays durch, also abc@email.de und dann noch def@email.de
  }
  $mailer->AddAddress($mailer->From); // Hier wird die Adminemail hinzugefügt, damit das Empfängerfeld der Email nicht leer ist
} else {
  foreach ($mail as $m) {
    $mailer->AddAddress(escape_for_email($mail));
  }
}



Zuletzt modifiziert von SFX am 11.07.2010 - 12:21:31


devteam.ps-evolution.in
Mairu
Coder
107 Beitragspunkte
Posts: 12618

User Pic

0 Mitglieder finden den Beitrag gut.

Merkliste 30.07.2010 - 09:47:33     Zitieren
Hast natürlich recht, dummer Fehler.


Und auch immer mal ein Blick auf die FAQ werfen. | Mairus Ilchseite
Rock@wulf
Ilch 2.0 Entwickler
13 Beitragspunkte
Posts: 1331

User Pic

0 Mitglieder finden den Beitrag gut.

Merkliste 12.08.2010 - 07:05:47     Zitieren
Wohl wahr so hab ich noch nicht betrachtet xD


MFG Rock@wulf
Unknown|User
Mitglied
 Beitragspunkte

0 Mitglieder finden den Beitrag gut.

Merkliste 12.08.2010 - 20:39:29     Zitieren
Joa, ganz schön schittisch programmiert xD
xCrystalx
Mitglied
3 Beitragspunkte
Posts: 498

User Pic

0 Mitglieder finden den Beitrag gut.

Merkliste 26.08.2010 - 09:55:50     Zitieren
bei mir gibt es das nicht mal in der datei^^

wollte es mal ändern aber ich habe nichts gefunden in der allg.php


Brain.exe Die Rundumlösung

[ Anmelden zum schreiben ]