def create
if request.put?
node = Node.from_xml(request.raw_post, true)
+ # FIXME remove debug
+ logger.debug request.raw_post
+ logger.debug node
if node
node.version = 0
- node.user_id = @user.id
+ #node.changeset_id = node.changeset
node.visible = true
node.save_with_history!
def read
begin
node = Node.find(params[:id])
- if node.visible
+ if node.visible?
response.headers['Last-Modified'] = node.timestamp.rfc822
render :text => node.to_xml.to_s, :content_type => "text/xml"
else
render :nothing => true, :status => :not_found
end
end
+
+ # Dump a specific version of the node based on the given params[:id] and params[:version]
+ def version
+ begin
+ node = Node.find(:first, :conditions => { :id => params[:id], :version => params[:version] } )
+ if node.visible
+ response.headers['Last-Modified'] = node.timestamp.rfc822
+ render :text => node.to_xml.to_s, :content_type => "text/xml"
+ else
+ render :nothing => true, :status => :gone
+ end
+ rescue ActiveRecord::RecordNotFound
+ render :nothing => true, :status => :not_found
+ end
+ end
# Update a node from given XML
def update
else
render :nothing => true, :status => :bad_request
end
- rescue OSM::APIVersionMismatchError ex
+ rescue OSM::APIVersionMismatchError => ex
render :text => "Version mismatch: Provided " + ex.provided.to_s +
", server had: " + ex.latest.to_s, :status => :bad_request
rescue ActiveRecord::RecordNotFound
def delete
begin
node = Node.find(params[:id])
+ # FIXME we no longer care about the user, (or maybe we want to check
+ # that the user of the changeset is the same user as is making this
+ # little change?) we really care about the
+ # changeset which must be open, and that the version that we have been
+ # given is the one that is currently stored in the database
node.delete_with_history(@user)
+
+ render :nothing => true
rescue ActiveRecord::RecordNotFound
render :nothing => true, :status => :not_found
- rescue OSM::APIAlreadyDeletedError
- render :text => "", :status => :gone
- rescue OSM::APIPreconditionFailedError
- render :text => "", :status => :precondition_failed
+ rescue OSM::APIError => ex
+ render ex.render_opts
end
end