Update Design/Clear Cache

[pschuchmann]

Hey zusammen,

ich habe das Minify etwas überarbeitet und ein paar kleinere Issues behoben. Weiter habe ich die Cache löschen Funktion in die Klasse ausgelagert. Dadurch ist es nun möglich einzelne Sets zu löschen als auch wie gehabt alle auf einmal. Weiter wird nun auch der EP CACHE_DELETED ausgelöst und beim Cache löschen unter den Systemeinstellungen wird nun auch der Minify Cache bereinigt.

[alxndr-w]

Du könntest dir überlegen, auf die rex_config_form umzusteigen, falls noch nicht geschehen. Ist kosmetische Fleißarbeit, aber wenn du schon dabei bist...

[pschuchmann]

Müsste ich nochmal sehen, was es damit genau auf sich hat. Erste schnelle Suche hatte nur ergeben, dass das sozusagen die rex_config ohne Speicherung in der DB ist, richtig?

[alxndr-w]

Umgekehrt, das ist rex_form für die Tabelle rex_config

[staabm]

Bitte in diesem PR nicht noch weitere themen umsetzen, er ist bereits sehr groß/unübersichtlich

[schuer]

Was Markus sagt. Kleiner Tipp: Man kann in der Diff-Ansicht die »Hide whitespace changes«-Option aktivieren, um etwas besser zu sehen, was genau im Code eigentlich geändert worden ist:

[pschuchmann]

Was Markus sagt. Kleiner Tipp: Man kann in der Diff-Ansicht die »Hide whitespace changes«-Option aktivieren, um etwas besser zu sehen, was genau im Code eigentlich geändert worden ist:

Bisher war per Tab eingerückt, in den geänderten Dateien lief mein CodeFormatter drüber und hat normale Spaces daraus gemacht ;) Hab versucht das an den Stellen wieder auf Tab zu ändern ;)

Nochmal was Anderes, wäre es interessant nochmal einen Prefixer einzubauen? Aus meiner Erfahrung raus braucht gerade der IE hier und da doch nochmal ein wenig Prefixe, gerade wenn es um Flexbox geht. Ich hatte Freitag Nachmittag nochmal ein wenig mit Composer experimentiert und da eine entsprechende Lib entdeckt. Idee wäre das einzubauen und konfigurierbar zu machen.

[schuer]

Bisher war per Tab eingerückt, in den geänderten Dateien lief mein CodeFormatter drüber und hat normale Spaces daraus gemacht ;)

@pschuchmann Ist auch total sinnvoll. Nur, damit ein PR möglichst einfach reviewt werden kann, sollte man im ersten Commit erstmal eine saubere Basis schaffen und erst in den nachfolgenden Commits die Änderungen einbringen. Dann bleibt es nachvollziehbar, und Leute müssen sich nicht durch Code wühlen.

Das Problem mit uneinheitlicher Formatierung haben leider einige FOR-AddOns, deshalb ist man häufig in der Situation, erstmal aufräumen zu müssen. Keine Kritik, nur als Info. Danke für deinen Einsatz! 💪

[pschuchmann]

@schuer Fürchte jetzt hab ich es endgültig verwurstet, aber es funktioniert. Ich hatte eben im Slack auch nochmal gefragt, ob es irgendwo einen Coding Standard für Redaxo gibt. Würde ja Sinn machen Addons so zu formatieren wie den Core. Dann finden sich allesamt besser zurecht!

Aber der Tipp mit den Whitespace Änderungen rausnehmen hilft auf jeden Fall schon mal viel ;)

[alxndr-w]

Können wir den hier mergen? Ich habe keinen Überblick, was genau nun die Änderungen sind...

[rkemmere]

Ja der kann gemerged werden. Haben wir bei 30 Installationen im Einsatz. Bisher keine Probleme.