bloodhound-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Olemis Lang <ole...@gmail.com>
Subject Re: svn commit: r1477551 - in /bloodhound/trunk: bloodhound_dashboard/ bloodhound_dashboard/bhdashboard/widgets/ bloodhound_dashboard/bhdashboard/widgets/templates/ bloodhound_theme/bhtheme/htdocs/ bloodhound_theme/bhtheme/templates/
Date Wed, 01 May 2013 01:06:35 GMT
On 4/30/13, Olemis Lang <olemis@gmail.com> wrote:
> On 4/30/13, matevz@apache.org <matevz@apache.org> 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.

Mime
View raw message