- <%= link_to l(:label_user_search_add),
+ <%= link_to sprite_icon('add', l(:label_search_for_watchers), size: 12),
new_dmsf_folder_permissions_path(project_id: @project, dmsf_folder_id: @folder),
remote: true,
method: :get %>
diff --git a/app/views/dmsf/trash.html.erb b/app/views/dmsf/trash.html.erb
index d6efeb9b..fd051282 100644
--- a/app/views/dmsf/trash.html.erb
+++ b/app/views/dmsf/trash.html.erb
@@ -22,7 +22,7 @@
<% if @file_delete_allowed %>
- <%= link_to sprite_icon('del', l(:label_empty_trash_bin)), empty_trash_path(id: @project), class: 'icon icon-del' %>
+ <%= delete_link empty_trash_path(id: @project), {}, l(:label_empty_trash_bin) %>
<% end %>
diff --git a/app/views/dmsf_context_menus/_revision_actions.html.erb b/app/views/dmsf_context_menus/_revision_actions.html.erb
index 44de9aaf..012c7912 100644
--- a/app/views/dmsf_context_menus/_revision_actions.html.erb
+++ b/app/views/dmsf_context_menus/_revision_actions.html.erb
@@ -30,8 +30,9 @@
obsolete_revision_path(revision),
data: { confirm: l(:text_are_you_sure) },
class: 'icon icon-close dmsf-revision-action-button' if file_manipulation_allowed && (revision.workflow == DmsfWorkflow::STATE_APPROVED) %>
-<%= link_to sprite_icon('del', l(:button_delete)),
- delete_revision_path(revision),
- data: { confirm: l(:text_are_you_sure) },
- title: l(:title_delete_revision),
- class: 'icon icon-del dmsf-revision-action-button' if file_delete_allowed && (file.dmsf_file_revisions.visible.count > 1) %>
+<% if file_delete_allowed && (file.dmsf_file_revisions.visible.count > 1) %>
+ <%= delete_link delete_revision_path(revision),
+ { data: { confirm: l(:text_are_you_sure) },
+ title: l(:title_delete_revision),
+ class: 'icon icon-del dmsf-revision-action-button' } %>
+<% end %>
\ No newline at end of file
diff --git a/app/views/dmsf_files/_version_selector.html.erb b/app/views/dmsf_files/_version_selector.html.erb
index e5393f3a..940afdf2 100644
--- a/app/views/dmsf_files/_version_selector.html.erb
+++ b/app/views/dmsf_files/_version_selector.html.erb
@@ -38,9 +38,9 @@
<% minor_version = '' %>
<% end %>
<%= select_tag select_tag_name_minor,
- options_for_select(DmsfUploadHelper::minor_version_select_options, minor_version),
- class: 'dmsf-select-version' %>
+ options_for_select(DmsfUploadHelper::minor_version_select_options, minor_version),
+ class: 'dmsf-select-version' %>
.
- <%= select_tag select_tag_name_patch, options_for_select(DmsfUploadHelper::patch_version_select_options,
- [h(' '), '']), class: 'dmsf-select-version' %>
+ <%= select_tag select_tag_name_patch, options_for_select(DmsfUploadHelper::patch_version_select_options, ' '),
+ class: 'dmsf-select-version' %>
<% end %>
diff --git a/assets/images/jstoolbar/bt_dmsf.png b/assets/images/jstoolbar/bt_dmsf.png
deleted file mode 100644
index 51ef5b19..00000000
Binary files a/assets/images/jstoolbar/bt_dmsf.png and /dev/null differ
diff --git a/assets/stylesheets/redmine_dmsf.css b/assets/stylesheets/redmine_dmsf.css
index ee9a5ad2..3de2a63f 100644
--- a/assets/stylesheets/redmine_dmsf.css
+++ b/assets/stylesheets/redmine_dmsf.css
@@ -394,5 +394,5 @@ div.dmsf-scroll {
/* Wiki toolbar */
.jstb_dmsf {
- background-image: url("../images/jstoolbar/bt_dmsf.png");
+ background-image: url("/document.png");
}
\ No newline at end of file
diff --git a/config/routes.rb b/config/routes.rb
index fa60f7fc..11795207 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -54,7 +54,7 @@ if Redmine::Plugin.installed? 'redmine_dmsf'
post '/projects/dmsf/append_email', to: 'dmsf#append_email', as: 'append_email_dmsf'
get '/projects/dmsf/autocomplete_for_user', to: 'dmsf#autocomplete_for_user'
put '/projects/:id/dmsf', controller: 'dmsf', action: 'drop'
- get '/projects/:id/dmsf/empty_trash', to: 'dmsf#empty_trash', as: 'empty_trash'
+ delete '/projects/:id/dmsf/empty_trash', to: 'dmsf#empty_trash', as: 'empty_trash'
get '/dmsf', to: 'dmsf#index', as: 'dmsf_index'
get '/dmsf/digest', to: 'dmsf#digest', as: 'dmsf_digest'
post '/dmsf/digest', to: 'dmsf#reset_digest', as: 'dmsf_reset_digest'
@@ -114,7 +114,7 @@ if Redmine::Plugin.installed? 'redmine_dmsf'
get '/dmsf/files/:id/unlock', controller: 'dmsf_files', action: 'unlock', as: 'unlock_dmsf_files'
post '/dmsf/files/:id/delete', controller: 'dmsf_files', action: 'delete', as: 'delete_dmsf_files'
post '/dmsf/files/:id/revision/create', controller: 'dmsf_files', action: 'create_revision'
- get '/dmsf/files/:id/revision/delete', controller: 'dmsf_files', action: 'delete_revision', as: 'delete_revision'
+ delete '/dmsf/files/:id/revision/delete', controller: 'dmsf_files', action: 'delete_revision', as: 'delete_revision'
get '/dmsf/files/:id/revision/obsolete', controller: 'dmsf_files',
action: 'obsolete_revision',
as: 'obsolete_revision'
diff --git a/test/functional/dmsf_controller_test.rb b/test/functional/dmsf_controller_test.rb
index 916c91a2..a25ac74a 100644
--- a/test/functional/dmsf_controller_test.rb
+++ b/test/functional/dmsf_controller_test.rb
@@ -106,7 +106,7 @@ class DmsfControllerTest < RedmineDmsf::Test::TestCase
def test_empty_trash
post '/login', params: { username: 'jsmith', password: 'jsmith' }
- get "/projects/#{@project1.id}/dmsf/empty_trash"
+ delete "/projects/#{@project1.id}/dmsf/empty_trash"
assert_equal 0, DmsfFolder.deleted.where(project_id: @project1.id).all.size
assert_equal 0, DmsfFile.deleted.where(project_id: @project1.id).all.size
assert_equal 0, DmsfLink.deleted.where(project_id: @project1.id).all.size
@@ -117,7 +117,7 @@ class DmsfControllerTest < RedmineDmsf::Test::TestCase
# Missing permissions
post '/login', params: { username: 'jsmith', password: 'jsmith' }
@role_manager.remove_permission! :file_delete
- get "/projects/#{@project1.id}/dmsf/empty_trash"
+ delete "/projects/#{@project1.id}/dmsf/empty_trash"
assert_response :forbidden
end