QA Plugindata

JuKu
13. März 2013, 21:06

Dieser Fehler kam, als ich code.contentlion.de aufgerufen hatte.

Zum Seitenanfang

stevieswebsite
14. März 2013, 18:31

Im Browser?

Zum Seitenanfang

JuKu
14. März 2013, 19:35

Jep.
Und mit TortoiseSVN konnte ich auch nicht auf das Repo zugreifen...
Habe auch noch nen Screenshot davon:

Zum Seitenanfang

stevieswebsite
15. März 2013, 12:43

Ahh ok. Die Oberfläche führt im Hintergrund auch nur SVN aus, hat aber andere Rechte als alle anderen. Hast du ne Url davon? Auf der Hauptseite komm ich drauf: http://code.contentlion.de/listing.php?r…703ca3be466fa8a

Kann auch sein, dass es noch im Cache liegt, den kann ich heute abend mal leeren.

Zum Seitenanfang

JuKu
15. März 2013, 14:35

Komisch, jetzt funktioniert wieder alles.

Zum Seitenanfang

stevieswebsite
15. März 2013, 19:27

BTT: Wie weit bist du jetzt mit den Sachen?

Zum Seitenanfang

JuKu
15. März 2013, 20:14

ich trage es gleich ein.

Muss nur mal die Änderungen vergleichen und das Escapen halt wieder reinnehmen.

Zum Seitenanfang

JuKu
16. März 2013, 09:42

So, das Repo ist nun wieder aktuell und die Escapes sind wieder drin.

Jetzt kannst du das nochmal testen und veröffentlichen.

Zum Seitenanfang

stevieswebsite
17. März 2013, 18:41

Fällt dir was auf?

	PHP-Quelltext
1	$last_update = "'".DataBase::Current()->EscapeString($area)."'";

Das $last_update würde ich auch als letzten Parameter nehmen, braucht man ja eigentlich nie, oder?

Die Funktion Current kannst du übrigens abkürzen:

	PHP-Quelltext
1 2 3 4 5 6 7	public static function Current() { if (is_null(self::$instance) { self::$instance = new Plugin_PluginData_Data(); } return self::$instance; }

Zum Seitenanfang

JuKu
18. März 2013, 20:33

Ja, die Zeile ist quatsch.
Kannst du das gleich mal ändern?

Muss nämlich noch was für nen kleines anderes Projekt machen...

Zum Seitenanfang

stevieswebsite
18. März 2013, 20:59

und das mit dem last_update?

Zum Seitenanfang

JuKu
19. März 2013, 14:58

Naja, was ist, wenn man ein älteres Datum speichern will?
(z.B. wenn man ein Board integriert hat und da das Reg.-Datum speichern will...)

Zum Seitenanfang

stevieswebsite
19. März 2013, 19:54

joa, aber wenigstens nach hinten verschieben.

Zum Seitenanfang

JuKu
25. März 2013, 18:08

Wie meinst du das?

Zum Seitenanfang

stevieswebsite
26. März 2013, 19:34

Du hast ja setData derzeit folgende Reihenfolge:

- $property
- $value
- $last_update
- $area

daraus würde ich dann folgendes machen:

- $property
- $value
- $area
- $last_update

so musst du dann nicht nen null für $last_update reinmachen, wenn du die area setzen möchtest (was wahrscheinlicher ist).

Am besten nimmst auch noch areaType (plugin/skin) auf ;-)