[
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.
