directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Emmanuel Lecharny (JIRA)" <>
Subject [jira] [Resolved] (FC-34) Improve the way Factories create instances
Date Tue, 25 Nov 2014 17:15:12 GMT


Emmanuel Lecharny resolved FC-34.
    Resolution: Fixed

> Improve the way Factories create instances
> ------------------------------------------
>                 Key: FC-34
>                 URL:
>             Project: FORTRESS-CORE
>          Issue Type: Improvement
>    Affects Versions: 1.0.0-RC39
>            Reporter: Shawn McKinney
>             Fix For: 1.0.0
> (this issue from old JIRA:
> Currently, the various Mgr factories are creating instances for Mgr depending on the
type of access we want (basically, direct access or via REST).
> We can imagine that we may have more kind of access in the future (remote, SOAP yukkk,
... )
> But in any case, I think we can simplify the way we create instances :
> accessClassName = AccessMgrImpl.class.getName();
> AccessMgr accessMgr = 
> (AccessMgr)ClassUtil.createInstance(accessClassName);
> could be written :
> AccessMgr accessMgr = new AccessMgrImpl();
> We know everything about the class, and we don't even have to pull a constructor FQCN
from the configuration, the only thing is that we will create a new instance depending on
some configuration parameter.
> (this is of course when the config does not provide the class name to instanciate)
> I also have some concern about the fact that we may want to mix REST and other kind of
access. Atm, either we have a REST access, or not, but it's global and can't be changed, as
the IS_EST flag is static and final. Each context might access the Mgr in different ways (at
least, this is my understanding, even if I'm pushing it a bit too far).
> Does it sound reasonnable ?

This message was sent by Atlassian JIRA

View raw message