[ 
https://issues.apache.org/jira/browse/STONEHENGE-81?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Avantika Agrawal updated STONEHENGE-81:
---------------------------------------

    Attachment: Stonehenge-81.patch

Here is the patch that makes some minor refinements to the ConfigService:
- removes unnecessary parts of Utility.ca, Settings.cs, and the config files
- replaces hardcoded strings with labels that are retrieved from the App.config 
files
- implements the getOPSConfig method to retrieve DB config info for the OPS
- creates a new method SQLHelper.GetAssemblyNamefromDBName(string DBName) which 
returns the DAL
- removes some unnecessary mappings of the endpoints and the switch statements



> Code Cleanup for the ConfigService
> ----------------------------------
>
>                 Key: STONEHENGE-81
>                 URL: https://issues.apache.org/jira/browse/STONEHENGE-81
>             Project: Stonehenge
>          Issue Type: Improvement
>            Reporter: Avantika Agrawal
>            Priority: Trivial
>         Attachments: Stonehenge-81.patch
>
>
> There is some unused code in Utility.cs, Settings.cs and the config files of 
> .NET StockTrader. This should be removed. In addition -we should try to 
> refactor the code to simplify the process of adding new endpoints - by 
> eliminating some switch statements, etc.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to