[#1350] url_for Aufrufe falsch

Date:
2010-03-29 18:28 		Priority:
2
State:
Open
Submitted by:
Bernhard Herzog (bh) 		Assigned to:
Torsten Irländer (torsten)
Web-Browser:
none		 Hardware:
none
Product:
none		 Operating System:
none
Component:
none		 Version:
none
Severity:
normal		 Resolution:
Accepted As Bug
URL:
Summary:
url_for Aufrufe falsch

Detailed description
Der MPuls Code enthält viele url_for Aufrufe der Form

h.url_for(controller='/auth', action='login')

oder, noch falscher:

h.url_for(controller="/case/select/%s" % id)

controller sollte wirklich nur der Name des Controllers sein, d.h. er enthält keinen Slash. Der erste Aufruf sollte lauten:

h.url_for(controller='auth', action='login')

der zweite vermutlich (ungetestet):

h.url_for(controller="case", action="select", id=id)

Die falschen Aufrufe werden probleme machen, wenn wir auf Routes > 1.11 wechseln sollten.
Message  ↓
Date: 2010-05-05 09:57
Sender: Bernhard Herzog

Meine Erfahrungen im März mit Routes 1.12 waren der Auslöser für dieses Issue. Das erste Beispiel lieferte in der tat mit Routes 1.12 die falsche URL zurück, wenn ich mich recht entsinne.

Date: 2010-05-05 09:40
Sender: Frank Koormann

Ich hatte heute Probleme mit der aktuellen Implementation des obigen Beispiels und Routes 1.12:

url_for(controller='/auth', action='loginAction')

d.h. auch das erste Beispiel stellt ein Problem dar?

Date: 2010-03-30 12:43
Sender: Torsten Irländer

Im zweiten Beispiel hast du in jedem Fall Recht!
Bei dem ersten Beipiel bin ich mir nicht sicher. AFAIR wirkt sich der führende slash auf das Cachingverhalten von Routes aus. Ohne slash wird versucht fehlende Teile eines URL aus dem Cache zu erzeugen, was nicht immer das gewünschte ist. Ein Slash löscht den Cache.

Wenn dies allerdings im Bezug zu Routes Probleme bereiten kann, dann müssen wir das natürlich im Auge behalten.

No related tasks

No attached documents

Field Old Value Date By
SeverityNone2010-04-15 15:28Torsten Irländer
priority32010-04-15 15:28Torsten Irländer
assigned_tonone2010-04-14 08:17Torsten Irländer
ResolutionNone2010-04-14 08:17Torsten Irländer