tbaddade / redaxo_url

REDAXO 5 AddOn zur URL-Generierung für eigene AddOns (ehemals Url Control, ehemals Frau Schultze)
MIT License
46 stars 22 forks source link

`is_user_path` müsste INT sein #266

Closed ynamite closed 1 year ago

ynamite commented 1 year ago

In einem aktuellen Projekt wird bei isUserPath immer false zurückgegeben, da $this->values['is_user_path'] vom Typ INT ist, aber mit einem String verglichen wird.

https://github.com/tbaddade/redaxo_url/blob/664a895ae07d5f19091418ac063916d4b7c39700/lib/Url/UrlManager.php#L229

Aktuellste Version von URL.

Wenn ich die '1' zu 1 ändere, klappt's. Soll ich einen PR machen?

tbaddade commented 1 year ago

Kannst du mal prüfen, ob das bei dir bereits bool ist?

ynamite commented 1 year ago

Ne, $this->values['is_user_path'] ist eine INT, wenn du das meinst.

tbaddade commented 1 year ago

Gern PR … schau mal paar Zeilen darüber isStructure … Müsste dann dort ebenfalls INT sein oder?

ynamite commented 1 year ago

Ich meine ja, bin aber nicht ganz sicher. Kann es kurz testen. Wozu ist isStructure denn gedacht bzw. wann ist es 1 (oder 0)?

tbaddade commented 1 year ago

Wozu ist isStructure denn gedacht

Das wäre die Auswahl im Profil "Unterkategorien anhängen?"

Du hast einen Artikel, der noch Unterkategorien hat. Beispiel Filme

Es werden dann pro Datensatz 2 Urls erstellt

ynamite commented 1 year ago

PR ist erstellt und ja, auch bei is_structure ist es eine INT :) https://github.com/tbaddade/redaxo_url/pull/267

tbaddade commented 1 year ago

close via #https://github.com/tbaddade/redaxo_url/pull/267