You are viewing a plain text version of this content. The canonical link for it is here.
Posted to stonehenge-dev@incubator.apache.org by "Avantika Agrawal (JIRA)" <ji...@apache.org> on 2009/07/14 20:01:14 UTC
[jira] Updated: (STONEHENGE-81) Code Cleanup for the ConfigService
[ 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.