> 1. pass :escape to Segment. I believe the offending method is > DynamicSegment's :interpolation_chunk. > 2. do no escaping in Segment; only do escaping in UrlHelper and/or > UrlWriter. > I'm not sure of the ramifications of either. Suggestions?
This has been like this for a while, so I'd like to hold off doing either of these things until we ship a 2.2 final release. However 'fundamentally' it seems wrong to me that you have to do escaping in your .to_param method, you should be able to return whatever from those and rely on the fact that it'll get escaped deeper in the routing machinery. Moving it out of the dynamic segment will probably be tricky as there's a lot of information there that other areas don't have access to. But basically, if we focus on the desired API, and choose our solution from that, we'll come up with a nice location for the fix. -- Cheers Koz --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
