| @@ -93,9 +93,6 class WikiController < ApplicationController | |||||
| 93 |
|
93 | |||
| 94 | # To prevent StaleObjectError exception when reverting to a previous version |
|
94 | # To prevent StaleObjectError exception when reverting to a previous version | |
| 95 | @content.version = @page.content.version |
|
95 | @content.version = @page.content.version | |
| 96 | rescue ActiveRecord::StaleObjectError |
|
|||
| 97 | # Optimistic locking exception |
|
|||
| 98 | flash[:error] = l(:notice_locking_conflict) |
|
|||
| 99 | end |
|
96 | end | |
| 100 |
|
97 | |||
| 101 | verify :method => :put, :only => :update, :render => {:nothing => true, :status => :method_not_allowed } |
|
98 | verify :method => :put, :only => :update, :render => {:nothing => true, :status => :method_not_allowed } | |
| @@ -131,7 +128,8 class WikiController < ApplicationController | |||||
| 131 |
|
128 | |||
| 132 | rescue ActiveRecord::StaleObjectError |
|
129 | rescue ActiveRecord::StaleObjectError | |
| 133 | # Optimistic locking exception |
|
130 | # Optimistic locking exception | |
| 134 | flash[:error] = l(:notice_locking_conflict) |
|
131 | flash.now[:error] = l(:notice_locking_conflict) | |
|
|
132 | render :action => 'edit' | |||
| 135 | end |
|
133 | end | |
| 136 |
|
134 | |||
| 137 | # rename a page |
|
135 | # rename a page | |
| @@ -155,6 +155,42 class WikiControllerTest < ActionController::TestCase | |||||
| 155 | assert_tag :tag => 'input', :attributes => {:id => 'content_version', :value => '1'} |
|
155 | assert_tag :tag => 'input', :attributes => {:id => 'content_version', :value => '1'} | |
| 156 | end |
|
156 | end | |
| 157 |
|
157 | |||
|
|
158 | def test_update_stale_page_should_not_raise_an_error | |||
|
|
159 | @request.session[:user_id] = 2 | |||
|
|
160 | c = Wiki.find(1).find_page('Another_page').content | |||
|
|
161 | c.text = 'Previous text' | |||
|
|
162 | c.save! | |||
|
|
163 | assert_equal 2, c.version | |||
|
|
164 | ||||
|
|
165 | assert_no_difference 'WikiPage.count' do | |||
|
|
166 | assert_no_difference 'WikiContent.count' do | |||
|
|
167 | assert_no_difference 'WikiContent::Version.count' do | |||
|
|
168 | put :update, :project_id => 1, | |||
|
|
169 | :id => 'Another_page', | |||
|
|
170 | :content => { | |||
|
|
171 | :comments => 'My comments', | |||
|
|
172 | :text => 'Text should not be lost', | |||
|
|
173 | :version => 1 | |||
|
|
174 | } | |||
|
|
175 | end | |||
|
|
176 | end | |||
|
|
177 | end | |||
|
|
178 | assert_response :success | |||
|
|
179 | assert_template 'edit' | |||
|
|
180 | assert_tag :div, | |||
|
|
181 | :attributes => { :class => /error/ }, | |||
|
|
182 | :content => /Data has been updated by another user/ | |||
|
|
183 | assert_tag 'textarea', | |||
|
|
184 | :attributes => { :name => 'content[text]' }, | |||
|
|
185 | :content => /Text should not be lost/ | |||
|
|
186 | assert_tag 'input', | |||
|
|
187 | :attributes => { :name => 'content[comments]', :value => 'My comments' } | |||
|
|
188 | ||||
|
|
189 | c.reload | |||
|
|
190 | assert_equal 'Previous text', c.text | |||
|
|
191 | assert_equal 2, c.version | |||
|
|
192 | end | |||
|
|
193 | ||||
| 158 | def test_preview |
|
194 | def test_preview | |
| 159 | @request.session[:user_id] = 2 |
|
195 | @request.session[:user_id] = 2 | |
| 160 | xhr :post, :preview, :project_id => 1, :id => 'CookBook_documentation', |
|
196 | xhr :post, :preview, :project_id => 1, :id => 'CookBook_documentation', | |
General Comments 0
You need to be logged in to leave comments.
Login now