terminal42 / contao-url-rewrite

Provides a back end module in Contao that allows you to specify url rewrites that are added to the Symfony Route collection
MIT License
15 stars 4 forks source link

Issue with |urlattr #28

Closed Tastaturberuf closed 2 years ago

Tastaturberuf commented 3 years ago

Unter Contao 4.9.20 wird das |urlattr automatisch an den Inserttag gehängt. Ich vermute dadurch dass es nur ein Teil einer Url ist, wird das jetzt nicht mehr richtig aufgelöst. Unter 4.9.16 lief das noch.

Antwort-URL:

Contao 4.9.16: https://contao.local/gm/{{anystores_detail::{store_id}:alias}}.html => https://contao.local/gm/mein_alias_von_store_id.html

Contao 4.9.20: https://contao.local/gm/{{anystores_detail::{store_id}:alias|urlattr}}.html => https://contao.local/gm/%7B%7Banystores_detail::%7Bstore_id}:alias|urlattr}}.html

Kann ich irgendwie verhindern das |urlattr automatisch angehängt wird?

Toflar commented 3 years ago

/cc @ausi :P

Tastaturberuf commented 3 years ago

Das hat unter Umständen nicht mit dem |urlattr zu tun, hab es über die DB rausgeworfen. Ergebnis: https://contao.local/gm/%7B%7Banystores_detail::%7Bstore_id}:alias}}.html

Er scheint nur die { zu maskieren.

ausi commented 3 years ago

{{anystores_detail::{store_id}:alias}} ist kein valider Insert-Tag, die Zeichen { und } dürfen darin nicht vorkommen. Was genau ist dieses {store_id}?

Tastaturberuf commented 3 years ago

Das ist Teil dieses Bundles hier und ist ein URL-Fragment(?).

Tastaturberuf commented 3 years ago

Wie gesagt, unter 4.9.16 läuft das noch.

ausi commented 3 years ago

Das ist Teil dieses Bundles hier

Dann sollte das Feld denke ich nicht rgxp => url nutzen, da die URL erst durch ersetzen der {store_id} platzhalter zusammengebaut wird.

Tastaturberuf commented 3 years ago

OK, das funktioniert. Ich kann das in meiner Installation über den DCA fixen. Ich weiß nicht viele noch eigene Inserttags zusammenbauen wollen und auch in das Problem schlittern. Könnte nen known issue sein.

Danke @ausi!

rabauss commented 2 years ago

Should be fixed with https://github.com/contao/contao/pull/3407

aschempp commented 2 years ago

also fixed by https://github.com/terminal42/contao-url-rewrite/commit/c33914242344070c55d41fc3165293798aa07ddb I would say