From e477f68a6c1da16b2b976ff97750a65bbd0e3dd9 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Wed, 5 Aug 2009 23:39:24 +0000
Subject: [PATCH] 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.

---
 app/controllers/changeset_controller.rb | 4 ++--
 app/views/changeset/_bbox.atom.builder  | 1 +
 app/views/changeset/_bbox.html.erb      | 1 +
 app/views/changeset/_user.atom.builder  | 1 +
 app/views/changeset/_user.html.erb      | 1 +
 5 files changed, 6 insertions(+), 2 deletions(-)
 create mode 100644 app/views/changeset/_bbox.atom.builder
 create mode 100644 app/views/changeset/_bbox.html.erb
 create mode 100644 app/views/changeset/_user.atom.builder
 create mode 100644 app/views/changeset/_user.html.erb

diff --git a/app/controllers/changeset_controller.rb b/app/controllers/changeset_controller.rb
index c02ced7c2..e451f6d04 100644
--- a/app/controllers/changeset_controller.rb
+++ b/app/controllers/changeset_controller.rb
@@ -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
diff --git a/app/views/changeset/_bbox.atom.builder b/app/views/changeset/_bbox.atom.builder
new file mode 100644
index 000000000..de3b1f112
--- /dev/null
+++ b/app/views/changeset/_bbox.atom.builder
@@ -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"))
diff --git a/app/views/changeset/_bbox.html.erb b/app/views/changeset/_bbox.html.erb
new file mode 100644
index 000000000..f9424befd
--- /dev/null
+++ b/app/views/changeset/_bbox.html.erb
@@ -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" %>
diff --git a/app/views/changeset/_user.atom.builder b/app/views/changeset/_user.atom.builder
new file mode 100644
index 000000000..61faea97f
--- /dev/null
+++ b/app/views/changeset/_user.atom.builder
@@ -0,0 +1 @@
+xml.a(user.display_name, :href => url_for(:controller => "user", :action => "view", :display_name => user.display_name))
diff --git a/app/views/changeset/_user.html.erb b/app/views/changeset/_user.html.erb
new file mode 100644
index 000000000..0e9507650
--- /dev/null
+++ b/app/views/changeset/_user.html.erb
@@ -0,0 +1 @@
+<%= link_to user.display_name, :controller => "user", :action => "view", :display_name => user.display_name %>
-- 
2.43.2