Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-04 Thread sreekanth
thanks Yasser for all you help and code. I would like to conclude this as a struts2 upgrade change, as BigDecimal field value was setting as BigDecimal.ZERO when field value passed as empty in struts2.5.10.x or < whereas 2.5.13 will set such empty BigDecimal field as null value (that is the

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-04 Thread Lukasz Lenart
2017-10-04 12:41 GMT+02:00 Yasser Zamani : >> 2.5.13. But now I also tested 2.5.10.1 and get following warn when try >> to set empty string value to a BigDecimal: >> >> Unexpected Exception caught setting 'myList[0].crAmt' on 'class >>

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-04 Thread Yasser Zamani
On 10/4/2017 10:45 AM, sreekanth wrote: > thanks Yasser, let me setup a sample too. I already went through fixes > introduced at WW-4581 as well and ran a test against it and found that's not > the reason for this behavior rather something related to form field binding. > > Here i'm adding your

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-04 Thread sreekanth
thanks Yasser, let me setup a sample too. I already went through fixes introduced at WW-4581 as well and ran a test against it and found that's not the reason for this behavior rather something related to form field binding. Here i'm adding your finding which i too believe the root cause for this

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-04 Thread Yasser Zamani
On 10/4/2017 9:19 AM, sreekanth wrote: > thanks Yasser, that info was great, so do you think so the way struts2 binds > BigDecimal changed between these two versions ? I found the difference at [1]. That is in order to fix WW-4581 [2]. [1]

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread sreekanth
thanks Yasser, that info was great, so do you think so the way struts2 binds BigDecimal changed between these two versions ? and also if possible kindly share your sample project in github so that i can fork it and check myself faster. -- Sent from:

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread Yasser Zamani
On 10/4/2017 8:07 AM, sreekanth wrote: > thanks Yasser, may i know with which version of struts you have tested it > 2.5.10.1 or 2.5.13 ? > 2.5.13. But now I also tested 2.5.10.1 and get following warn when try to set empty string value to a BigDecimal: Unexpected Exception caught setting

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread sreekanth
thanks Yasser, may i know with which version of struts you have tested it 2.5.10.1 or 2.5.13 ? -- Sent from: http://struts.1045723.n5.nabble.com/Struts-User-f3426046.html - To unsubscribe, e-mail:

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread Yasser Zamani
On 10/3/2017 10:33 PM, sreekanth wrote: > Thanks Yasser, if my previous doubts get cleared then probably i can assume > there is behavioral change in binding empty value. If thats not the case > then will try to replicate as you have mentioned. > In my yesterday test application, empty string

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread sreekanth
Thanks Yasser, if my previous doubts get cleared then probably i can assume there is behavioral change in binding empty value. If thats not the case then will try to replicate as you have mentioned. -- Sent from: http://struts.1045723.n5.nabble.com/Struts-User-f3426046.html

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread sreekanth
Hi Lukasz, what bothering me is, we haven't changed this particular jsp since an year or more and its working fine till 2.5.10.x. I have the following doubt, if it can be answered then i can assume whats going wrong in my code. What was the behaviour of struts 2.5.10.1 while

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread Yasser Zamani
On 10/3/2017 9:56 PM, sreekanth wrote: > The value not getting set at all, its coming as null. By the time i will also > see if i can provide some log as you mentioned. > I tested similar action with a list of A where A has two BigDecimals; posted values (myList[x].y] getting set here however

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread Lukasz Lenart
2017-10-03 20:03 GMT+02:00 sreekanth : > Thanks Lukasz, can you please point me some wiki page or what change i have > to do to make my existing code working ? Hm... as far I recall there is no special settings, just use format according to user locale, eg.: US

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread sreekanth
The value not getting set at all, its coming as null. By the time i will also see if i can provide some log as you mentioned. -- Sent from: http://struts.1045723.n5.nabble.com/Struts-User-f3426046.html - To unsubscribe,

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread Yasser Zamani
On 10/3/2017 8:26 PM, sreekanth wrote: > Hi, > We have a List object which is getting set at JSP (element added > dynamically in front end), while saving this form the object's value for > BigDecimal fields are getting null. > > eg: > > Class A { > String strVal; > BigDecimal drAmt; >

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread sreekanth
Thanks Lukasz, can you please point me some wiki page or what change i have to do to make my existing code working ? -- Sent from: http://struts.1045723.n5.nabble.com/Struts-User-f3426046.html - To unsubscribe, e-mail:

Re: Iusse in BigDecimal conversion after upgrading from 2.5.10.1 to 2.5.13

2017-10-03 Thread Lukasz Lenart
2017-10-03 18:56 GMT+02:00 sreekanth : > Hi, > We have a List object which is getting set at JSP (element added > dynamically in front end), while saving this form the object's value for > BigDecimal fields are getting null. > > eg: > > Class A { > String