jamesfredley opened a new pull request, #15726:
URL: https://github.com/apache/grails-core/pull/15726

   ## Summary
   
   Fixes #15714 by accepting browser-submitted ASCII hyphen-minus values in 
locale-aware number binding for locales whose DecimalFormat minus sign is 
U+2212, such as nb_NO.
   
   The earlier PR #15475 correctly scoped the rendering fix to form field input 
output, but #15714 shows the remaining failure is on the request binding side. 
The sample application's proposed direction was right: convert a leading ASCII 
minus to the active locale minus before calling NumberFormat.parse, instead of 
changing display tags.
   
   ## Changes
   
   - Normalize only a leading ASCII minus in LocaleAwareNumberConverter before 
parsing.
   - Leave g:formatNumber, g:fieldValue, and form field rendering behavior 
unchanged.
   - Add Spock coverage for Long, BigDecimal, and BigInteger converters in 
nb_NO and en_US.
   
   ## Verification
   
   - ./gradlew :grails-databinding:test --tests 
"org.grails.databinding.converters.web.LocaleAwareNumberConverterSpec"
   - ./gradlew :grails-databinding:test
   - Oracle review-gate verdict: GREEN


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to