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

Reply via email to