Render the links using partial so we can do the appropriate rendering

for HTML and Atom versions and hopefully get the escaping right in the
feed. Closes #2135.
This commit is contained in:
Tom Hughes 2009-08-05 23:39:24 +00:00
parent c0c506adde
commit e477f68a6c
5 changed files with 6 additions and 2 deletions

View file

@ -281,11 +281,11 @@ class ChangesetController < ApplicationController
if bbox
conditions = cond_merge conditions, conditions_bbox(bbox)
bbox = BoundingBox.from_s(bbox)
bbox_link = "<a href='#{url_for(:controller => "site", :action => "index", :minlon => bbox.min_lon, :minlat => bbox.min_lat, :maxlon => bbox.max_lon, :maxlat => bbox.max_lat, :box => "yes")}'>#{bbox.to_s}</a>"
bbox_link = render_to_string :partial => "bbox", :object => bbox
end
if user
user_link = "<a href='#{url_for(:controller => "user", :action => "view", :display_name => user.display_name)}'>#{user.display_name}</a>"
user_link = render_to_string :partial => "user", :object => user
end
if user and bbox

View file

@ -0,0 +1 @@
xml.a(bbox.to_s, :href => url_for(:controller => "site", :action => "index", :minlon => bbox.min_lon, :minlat => bbox.min_lat, :maxlon => bbox.max_lon, :maxlat => bbox.max_lat, :box => "yes"))

View file

@ -0,0 +1 @@
<%= link_to bbox.to_s, :controller => "site", :action => "index", :minlon => bbox.min_lon, :minlat => bbox.min_lat, :maxlon => bbox.max_lon, :maxlat => bbox.max_lat, :box => "yes" %>

View file

@ -0,0 +1 @@
xml.a(user.display_name, :href => url_for(:controller => "user", :action => "view", :display_name => user.display_name))

View file

@ -0,0 +1 @@
<%= link_to user.display_name, :controller => "user", :action => "view", :display_name => user.display_name %>