From: Ævar Arnfjörð Bjarmason Date: Sun, 7 Jun 2009 03:06:09 +0000 (+0000) Subject: The list_bbox_rss won't work at all without a bbox. Thow an error when it's not supplied X-Git-Tag: live~7153 X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/e9ab43cd302926f5741f9f1cd16ffb79e0f88482?hp=b1117365d24398aa30ba7cb2cf2b7f31c41d0f01 The list_bbox_rss won't work at all without a bbox. Thow an error when it's not supplied --- diff --git a/app/controllers/changeset_controller.rb b/app/controllers/changeset_controller.rb index 959957463..3d5c8dd4b 100644 --- a/app/controllers/changeset_controller.rb +++ b/app/controllers/changeset_controller.rb @@ -374,12 +374,7 @@ class ChangesetController < ApplicationController elsif params['minlon'] and params['minlat'] and params['maxlon'] and params['maxlat'] bbox = h(params['minlon']) + ',' + h(params['minlat']) + ',' + h(params['maxlon']) + ',' + h(params['maxlat']) else - #TODO: fix bugs in location determination for history tab (and other tabs) then uncomment this redirect - #redirect_to :action => 'list' - - # For now just render immediately, and skip the db - render - return + raise OSM::APIBadUserInput.new("Bounding box must be supplied for /history/rss") end conditions = conditions_bbox(bbox);