X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/495bd7f1f077e5cae4428fab8a780f0f479893d0..a90be5e69a478e2b49ae676b649a78589f85a50e:/app/controllers/changeset_controller.rb diff --git a/app/controllers/changeset_controller.rb b/app/controllers/changeset_controller.rb index e16a4e9b3..181c827b3 100644 --- a/app/controllers/changeset_controller.rb +++ b/app/controllers/changeset_controller.rb @@ -53,7 +53,11 @@ class ChangesetController < ApplicationController raise OSM::APIUserChangesetMismatchError end - changeset.open = false + # to close the changeset, we'll just set its closed_at time to + # now. this might not be enough if there are concurrency issues, + # but we'll have to wait and see. + changeset.closed_at = DateTime.now + changeset.save! render :nothing => true rescue ActiveRecord::RecordNotFound @@ -361,10 +365,10 @@ class ChangesetController < ApplicationController raise OSM::APIBadUserInput.new("bad time range") if times.size != 2 from, to = times.collect { |t| DateTime.parse(t) } - return ['created_at > ? and created_at < ?', from, to] + return ['closed_at >= ? and created_at <= ?', from, to] else # if there is no comma, assume its a lower limit on time - return ['created_at > ?', DateTime.parse(time)] + return ['closed_at >= ?', DateTime.parse(time)] end else return nil @@ -380,7 +384,7 @@ class ChangesetController < ApplicationController ## # restrict changes to those which are open def conditions_open(open) - return open.nil? ? nil : ['open = ?', true] + return open.nil? ? nil : ['closed_at >= ?', DateTime.now] end end