jro_apps/redmine Commit - r7828:885605b439de

Resourcified attachments....

Jean-Philippe Lang -

r7828:885605b439de

parent child

Context file:

r7828:885605b439de

Collapse all files

app/controllers/attachments_controller.rb +1 -1

               end
-              verify :method => :post, :only => :destroy
+              verify :method => :delete, :only => :destroy
               def destroy
                 # Make sure association callbacks are called
                 @attachment.container.attachments.delete(@attachment)

app/views/attachments/_links.html.erb +2 -2

             <%= h(" - #{attachment.description}") unless attachment.description.blank? %>
               <span class="size">(<%= number_to_human_size attachment.filesize %>)</span>
               <% if options[:deletable] %>
-                <%= link_to image_tag('delete.png'), {:controller => 'attachments', :action => 'destroy', :id => attachment},
+                <%= link_to image_tag('delete.png'), attachment_path(attachment),
                                                      :confirm => l(:text_are_you_sure),
-                                                     :method => :post,
+                                                     :method => :delete,
                                                      :class => 'delete',
                                                      :title => l(:button_delete) %>
               <% end %>

app/views/files/index.html.erb +2 -2

                 <td class="downloads"><%= file.downloads %></td>
                 <td class="digest"><%= file.digest %></td>
                 <td align="center">
-                <%= link_to(image_tag('delete.png'), {:controller => 'attachments', :action => 'destroy', :id => file},
+                <%= link_to(image_tag('delete.png'), attachment_path(file),
-                                                     :confirm => l(:text_are_you_sure), :method => :post) if delete_allowed %>
+                                                     :confirm => l(:text_are_you_sure), :method => :delete) if delete_allowed %>
                 </td>
               </tr>
               <% end

config/routes.rb +3 -3

                 repositories.connect 'projects/:id/repository/:action', :conditions => {:method => :post}
               end
-              map.connect 'attachments/:id', :controller => 'attachments', :action => 'show', :id => /\d+/
+              map.resources :attachments, :only => [:show, :destroy]
-              map.connect 'attachments/:id.:format', :controller => 'attachments', :action => 'show', :id => /\d+/
+              # additional routes for having the file name at the end of url
               map.connect 'attachments/:id/:filename', :controller => 'attachments', :action => 'show', :id => /\d+/, :filename => /.*/
               map.connect 'attachments/download/:id/:filename', :controller => 'attachments', :action => 'download', :id => /\d+/, :filename => /.*/

test/functional/attachments_controller_test.rb +8 -6

                 @request.session[:user_id] = 2
                 assert_difference 'issue.attachments.count', -1 do
-                  post :destroy, :id => 1
+                  delete :destroy, :id => 1
                 end
                 # no referrer
                 assert_redirected_to '/projects/ecookbook'
               def test_destroy_wiki_page_attachment
                 @request.session[:user_id] = 2
                 assert_difference 'Attachment.count', -1 do
-                  post :destroy, :id => 3
+                  delete :destroy, :id => 3
                   assert_response 302
                 end
                 set_tmp_attachments_directory
               def test_destroy_project_attachment
                 @request.session[:user_id] = 2
                 assert_difference 'Attachment.count', -1 do
-                  post :destroy, :id => 8
+                  delete :destroy, :id => 8
                   assert_response 302
                 end
                 set_tmp_attachments_directory
               def test_destroy_version_attachment
                 @request.session[:user_id] = 2
                 assert_difference 'Attachment.count', -1 do
-                  post :destroy, :id => 9
+                  delete :destroy, :id => 9
                   assert_response 302
                 end
                 set_tmp_attachments_directory
               end
               def test_destroy_without_permission
-                post :destroy, :id => 3
+                assert_no_difference 'Attachment.count' do
-                assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fdestroy%2F3'
+                  delete :destroy, :id => 3
+                end
+                assert_response 302
                 assert Attachment.find_by_id(3)
                 set_tmp_attachments_directory
               end

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages