It doesn't break the calendar. The calendar wasn't "broken" before your change. Your change was broken.
-Adrian --- On Tue, 8/4/09, Hans Bakker <[email protected]> wrote: > From: Hans Bakker <[email protected]> > Subject: Re: svn commit: r801024 - > /ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy > To: [email protected] > Date: Tuesday, August 4, 2009, 11:15 PM > 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 <[email protected]> > wrote: > > > > > From: Hans Bakker <[email protected]> > > > Subject: Re: svn commit: r801024 - > /ofbiz/trunk/applications/workeffort/webapp/workeffort/WEB-INF/actions/calendar/Month.groovy > > > To: [email protected] > > > Date: Tuesday, August 4, 2009, 6:35 PM > > > Thank you for your help Adrian. > > > > > > On Tue, 2009-08-04 at 23:40 +0000, [email protected] > > > 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 > >
