saviour 0.5.7 → 0.5.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2c4a0898f2a80dfb611406f5d9ad72a6abdd8a1dc36716013f7d6c375041a603
-  data.tar.gz: b394cc24a4e380a64948026543489c2f9f40cdc91023d6009642b351e3162dea
+  metadata.gz: 7f06409c24b72526d31e275f0960f5bf754013f3d53497a81a94bef09c574407
+  data.tar.gz: 318b4ab1d8640c87302483853419b03cea9495af7e299c124f3fab8f6be5b8df
 SHA512:
-  metadata.gz: 33b0babd6aac01fda54898026e70fb717b65819c7cbddfc721baa6c0ec5f2bff19b05edbe7b465ff972e3c9ce14db9a9a8ceee180f073d92f92b8c7741fc3748
-  data.tar.gz: ac423bdfc6ed48c368ca92e6e4aeca395ceb51ae4d14d363971b27d656777e8974c5514a6db959ca89154c95c5a8fc484cdad26e478198ea17775abda466250a
+  metadata.gz: dcb4f0d8c74ff3d2bf6354c4012f65e5869009a75ca0d23b773156527767b880c2f328de365a6eb8dc5e371f074dedd37785436c545edf6f3fd034971f819acf
+  data.tar.gz: ff1b6c64937fd164ab1bdc1e5e7d76c81d0116c303696c17b86b40f8514b6e9bb58de18b4fa53bcc79f6ec67a991c9878a6db0e396d06f4db2921c5b9fc6c9ad

data/lib/saviour/file.rb

@@ -54,8 +54,8 @@ module Saviour
       new_file
     end
 
-    def dup
-      new_file = Saviour::File.new(@uploader_klass, @model, @attached_as)
+    def dup(new_model)
+      new_file = Saviour::File.new(@uploader_klass, new_model, @attached_as)
 
       if persisted?
         new_file.assign(Saviour::StringSource.new(read, filename))

data/lib/saviour/model.rb

@@ -24,7 +24,8 @@ module Saviour
       duped = super
 
       self.class.attached_files.each do |attach_as|
-        duped.instance_variable_set("@__uploader_#{attach_as}", send(attach_as).dup)
+        duped[attach_as] = nil
+        duped.instance_variable_set("@__uploader_#{attach_as}", send(attach_as).dup(duped))
       end
 
       duped

data/lib/saviour/version.rb

@@ -1,3 +1,3 @@
 module Saviour
-  VERSION = "0.5.7"
+  VERSION = "0.5.8"
 end

data/spec/feature/crud_workflows_spec.rb

@@ -215,22 +215,28 @@ describe "CRUD" do
       expect(Saviour::Config.storage.exists?(b[:file])).to be_truthy
     end
 
-    it "creates a non persisted file attachment" do
+    it "creates a non persisted file attachment with initially clear db paths" do
       a = klass.create! file: Saviour::StringSource.new("contents", "file.txt")
       expect(Saviour::Config.storage.exists?(a[:file])).to be_truthy
 
       b = a.dup
       expect(b).to_not be_persisted
       expect(b.file).to_not be_persisted
+      expect(b[:file]).to be_nil
     end
 
     it "can be saved" do
       a = klass.create! file: Saviour::StringSource.new("contents", "file.txt")
+      path_a = a[:file]
+
       b = a.dup
       b.save!
+      path_b = b[:file]
 
-      expect(Saviour::Config.storage.exists?(b[:file])).to be_truthy
-      expect(Saviour::Config.storage.read(a[:file])).to eq Saviour::Config.storage.read(b[:file])
+      expect(path_a).to eq "/store/dir/#{a.id}/file.txt"
+      expect(path_b).to eq "/store/dir/#{b.id}/file.txt"
+      expect(Saviour::Config.storage.exists?(path_b)).to be_truthy
+      expect(Saviour::Config.storage.read(path_a)).to eq Saviour::Config.storage.read(path_b)
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: saviour
 version: !ruby/object:Gem::Version
-  version: 0.5.7
+  version: 0.5.8
 platform: ruby
 authors:
 - Roger Campos
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-10 00:00:00.000000000 Z
+date: 2018-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord