Thanks Adrian again...i checked it and see that october is one day short
in the french locale....but if that really 'breaks' the calendar?

On Tue, 2009-08-04 at 19:30 -0700, Adrian Crum wrote:
> A good demo of the bug before the online demo is updated:
> 
> Using the online demo, go to the work effort month view calendar. Click Next 
> Month until you reach October. Notice the last week. Switch the locale to 
> French and look at the last week again.
> 
> -Adrian
> 
> 
> --- On Tue, 8/4/09, Hans Bakker <mailingl...@antwebsystems.com> wrote:
> 
> > From: Hans Bakker <mailingl...@antwebsystems.com>
> > Subject: Re: svn commit: r801024 - 
> > /ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy
> > To: dev@ofbiz.apache.org
> > Date: Tuesday, August 4, 2009, 6:35 PM
> > Thank you for your help Adrian.
> > 
> > On Tue, 2009-08-04 at 23:40 +0000, adri...@apache.org
> > wrote:
> > > Author: adrianc
> > > Date: Tue Aug  4 23:40:00 2009
> > > New Revision: 801024
> > > 
> > > URL: http://svn.apache.org/viewvc?rev=801024&view=rev
> > > Log:
> > > Fixed a bug in the Month view work effort calendar.
> > > 
> > > Modified:
> > > 
> >    
> > ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy
> > > 
> > > Modified:
> > ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy
> > > URL: 
> > > http://svn.apache.org/viewvc/ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy?rev=801024&r1=801023&r2=801024&view=diff
> > >
> > ==============================================================================
> > > ---
> > ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy
> > (original)
> > > +++
> > ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy
> > Tue Aug  4 23:40:00 2009
> > > @@ -51,24 +51,30 @@
> > >  context.prevMillis = new
> > Long(prev.getTime()).toString();
> > >  next = UtilDateTime.getDayStart(start,
> > numDays+1, timeZone, locale);
> > >  context.nextMillis = new
> > Long(next.getTime()).toString();
> > > -end = UtilDateTime.getDayStart(start, numDays,
> > timeZone, locale);
> > > +end = UtilDateTime.getMonthEnd(start, timeZone,
> > locale);
> > >  
> > >  //Find out what date to get from
> > >  getFrom = null;
> > >  prevMonthDays = 
> > tempCal.get(Calendar.DAY_OF_WEEK) -
> > tempCal.getFirstDayOfWeek();
> > > -if (prevMonthDays < 0) prevMonthDays = 7 +
> > prevMonthDays;
> > > -tempCal.add(Calendar.DATE,-(prevMonthDays));
> > > +if (prevMonthDays < 0) prevMonthDays += 7;
> > > +tempCal.add(Calendar.DATE, -prevMonthDays);
> > >  numDays += prevMonthDays;
> > >  getFrom = new
> > Timestamp(tempCal.getTimeInMillis());
> > >  firstWeekNum =
> > tempCal.get(Calendar.WEEK_OF_YEAR);
> > >  context.put("firstWeekNum", new
> > Integer(firstWeekNum));
> > >  
> > >  // also get days until the end of the week at
> > the end of the month
> > > -lastWeekCal =
> > UtilDateTime.toCalendar(UtilDateTime.addDaysToTimestamp(start,
> > numDays - prevMonthDays), timeZone, locale);
> > > -followingMonthDays = 7 +
> > lastWeekCal.getFirstDayOfWeek() -
> > lastWeekCal.get(Calendar.DAY_OF_WEEK);
> > > -if (followingMonthDays < 7) numDays +=
> > followingMonthDays; 
> > > +lastWeekCal = UtilDateTime.toCalendar(end, timeZone,
> > locale);
> > > +monthEndDay = lastWeekCal.get(Calendar.DAY_OF_WEEK);
> > > +getTo = UtilDateTime.getWeekEnd(end, timeZone,
> > locale);
> > > +lastWeekCal = UtilDateTime.toCalendar(getTo,
> > timeZone, locale);
> > > +followingMonthDays =
> > lastWeekCal.get(Calendar.DAY_OF_WEEK) - monthEndDay;
> > > +if (followingMonthDays < 0) {
> > > +    followingMonthDays += 7;
> > > +}
> > > +numDays += followingMonthDays; 
> > >  
> > > -serviceCtx = UtilMisc.toMap("userLogin", userLogin,
> > "start", getFrom,"numPeriods", new Integer(numDays),
> > "periodType", new Integer(Calendar.DATE));
> > > +serviceCtx = UtilMisc.toMap("userLogin", userLogin,
> > "start", getFrom, "numPeriods", new Integer(numDays),
> > "periodType", new Integer(Calendar.DATE));
> > >  serviceCtx.putAll(UtilMisc.toMap("partyId",
> > partyId, "facilityId", facilityId, "fixedAssetId",
> > fixedAssetId, "workEffortTypeId", workEffortTypeId,
> > "locale", locale, "timeZone", timeZone));
> > >  if (entityExprList) {
> > >     
> > serviceCtx.putAll(["entityExprList" : entityExprList]);
> > > 
> > > 
> > -- 
> > Antwebsystems.com: Quality OFBiz services for competitive
> > rates
> > 
> > 
> 
> 
>       
-- 
Antwebsystems.com: Quality OFBiz services for competitive rates

Reply via email to