includes install.php

[skerbis]

and replaces $this by $addon

so ist immer sichergestellt das die Tabelle unseren Wünschen entspricht.

[tyrant88]

Ich mag das nicht, wenn ich gleichzeitig über mehrer Dinge entschieden muss... einfach nur das include hätte gereicht.

[xong]

Das ist so nicht okay. Wenn, dann sollte die install.php die update.php einbinden.

[tyrant88]

Nee, es geht ja darum, dass auch beim Update die gewünschte Tabellen-Struktur (wie in install.php) hergestellt wird.

[xong]

Ja, und weil das bei jedem Update passieren muss, sollte das auch in die update.php. Dadurch können bei der Installation auch andere Aufgaben ausgeführt werden, die nicht bei jedem Update durchgeführt werden müssen.

[tyrant88]

Ist ja jetzt aber erstmal gewährleistet, dass ein Update die Änderungen durchführt, oder wo siehst du ein konkretes Problem?

Ich hatte mich gestern auch gefragt, wo die Unterschiede liegen und glaubte mich noch schemenhaft an eine Unterhaltung mit Gregor zu erinnern, wo er erwähnte es gäbe Überlegungen das eh auf nur eine Datei zu konsolidieren...

Dann hatte ich mich gestern von skerbis überreden lassen, weil viele andere AddOns das auch machen und mir aktuell kein Thema einfiel, wo man das explizit anders bräuchte.

[skerbis]

In der install macht search_it aktuell nichts anderes als die db anzulegen. In der Update finden aber schon Änderungen statt für vorversionen. Daher dieser Weg. In der nächsten Version kann man es wieder drehen und diese Version voraussetzen