On 2/07/2010, at 9:56 AM, BJ Freeman wrote: > [ I'm sure it doesn't need to be stated but the trunk is not dumping ground > for you to stash incomplete work in.] > I find that a curious statement since I have seen this type of process in > ofbiz for a long time. > > It is also something I commented on, like you, and got told that was the way > it was done
I can't respond to a general statement, if you have a link to your previous discussions I'd be glad to give my opinion. I have no problem with code being committed even if it's not feature complete, what I do have a problem with is code being rushed in "because I need it in the trunk" (what does that even mean?) and "it was convenient to temporary put it there". [snip, ever considered shortening your footer?] > > Scott Gray sent the following on 7/1/2010 2:47 PM: >> On 2/07/2010, at 9:05 AM, Jacques Le Roux wrote: >> >>> Scott Gray wrote: >>>> Why is this in common instead of product? >>> >>> You are very curious :o) >>> >>> Because I quickly followed the path already opened by >>> getAssociatedStateList. >> >> Geo is a framework level entity, that's why a method such as this is in >> common. >> >>> Tomorrow I will commit the related service and js script in Product and >>> will then move it to Product too. >>> I committed as is for now because I had bigger fish to fry this evening but >>> needed this in trunk already and it was convenient to >>> temporary put it there . >> >> I'm sure it doesn't need to be stated but the trunk is not dumping ground >> for you to stash incomplete work in. >> >>>> I'm also willing to bet that a method already exists that does this. >>> >>> I'd more than happy to use it if it already exists, just show me the way... >> >> Is that really how you intend on proceeding? Just duplicate functionality >> as you need to instead of checking if something already exists? It's not my >> job to do your work for you. >> >>> Jacques >>> >>>> >>>> Regards >>>> Scott >>>> >>>> HotWax Media >>>> http://www.hotwaxmedia.com >>>> >>>> On 2/07/2010, at 2:05 AM, [email protected] wrote: >>>> >>>>> Author: jleroux >>>>> Date: Thu Jul 1 14:05:21 2010 >>>>> New Revision: 959673 >>>>> >>>>> URL: http://svn.apache.org/viewvc?rev=959673&view=rev >>>>> Log: >>>>> Adds a getAssociatedProductsList method. I will certainly add the >>>>> companions service and request later in the Product >>>>> application. >>>>> I have also the vague desire to generalize this more, as I have been able >>>>> to quickly use it in a widget form following the >>>>> Freemarker way already used. >>>>> >>>>> Modified: >>>>> ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonWorkers.java >>>>> >>>>> Modified: >>>>> ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonWorkers.java >>>>> URL: >>>>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonWorkers.java?rev=959673&r1=959672&r2=959673&view=diff >>>>> ============================================================================== >>>>> --- >>>>> ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonWorkers.java >>>>> (original) +++ >>>>> ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonWorkers.java Thu >>>>> Jul 1 14:05:21 2010 @@ -33,6 +33,7 @@ import >>>>> org.ofbiz.entity.GenericValue; >>>>> import org.ofbiz.entity.condition.EntityCondition; >>>>> import org.ofbiz.entity.condition.EntityExpr; >>>>> import org.ofbiz.entity.condition.EntityOperator; >>>>> +import org.ofbiz.entity.util.EntityUtil; >>>>> >>>>> /** >>>>> * Common Workers >>>>> @@ -135,6 +136,35 @@ public class CommonWorkers { >>>>> return geoList; >>>>> } >>>>> >>>>> + public static List<GenericValue> >>>>> getAssociatedProductsList(Delegator delegator, String productCategoryId) { >>>>> + return getAssociatedProductsList(delegator, productCategoryId, >>>>> null); >>>>> + } >>>>> + >>>>> + /** >>>>> + * Returns a list of active related products for a product category >>>>> + */ >>>>> + public static List<GenericValue> >>>>> getAssociatedProductsList(Delegator delegator, String productCategoryId, >>>>> String >>>>> listOrderBy) { + List<GenericValue> products = >>>>> FastList.newInstance(); >>>>> + if (UtilValidate.isNotEmpty(productCategoryId)) { >>>>> + EntityCondition productsFindCond = >>>>> EntityCondition.makeCondition( >>>>> + EntityCondition.makeCondition("productCategoryId", >>>>> productCategoryId)); >>>>> + >>>>> + if (UtilValidate.isEmpty(listOrderBy)) { >>>>> + listOrderBy = "sequenceNum"; >>>>> + } >>>>> + List<String> sortList = UtilMisc.toList(listOrderBy); >>>>> + >>>>> + try { >>>>> + products = delegator.findList("ProductCategoryMember", >>>>> productsFindCond, null, sortList, null, true); >>>>> + products = EntityUtil.filterByDate(products); >>>>> + } catch (GenericEntityException e) { >>>>> + Debug.logError(e, "Cannot lookup ProductCategoryMember", >>>>> module); >>>>> + } >>>>> + } >>>>> + >>>>> + return products; >>>>> + } >>>>> + >>>>> /** >>>>> * A generic method to be used on Type enities, e.g. ProductType. >>>>> Recurse to the root level in the type hierarchy >>>>> * and checks if the specified type childType has parentType as its >>>>> parent somewhere in the hierarchy. >>> >>> >> >
smime.p7s
Description: S/MIME cryptographic signature
