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

2009-07-17 Thread BJ Freeman
+1 for interfaces. in Eclipse you notified when there is changes to the interface. Eclipse will add the missing items to the classes that depend on the interface. Plus they are good place to do Docs on the usage. Adrian Crum sent the following on 7/17/2009 5:07 PM: > --- On Fri, 7/17/09, David E J

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

2009-07-17 Thread Adrian Crum
--- On Fri, 7/17/09, David E Jones wrote: > There is a basic reason for this, and it's because I'm lazy > and also not sure how many of these "lower level" objects we > even want interfaces for. My preference would be to change all of it to interfaces. Higher level code should interact with int

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

2009-07-17 Thread David E Jones
There is a basic reason for this, and it's because I'm lazy and also not sure how many of these "lower level" objects we even want interfaces for. By doing it this way the ModelEntityInterface object is not meant to be used as a straight replacement of ModelEntity and does not have all