YRewrite Integration

[MC-PMOE]

Das war hier schon mal Thema: #53
Wenn generell Interesse daran besteht bzw. es keine Einwände gibt, könnte ich wenn ich die Zeit finde einen PR dazu beisteuern.
Würde das dann vermutlich technisch so lösen wollen, dass bei vorhandensein von YRewrite die DB Query zur Abfrage der Tabelle rex_consent_manager_domain um die entsprechende Abfrage der YRewrite Tabelle erweitert wird, und eventuell doppelt vorhandene (Beim consent_manager und YRewrite eingetragene) zusammengefügt werden.
Gibt es da vielleicht Gründe die dagegen sprechen würden?

[IngoWinter]

Zu deiner anvisierten technischen Umsetzung kann ich nix sagen, aber grundsätzlich spricht nicht gegen einen solchen PR.

[MC-PMOE]

Vom Ansatz her würde ich damit gerne möglichst neue Optionen in der Konfiguration vermeiden, so dass die Domains aus YRewrite einfach "da" sind und zur Auswahl stehen wenn vorhanden.
Ich habe aber noch nicht so den Überblick über das AddOn und eine Befürchtung von mir wäre, dass es zu Problemen kommen könnte sofern die rex_consent_manager_domain irgendwo anders abgerufen und genutzt wird und dann Domains konfiguriert sind die so in der Tabelle nicht existieren und von YRewrite kommen.
Vielleicht wäre es dann besser eher einen Import von YRewrite zu machen und den dann immer anzustoßen wenn bei YRewrite eine Domain hinzugefügt wird oder entfernt.

[aeberhard]

@MC-PMOE
Phillip gerne PR dazu. Denke mal das ist aber nicht so einfach umzusetzen.

[alxndr-w]

@MC-PMOE ist das immer noch ein Problem aktuell?

[MC-PMOE]

@alxndr-w Problem nicht, wäre eher ein "nice to have". Im Alltag aber vllt. auch eher zu vernachlässigen, da man ja selten Domains anpasst und man dann halt einmalig so anlegt.
Falls das issue stört weil es hier so vor sich hinvegetiert kann es auch gerne geschlossen werden. Wenn dann doch mal Zeit da ist kann ich ja immer noch einen PR hier einreichen.

[alxndr-w]

@MC-PMOE Ich hab' das Ganze in meiner Consent-Lösung in ein YForm-Value gepackt - das geht hier zwar nicht, aber die Domains sind ja schnell "hergezaubert", falls du's dir abschauen möchtest:

https://github.com/alexplusde/wenns_sein_muss/blob/65e79ccf5c9e510f536dd74b9a8b957743e6732b/lib/yform/value/domain.php#L5-L16

(Prüft, ob YRewrite installiert ist und gibt immer die Default-Domain mit an - einen EP gibt's meines Wissens nicht)