Author: lektran
Date: Thu Mar 18 00:57:06 2010
New Revision: 924577
URL:
http://svn.apache.org/viewvc?rev=924577&view=revLog:
Fixed a FIXME, commented code was attempting to access a null array, month is a multiple of one so I just removed the multiplication attempt.
Modified:
ofbiz/trunk/applications/product/src/org/ofbiz/product/subscription/SubscriptionServices.java
Modified: ofbiz/trunk/applications/product/src/org/ofbiz/product/subscription/SubscriptionServices.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/subscription/SubscriptionServices.java?rev=924577&r1=924576&r2=924577&view=diff==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/product/subscription/SubscriptionServices.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/product/subscription/SubscriptionServices.java Thu Mar 18 00:57:06 2010
@@ -119,8 +119,7 @@ public class SubscriptionServices {
calendar.add(times[0], (useTime.intValue() * times[1]));
} else {
Debug.logWarning("Don't know anything about useTimeUomId [" + useTimeUomId + "], defaulting to month", module);
- // FIXME: times[] can only be null here
- //calendar.add(Calendar.MONTH, (useTime.intValue() * times[1]));
+ calendar.add(Calendar.MONTH, useTime);
}
thruDate = new Timestamp(calendar.getTimeInMillis());