Chintana is welcome to comment, but I think we should go with my patch. -Ben
________________________________________ From: Andrew (Drew) (JIRA) [[email protected]] Sent: Friday, June 12, 2009 7:51 PM To: [email protected] Subject: [jira] Commented: (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:comment-tabpanel&focusedCommentId=12719023#action_12719023 ] Andrew (Drew) commented on STONEHENGE-55: ----------------------------------------- Chintana, Ben It seems the two patches have a merge conflict. for SH-55. Please comment on how best to merge these to patches > 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.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.
