Hi,

Am Dienstag, 28. Aug 2007, 23:00:08 +0900 schrieb eggie5:
> How can this be more succinct? Can I somehow put the nil check inline
> with the each?
> 
> unless(params[:categories].nil?)
>             params[:categories].each do |id|
>                 @subscriber.subscriptions.create :category_id =>
> id, :timestamp_start => Time.now.to_s, :is_subscribed => true
>             end
>         end

Maybe:

  pc = params[:categories]
  pc and pc.each do |id|
    ...
  end

Bertram


-- 
Bertram Scharpf
Stuttgart, Deutschland/Germany
http://www.bertram-scharpf.de