Using attachment_fu in existing user model with edit/update

Hi all,

I currently have a working development site with the user model already
setup. I did however hold out on user images/avatars until the end and
am now thinking it wasn’t such a good idea.

I’m using attachment_fu with imagescience to do the uploads. I have
installed everything and setup a test scaffold following the screencast
at
http://www.rubyplus.org/episodes/31-How-to-upload-images-in-Rails-2-.html
and got that up and running without a problem.

My issue is this:

I have a user model which when a user registers (new/create action) I
want to just set his account up with the least amount of information
(username, email, and password). I don’t want the user to worry about
uploading the image at the time of registration. I’ve been trying to set
up the code so that the user can upload the image when editing his/her
profile but I’m getting different errors even though I’m following the
same logic as the screencast I previously mentioned (same exact editing
code actually). I’ve also added the necessary fields for the database
fields to the user table. But I’ve had no luck.

Does attachment_fu only work if you upload the images at the time of
using the new/create actions? There must be a way to make this work by
just editing a user. I’m sure I’m just overlooking something, any help
is appreciated!!

Let me know if there is something else you need to know. Thank you for
your time!

Best regards,
Tony

Tony -

On 28-Aug-08, at 11:45 PM, Tony T. wrote:

installed everything and setup a test scaffold following the
uploading the image at the time of registration. I’ve been trying to
just editing a user. I’m sure I’m just overlooking something, any help
is appreciated!!

Let me know if there is something else you need to know. Thank you for
your time!

best to place your images in a separate model.

has_many :images

but best if images has a polymorphic relationship - that way you can
add images to other models

Class User < ActiveRecord::Base

has_many :images, :as => :attacheable, :dependent => :destroy

end

Class Image < ActiveRecord::Base

belongs_to :attacheable, :polymorphic => true
has_attachment :storage => :file_system, …

end

Jodi

Jodi S. wrote:

Tony -

On 28-Aug-08, at 11:45 PM, Tony T. wrote:

installed everything and setup a test scaffold following the
uploading the image at the time of registration. I’ve been trying to
just editing a user. I’m sure I’m just overlooking something, any help
is appreciated!!

Let me know if there is something else you need to know. Thank you for
your time!

best to place your images in a separate model.

has_many :images

but best if images has a polymorphic relationship - that way you can
add images to other models

Class User < ActiveRecord::Base

has_many :images, :as => :attacheable, :dependent => :destroy

end

Class Image < ActiveRecord::Base

belongs_to :attacheable, :polymorphic => true
has_attachment :storage => :file_system, …

end

Jodi

Hi Jodi,

Thanks for the reply. I tried adding the additional table relationships
as you suggested but I’m getting an error on the user update action:
undefined local variable or method `filename’ for #User:0x2564508

I have the standard update code in my controller:

def update
@user = User.find(params[:id])

respond_to do |format|
  if @user.update_attributes(params[:user])
    flash[:notice] = 'User was successfully updated.'
    format.html { redirect_to(@user) }
    format.xml  { head :ok }
  else
    format.html { render :action => "edit" }
    format.xml  { render :xml => @user.errors, :status => 

:unprocessable_entity }
end
end
end

and the view code for the upload is:
<%= file_field_tag :uploaded_data %>

Any help is appreciated! Thanks.

-Tony

Hi Tony,
The problem is you’re using the user’s form and including attributes
of another model in it. I guess your params look like ‘user’ =>
{ ‘name’ => ‘something’, …, ‘filename’ => ‘image.jpg’}, but filename
is not an attribute of User model but of Image model.

Jodi’s tip of creating a separate model has more implications than
just setting up the relationship: You should create a separate
controller for creating the image: In your user’s form, surely when he
wants to add an image, he gets to an image upload view. This would
call a create action in the ImageController:
def create # in ImageController, not UserController
@user = current_user # or User.find(params[:user_id]) if you post
the user_id in the form for the image.
@image = Image.new(params[:image]) #this is where the filename
attribute gets set
if @image.save
@user.image << @image #associate image to the user

else

end
end

If you prefer to do the association only when the user saves his
profile, you could just pass the image_id back to the form for the
user’s profile and then add the association with the UserController
update action. As image_id is an attribute of user, you can just mass-
assign it like you show in @user.update_attributes(params[:user]).

Thanx for jumping in Degee -

On 29-Aug-08, at 11:23 AM, deegee wrote:

wants to add an image, he gets to an image upload view. This would

On 29 aug, 14:40, Tony T. [email protected] wrote:

is appreciated!!
add images to other models
has_attachment :storage => :file_system, …
undefined local variable or method `filename’ for #User:0x2564508
format.xml { head :ok }

Any help is appreciated! Thanks.

-Tony

Posted viahttp://www.ruby-forum.com/.

I prefer to create images within the (in this case) the UserController.

ripped out of a project

View -

<%= file_field_tag “photo[]”, :id => “photo_image”, :class =>
“attach_photo” %>
<%= text_field_tag “caption[]”, nil, {:id => “photo_title”, :class =>
“attach_photo”} %>

User controller - after @user object is valid + saved

 #attached uploaded pics - an array of photos + captions
 if params[:photo]
   params[:photo].each_with_index do |image, caption_index|
     caption = ''
     caption = params[:caption][caption_index] if params[:caption]

&& params[:caption][caption_index]

     @user.images.create!(:uploaded_data => image, :caption =>

caption) if image.size != 0
end
end

Jodi

It’s a bit difficult to help you without the code here. Looks like
you’re not performing the correct assignment in your controller.
content_type is a method that can only be called on a UploadedStringIO
object and somehow you’re typecasting it to Array it seems.

Please show us the code for your controller update action and your
edit view. And it also helps to tell us which params are being passed
when you call the action (from your development.log).

Jodi S. wrote:

Thanx for jumping in Degee -

On 29-Aug-08, at 11:23 AM, deegee wrote:

wants to add an image, he gets to an image upload view. This would

On 29 aug, 14:40, Tony T. [email protected] wrote:

is appreciated!!
add images to other models
has_attachment :storage => :file_system, …
undefined local variable or method `filename’ for #User:0x2564508
format.xml { head :ok }

Any help is appreciated! Thanks.

-Tony

Posted viahttp://www.ruby-forum.com/.

I prefer to create images within the (in this case) the UserController.

ripped out of a project

View -

<%= file_field_tag “photo[]”, :id => “photo_image”, :class =>
“attach_photo” %>
<%= text_field_tag “caption[]”, nil, {:id => “photo_title”, :class =>
“attach_photo”} %>

User controller - after @user object is valid + saved

 #attached uploaded pics - an array of photos + captions
 if params[:photo]
   params[:photo].each_with_index do |image, caption_index|
     caption = ''
     caption = params[:caption][caption_index] if params[:caption]

&& params[:caption][caption_index]

     @user.images.create!(:uploaded_data => image, :caption =>

caption) if image.size != 0
end
end

Jodi

Thanks for all the help guys.

I finally got back to work after being busy from my day job. I got the
filename situation cleared but now I get the following error:

undefined method `content_type’ for
[#ActionController::UploadedStringIO:0x23f8e1c, 0]:Array

I’m doing things the “Jodi way”. Maybe I’m just tired and can’t see the
obvious solution but any advice is really appreciated as always.

And yes, I realize that this info is being spoon fed to me and I STILL
can’t get it. Trust me, it’s more frustrating to me than it is to anyone
else! :-\

Thanks again all!!!

-Tony

Hi Jodi,

Thanks for the reply. I tried adding the additional table relationships
as you suggested but I’m getting an error on the user update action:
undefined local variable or method `filename’ for #User:0x2564508

I have the standard update code in my controller:

def update
@user = User.find(params[:id])

respond_to do |format|
  if @user.update_attributes(params[:user])
    flash[:notice] = 'User was successfully updated.'
    format.html { redirect_to(@user) }
    format.xml  { head :ok }
  else
    format.html { render :action => "edit" }
    format.xml  { render :xml => @user.errors, :status => 

:unprocessable_entity }
end
end
end

and the view code for the upload is:
<%= file_field_tag :uploaded_data %>

Any help is appreciated! Thanks.

-Tony

I know this is an old post, but I was trying to find the solution to:
undefined local variable or method `filename’
and I arrived here.

I got this error because I made a mistake creating the table. I used
“file_name” instead of “filename”. Changing this solved the problem for
me.

Cheers
Aukan

This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.

| Privacy Policy | Terms of Service | Remote Ruby Jobs