way = Way.from_xml(request.raw_post, true)
if way
- if !way.preconditions_ok?
+ unless way.preconditions_ok?
render :text => "", :status => :precondition_failed
else
- way.user_id = @user.id
+ way.version = 0
way.save_with_history!
render :text => way.id.to_s, :content_type => "text/plain"
begin
way = Way.find(params[:id])
+ response.headers['Last-Modified'] = way.timestamp.rfc822
+
if way.visible
render :text => way.to_xml.to_s, :content_type => "text/xml"
else
render :text => "", :status => :gone
end
+ rescue OSM::APIError => ex
+ render ex.render_opts
rescue ActiveRecord::RecordNotFound
render :nothing => true, :status => :not_found
end
new_way = Way.from_xml(request.raw_post)
if new_way and new_way.id == way.id
- if !new_way.preconditions_ok?
- render :text => "", :status => :precondition_failed
- else
- way.user_id = @user.id
- way.tags = new_way.tags
- way.nds = new_way.nds
- way.visible = true
- way.save_with_history!
-
- render :nothing => true
- end
+ way.update_from(new_way, @user)
+ render :text => way.version.to_s, :content_type => "text/plain"
else
render :nothing => true, :status => :bad_request
end
+ rescue OSM::APIError => ex
+ render ex.render_opts
rescue ActiveRecord::RecordNotFound
render :nothing => true, :status => :not_found
end
end
+ # This is the API call to delete a way
def delete
begin
way = Way.find(params[:id])
+ way.delete_with_history(@user)
- if way.visible
- if RelationMember.find(:first, :joins => "INNER JOIN current_relations ON current_relations.id=current_relation_members.id", :conditions => [ "visible = 1 AND member_type='way' and member_id=?", params[:id]])
- render :text => "", :status => :precondition_failed
- else
- way.user_id = @user.id
- way.tags = []
- way.nds = []
- way.visible = false
- way.save_with_history!
-
- render :nothing => true
- end
- else
- render :text => "", :status => :gone
- end
+ # if we get here, all is fine, otherwise something will catch below.
+ render :nothing => true
+ rescue OSM::APIError => ex
+ render ex.render_opts
rescue ActiveRecord::RecordNotFound
render :nothing => true, :status => :not_found
- rescue => ex
- puts ex
end
end
end
def ways_for_node
- wayids = WayNode.find(:all, :conditions => ['node_id = ?', params[:id]]).collect { |ws| ws.id }.uniq
+ wayids = WayNode.find(:all, :conditions => ['node_id = ?', params[:id]]).collect { |ws| ws.id[0] }.uniq
doc = OSM::API.new.get_xml_doc