Merge pull request #1544 from danielevans/bug/activerecord_dup

Flush mounters on dup of active record model
carrierwaveuploader · Feb 26, 2015 · 79b8e24 · 79b8e24
2 parents bbef19c + b416406
commit 79b8e24
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/lib/carrierwave/orm/activerecord.rb b/lib/carrierwave/orm/activerecord.rb
@@ -85,6 +85,12 @@ def reload(*)
           @_mounters = nil
           super
         end
+
+        # Reset cached mounter on record dup
+        def initialize_dup(other)
+          @_mounters = nil
+          super
+        end
       RUBY
     end
 

diff --git a/spec/orm/activerecord_spec.rb b/spec/orm/activerecord_spec.rb
@@ -1495,4 +1495,12 @@ def filename
       expect { @event.with_lock {} }.to_not raise_error
     end
   end
+
+  describe "#dup" do
+    it "appropriately removes the model reference from the new models uploader" do
+      @event.save
+      new_event = @event.dup
+      expect(new_event.image.model).not_to eq @event
+    end
+  end
 end