From: Tom Hughes Date: Tue, 25 Oct 2022 18:28:10 +0000 (+0100) Subject: Merge remote-tracking branch 'upstream/pull/3768' X-Git-Tag: live~1535 X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/3d0b94abdafeb04cbdece55ab21d3fb725d59137?hp=a544895e6c82f29ff20dbcb588fbb9359120d8d1 Merge remote-tracking branch 'upstream/pull/3768' --- diff --git a/app/controllers/api/notes_controller.rb b/app/controllers/api/notes_controller.rb index c152c6272..d915d8126 100644 --- a/app/controllers/api/notes_controller.rb +++ b/app/controllers/api/notes_controller.rb @@ -35,6 +35,10 @@ module Api # Check the the bounding box is not too big bbox.check_size(Settings.max_note_request_area) + @min_lon = bbox.min_lon + @min_lat = bbox.min_lat + @max_lon = bbox.max_lon + @max_lat = bbox.max_lat # Find the notes we want to return @notes = notes.bbox(bbox).order("updated_at DESC").limit(result_limit).preload(:comments) @@ -190,6 +194,10 @@ module Api bbox.check_size(Settings.max_note_request_area) notes = notes.bbox(bbox) + @min_lon = bbox.min_lon + @min_lat = bbox.min_lat + @max_lon = bbox.max_lon + @max_lat = bbox.max_lat end # Find the comments we want to return diff --git a/test/controllers/api/notes_controller_test.rb b/test/controllers/api/notes_controller_test.rb index f7e7c77f4..31c55b3a8 100644 --- a/test/controllers/api/notes_controller_test.rb +++ b/test/controllers/api/notes_controller_test.rb @@ -632,6 +632,7 @@ module Api assert_equal "application/rss+xml", @response.media_type assert_select "rss", :count => 1 do assert_select "channel", :count => 1 do + assert_select "description", :text => /1\.2/, :count => 1 assert_select "item", :count => 2 end end @@ -1032,6 +1033,7 @@ module Api assert_equal "application/rss+xml", @response.media_type assert_select "rss", :count => 1 do assert_select "channel", :count => 1 do + assert_select "description", :text => /1\.2/, :count => 1 assert_select "item", :count => 2 end end