Support for cloning file attachments about deep_cloneable

[fro]

It seems it's working... if you override the attribute:

def duplicate
  resource_dup = self.dup :include => :something
  resource_dup.pdf_file = self.pdf_file
  resource_dup.save!
end

[moiristo]

Yes, I've done it like this in one of my projects as well. But I think I created this suggestion with the idea in mind that it should be possible to do this without overriding the dup.

[moiristo]

I think the addition of cloning with a block is sufficient to support cloning file attachments, therefore I'm closing this ticket.

[fro]

Well, it seems it's not working while cloning nested resources with attachments...

[bcackerman]

Agree with Fro, any fixes?

[moiristo]

• What attachment library are you using?
• Do you have a code sample?

[bcackerman]

Hi, I'm using CarrierWave and my code looked like this: http://pastie.org/5556680

[moiristo]

As the implementation for every attachment gem is different, I still don't think that this should be in this gem. My solution in this case would be to pass a block to the dup method to modify the copy. Below is an example. I haven't tried this before (the block option was added by someone else), so I can't guarantee it just works now. I'll add a test for it soon.

def self.duplicate_it(invoice)
  new_invoice = invoice.dup :include => {:lineitems => :images} do |original, kopy|
    kopy.image = original.image if kopy.is_a?(Image)
  end 

  new_invoice.save!
  return new_invoice
end

[bcackerman]

That worked!!!

[bcackerman]

Although this reuploads the file each time which just doesn't work for a lot of records...damn

[moiristo]

I don't think that's something that can be solved easily.. I'd probably go for a solution in which you only copy the 'image' database attribute and perform the file copy in the uploads dir manually.