9.1d: Calendar getBaseStyle returns non-workdaystyle

[martintaal]

Hi,
Another thing I noticed with the 9.1d build of 10-10 is that the getBaseStyle seems to compute a wrong base style. First when opening the calendar it shows no non-workdaystyle, then when moving to the previous date all cells are non-workdaystyle.

See the attached images.

I checked/debugged the source code. In the getBaseStyle there is this line:
currRowTime = wStart.duplicate(),

which is later used like this:

        if (dayIsWorkday && (!cal.showDayLanes || 
                (wStart.getTime() <= currRowTime && currRowTime < wEnd.getTime())))
        { 
            return cal.workdayBaseStyle;
        } else {
            return this.baseStyle;
        }

It seems that the currRowTime is not the current row time but the chosen date for that day. It seems that the cell date is a better approach to get the date. So something like:
currRowTime = date.duplicate(),
(date is the celldate)

But possibly there is a better solution.

gr. Martin

[Isomorphic]

Ok, we'll take a look and update this thread when we've fixed it.

[Isomorphic]

Ok, this is fixed and will hit nightly builds tomorrow