DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=41018>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=41018





------- Additional Comments From [EMAIL PROTECTED]  2006-11-22 09:50 -------
(In reply to comment #2)
> -1 for the patch. It's simply that Context.OBJECT_FACTORIES usage is not
> supported at the moment (same for all the other reference types).

I think it's rather strange to program a bug in getFactoryClassName() because of
Context.OBJECT_FACTORIES being unsupported.

How about changing it into:

public String getFactoryClassName() {
  String factory = super.getFactoryClassName();
  if (factory != null) {
    return factory;
  } else {
    return DEFAULT_FACTORY;
  }
}

That would work well in case a custom factory is passed to the constructor, and
in case no factory (or null) is specified.

On a side note: I'm curious why this issue was never encountered, the
documentation contains plenty of documentation about the global resources
feature, is it not recommended some way, or is it simply because no one uses it?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to