Thema: WP Disclaimer
Einzelnen Beitrag anzeigen
Alt 18.04.2008, 11:29   #3 (permalink)
Arno Simon
PostRank: 10
 
Benutzerbild von Arno Simon
 
Registriert seit: 30.11.2005
Ort: Oberhausen (Rhld.)
Beiträge: 1.620
Das Plugin liest sich nicht schlecht, habe aber ein paar Anmerkungen zur Umsetzung - insbesondere auch zu Deinem Hinweis bezüglich des Mailtextes auf der Plugin-Seite.....

- das Hinzufügen der Action-Hooks sollte meines Erachtens an das Ende des Codings gestellt werden. Sofern in Zukunft beim Einstellen der ActionHooks eine Prüfung auf Existenz der anzumeldenden Funktionen durchgeführt wird, bekommst Du beim Hinzufügen am Anfang des Codings einen Fehler. Ist einfach nur Zukunftssicherer

- Anstatt den Text manuell in die entsprechende PHP-Datei klimpern zu lassen, währe es IMHO ratsam - wenn man denn schon zu einem Hack greifen muss! - dort einen eigenen Hook einzubringen und diesen dann im Plugin zu benutzen um den Text über eine entsprechende Funktion (ggf. als NurText!?) in die Mail übertragen zu lassen Erspart dem Anwender die lästige Tipperei bei Upgrades zum größten Teil - er muss dann nur noch den entsprechenden Hook in den Core-Bereich eintragen und nicht mehr den kompletten Text

@Keiran,

dem Coding des Plugins nach sollte es sich um einen Fehler auf Deiner Website handeln. Der Wert der Checkbox wird mit "accept" (gemäß Coding) zurückgeliefert. So Du diesen angepasst haben solltest, kann das Plugin nicht mehr funktionieren. Wenn nicht, musst du mal schauen, was denn im Plugin ankommt (echo).

vG

Arno
__________________
Feedback ist Wichtig!|FAQ|Rückfragen im Forum!|Wenn ich mal was vergesse.... einfach mal sagen...
Arno Simon ist offline   Mit Zitat antworten