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 2009/11/06 08:54:57 UTC

svn commit: r833307 - /ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java

Author: hansbak
Date: Fri Nov  6 07:54:57 2009
New Revision: 833307

URL: http://svn.apache.org/viewvc?rev=833307&view=rev
Log:
fix for revision 832581 as reported in the mailing list and waiting for adirans attention

Modified:
    ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java

Modified: ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java?rev=833307&r1=833306&r2=833307&view=diff
==============================================================================
--- ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java (original)
+++ ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java Fri Nov  6 07:54:57 2009
@@ -637,7 +637,7 @@
                             for (DateRange periodRange : periodRanges) {
                                 if (periodRange.includesDate(occurrence)) {
                                     GenericValue cloneWorkEffort = (GenericValue) workEffort.clone();
-                                    TimeDuration duration = TimeDuration.fromNumber(workEffort.getLong("estimatedMilliSeconds"));
+                                    TimeDuration duration = TimeDuration.fromNumber(workEffort.getDouble("estimatedMilliSeconds").longValue());
                                     if (!duration.isZero()) {
                                         Calendar endCal = UtilDateTime.toCalendar(occurrence, timeZone, locale);
                                         Date endDate = duration.addToCalendar(endCal).getTime();



Re: svn commit: r833307 - /ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java

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

Thank you for fixing that. I don't know how that escaped my tests.

-Adrian

--- On Thu, 11/5/09, hansbak@apache.org <ha...@apache.org> wrote:

> From: hansbak@apache.org <ha...@apache.org>
> Subject: svn commit: r833307 - /ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java
> To: commits@ofbiz.apache.org
> Date: Thursday, November 5, 2009, 11:54 PM
> Author: hansbak
> Date: Fri Nov  6 07:54:57 2009
> New Revision: 833307
> 
> URL: http://svn.apache.org/viewvc?rev=833307&view=rev
> Log:
> fix for revision 832581 as reported in the mailing list and
> waiting for adirans attention
> 
> Modified:
>    
> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java
> 
> Modified:
> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java?rev=833307&r1=833306&r2=833307&view=diff
> ==============================================================================
> ---
> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java
> (original)
> +++
> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortServices.java
> Fri Nov  6 07:54:57 2009
> @@ -637,7 +637,7 @@
>                
>              for
> (DateRange periodRange : periodRanges) {
>                
>              
>    if (periodRange.includesDate(occurrence))
> {
>                
>                
>      GenericValue cloneWorkEffort =
> (GenericValue) workEffort.clone();
> -               
>                
>     TimeDuration duration =
> TimeDuration.fromNumber(workEffort.getLong("estimatedMilliSeconds"));
> +               
>                
>     TimeDuration duration =
> TimeDuration.fromNumber(workEffort.getDouble("estimatedMilliSeconds").longValue());
>                
>                
>      if (!duration.isZero()) {
>                
>                
>          Calendar endCal =
> UtilDateTime.toCalendar(occurrence, timeZone, locale);
>                
>                
>          Date endDate =
> duration.addToCalendar(endCal).getTime();
> 
> 
>