+1 Cheers, Tim -- Tim Ruppert HotWax Media http://www.hotwaxmedia.com
o:801.649.6594 f:801.649.6595 On Aug 5, 2009, at 5:21 PM, Adrian Crum wrote:
I'm bumping this because I might have some time this weekend to help.David - I would like to work on converting some of the frequently used lower-level concrete classes to interfaces. You didn't reply when I suggested it before. Do you have any objections?Also, if that conversion is done, it could be done in the trunk - negating the need for a branch. In other words, once the higher level code is using interfaces, you can muck around with the implementations all you want.-Adrian --- On Fri, 7/17/09, Adrian Crum <adrian.c...@yahoo.com> wrote:From: Adrian Crum <adrian.c...@yahoo.com>Subject: Re: svn commit: r795024 [1/6] - in /ofbiz/branches/ executioncontext20090716: ./ applications/content/src/org/ofbiz/ content/content/ applications/order/src/org/ofbiz/order/order/ applications/party/src/org/ofbiz/party/party/ applications/product/ src/org/ofb...To: dev@ofbiz.apache.org Date: Friday, July 17, 2009, 5:07 PM --- On Fri, 7/17/09, David E Jones <d...@me.com> wrote:There is a basic reason for this, and it's because I'mlazyand also not sure how many of these "lower level"objects weeven want interfaces for.My preference would be to change all of it to interfaces. Higher level code should interact with interfaces - not concrete classes (dependency inversion). Keep in mind you're not alone in this effort - I'm available to help. -Adrian
smime.p7s
Description: S/MIME cryptographic signature