Patch #42394 » 0001-force-format-to-html-for-the-download_named_attachme.patch
config/routes.rb | ||
---|---|---|
316 | 316 | |
317 | 317 |
# additional routes for having the file name at the end of url |
318 | 318 |
get 'attachments/:id/:filename', :to => 'attachments#show', :id => /\d+/, :filename => /.*/, :as => 'named_attachment', :format => 'html' |
319 |
get 'attachments/download/:id/:filename', :to => 'attachments#download', :id => /\d+/, :filename => /.*/, :as => 'download_named_attachment' |
|
319 |
get 'attachments/download/:id/:filename', :to => 'attachments#download', :id => /\d+/, :filename => /.*/, :as => 'download_named_attachment', format: 'html'
|
|
320 | 320 |
get 'attachments/download/:id', :to => 'attachments#download', :id => /\d+/ |
321 | 321 |
get 'attachments/thumbnail/:id(/:size)', :to => 'attachments#thumbnail', :id => /\d+/, :size => /\d+/, :as => 'thumbnail' |
322 | 322 |
resources :attachments, :only => [:show, :update, :destroy] |
test/integration/attachments_test.rb | ||
---|---|---|
267 | 267 |
end |
268 | 268 |
end |
269 | 269 | |
270 |
def test_unauthorized_named_download_link_should_redirect_to_login |
|
271 |
with_settings login_required: '1' do |
|
272 |
get "/attachments/download/1" |
|
273 |
assert_redirected_to "/login?back_url=http%3A%2F%2Fwww.example.com%2Fattachments%2Fdownload%2F1" |
|
274 | ||
275 |
get "/attachments/download/1/error281.txt" |
|
276 |
assert_redirected_to "/login?back_url=http%3A%2F%2Fwww.example.com%2Fattachments%2Fdownload%2F1%2Ferror281.txt" |
|
277 |
end |
|
278 |
end |
|
279 | ||
270 | 280 |
private |
271 | 281 | |
272 | 282 |
def ajax_upload(filename, content, attachment_id=1) |