X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/226c41be692452129227f30d67f2d847e510015c..8ae5d94b2f16d6f2cf1739e19ebc3793a18a0a4a:/app/models/relation.rb diff --git a/app/models/relation.rb b/app/models/relation.rb index dd4d49258..d5a6d5dc0 100644 --- a/app/models/relation.rb +++ b/app/models/relation.rb @@ -220,8 +220,7 @@ class Relation < ActiveRecord::Base self.lock! check_consistency(self, new_relation, user) # This will check to see if this relation is used by another relation - rel = RelationMember.find(:first, :joins => :relation, - :conditions => [ "visible = ? AND member_type='Relation' and member_id=? ", true, self.id ]) + rel = RelationMember.joins(:relation).where("visible = ? AND member_type = 'Relation' and member_id = ? ", true, self.id).first raise OSM::APIPreconditionFailedError.new("The relation #{new_relation.id} is used in relation #{rel.relation.id}.") unless rel.nil? self.changeset_id = new_relation.changeset_id @@ -279,7 +278,7 @@ class Relation < ActiveRecord::Base # use reflection to look up the appropriate class model = Kernel.const_get(m[0].capitalize) # get the element with that ID - element = model.find(:first, :conditions =>["id = ?", m[1]]) + element = model.where(:id => m[1]).first # and check that it is OK to use. unless element and element.visible? and element.preconditions_ok?