You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ha...@apache.org on 2007/12/17 16:24:47 UTC

svn commit: r604898 - /ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java

Author: hansbak
Date: Mon Dec 17 07:24:46 2007
New Revision: 604898

URL: http://svn.apache.org/viewvc?rev=604898&view=rev
Log:
another small day calculation class to be used in minilanguage

Modified:
    ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java

Modified: ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java?rev=604898&r1=604897&r2=604898&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java (original)
+++ ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java Mon Dec 17 07:24:46 2007
@@ -83,6 +83,10 @@
         return new Timestamp(start.getTime() + (24*60*60*1000*days));
     }
 
+    public static Timestamp addDaysToTimestamp(Timestamp start, Double days) {
+        return new Timestamp(start.getTime() + ((int) (24*60*60*1000*days)));
+    }
+
     public static double getInterval(Timestamp from, Timestamp thru) {
         return thru != null ? thru.getTime() - from.getTime() + (thru.getNanos() - from.getNanos()) / 1000000 : 0;
     }



Re: svn commit: r604898 - /ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java

Posted by David E Jones <jo...@hotwaxmedia.com>.
I was going to comment on this too. As Adrian has expressed a number  
of times using millisecond calculations is not reliable and the  
Calendar or similar objects should be used instead.

-David


On Dec 17, 2007, at 8:40 AM, Adrian Crum wrote:

> Hans,
>
> Did you read my last email on this subject? Your methods will not  
> work properly. Please use the
>
> UtilDateTime.adjustTimestamp(Timestamp stamp, int adjType, int  
> adjQuantity, TimeZone timeZone, Locale locale)
>
> method.
>
> -Adrian
>
> hansbak@apache.org wrote:
>
>> Author: hansbak
>> Date: Mon Dec 17 07:24:46 2007
>> New Revision: 604898
>> URL: http://svn.apache.org/viewvc?rev=604898&view=rev
>> Log:
>> another small day calculation class to be used in minilanguage
>> Modified:
>>    ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ 
>> UtilDateTime.java
>> Modified: ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ 
>> UtilDateTime.java
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java?rev=604898&r1=604897&r2=604898&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ 
>> UtilDateTime.java (original)
>> +++ ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ 
>> UtilDateTime.java Mon Dec 17 07:24:46 2007
>> @@ -83,6 +83,10 @@
>>         return new Timestamp(start.getTime() + (24*60*60*1000*days));
>>     }
>> +    public static Timestamp addDaysToTimestamp(Timestamp start,  
>> Double days) {
>> +        return new Timestamp(start.getTime() + ((int)  
>> (24*60*60*1000*days)));
>> +    }
>> +
>>     public static double getInterval(Timestamp from, Timestamp  
>> thru) {
>>         return thru != null ? thru.getTime() - from.getTime() +  
>> (thru.getNanos() - from.getNanos()) / 1000000 : 0;
>>     }
>


Re: svn commit: r604898 - /ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java

Posted by Adrian Crum <ad...@hlmksw.com>.
Hans,

Did you read my last email on this subject? Your methods will not work properly. Please use the

UtilDateTime.adjustTimestamp(Timestamp stamp, int adjType, int adjQuantity, TimeZone timeZone, 
Locale locale)

method.

-Adrian

hansbak@apache.org wrote:

> Author: hansbak
> Date: Mon Dec 17 07:24:46 2007
> New Revision: 604898
> 
> URL: http://svn.apache.org/viewvc?rev=604898&view=rev
> Log:
> another small day calculation class to be used in minilanguage
> 
> Modified:
>     ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java
> 
> Modified: ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java?rev=604898&r1=604897&r2=604898&view=diff
> ==============================================================================
> --- ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java (original)
> +++ ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java Mon Dec 17 07:24:46 2007
> @@ -83,6 +83,10 @@
>          return new Timestamp(start.getTime() + (24*60*60*1000*days));
>      }
>  
> +    public static Timestamp addDaysToTimestamp(Timestamp start, Double days) {
> +        return new Timestamp(start.getTime() + ((int) (24*60*60*1000*days)));
> +    }
> +
>      public static double getInterval(Timestamp from, Timestamp thru) {
>          return thru != null ? thru.getTime() - from.getTime() + (thru.getNanos() - from.getNanos()) / 1000000 : 0;
>      }
> 
> 
>