Ben - I have committed this patch. Please sync and make sure it is ok (I have still not seen the commit mail go by). You can resolve the issue.
Kamal -----Original Message----- From: Ben Dewey (JIRA) [mailto:[email protected]] Sent: Thursday, June 25, 2009 9:58 PM To: [email protected] Subject: [jira] Updated: (STONEHENGE-55) Trying to buy or sell anything an amount larger than 3 digits throws an error in .Net stocktrader [ https://issues.apache.org/jira/browse/STONEHENGE-55?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Dewey updated STONEHENGE-55: -------------------------------- Attachment: Stonehenge-55-part2.patch Drew, There is an extra part2 to the patch that needs to be applied for it to work. Thanks to avantika for testing and finding this. -Ben > Trying to buy or sell anything an amount larger than 3 digits throws an error > in .Net stocktrader > ------------------------------------------------------------------------------------------------- > > Key: STONEHENGE-55 > URL: https://issues.apache.org/jira/browse/STONEHENGE-55 > Project: Stonehenge > Issue Type: Bug > Components: DOTNET_CLIENT > Affects Versions: M1 > Reporter: Chintana Wilamuna > Fix For: M2 > > Attachments: remove-comma.patch, Stonehenge-55-part2.patch, > STONEHENGE-55.patch > > > Steps to reproduce, > 1) Start .Net stocktrader services > 2) Login and try to buy or sell an amount larger than 3 digits > This throws an ASP error. > Cause: If the value the user tries to buy or sell, let's say 1000. This is > represented as "1,000" causing the Integer conversion to fail. > Quick fix: In StockTrade.aspx.cs line 72 and 73 change to the following, > getting rid of the comma: > SellQuantityRangeValidator.MaximumValue = > holding.quantity.Replace(",", ""); > SellQuantity.Text = holding.quantity.Replace(",", ""); -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
