yes daan. it need not be static as its singleton. PR in progress.
~Rajani
On Fri, Jul 24, 2015 at 2:16 PM, Daan Hoogland
wrote:
> Rajani, applicationCtx is static but being set by the override
> setApplicationContext(..) which is not static. Does is need to be
> static? If so an extra wrapper m
Rajani, applicationCtx is static but being set by the override
setApplicationContext(..) which is not static. Does is need to be
static? If so an extra wrapper method will kill the warning (not the
issue).
As the LdapUserManagerFactory is a singleton I don't think the appctx
needs to be static. Ag