DELETED

Cineplex · Beliebter [**User**] Anmeldung: 22.01.05 Beiträge: 250

DELETED

Metaman · Power-Poster [**User**] Anmeldung: 20.11.05 Beiträge: 636

ok, da ich um Kritik gebeten wurde, will ich mal loslegen Verlegen

Das Script an sich scheint zu funktionieren,
sollte jedoch in diesem Zustand nicht benutzt werden.

Denn fast alles was der User da in die Formularfelder einträgt wird so wie es ist an die Datenbank übergeben.
und es gibt User die da nicht nur ihren Namen und Email eintragen.

such mal in Google nach infos über SQL Injection !!!!

so neben diesem Problem sollte natürlich eine Prüfung mit rein, ob die eingetragene Emailadresse auch wirklich eine Emailadresse ist !!

Dann darf es nicht möglich sein, eine Emailadresse einfach so zum Newsletter anzumelden Smilie

sonnst könnte da ja jeden eintragen die ich kenne.
also eine neue Emailadresse sollte erstmal "auf Halde" gelegt werden.
das heit die Email wird in der DB gespeichert, wird aber beim Newsletter übersprungen !!!
mit der Eintragung sollte dein Script automatisch eine Mail an die genante Adresse verschicken, mit einem Bestäigungslink.
erst wenn dieser Link aufgerufen wird wird die Email aktiv und kann den Newsletter erhalten.

am einfachsten geht das wenn du einfach den time() Wert in einer weiteren Spalte in der DB ablegst.
der Bestätigungslink für den User, enthällt dann diesen Wert+ die Userid
wird der Link dann aufgerufen, wird einfach nur der time Wert aus der DB entfernt und die adresse ist aktiv

zur SQL Tabelle, hier solltest du den SQL Dump ändern, denn sowas wie
collate latin1_general_ci führt online auf einigen Datenbanken zu problemen und die Tabelle würde nicht angelegt werden.

sooo
die Session im Admin ist etwas unsauber, aber das ist kein direktes Problem, könnte jedoch auf einigen Servern dazu führen das die Session beim arbeiten im Admin verloren geht !!

Dann zum versenden des newsletters und der anzeige der Adressen im Admin.
hier wirds schnell probleme geben.
100 oder 200 Adressen sind hier sicher kein PRoblen, jedoch alles darüber könnte zuviel werden.

Denn bei beiden Funktionen liest Du komplett alle Emailadresse aus, und arbeitest die in einer schleife ab.
Beim Versenden könnte dies jedoch dazu führen das die maximale Laufzeit des Scriptes überschritten wird, was in einem Server Error endet...
und dann hat man keine Kontrolle an wen die Mail schon rausgegangen ist.

Darum:
beim Anzeigen der Emails verteile die über mehrere Seiten (Blätterfunktion)

und etwas ähnliches muß Du auch beim versenden benutzen.
hier würde ich immer nur 100 Mails in einer schleife verarbeiten
nach dem durchlauf der Schleife machst du dann eine ausgabe auf dem Bildschirm, und könntest dann dann z.b. mit einem simplen Meta Refresh das script erneut aufrufen und dann die nächsten 100 Mails versenden..
und das bis zum ende.
also bei 1000 Adressen würde das Script 10 mal neu starten.

so das sollte wohl erstmal reichen

Und damit mich nicht wieder alle für einen Miesmacher halten Lachen

Ich wurde um ein Feedback gebeten und Kritik wurde ausdrücklich erwünscht...

Cineplex · Beliebter [**User**] Anmeldung: 22.01.05 Beiträge: 250

DELETED

Metaman · Power-Poster [**User**] Anmeldung: 20.11.05 Beiträge: 636

ich würde im Admin die Session direkt noch mit einem Cookie verbinden oder die Session ID direkt mit jedem Link übergeben...

Cineplex · Beliebter [**User**] Anmeldung: 22.01.05 Beiträge: 250

DELETED

dein_opi · Verfasst am: 27.05.2006, 11:50 Titel:

ich hab ja schon in einem anderen forum gefragt ob der newsletter als html oder als text verschickt wird..

du meintest jetzt txt, version 2 html..

wie wäre es mit einer config_option ob html ODER text

Cineplex · Beliebter [**User**] Anmeldung: 22.01.05 Beiträge: 250

DELETED

Cineplex · Beliebter [**User**] Anmeldung: 22.01.05 Beiträge: 250

DELETED

media_ct · Verfasst am: 06.06.2006, 10:52 Titel:

Hallo Cineplex,

ohne mir dein Script jetzt näher angeschaut zu haben, eine "Mailadressen exportieren - Funktion" wäre ganz interessant.

Dabei sollten sämtliche Mailadressen einfach untereinander gelistet in einer txt-Datei ausgegeben werden.

Grund: Man verwendet das Script nur zur An- u. Abmeldung, jedoch nicht zum Versand.

Cineplex · Beliebter [**User**] Anmeldung: 22.01.05 Beiträge: 250

DELETED

media_ct · Verfasst am: 06.06.2006, 13:47 Titel:

Hi,

nene - so war das nicht gemeint. Smilie

Ich meinte damit, dass es einfach einen Link gibt, der heist zb. "Mailadressen exportieren"
Wenn man den anklickt, dann sollen alle vorhandenen Mailadresses der Abonnenten aus der Datenbank ausgelesen werden, und in einem Text-File geöffnet werden.

Somit hat man alle schön untereinander gelistet, kann das File irgendwo abspeichern und die Adressen in andere Programme/Software importieren, das eben dadurch mit sehr wenig Aufwand.

Wäre wie gesagt interessant, wenn man dein Script nur zur An- und Abmeldung verwendet, nicht aber zum Versenden, weil man dazu bereits andere Software hat oder was auch immer.
Jetzt verständlich? Smilie