diff --git a/app/controllers/dmsf_upload_controller.rb b/app/controllers/dmsf_upload_controller.rb
index 4af91c1e..e29c2f4c 100644
--- a/app/controllers/dmsf_upload_controller.rb
+++ b/app/controllers/dmsf_upload_controller.rb
@@ -122,6 +122,7 @@ class DmsfUploadController < ApplicationController
         if upload
           uploaded_file[:disk_filename] = upload.disk_filename
           uploaded_file[:tempfile_path] = upload.tempfile_path
+          uploaded_file[:size] = upload.size
         end
       end
       commit_files_internal uploaded_files
diff --git a/app/views/dmsf/_dir_trash.html.erb b/app/views/dmsf/_dir_trash.html.erb
index 563148b1..0d5425a9 100644
--- a/app/views/dmsf/_dir_trash.html.erb
+++ b/app/views/dmsf/_dir_trash.html.erb
@@ -75,7 +75,7 @@
         :data => {:confirm => l(:text_are_you_sure)},
         :method => :delete,
         :title => l(:title_delete),
-        :class => 'icon-only icon-delete') %>
+        :class => 'icon-only icon-del') %>
     <% else %>
       <%= link_to('', delete_dmsf_path(:id => project, :folder_id => subfolder, :commit => 'yes'),
         :data => {:confirm => l(:text_are_you_sure)},
diff --git a/app/views/dmsf/_url.html.erb b/app/views/dmsf/_url.html.erb
index d3ff39b0..80d67d4e 100644
--- a/app/views/dmsf/_url.html.erb
+++ b/app/views/dmsf/_url.html.erb
@@ -78,7 +78,7 @@
     <span class="icon"></span>
   <% end %>
 </td>
-<td class="hol"><%= position %> </td>
-<td class="hol"></td>
-<td class="hol">link.updated_at.to_i</td>
-<td class="hol"></td>
+<td class="none"><%= position %> </td>
+<td class="none"></td>
+<td class="none">link.updated_at.to_i</td>
+<td class="none"></td>
diff --git a/app/views/dmsf/_url_trash.html.erb b/app/views/dmsf/_url_trash.html.erb
index 5d73b077..9ffb09ba 100644
--- a/app/views/dmsf/_url_trash.html.erb
+++ b/app/views/dmsf/_url_trash.html.erb
@@ -64,10 +64,10 @@
   <% if @file_delete_allowed %>
     <%= link_to_if(@file_delete_allowed, '', dmsf_link_path(:id => link, :commit => 'yes'),
       :data => {:confirm => l(:text_are_you_sure)}, :method => :delete,
-      :title => l(:title_delete), :class => "icon icon-delete" ) %>
+      :title => l(:title_delete), :class => "icon icon-del" ) %>
   <% end %>
 </td>
-<td class="hol">1</td>
-<td class="hol"></td>
-<td class="hol">link.updated_at.to_i</td>
-<td class="hol"></td>
+<td class="none">1</td>
+<td class="none"></td>
+<td class="none">link.updated_at.to_i</td>
+<td class="none"></td>
diff --git a/app/views/dmsf/trash.html.erb b/app/views/dmsf/trash.html.erb
index 93f2c5f2..28991560 100644
--- a/app/views/dmsf/trash.html.erb
+++ b/app/views/dmsf/trash.html.erb
@@ -81,10 +81,10 @@
             <% end %>
         <% end %>
         <th class ="dmsf_th"><%# controls %></th>
-        <th class="hol"><%# position %></th>
-        <th class="hol"><%# size %></th>
-        <th class="hol"><%# updated %></th>
-        <th class="hol"><%# version %></th>
+        <th class="none"><%# position %></th>
+        <th class="none"><%# size %></th>
+        <th class="none"><%# updated %></th>
+        <th class="none"><%# version %></th>
       </tr>
     </thead>
   <tbody>
diff --git a/extra/api/file.xml b/extra/api/file.xml
index f1c19385..eec8bb38 100644
--- a/extra/api/file.xml
+++ b/extra/api/file.xml
@@ -2,7 +2,6 @@
 <attachments>
   <folder_id>6118</folder_id>
   <uploaded_file>
-    <disk_filename>cat.gif</disk_filename>
     <name>cat.gif</name>
     <title>cat.gif</title>
     <description>REST API</description>
diff --git a/test/integration/rest_api/dmsf_file_api_test.rb b/test/integration/rest_api/dmsf_file_api_test.rb
index 45535e91..719e8d2e 100644
--- a/test/integration/rest_api/dmsf_file_api_test.rb
+++ b/test/integration/rest_api/dmsf_file_api_test.rb
@@ -99,7 +99,6 @@ class DmsfFileApiTest < RedmineDmsf::Test::IntegrationTest
       <attachments>
        <folder_id/>
        <uploaded_file>
-         <disk_filename>test.txt</disk_filename>
          <name>test.txt</name>
          <title>test.txt</title>
          <description>REST API</description>
@@ -109,16 +108,20 @@ class DmsfFileApiTest < RedmineDmsf::Test::IntegrationTest
        </uploaded_file>
       </attachments>
     }
-    post "/projects/#{@project1.id}/dmsf/commit.xml?&key=#{token.value}", payload, {"CONTENT_TYPE" => 'application/xml'}
+    assert_difference 'DmsfFileRevision.count', +1 do
+      post "/projects/#{@project1.id}/dmsf/commit.xml?&key=#{token.value}", payload, {"CONTENT_TYPE" => 'application/xml'}
+    end
     #<?xml version="1.0" encoding="UTF-8"?>
     #<dmsf_files total_count="1" type="array">
     # <file>
     #   <id>17229</id>
     #   <name>test.txt</name>
     # </file>
-    # </dmsf_files>
+    # </dmsf_files> #
     assert_select 'dmsf_files > file > name', :text => 'test.txt'
     assert_response :success
+    revision = DmsfFileRevision.order(:id).last
+    assert revision && revision.size > 0
     begin
       FileUtils.rm_rf Setting.plugin_redmine_dmsf['dmsf_storage_directory']
     rescue Exception => e