Right Ashish, it's a bad old habit :/ Fortunately it's easily fixed

Thanks for the reminder, done

Jacques

From: "Ashish Vijaywargiya" <[email protected]>
It is always better to specify brief details of the bug fix in the
commit log, isn't it? ;) :-)

--
Ashish

On Sat, Aug 14, 2010 at 5:33 PM,  <[email protected]> wrote:
Author: jleroux
Date: Sat Aug 14 12:03:11 2010
New Revision: 985471

URL: http://svn.apache.org/viewvc?rev=985471&view=rev
Log:
Fix a bug reported by rrhati on user ML

Modified:
ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/cart/ShowPromotionDetails.groovy

Modified: 
ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/cart/ShowPromotionDetails.groovy
URL: http://svn.apache.org/viewvc/ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/cart/ShowPromotionDetails.groovy?rev=985471&r1=985470&r2=985471&view=diff
==============================================================================
--- ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/cart/ShowPromotionDetails.groovy (original) +++ ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/cart/ShowPromotionDetails.groovy Sat Aug 14 12:03:11 2010
@@ -22,6 +22,7 @@ import org.ofbiz.entity.util.EntityUtil;
import org.ofbiz.order.shoppingcart.product.ProductPromoWorker;

productPromoId = request.getParameter("productPromoId");
+if (!productPromoId) productPromoId = parameters.productPromoId;
productPromo = delegator.findByPrimaryKey("ProductPromo", [productPromoId : 
productPromoId]);

promoAutoDescription = ProductPromoWorker.makeAutoDescription(productPromo, 
delegator, locale);






Reply via email to