Merge pull request #112 from die-antwort/fix-remove-file-in-embedded-…

…document Fix removing uploaded files in embedded documents.
carrierwaveuploader · Sep 3, 2013 · 83d9961 · 83d9961
2 parents bb18692 + 6b2b516
commit 83d9961
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/lib/carrierwave/mongoid.rb b/lib/carrierwave/mongoid.rb
@@ -43,6 +43,12 @@ def #{column}=(new_file)
           super
         end
 
+        def remove_#{column}=(arg)
+          column = _mounter(:#{column}).serialization_column
+          send(:"\#{column}_will_change!")
+          super
+        end
+
         def remove_#{column}!
           super unless respond_to?(:paranoid?) && paranoid? && flagged_for_destroy?
         end

diff --git a/spec/mongoid_spec.rb b/spec/mongoid_spec.rb
@@ -599,6 +599,12 @@ def filename
         @doc.mongo_locations.first[:image].should == 'test.jpeg'
       end
 
+      it "removes a file" do
+        @doc.update_attributes mongo_locations_attributes: { '0' => { _id: @embedded_doc._id, remove_image: "1" } }
+        @doc.reload
+        @doc.mongo_locations.first[:image].should_not be_present
+      end
+
       describe 'with double embedded documents' do
 
         before do