[ http://issues.apache.org/jira/browse/JAMES-654?page=all ]
Stefano Bagnara updated JAMES-654:
----------------------------------
Summary: Remove avalon "Context" usage when possible: deprecate
AvalonContextUtils/Constants in favor of a FileSystem Service (we only need the
getFile now) (was: Deprecate AvalonContextUtils/Constants in favor of a
JamesContextService (we only need the getFile now))
Description:
We should remove the AvalonContextUtils that we currently use only to retrieve
files stored in the apps folder.
If we introduce a new Service for that we can also remove many uses of
Context/Contextualizable and make the whole thing less Avalon-ized.
---
In James.initialize (or initializeServernamesAndPostmaster in trunk) we create
a new DefaultContext that we store in a field, but we never use it.
We could remove it and remove the constants currently used only by that call.
This unused variable seems to be there unchanged since 2.2.0: it is referenced
in UsersLDAPRepository but it never worked at least since 2.2.0 so we should
remove it and eventually create the service chain needed to service the
usersldaprepository.
---
In the remaining James we use the context almost only to access a file in the
context: if we introduce this service we can remove Context/Contextualizable
from James.
was:
We should remove the AvalonContextUtils that we currently use only to retrieve
files stored in the apps folder.
If we introduce a new Service for that we can also remove many uses of
Context/Contextualizable and make the whole thing less Avalon-ized.
Made the issue more generic so to involve more changes.
> Remove avalon "Context" usage when possible: deprecate
> AvalonContextUtils/Constants in favor of a FileSystem Service (we only need
> the getFile now)
> ---------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: JAMES-654
> URL: http://issues.apache.org/jira/browse/JAMES-654
> Project: James
> Issue Type: Improvement
> Affects Versions: Trunk
> Reporter: Stefano Bagnara
> Assigned To: Stefano Bagnara
> Priority: Minor
> Fix For: Trunk
>
>
> We should remove the AvalonContextUtils that we currently use only to
> retrieve files stored in the apps folder.
> If we introduce a new Service for that we can also remove many uses of
> Context/Contextualizable and make the whole thing less Avalon-ized.
> ---
> In James.initialize (or initializeServernamesAndPostmaster in trunk) we
> create a new DefaultContext that we store in a field, but we never use it.
> We could remove it and remove the constants currently used only by that call.
> This unused variable seems to be there unchanged since 2.2.0: it is
> referenced in UsersLDAPRepository but it never worked at least since 2.2.0 so
> we should remove it and eventually create the service chain needed to service
> the usersldaprepository.
> ---
> In the remaining James we use the context almost only to access a file in the
> context: if we introduce this service we can remove Context/Contextualizable
> from James.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
