class OldNodeController < ApplicationController
require 'xml/libxml'
- session :off
+ skip_before_filter :verify_authenticity_token
before_filter :check_api_readable
after_filter :compress_output
+ around_filter :api_call_handle_error, :api_call_timeout
def history
- begin
- node = Node.find(params[:id])
-
- doc = OSM::API.new.get_xml_doc
-
- node.old_nodes.each do |old_node|
- doc.root << old_node.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
+ node = Node.find(params[:id])
+
+ doc = OSM::API.new.get_xml_doc
+
+ node.old_nodes.each do |old_node|
+ doc.root << old_node.to_xml_node
end
+
+ render :text => doc.to_s, :content_type => "text/xml"
end
def version
- begin
- old_node = OldNode.find(:first, :conditions => {:id => params[:id], :version => params[:version]} )
-
- response.headers['Last-Modified'] = old_node.timestamp.rfc822
+ if old_node = OldNode.where(:node_id => params[:id], :version => params[:version]).first
+ response.last_modified = old_node.timestamp
doc = OSM::API.new.get_xml_doc
doc.root << old_node.to_xml_node
render :text => doc.to_s, :content_type => "text/xml"
- rescue ActiveRecord::RecordNotFound
+ else
render :nothing => true, :status => :not_found
- rescue
- render :nothing => true, :status => :internal_server_error
end
end
end