rex_delete_cache: rex_config nicht neu laden

[gharlan]

closes #1819

Eingeführt hatte ich das in #1624, da mir aufgefallen war, dass nach einem DB-Import rex_config noch die alten Werte enthält.
Ich denke, ich würde das nun aber doch aus rex_delete_cache wieder ganz rausnehmen, da es eben Probleme macht. Stattdessen muss man rex_config::refresh() selbst aufrufen, wenn man direkt was an der rex_config-Tabelle ändert.

[bloep]

Ungetestet

[tbaddade]

Wenn man etwas in die Config schreibt, sollten doch im Anschluss auch die neuen Werte bereitstehen. Ich finde das jetzige Verhalten korrekt und man sollte das Problem versuchen anders zu lösen.

[gharlan]

Wenn man etwas in die Config schreibt, sollten doch im Anschluss auch die neuen Werte bereitstehen.

Wenn man mit rex_config arbeitet, passt alles.
Es geht ja nur darum, wenn man direkt die Tabelle bearbeitet, über rex_sql, backup-addon, adminer, phpmyadmin, oder sonstiges.
Wobei das backup-Addon nun selbst dafür sorgen würde, dass auch da alles passt.
Es ginge dann also nur noch um die anderen Fälle. Wenn man nach einer solchen "händischen" Änderung in rex_config anschließend den Cache löscht, wirken sich die rex_config-Änderungen erst ab dem nächsten Call aus, noch nicht in dem Call, wo der Cache gelöscht wird.

Das Verhalten wäre wieder so, wie in 5.0 bis 5.5, wo nie irgendjemand ein Problem diesbezüglich gemeldet hat. Plus dem neuen Fix im Backup-Addon. Auch da hatte aber nie jemand ein konkretes Problem gemeldet, sondern mir war nur aufgefallen, dass dort theoretisch eines war.

Ich bin aber offen für noch bessere Lösungsideen, ich selbst habe aber aktuell keine.

[gharlan]

Noch mal zur Klarstellung: Ich bin auch nicht 100% zufrieden mit dem Vorschlag hier im PR. Aber mir fällt nichts besseres ein, und es hat halt eigentlich dann doch in 5.0 bis 5.5 soweit funktioniert. Und das Problem im backup-Addon wurde ja trotzdem gelöst.

Alternativ könnte man es so lassen, wie es (in 5.6) ist.
Wenn man rex_config ändert, sollte man dann explizit rex_config::save() aufrufen, damit bei einem möglichen Aufruf von rex_delete_cache() die neuen Werte nicht verloren gehen.
So machen es nun manche Addons ja schon, um das Problem zu umgehen.

Im Issue wurde vorgeschlagen, rex_config::save() in rex_delete_cache automatisch aufzurufen.
Das ist aber eigentlich nicht das, was man allgemein beim Cache löschen machen möchte. Zum Beispiel beim Backup-Import sollen die Werte defintiv aus dem Backup kommen, und nicht dann beim anschließenden Cache löschen eventuell von zuvor gesetzen Werten wieder überschrieben werden.

[staabm]

ich finde den vorschlag besser, als was wir in 5.6 haben. daher passt das für mich.

[tbaddade]

@gharlan Da habe ich das Problem falsch interpretiert. Danke für die genaue Erklärung. Dann passt das auch für mich.