[
http://jira.amdatu.org/jira/browse/AMDATU-103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=10096#action_10096
]
Angelo van der Sijpt commented on AMDATU-103:
---------------------------------------------
Completed up to FriendsGadgetServiceImpl.
> Services that read (and require) their config from ConfigAdmin should use a
> ManagedService
> ------------------------------------------------------------------------------------------
>
> Key: AMDATU-103
> URL: http://jira.amdatu.org/jira/browse/AMDATU-103
> Project: Amdatu
> Issue Type: Task
> Components: Core
> Affects Versions: 0.0.5
> Reporter: Ivo Ladage - van Doorn
> Assignee: Angelo van der Sijpt
> Fix For: 0.0.5
>
>
> Amdatu contains several services that use the ConfigAdmin service to store
> their (own) configuration. Most service also require this configuration to be
> available. In some cases some special logic is added to ensure this
> availability. It would be better to use the ManagedService approach for this
> with a req service dependency. Managed services should be introduced in the
> following classes:
> CassandraConfigurationServiceImpl
> GadgetManagementServiceImpl (should be refactored, need of initialized is not
> necessary when GadgetStore.addCategory is slightly refactored)
> CourseGadgetServiceImpl
> CourseServiceImpl (config should be added, doesn't exist yet)
> FriendsGadgetServiceImpl
> LoginServiceImpl
> SPARQLEndpointServiceImpl
> AuthorizationServiceImpl
> CassandraConfigurationServiceImpl
> ConfigTemplateManagerImpl
> ConsoleLogHandler (without service dependency, availability of configuration
> should not be required)
> SesameServiceImpl (ConfigAdmin should not be used at all since the file path
> cannot be changed)
> ShindigConfigurator (ConfigAdmin should not be used at all since the file
> path cannot be changed)
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.amdatu.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
