jaculabilis/danbooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

api: fix legacy /post/index and /tag/index endpoints.

Browse Source 
Fixup for a1cd9d2b5. The route order matters here, the legacy endpoints
need to go first.
This commit is contained in:
evazion
2020-12-25 02:43:56 -06:00
parent d9db32640a
commit ae49ed2b1a
2 changed files with 31 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
config/routes.rb
View File

@@ -271,6 +271,12 @@ Rails.application.routes.draw do
end end
end end
# Legacy Danbooru 1 API endpoints
get "/tag/index.xml", :controller => "legacy", :action => "tags", :format => "xml"
get "/tag/index.json", :controller => "legacy", :action => "tags", :format => "json"
get "/post/index.xml", :controller => "legacy", :action => "posts", :format => "xml"
get "/post/index.json", :controller => "legacy", :action => "posts", :format => "json"
# Legacy Danbooru 1 redirects. # Legacy Danbooru 1 redirects.
get "/artist" => redirect {|params, req| "/artists?page=#{req.params[:page]}&search[name]=#{CGI.escape(req.params[:name].to_s)}"} get "/artist" => redirect {|params, req| "/artists?page=#{req.params[:page]}&search[name]=#{CGI.escape(req.params[:name].to_s)}"}
get "/artist/show/:id" => redirect("/artists/%{id}") get "/artist/show/:id" => redirect("/artists/%{id}")
@@ -294,12 +300,6 @@ Rails.application.routes.draw do
get "/wiki/show" => redirect {|params, req| "/wiki_pages?title=#{CGI.escape(req.params[:title].to_s)}"} get "/wiki/show" => redirect {|params, req| "/wiki_pages?title=#{CGI.escape(req.params[:title].to_s)}"}
get "/help/:title" => redirect {|params, req| "/wiki_pages?title=#{CGI.escape('help:' + req.params[:title])}"} get "/help/:title" => redirect {|params, req| "/wiki_pages?title=#{CGI.escape('help:' + req.params[:title])}"}
# Legacy Danbooru 1 API endpoints
get "/tag/index.xml", :controller => "legacy", :action => "tags", :format => "xml"
get "/tag/index.json", :controller => "legacy", :action => "tags", :format => "json"
get "/post/index.xml", :controller => "legacy", :action => "posts", :format => "xml"
get "/post/index.json", :controller => "legacy", :action => "posts", :format => "json"
get "/login", to: "sessions#new", as: :login get "/login", to: "sessions#new", as: :login
get "/logout", to: "sessions#sign_out", as: :logout get "/logout", to: "sessions#sign_out", as: :logout
get "/profile", to: "users#profile", as: :profile get "/profile", to: "users#profile", as: :profile

25
test/functional/legacy_controller_test.rb Normal file
View File

@@ -0,0 +1,25 @@
require 'test_helper'
class LegacyControllerTest < ActionDispatch::IntegrationTest
context "The legacy controller" do
context "post action" do
should "work" do
get "/post/index.xml"
assert_response :success
get "/post/index.json"
assert_response :success
end
end
context "tag action" do
should "work" do
get "/tag/index.xml"
assert_response :success
get "/tag/index.json"
assert_response :success
end
end
end
end