Tidy up a few details.
authorTom Hughes <tom@compton.nu>
Sun, 20 Apr 2008 12:29:50 +0000 (12:29 +0000)
committerTom Hughes <tom@compton.nu>
Sun, 20 Apr 2008 12:29:50 +0000 (12:29 +0000)
app/controllers/browse_controller.rb
app/views/browse/_common.rhtml
app/views/browse/_map.rhtml

index dcc4cd3f31504291ef19f90b975903ad3c493c18..c35bb821a00e587b23da28457778faac762992b9 100644 (file)
@@ -1,32 +1,32 @@
 class BrowseController < ApplicationController
   before_filter :authorize_web  
   layout 'site'
 class BrowseController < ApplicationController
   before_filter :authorize_web  
   layout 'site'
+
   def way_view 
     begin
   def way_view 
     begin
-      way = Way.find(params[:id])
+      @way = Way.find(params[:id])
      
      
-      @way = way
       @name = @way.tags['name'].to_s 
       if @name.length == 0:
        @name = "#" + @way.id.to_s
       @name = @way.tags['name'].to_s 
       if @name.length == 0:
        @name = "#" + @way.id.to_s
-      end      
+      end
+       
       @title = 'Way | ' + (@name)
       @title = 'Way | ' + (@name)
-      
     rescue ActiveRecord::RecordNotFound
       render :nothing => true, :status => :not_found
     end
   end
     rescue ActiveRecord::RecordNotFound
       render :nothing => true, :status => :not_found
     end
   end
+
   def node_view 
     begin
   def node_view 
     begin
-      node = Node.find(params[:id])
+      @node = Node.find(params[:id])
      
      
-      @node = node
       @name = @node.tags_as_hash['name'].to_s 
       if @name.length == 0:
        @name = "#" + @node.id.to_s
       @name = @node.tags_as_hash['name'].to_s 
       if @name.length == 0:
        @name = "#" + @node.id.to_s
-      end      
+      end
+       
       @title = 'Node | ' + (@name)
       @title = 'Node | ' + (@name)
-      
     rescue ActiveRecord::RecordNotFound
       render :nothing => true, :status => :not_found
     end
     rescue ActiveRecord::RecordNotFound
       render :nothing => true, :status => :not_found
     end
index 70b96590b834e0341d84a94d8158f2682ba14fb5..d400405c37ada15e9b63da9f78d4624b56c929ae 100644 (file)
@@ -3,7 +3,7 @@
 <li><%= link_to h("API"), :controller => "api/#{API_VERSION}/"+type, :id => obj.id %></li>
 <li><%= link_to h("History"), :controller => "api/#{API_VERSION}/"+type, :id => obj.id, :action => "history" %></li>
 </ul>
 <li><%= link_to h("API"), :controller => "api/#{API_VERSION}/"+type, :id => obj.id %></li>
 <li><%= link_to h("History"), :controller => "api/#{API_VERSION}/"+type, :id => obj.id, :action => "history" %></li>
 </ul>
-Last edited: <%= h(obj.timestamp) %><% if obj.user.data_public %>,
-  by <%= link_to h(obj.user.display_name), :controller => 'user', :action => 'view' , :display_name => obj.user.display_name %>
+Last edited: <%= h(obj.timestamp) %>
+<% if obj.user.data_public %>
+by <%= link_to h(obj.user.display_name), :controller => 'user', :action => 'view' , :display_name => obj.user.display_name %>
 <% end %>
 <% end %>
-
index 3128108d915a9bd19d9d813c927b381558630b03..a90a3fffd5a6710c3c306271d9379df70b31e862 100644 (file)
@@ -16,8 +16,8 @@
   var osm_layer = new OpenLayers.Layer.GML("OSM", url, {format: OpenLayers.Format.OSM, projection: new OpenLayers.Projection("EPSG:4326")});
   osm_layer.events.register("loadend", osm_layer, function() {
     $("loading").innerHTML = ""; 
   var osm_layer = new OpenLayers.Layer.GML("OSM", url, {format: OpenLayers.Format.OSM, projection: new OpenLayers.Projection("EPSG:4326")});
   osm_layer.events.register("loadend", osm_layer, function() {
     $("loading").innerHTML = ""; 
-    this.map.zoomToExtent( this.features[0].geometry.getBounds());
-    var center = map.getCenter().clone().transform(this.map.getProjectionObject(), new OpenLayers.Projection("EPSG:4326"));
+    this.map.zoomToExtent(this.features[0].geometry.getBounds());
+    var center = getMapCenter();
     $("larger_map").href = '/?lat='+center.lat+'&lon='+center.lon+'&zoom='+this.map.getZoom();
     $("larger_map").innerHTML = "View Larger Map";
   })   
     $("larger_map").href = '/?lat='+center.lat+'&lon='+center.lon+'&zoom='+this.map.getZoom();
     $("larger_map").innerHTML = "View Larger Map";
   })