The list_bbox_rss won't work at all without a bbox. Thow an error when it's not supplied
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Sun, 7 Jun 2009 03:06:09 +0000 (03:06 +0000)
committerÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Sun, 7 Jun 2009 03:06:09 +0000 (03:06 +0000)
app/controllers/changeset_controller.rb

index 959957463097d53ae93de2588c52ba4bc94f1c32..3d5c8dd4bf1ebd7d278d0ec595466fbe15b48c94 100644 (file)
@@ -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);