Default value depending on the controller action

The solution that is good for me:

<% if controller.action_name == ‘new’ %>
<%= f.text_field :image_url, :value => ‘no image’ %>
<% else %>
<%= f.text_field :image_url %>
<% end %>

and then in index.erb set smth like that
<%= product.image_url == ‘no image’ ? image_tag("/images/
empty.png", :alt => ‘Book cover’, :class => ‘list_image’) :
image_tag("/images/#{product.image_url}", :alt => ‘Book cover’, :class
=> ‘list_image’) %>
to set work on the default value for this.

You’re not really asking a question, but if you’re looking for
feedback on your code, I’m happy to provide my 2 cents.

It appears that image_url just returns a string as a path to an image
under /images/ on your web server.

I would override the default accessor for image_url in your model code
eg.

def image_url
url_string = read_attribute(:image_url)
url_string.blank? ? “defaultpath.png” : url_string
end

This way you can check the state of the model to determine what to
render in the view instead of testing which controller action is being
called. For example, what if you had a product without a url ? Your
code would break when you call the edit action even though you would
want to render the default value for product. This could could be
taken a lot further, but I’ll wait to see if this is helpful to you.

Also, image_tag references images relative to /images/ on your web
server, so the image you’d be looking for would be under the page /
images/images/#{image_url}

Hope that helps,

Luke

Oops, I’m sorry. I thought I answered to some open topic with this,
and didn’t know I started a new one.

Well, thank you for you idea, Luke. I though about putting work with
data to model, and I think you are right.
As for empty field, I add validates to model to check that this field
isn’t empty, and also add explanation to the error note.
Sure, I can allow to leave it just blank for ‘no image’, but validate
for presence will make user sure he didn’t forget to put the image
url, and along with format validation with regular expression, it will
minimize the number of mistakes I think.

Well, actually that was a test and I’m just learning Rails, so I guess
my way is not really good on the point of usability, and it’s not a
right thing, but i’m just testing :slight_smile:

Peace,
Vit