Fix up issues with notes in the data browser

diff --git a/app/assets/javascripts/browse.js b/app/assets/javascripts/browse.js
index 08b35599163c731aec488d71f36344d0d8a6a024..906bb0baeb9f9749bbfda2eeccf2d12c0cf152b7 100644 (file)
--- a/app/assets/javascripts/browse.js
+++ b/app/assets/javascripts/browse.js
@@ -47,7 +47,7 @@ $(document).ready(function () {
     setMapCenter(centre, 16);
     addMarkerToMap(centre);
 
-    var bbox = getMapExtent();
+    var bbox = unproj(map.getExtent());
 
     $("#loading").hide();
     $("#browse_map .geolink").show();

diff --git a/app/views/browse/_map.html.erb b/app/views/browse/_map.html.erb
index 8c28644b11f06df0540a12af7d8f2f7a5ec1bbb5..f694d04644ebfcdd0228d39d701b98ada8409481 100644 (file)
--- a/app/views/browse/_map.html.erb
+++ b/app/views/browse/_map.html.erb
@@ -2,7 +2,7 @@
 </iframe>
 
 <div id="browse_map">
-  <% if map.instance_of? Changeset or (map.instance_of? Node and map.version > 1) or map.visible %>
+  <% if map.instance_of? Changeset or (map.instance_of? Node and map.version > 1) or map.visible? %>
 
   <% content_for :head do %>
     <%= javascript_include_tag "browse" %>
@@ -23,7 +23,7 @@
        data = {
          :type => "note",
          :lon  => map.lon,
-         :lon  => map.lat
+         :lat  => map.lat
        }
      else
        data = {
@@ -39,7 +39,7 @@
 
   <% if map.instance_of? Note -%>
     <%= link_to t("browse.map.larger.area"),
-                root_path(:note => "yes"),
+                root_path(:notes => "yes"),
                 :id => "area_larger_map",
                 :class => "geolink bbox" %>
   <% else -%>

diff --git a/app/views/browse/note.html.erb b/app/views/browse/note.html.erb
index 59c96c4aa2bc78c791c0ec5e8e347c43ca20f483..4137c33c1dd5f0e351e21393d6e28bfdc3344717 100644 (file)
--- a/app/views/browse/note.html.erb
+++ b/app/views/browse/note.html.erb
@@ -11,18 +11,18 @@
 
   <tr>
     <th><%= t "browse.note.opened" %></th>
-    <td><%= t "browse.note.at_by", :when => friendly_date(@note.created_at), :user => note_author(@note) %></td>
+    <td><%= t "browse.note.at_by_html", :when => friendly_date(@note.created_at), :user => note_author(@note) %></td>
   </tr>  
 
   <% if @note.status == "closed" %>
     <tr>
       <th><%= t "browse.note.closed" %></th>
-      <td><%= t "browse.note.at_by", :when => friendly_date(@note.closed_at), :user => note_author(@note.comments.last) %></td>
+      <td><%= t "browse.note.at_by_html", :when => friendly_date(@note.closed_at), :user => note_author(@note.comments.last) %></td>
     </tr>  
   <% elsif @note.comments.length > 1 %>
     <tr>
       <th><%= t "browse.note.last_modified" %></th>
-      <td><%= t "browse.note.at_by", :when => friendly_date(@note.updated_at), :user => note_author(@note.comments.last) %></td>
+      <td><%= t "browse.note.at_by_html", :when => friendly_date(@note.updated_at), :user => note_author(@note.comments.last) %></td>
     </tr>  
   <% end %>
 
@@ -33,7 +33,7 @@
 
   <tr>
     <th><%= t "browse.node_details.coordinates" %></th>
-    <td><div class="geo"><%= link_to ("<span class='latitude'>#{number_with_delimiter(@note.lat)}</span>, <span class='longitude'>#{number_with_delimiter(@note.lon)}</span>"), {:controller => 'site', :action => 'index', :lat => h(@note.lat), :lon => h(@note.lon), :zoom => "18"} %></div></td>
+    <td><div class="geo"><%= link_to ("<span class='latitude'>#{number_with_delimiter(@note.lat)}</span>, <span class='longitude'>#{number_with_delimiter(@note.lon)}</span>".html_safe), {:controller => 'site', :action => 'index', :lat => h(@note.lat), :lon => h(@note.lon), :zoom => "18"} %></div></td>
   </tr>
 
   <% if @note.comments.length > 1 %>
@@ -46,7 +46,7 @@
               <td>
                 <%= h(comment.body) %>
                 <br />
-                <span class="by"><%= t "browse.note.at_by", :when => friendly_date(comment.created_at), :user => note_author(comment) %></span>
+                <span class="by"><%= t "browse.note.at_by_html", :when => friendly_date(comment.created_at), :user => note_author(comment) %></span>
               </td>
             </tr>
           <% end %>

diff --git a/config/locales/en.yml b/config/locales/en.yml
index 38f6757ae0fd1b01b292a1ba47e562f1ae7b1dbe..bacf8a4e630983c95a718f880e400930a3886be9 100644 (file)
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -291,7 +291,7 @@ en:
       opened: "Opened:"
       last_modified: "Last modified:"
       closed: "Closed:"
-      at_by: "%{when} ago by %{user}"
+      at_by_html: "%{when} ago by %{user}"
       description: "Description:"
       comments: "Comments:"
   changeset: