Rick,
I realized I need to take a look at Ignacio's patch since I'm sure it
won't apply cleanly with the SPI changes in place. So, if you don't
mind, could you take a look at adding optional autowires?
Here is what needs to be done:
1. Change the autowire annotation to default to required true (obvious)
2. The implementation processing framework needs to be updated to
introspect autowire annotations and update
ReferenceDefinition.setRequired(). ReferenceDefinition is used to
represent references in general, and autowires in particular, which
are treated as a type. There are several processors which may need
updating. One is ImplementationProcessorServiceImpl.
3. SystemComponentBuilder needs to be modified to deal with
ReferenceDefinition.isRequired() for autowires and pass that to the
SystemOutboundAutowire constructor (which will need to be modified to
add a required field) as it creates system wires.
4. SystemOutboundAutowire when it resolves the autowire will throw an
exception if its required field is true.
Let me know if I can help out.
Jim
On Aug 30, 2006, at 2:06 PM, Jim Marino wrote:
On Aug 30, 2006, at 1:41 PM, Rick wrote:
Sure why meddle with the master! :-)
I'd actually not mind avoiding doing it myself but I prefer that to
having to reconcile changes
Seriously, the only request I would have is if convenient could
you make it a separate svn checkin? Just like to review what you
did for just this.
Yep. That's what I intend to do
The autowire information as far as I see is lost in the connection
building phase, unless I missed it. Was wondering if it could not
be added directly to the ServiceContract.
It shouldn't be on the service contract since it is part of the
component type (autowire is really a type of reference).
ReferenceDefinition already contains the information needed and can
be accessed from the system component builder.
Jim
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]