New subject: [xwiki-devs] [VOTE] Use ComponentManager.getInstance*() instead of lookupComponent/LookupList/LookupMap

6 Apr 2012

Hi devs,

In another thread we had a discussion about the new name of the method for looking up
components. We had to provide a new one since we introduced support for Types in
http://jira.xwiki.org/jira/browse/XCOMMONS-121

Thomas has introduced a new lookupComponent() method and has asked in a mail to speak up
if we wanted a different name. I have replied to this thread and explained why I preferred
a new one (see http://markmail.org/message/lvah3d6f37ak66p3). 

I'm copying my rationale here:

"
All looks good except CM.lookupComponent() which Ï don't like a lot for the
following reasons:
* It's a bit long. It's longer than before and it would be nicer if it were as
short as before or even shorter ;)
* It's not symmetrical with other lookups like lookupList and lookupMap, which should
theoretically be lookupComponentList() and lookupComponentMap() which are even longer
"

I've proposed several possibilities and some committers have expressed their
preferences for getInstance*() so I'm officially proposing it here as a VOTE since
it's an important API that we're going to keep for a long time ;)

So this vote is about using:

* CM.getInstance(…)
* CM.getInstanceList(…)
* CM.getInstanceMap(…)

Rationale:
* Shorter than lookupComponent, LookupComponentList, LookupComponentMap
* This is what is used in Guice and Picocontainer and seems to be a well-known name

Here's my +1

IMPORTANT: Please vote quickly since we need to agree before the release of 4.0 final and
4.0RC1 is planned for this coming Monday so it would be ideal to speed up this vote and do
the change today if we agree about it.

Thanks
-Vincent