Klaus Kaempf write: > * 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
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 time -- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
