ilch Forum » Ilch Clan 1.1 » Kritik und Verbesserungen » Fehler in icmail()

Geschlossen
  1. #1
    User Pic
    SFX Mitglied
    Registriert seit
    31.07.2009
    Beiträge
    52
    Beitragswertungen
    0 Beitragspunkte
    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
    0 Mitglieder finden den Beitrag gut.
  2. #2
    User Pic
    Rock@wulf Hall Of Fame
    Registriert seit
    03.06.2004
    Beiträge
    3.282
    Beitragswertungen
    239 Beitragspunkte
    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
    Meine Postings repräsentieren meine Meinung wenn nicht anders gekennzeichnet.
    MFG Rock@wulf
    0 Mitglieder finden den Beitrag gut.
  3. #3
    User Pic
    SFX Mitglied
    Registriert seit
    31.07.2009
    Beiträge
    52
    Beitragswertungen
    0 Beitragspunkte
    [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
    0 Mitglieder finden den Beitrag gut.
  4. #4
    User Pic
    Mairu Coder
    Registriert seit
    16.06.2006
    Beiträge
    15.334
    Beitragswertungen
    386 Beitragspunkte
    Hast natürlich recht, dummer Fehler.
    Und auch immer mal ein Blick auf die FAQ werfen. | Mairus Ilchseite
    0 Mitglieder finden den Beitrag gut.
  5. #5
    User Pic
    Rock@wulf Hall Of Fame
    Registriert seit
    03.06.2004
    Beiträge
    3.282
    Beitragswertungen
    239 Beitragspunkte
    Wohl wahr so hab ich noch nicht betrachtet xD
    Meine Postings repräsentieren meine Meinung wenn nicht anders gekennzeichnet.
    MFG Rock@wulf
    0 Mitglieder finden den Beitrag gut.
  6. #6
    User Pic
    Unknown|User gelöschter User
    Joa, ganz schön schittisch programmiert xD
    0 Mitglieder finden den Beitrag gut.
  7. #7
    User Pic
    xCrystalx Mitglied
    Registriert seit
    31.12.2008
    Beiträge
    505
    Beitragswertungen
    4 Beitragspunkte
    bei mir gibt es das nicht mal in der datei^^

    wollte es mal ändern aber ich habe nichts gefunden in der allg.php
    0 Mitglieder finden den Beitrag gut.
Geschlossen

Zurück zu Kritik und Verbesserungen

Optionen: Bei einer Antwort zu diesem Thema eine eMail erhalten