Issue Details (XML | Word | Printable)

Key: SHALE-372
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Joost Schouten
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Shale

messages_nl.properties

Created: 22/Dec/06 09:13 PM   Updated: 23/Jan/07 04:40 PM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: 1.0.4

File Attachments:
  Size
File Licensed for inclusion in ASF works messages_nl.properties 2006-12-22 09:14 PM Joost Schouten 1 kB


 Description  « Hide
Hi,

Here is the nl translation for the /shale/framework/trunk/shale-validator/src/main/resources/org/apache/shale/validator/messages.properties

I'm not quite sure if the " should be escaped or not. Also, there is no propper translation for float, integer etc. So I refered to them in their english name. However, I think it would make sence to add some sort of pre-validator checking if the the parameter is a number or not. If not, return a message like "{0} is not a number". This message is understood by the endusers better than should be a float, integer etc. Once the validator detected a number of the wrong kind display the error message indicating it is not the right type of number.

I hope this all makes sence, and please correct me if I made a mistake in how I am contributing.

Regards,
Joost

--------------------------------------------

# Licensed to the Apache Software Foundation (ASF) under one or more
# contributor license agreements. See the NOTICE file distributed with
# this work for additional information regarding copyright ownership.
# The ASF licenses this file to you under the Apache License, Version 2.0
# (the "License"); you may not use this file except in compliance with
# the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
errors.required={0} is vereist.
errors.minlength={0} mag niet minder dan {1} karakter(s) hebben.
errors.maxlength={0} mag niet meer dan {1} karakter(s) hebben.
errors.invalid={0} is ongeldig.

errors.byte={0} moet een "byte" zijn.
errors.short={0} moet een "short" zijn.
errors.integer={0} moet een "integer" zijn.
errors.long={0} moet een "long" zijn.
errors.float={0} moet een "float" zijn.
errors.double={0} moet een "double" zijn.

errors.date={0} is geen datum.
errors.range={0} valt buiten het bereik van {1} tot en met {2}.
errors.creditcard={0} is een onjuist creditcardnummer.
errors.email={0} is een onjuist e-mailadres.
errors.url={0} is een onjuist url-adres.

 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Craig McClanahan added a comment - 27/Dec/06 10:38 PM
I've added your translations to the trunk, and they will be present in the 20061228 nightly build, as well as the 1.0.4 release when it's done. Thanks for the patch!

Regarding quotes, I think they do indeed need to be escaped ... the default processing seems to remove them, and other translations for these messages do not seem to use them. Regarding preprocessing, that would be an interesting idea. But, if I type "abc" into a field that expects an integer, and get back "{0} is not a number", that does not give me enough information, right? I can type "123.456" and get yet another error ("{0} is not an integer"), and I'll start grumbling "why didn't you tell me that in the first place?"

Joost Schouten added a comment - 27/Dec/06 11:38 PM
I agree with you that just one message is the quickest solution, however, you and I know what integers, floats etc. mean. Most of my application users do not. I want to present them with a message they understand. Also, currently when two validators are in place (eg. requires and email), they also have an order which might result in two messages after each other ("requred" and on second submit "wrong format").

Maybe it would be best to create an new validator type for number. So that if you want you can add the number validator in front of your integer validator. What do you think?

Joost Schouten added a comment - 28/Dec/06 09:35 AM
I currently solved my number chacking problem as follows:

<h:inputText id="myInteger" value="#{contact.myInteger}">
<s:commonsValidator type="float" message="{0} should be a number" arg="#{labels['labels.user.myInteger']}" server="true" client="true"/>
        <!-- would like to use a double, but that fails. posted a message on the shale user mailinglist-->
<s:commonsValidator type="integer" arg="#{labels['labels.user.myInteger']}" server="true" client="true"/>
</h:inputText>
<h:message for="myInteger" styleClass="error"/>

This results in a message telling my users to use a number, if they entered text or a wrong format. Once that is correct, and it is a float in stead of an integer, they'll get told to us an integer in stead.

Obviously I would propose to add a numbet validator type to the s:commonsValidator and a numberValidator to the commons package. Do more people support this? If so I can build the validators and submit them.