Author: lektran
Date: Thu Mar 18 00:57:06 2010
New Revision: 924577

URL: http://svn.apache.org/viewvc?rev=924577&view=rev
Log:
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());


Reply via email to