* Josef Reidinger <[email protected]> [Sep 22. 2009 10:29]:
>      else
>        logger.warn("No argument to update")
> -      raise InvalidParameters.new [{:name => "language", :error => 
> "Missing"}]
> +      raise InvalidParameters.new :language => "Missing"

This is looking way better now, thanks Josef.

Now I wonder if the "Missing" string shouldn't be replaced by a
constant to make its semantic meaning clearer ?!

Klaus
---
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg)

-- 
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to