Discussion:
[castor-dev] [StudenProgram] proposal for changes in ResolverStrategy structure
Michael Schröder
2009-12-13 16:41:59 UTC
Permalink
So Werner,



we recently thought about the BaseResolver class you told us to implement in
order to refactor the ResolverStrategy. The attachment contains our proposal
for a possible new class structure. We weren’t sure whether you want us to
refactor everything or only create this new class in order to define the new
isCascading… methods on a good and central spot.



The question now is, whether our task also is to refactor all the
implementations of possible ResolverStrategies and try to pull out methods
which can also be implemented in this new abstract class or only to use the
new class for the four new Methods caring about the cascading functionality?



Lg Ivo
Werner Guttmann
2009-12-13 23:04:01 UTC
Permalink
Hi Michael,

as always, start with the absolute minimum required (i.e. the four new
methods discussed and everything required to support those), and in the
future build on this.

If you find something that's easy to be achieved (refactored) in this
case, do not hesitate to go about this now.

Cheers
Werner
Post by Michael Schröder
So Werner,
we recently thought about the BaseResolver class you told us to implement in
order to refactor the ResolverStrategy. The attachment contains our proposal
for a possible new class structure. We weren’t sure whether you want us to
refactor everything or only create this new class in order to define the new
isCascading… methods on a good and central spot.
The question now is, whether our task also is to refactor all the
implementations of possible ResolverStrategies and try to pull out methods
which can also be implemented in this new abstract class or only to use the
new class for the four new Methods caring about the cascading functionality?
Lg Ivo
------------------------------------------------------------------------
------------------------------------------------------------------------
---------------------------------------------------------------------
http://xircles.codehaus.org/manage_email
---------------------------------------------------------------------
To unsubscribe from this list, please visit:

http://xircles.codehaus.org/manage_email

Loading...