X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/fd6f00b069af5a7b87227ea4c3401dc461d7311a..1db6b392a6c9f255d74669d8ae6db67934a2ab54:/app/controllers/old_way_controller.rb?ds=sidebyside diff --git a/app/controllers/old_way_controller.rb b/app/controllers/old_way_controller.rb index a42496687..1e3546a0d 100644 --- a/app/controllers/old_way_controller.rb +++ b/app/controllers/old_way_controller.rb @@ -4,39 +4,33 @@ class OldWayController < ApplicationController session :off before_filter :check_api_readable after_filter :compress_output + around_filter :api_call_handle_error, :api_call_timeout def history - begin - way = Way.find(params[:id]) + way = Way.find(params[:id]) - doc = OSM::API.new.get_xml_doc - - way.old_ways.each do |old_way| - doc.root << old_way.to_xml_node - end - - render :text => doc.to_s, :content_type => "text/xml" - rescue ActiveRecord::RecordNotFound - render :nothing => true, :status => :not_found - rescue - render :nothing => true, :status => :internal_server_error + doc = OSM::API.new.get_xml_doc + + way.old_ways.each do |old_way| + doc.root << old_way.to_xml_node end + + render :text => doc.to_s, :content_type => "text/xml" end def version - begin - old_way = OldWay.find(:first, :conditions => {:id => params[:id], :version => params[:version]} ) - - response.headers['Last-Modified'] = old_way.timestamp.rfc822 - - doc = OSM::API.new.get_xml_doc - doc.root << old_way.to_xml_node - - render :text => doc.to_s, :content_type => "text/xml" - rescue ActiveRecord::RecordNotFound + old_way = OldWay.find(:first, :conditions => {:id => params[:id], :version => params[:version]} ) + if old_way.nil? + # (RecordNotFound is not raised with find :first...) render :nothing => true, :status => :not_found - rescue - render :nothing => true, :status => :internal_server_error + return end + + response.headers['Last-Modified'] = old_way.timestamp.rfc822 + + doc = OSM::API.new.get_xml_doc + doc.root << old_way.to_xml_node + + render :text => doc.to_s, :content_type => "text/xml" end end