Changing CSS class based on model if statement RoR

Question

I have an app where a product can have a regular price or a discounted price both of which are saved in two seperate columns in my database as: price and offer.

I have a controller called 'generals' which is for all general products.

What I would like to happen is that if 'general.offer' is blank then the div that wraps the price will have a class of 'product-price'. If however there is a value present in general.offer then the div will have a class of 'product-special-price'.

Currently I have the following:

#View

<div class="<%= product_price %>">
  <%= general.price %>
</div> 


#Application Helper

def product_price
   if general.offer.blank?
      return "product-price"
   else
      return "product-special-price"
   end
end

At the moment I am receiving a wrong number of arguments error. Any advice people have on how to fix this would be much appreciated :)

Hi Michael, Sorry the error I am receiving is undefined local variable or method general for #<#<Class:0x61fc908>:0x5dce1d8>. Thanks so much for your help! — Tom Pinchen
– Tom Pinchen, Commented Jul 22, 2013 at 14:13

Mike Szyndel · Accepted Answer · 2013-07-22 14:07:21Z

1

I think you should pass general to product_price but depends on the error you're getting.

answered Jul 22, 2013 at 14:07

Mike Szyndel

10.6k11 gold badges51 silver badges64 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Tom Pinchen Over a year ago

Thanks Michael, How would I pass general into product_price. Would it be using @general defined in my generals controller ie. @general.offer.blank? When doing this I am left with error undefined method offer for nil:NilClass

Mike Szyndel Over a year ago

There's no magic in Rails! You need to pass it explicitly product_price(general), both in declaration and when you call it.

Collectives™ on Stack Overflow

Changing CSS class based on model if statement RoR

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related