diff options
author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2014-03-15 11:41:24 +0000 |
---|---|---|
committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2014-03-15 11:41:24 +0000 |
commit | c075626462e9cdfa607cd152ccfa661c9a602b2d (patch) | |
tree | 511a4a728c2eb8ac8833ece8073af350a3aeb182 | |
parent | 218e5f40a15932206cf243e83fbdcfad5cc38cd6 (diff) | |
download | redmine-c075626462e9cdfa607cd152ccfa661c9a602b2d.tar.gz redmine-c075626462e9cdfa607cd152ccfa661c9a602b2d.zip |
Fixed that requesting a specific version of a non-existent wiki page raises an error (#16255).
git-svn-id: http://svn.redmine.org/redmine/trunk@12971 e93f8b46-1217-0410-a6f0-8f06a7374b81
-rw-r--r-- | app/models/wiki_page.rb | 8 | ||||
-rw-r--r-- | test/functional/wiki_controller_test.rb | 5 |
2 files changed, 10 insertions, 3 deletions
diff --git a/app/models/wiki_page.rb b/app/models/wiki_page.rb index ba418118f..fc56d9d6b 100644 --- a/app/models/wiki_page.rb +++ b/app/models/wiki_page.rb @@ -104,9 +104,11 @@ class WikiPage < ActiveRecord::Base end def content_for_version(version=nil) - result = content.versions.find_by_version(version.to_i) if version - result ||= content - result + if content + result = content.versions.find_by_version(version.to_i) if version + result ||= content + result + end end def diff(version_to=nil, version_from=nil) diff --git a/test/functional/wiki_controller_test.rb b/test/functional/wiki_controller_test.rb index 79ccb1c77..3e25be08e 100644 --- a/test/functional/wiki_controller_test.rb +++ b/test/functional/wiki_controller_test.rb @@ -161,6 +161,11 @@ class WikiControllerTest < ActionController::TestCase assert_template 'edit' end + def test_show_specific_version_of_an_unexistent_page_without_edit_right + get :show, :project_id => 1, :id => 'Unexistent page', :version => 1 + assert_response 404 + end + def test_show_unexistent_page_with_parent_should_preselect_parent @request.session[:user_id] = 2 get :show, :project_id => 1, :id => 'Unexistent page', :parent => 'Another_page' |