ruby on rails: duplicate images uploaded with each form submission

Alexander Rousmaniere

January 13, 2015 00:59

I'm uploading to Cloudinary from an external URL using this Ruby On Rails code:

class Item < ActiveRecord::Base

validates :picture_link, presence: true

 before_save :get_cloudinary_url

def get_cloudinary_url

_ upload = Cloudinary::Uploader.upload(self.picture_link.to_s)_

_ self.secure_url = upload["secure_url"]_

_ self.public_id = upload["public_id"]_

 

This works well, but EVERY time the form is submitted, a duplicate image gets uploaded. Ideally I'd like to tell rails to only the get_cloudinary_url _function in the case that the _picture_link has been actually changed. 

0

• 

  Alexander Rousmaniere

  January 13, 2015 01:26

  I tried adjusting the function to this

   

  def get_cloudinary_url

  #get from cloudinary the secure_url and add it to the item.
  
  _ if self.changed?_
  
  _ upload = Cloudinary::Uploader.upload(self.picture_link.to_s)_
  
  _ self.secure_url = upload["secure_url"]_
  
  _ self.public_id = upload["public_id"] _
  
  _ end_
  
  _ end_

   

   

  but still duplicate images are uploaded.

  0
• 

  Itay Taragano

  January 13, 2015 15:19

  Can you please try the following and let me know if this helps? :

  if self._picture_link__changed?
  
  _ ...
  _

  You might want to consider using our CarrierWave integration to simplify this process.

  0
• 

  Alexander Rousmaniere

  January 13, 2015 23:22

  I had tried the _changed method, but I think that returns an array, not a boolean -- or maybe it was something else causing problems.

  In the end, I used:

   

  if self.picture_link_was != self.picture_link

  ...... and so far this is working

   

  0
• 

  Itay Taragano

  January 14, 2015 13:59

  The _changed? method is a boolean. The .changes  method returns an array. See:  http://api.rubyonrails.org/classes/ActiveModel/Dirty.html

  However, I'm glad it works for you now :)

  0

Post is closed for comments.