Comments on: Attributes not saving for models with attachments

By: Kostas

Kostas — Wed, 08 Jul 2009 07:58:50 +0000

I believe that the best course of action is to comment out the line from the plugin and add it manually to whichever model actually needs it.
This way you keep security and control on your code.

By: Lex

Lex — Sun, 31 May 2009 04:29:18 +0000

I had a similar issue where I had set an attribute (id) that was not a model accessor. I chose to manually set the id attribute and then remove it from the hash — rather than make the id an accessor:

1. def image_attributes=(image_attributes)
2. image_attributes.each do |attributes|
3. if attributes[:id].blank?
4. photos.build(attributes)
5. else
6. photo = photos.detect { |t| t.id == attributes[:id].to_i}
7. photo.id = attributes[:id]
8. attributes.delete(“id”)
9. photo.attributes = attributes
10. end
11. end
12. end

We get alot of speed with the Rails framework, e.g., Mass Assignment Hashes…but can spend alot of time debugging it, too.

By: nahum

nahum — Mon, 16 Feb 2009 03:06:11 +0000

Doing that is still a huge pain to maintain though as the same situation could occur later when someone else adds a new field to the model. I’m not working for that client anymore, so I’ve told them what I did and left some hard to miss comments but ….

When it comes down to it I don’t think a plugin should be doing that as it’s business logic which should lie on the model itself rather than hidden somewhere else.

By: dw

dw — Sat, 14 Feb 2009 22:21:43 +0000

Instead of commenting out the line, just list the attributes that you want to be set via mass assignment. This protects the rest of the stuff (parent_id, height, etc). Restart your server afterwards to make sure the file is initialized with your changes.

Line 179:
base.attr_accessible :uploaded_data, :your_colum_name, etc

By: Daniel

Daniel — Mon, 17 Nov 2008 07:17:34 +0000

Hey thanks for this insight I was having the same issue and I commented out line 171 in the file attchment_fu.rb located in the vendor plugin folder. Seems to have done the trick, I am only uploading images and I haven’t run into any problems yet with that one line commented out. I will post here if I do run into any troubles. Thanks again