From: Devin Mullins <twifkak@...>
Date: 2007-02-08T23:26:28+09:00
Subject: Re: DRY or not ...

Devin Mullins wrote:
> Josselin wrote:
> 
>>    if params[:new_title].blank?
>>      params[:new_title] = nil
>>    elsif params[:new_title] == "null"
>>      params[:new_title] = ""
>>    end
> 
>   params[:new_title] = if params[:new_title].blank?
>     nil
>   elsif params[:new_title] == "null"
>     ""
>   end

Heh. Big fat bug. Right at the beginning. *cough*

   params[:new_title] = case params[:new_title]
     when '': nil
     when 'null': ''
     else params[:new_title]
   end
or:
   params[:new_title] = {
     '' => nil, 'null' => '', params[:new_title] => params[:new_title]
   }[params[:new_title]] #yikes!

Devin