On 4/30/13, Olemis Lang <[email protected]> wrote: > On 4/30/13, [email protected] <[email protected]> wrote: >> Author: matevz >> Date: Tue Apr 30 10:58:59 2013 >> New Revision: 1477551 >> >> URL: http://svn.apache.org/r1477551 >> Log: >> Ticket relations widget - initial implementation >> > > I'm really excited to see this > :) > > Is there a chance to move these ticket relations widgets on to BH > Relations plugin ? This approach might lead a circular package > dependency scenario > > BH Relations > +- BH Theme > +- BH Dashboard > +- BH Relations ... > > whereas IMHO this should look like > > BH Relations > +- BH Theme > +- BH Dashboard > +- BH Dashboard > > Besides , keeping such widgets in BH Dashboard plugin when BH > Relations is not available IMHO is against package cohesion . > > http://en.wikipedia.org/wiki/Cohesion_(computer_science) > http://en.wikipedia.org/wiki/Coupling_(computer_programming) > http://teleki.net/download/talks/2012.02.07.coupling.cohesion.pdf
Modularity means processes with minimum coupling and maximum cohesion. ( http://en.wikipedia.org/wiki/ISO/IEC_12207 ) http://en.wikipedia.org/wiki/Package_Principles ... and afaict also an extension of Single responsibility principle to package dependencies will apply . http://en.wikipedia.org/wiki/Single_responsibility_principle -- Regards, Olemis.
