Deutschland United States United Kingdom
ContentLion - Open Source CMS

QA Plugindata

QA Plugindata

Ich möchte jetzt in die Richtung gehen,
die zu veröffentlichten Plugins vorher hier vorzustellen. Dabei Stelle ich dann die Änderungswünsche und Fragen am Plugin vor. Das mache ich dann nicht, um die Fehler des Entwicklers zu präsentieren, sondern daran, dass alle daraus lernen können. Vielleicht findet ihr ja auch Fehler in den Codes.

Fangen wir mal an mit dem Plugin Plugindata. Die erste Frage ist, was macht das genau? Die Klasse Plugin_PluginData_Data sieht für mich so aus, als wäre es wie die unsere Settingsklasse, nur das Eigenschaften nicht vererbt werden können.

Meine Änderungen im Code:
- trunk und branch hinzugefügt
- Escapes eingefügt, um SQL Injektions zu vermeiden

Zum Plugin:
Man kann es mit den Einstellungen vergleichen, wobei es aber einige Unterschiede gibt.
Bei den Daten aus dem Plugin geht es nicht um Einstellungen, die man als User anpassen können soll. ;)
Sondern Plugin-Interne Sachen, z.B. kann man darin speichern, wann das eigene Plugin installiert wurde oder den Namen einer Klasse. So kann man dann später über

PHP-Quelltext

1
2
$class Plugin_PluginData_Data::getData("plugin_test_klasse1");
$instance = new $class();
ganz einfach die Klasse ändern, da diese ja variabel in der Datenbank gespeichert ist. Oder Methoden-Namen. Dieses Plugin ist also auch vorallem für die Anpassbarkeit. So kann ein anderes Plugin eine Klasse einfach "überschreiben" und neue Funktionen hinzufügen. :D

Die Settings-Daten muss man auch nicht über die Oberfläche anpassen ;-)

Aber ok, kann die Unterschiede dann mit in die Pluginbeschreibung nehmen.

Hast du es schon veröffentlicht? :)

Danke! :)
Jetzt bin ich wieder zufrieden. ^^

In der Datei ist sie 2x mal drinn :D also
<?php code ?>
<?php gleicher code ?>

Habe mal die aktuelle Datei hochgeladen. ;)

Wieso hast du jetzt die Escapes rausgenommen und somit SQL-Injections zugelassen? last_update würde ich übrigens oben ganz rauslassen. Macht ja keinen Sinn, dass anders zu setzen.

http://code.contentlion.de/comp.php?repn…compare[]=%2F@6

Ups! 8|
Das wollte ich gar nicht! Da hatte sich nur irgendetwas überschnitten...

@last_update:
Naja, wenn man das Registrierungs- / Installations - Datum z.B. ändern möchte, kann man das ja dann angeben.
Ob das Sinn macht, weiß ich selber noch nicht so genau.^^

Welche Frage? 8|
Es war doch gar keine dabei! ^^

Ok, anders: Verstehe deine Aussage nicht.

Ich werde die Escape-Anweisungen natürlich wieder reinnehmen, diese sollten eig. gar nicht entfernt werden. ;)

Der SVN-Server ist irgendwie gerade down?!? ?(

PS:
Da kommt der Fehler:

Quellcode

1
2
Fehler beim Ausführen des Befehls: svn --non-interactive --config-dir /tmp --username '***' --password '***' log --xml --quiet 'http://code.contentlion.de/svn/contentlion-skin-simplistic_white/@' --limit 1
svn: OPTIONS of 'http://code.contentlion.de/svn/contentlion-skin-simplistic_white': Could not resolve hostname `code.contentlion.de': Temporary failure in name resolution (http://code.contentlion.de)

Was hast du denn mit dem Skin zu tun?