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

posts: fix error on /posts?md5=<does_not_exist>.

Browse Source
This commit is contained in:
evazion
2019-08-13 21:30:21 -05:00
parent 0f98631908
commit dbfd6185f1
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/controllers/posts_controller.rb
View File

@@ -4,7 +4,7 @@ class PostsController < ApplicationController
def index def index
if params[:md5].present? if params[:md5].present?
@post = Post.find_by_md5(params[:md5]) @post = Post.find_by!(md5: params[:md5])
respond_with(@post) do |format| respond_with(@post) do |format|
format.html { redirect_to(@post) } format.html { redirect_to(@post) }
end end

5
test/functional/posts_controller_test.rb
View File

@@ -132,6 +132,11 @@ class PostsControllerTest < ActionDispatch::IntegrationTest
get posts_path, params: { md5: @post.md5 } get posts_path, params: { md5: @post.md5 }
assert_redirected_to(@post) assert_redirected_to(@post)
end end
should "return error on nonexistent md5" do
get posts_path(md5: "foo")
assert_response 404
end
end end
context "with a random search" do context "with a random search" do