Mailinglist Archive: yast-devel (163 mails)

< Previous Next >
Re: [yast-devel] Re: [yast-commit] <rest-service> master : Change calling conventions for Invalid parameters to follow DRY
  • From: Josef Reidinger <jreidinger@xxxxxxx>
  • Date: Tue, 22 Sep 2009 10:48:07 +0200
  • Message-id: <200909221048.07164.jreidinger@xxxxxxx>
Klaus Kaempf write:
* Josef Reidinger <jreidinger@xxxxxxx> [Sep 22. 2009 10:29]:
logger.warn("No argument to update")
- raise [{:name => "language", :error =>
"Missing"}] + raise :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 ?!


Yes, it is possible, but identifier is for frontend (so frontend must know
this string) and it must also have meaning, because this string is used in
rest response, so for anyone who want directly use REST backend via curl it
should have some meaning. But I think that for missing value we should
introduce constant.

Josef Reidinger
YaST team
maintainer of perl-Bootloader, YaST2-Repair, webyast modules language and
To unsubscribe, e-mail: yast-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: yast-devel+help@xxxxxxxxxxxx

< Previous Next >