new patch
On 4/26/07, Matthias Wessendorf <[EMAIL PROTECTED]> wrote:
On 4/26/07, Adam Winer <[EMAIL PROTECTED]> wrote:
> I'm confused about this patch:
>
> - if(this._isConvertible())
> + if(!(numberString.indexOf(',') != -1 || numberString.indexOf(',') !=
> -1) && this._isConvertible())
> {
> return _decimalParse(numberString,
> this._messages,
>
> Why is numberString.indexOf(',') != -1 being checked twice?
typo ...
'.' and ','
> And, if you're trying to use the numeric separator, shouldn't
> this be localized, not a hardcoded character?
yes, I'll check that as well
>
> -- Adam
>
>
> On 4/26/07, Matthias Wessendorf <[EMAIL PROTECTED]> wrote:
> > Hi,
> >
> > there is a minimalistic version of the client side converter for the
> > NumberConverter, but this client side converter is causing some
> > issues. There might be some *tool* that help us (me) to get a better
> > version. However, does anyone mind to apply the patch for
> >
> > https://issues.apache.org/jira/browse/ADFFACES-468
> >
> > in the mean time ?
> > (it's only a subtask of a bug, saying, we need a client side number
converter)
> >
> > -M
> >
> > --
> > Matthias Wessendorf
> > http://tinyurl.com/fmywh
> >
> > further stuff:
> > blog: http://jroller.com/page/mwessendorf
> > mail: mwessendorf-at-gmail-dot-com
> >
>
--
Matthias Wessendorf
http://tinyurl.com/fmywh
further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com
--
Matthias Wessendorf
http://tinyurl.com/fmywh
further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com