Social Media
Foren
QA Adminmail
- 1
- 2
|
QA Adminmail
So, nun habe ich mir die Adminmail angesehen. Quellcode: http://code.contentlion.de/Plugin+Adminmail/
Folgendes ist mir noch aufgefallen:
- error_reporting in der activate.php muss auf jeden Fall noch raus. Das wird nur von ContentLion verwaltet
- beim Initialisieren die E-Mail Adresse erstmal leerlassen. Ich kenne eigentlich keine Seite, die admin@ nutzt. Da eher info@. Aber wie gesagt, besser leer lassen, als dort eine falsche Mail Adresse stehen zu haben
- Der ApiKey von ContentLion wird nicht nach außen gegeben! Damit würdest du Zugriffsrechte auf die API im Context von anderen Apps bekommen. Das geht gar nicht. Gleiches gilt selbstverständlich für die Datenbankeinstellungen, aber das hast ja auch nicht gemacht. Statistiken über die Nutzung wird es bald auch von ContentLion geben, wenn die Plugins richtig veröffenlticht sind, hast dann bald eine Übersicht der Download-Statistiken. Evtl. später auch Deinstallationsstatistiken.
- Settungs nicht per INSERT Query, sondern über das Settings-Objekt einfügen. Area muss dabei "adminmail" und areaType muss "plugin" sein.
- Wofür ist der Query in die packages-Tabelle??
- Wie sieht's aus mit Validierung der Mail-Adresse?
- Warum machst du eine HTML-Mail, ohne (außer <br />) richtige HTML-Tags zu verwenden? Sehe da keinen Sinn hinter
- Auskommentierterten Code rausnehmen. Den kannst du im SVN Log nachschauen.
- Warum nutzt du für Texte nicht die Language-Klasse (bei "Diese Mail wurde automatisch generiert. Bitte antworten Sie nicht darauf...")?
- Deine Events vor und nach der Mail haben keine Namespaces
Sorry wenn ich so streng dabei bin, ich möchte aber das der Enduser auch hochwertige Plugins bekommt. Da muss man dann schon etwas strenger bei der QA sein.
Folgendes ist mir noch aufgefallen:
- error_reporting in der activate.php muss auf jeden Fall noch raus. Das wird nur von ContentLion verwaltet
- beim Initialisieren die E-Mail Adresse erstmal leerlassen. Ich kenne eigentlich keine Seite, die admin@ nutzt. Da eher info@. Aber wie gesagt, besser leer lassen, als dort eine falsche Mail Adresse stehen zu haben
- Der ApiKey von ContentLion wird nicht nach außen gegeben! Damit würdest du Zugriffsrechte auf die API im Context von anderen Apps bekommen. Das geht gar nicht. Gleiches gilt selbstverständlich für die Datenbankeinstellungen, aber das hast ja auch nicht gemacht. Statistiken über die Nutzung wird es bald auch von ContentLion geben, wenn die Plugins richtig veröffenlticht sind, hast dann bald eine Übersicht der Download-Statistiken. Evtl. später auch Deinstallationsstatistiken.
- Settungs nicht per INSERT Query, sondern über das Settings-Objekt einfügen. Area muss dabei "adminmail" und areaType muss "plugin" sein.
- Wofür ist der Query in die packages-Tabelle??
- Wie sieht's aus mit Validierung der Mail-Adresse?
- Warum machst du eine HTML-Mail, ohne (außer <br />) richtige HTML-Tags zu verwenden? Sehe da keinen Sinn hinter
- Auskommentierterten Code rausnehmen. Den kannst du im SVN Log nachschauen.
- Warum nutzt du für Texte nicht die Language-Klasse (bei "Diese Mail wurde automatisch generiert. Bitte antworten Sie nicht darauf...")?
- Deine Events vor und nach der Mail haben keine Namespaces
Sorry wenn ich so streng dabei bin, ich möchte aber das der Enduser auch hochwertige Plugins bekommt. Da muss man dann schon etwas strenger bei der QA sein.
WBB hat gerade meinen ganz langen Beitrag geschluckt?!?
Hier nochmal in Kurzform:
1. & 2.: behoben
3.: Sorry, hatte was falsch verstanden.
Aber die Datei dahinter (API) war eh noch leer.
EDIT:
Aber du machst das, z.B. beim Einreichen-Button so.
Da wird immer der API-Key mit übertragen.
@Settings:
Ach so, dache schon, dass sei überflüssig.
Wie macht man das?
Über die set-Methode?
@Packages:
Du sagtest doch, dass jedes Plugin, welches später Updates ermöglichen soll, dort eingetragen werden soll?
@Validierung:
Schwer realisierbar, da beim setzen der Einstellung kein Event geworfen wird.
Habs aber mal bei der send-Methode hinzugefügt.
@HTML-Mail:
Habe nen Parameter hinzugefügt.
Es soll ja auch möglich sein, HTML-Mails zu verschicken, schon alleine, wenn man z.B. Links für's direkte Freischalten von Kommentaren verschicken will.
@Auskommentierterten Code:
Behoben.
@Lang-Klasse:
hab ich gemacht. --> Behoben!
@Events:
Habs mal angepasst.
Kein Ding!
Find ich gut!
|
@Apikey:
Der Apikey ist ja extra zur Autorisierung an ContentLion.de. Würde ich ihn nicht verwenden, hätte er ja gar keine Existenzberechtigung
Die Packages werden von Content
@Settings:
Ja genau.
@Packages
Jetzt weiß ich wieder wofür das war Das brauchst nicht mehr zu Pflegen, der Updater der auf die Dauer kommen wird, macht das komplett anders.
@Validierung
Am besten nimmst du dafür die Plugin-Settings. Da kannst du unter Datei > Einstellungen eine komplett eigene Seite machen, wo du alle Einstellungen hinterlegen kannst.
@HTML-Mail:
Links gehen auch in Textmals ;-) Aber ok, könnte man so machen. Wenn dann möchte ich aber auch den ganzen HTML-Inhalt senden können.
@Verschwundener Beitrag: Keine Ahnung, was da passiert seien könnte. Ich schreib allgemein sehr lange Beiträge (auch auf Facebook oder so) vorher im Notepad und speicher zwischendurch (mein rechner stürzt derzeit leider manchmal ab)
Der Apikey ist ja extra zur Autorisierung an ContentLion.de. Würde ich ihn nicht verwenden, hätte er ja gar keine Existenzberechtigung
Die Packages werden von Content
@Settings:
Ja genau.
@Packages
Jetzt weiß ich wieder wofür das war Das brauchst nicht mehr zu Pflegen, der Updater der auf die Dauer kommen wird, macht das komplett anders.
@Validierung
Am besten nimmst du dafür die Plugin-Settings. Da kannst du unter Datei > Einstellungen eine komplett eigene Seite machen, wo du alle Einstellungen hinterlegen kannst.
@HTML-Mail:
Links gehen auch in Textmals ;-) Aber ok, könnte man so machen. Wenn dann möchte ich aber auch den ganzen HTML-Inhalt senden können.
@Verschwundener Beitrag: Keine Ahnung, was da passiert seien könnte. Ich schreib allgemein sehr lange Beiträge (auch auf Facebook oder so) vorher im Notepad und speicher zwischendurch (mein rechner stürzt derzeit leider manchmal ab)
|
http://www.contentlion.de/doku/settings-…stellungen.html
- Wenn noch was nicht verständlich ist, einfach Bescheid geben.
- Wenn noch was nicht verständlich ist, einfach Bescheid geben.
- 1
- 2