today favorite refactor

Eight lines to 1 😉

Before:

if params[:full_details]
  if params[:full_details] == 'true'
    session[:full_details] = 'true'
  else
    session[:full_details] = 'false'
else
  session[:full_details] = 'false'
end

After:

session[:full_details] = (params[:full_details] == 'true') ? 'true' : 'false' rescue 'false'
Advertisements

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out /  Change )

Google photo

You are commenting using your Google account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )

Connecting to %s