dateGetWeek js routine

Hi.
I think that dateGetWeek routine is wrong. If you try weekSpinner behaviour you will see strange things. I modify that routine this way
dateGetWeek = function (paramDate, dowOffset) {
/*
* getWeek() was developed by Antonio Giarratana:
* www.meanfreepath.com
*/
dowOffset = (dowOffset === null) ? 1 : dowOffset; // default dowOffset to 1
// (ISO 8601)
var jan1ThisYear = new Date(paramDate.getFullYear(), 0, 1);
var jan1ThisYearDow = jan1ThisYear.getDay()-dowOffset;
if (jan1ThisYearDow < 0) {
jan1ThisYearDow += 7;
}
var thisDow = paramDate.getDay()-dowOffset;
if (thisDow < 0) {
thisDow += 7;
}
var thisDoY = Math.floor((paramDate.getTime() - jan1ThisYear.getTime() - (paramDate.getTimezoneOffset() - jan1ThisYear.getTimezoneOffset()) * 60000) / 86400000) + 1;
var weeknum = Math.floor((thisDoY + jan1ThisYearDow - 1) / 7) + 1;
return weeknum;
};

Now Spinner seems to work.

Hi.
Try on the page of Work - weekly Report. As soon as you increase week up 50, you get 1 2019 instead of 51 2018. It's the result of recent patch you added in spinner script; it was
if (this.value>getWeek(31, 12, year) ) {
now
if (this.value>getWeek(31, 12, year) && this.value>50) {
At the moment getWeek(31, 12, 2018) returns 1 as week number !!
In my routine I ignore ISO standard but actually I assume first week the first one containing Sunday ( Italian standard)
I tried dates from jan 1 2019 up to jan 8 2019 and I get week 1 up to 6, week 2 later

The routine should be fixed too : try it for first days in 2016 and you will get 0, as 0 result is not handled. Or you can get 55 instead of 53 (look weeknum = nday <= 4 ? 1 : 55;
Anyway, i think that spinner cna be fixed using the following (ISO standard compatible)
lastWeekOfTheYearTemp = function (year, dowOffset) {
/*
* lastWeekOfTheYearTemp() was developed by Antonio Giarratana:
*
*/
dowOffset = (dowOffset === null) ? 1 : dowOffset; // default dowOffset to 1
// (ISO 8601)
var jan1ThisYear = new Date(year, 0, 1);
var jan1ThisYearDow = jan1ThisYear.getDay() - dowOffset; // the day of week the year begins
jan1ThisYearDow = (jan1ThisYearDow >= 0 ? jan1ThisYearDow : jan1ThisYearDow + 7);
var jan1NextYear = new Date(year + 1, 0, 1);
var jan1NextYearDow = jan1NextYear.getDay() - dowOffset; // the day of week the year begins
jan1NextYearDow = (jan1NextYearDow >= 0 ? jan1NextYearDow : jan1NextYearDow + 7);
var weeknum = 52;
// if the year starts before the middle of a week
if (jan1ThisYearDow < 4 && jan1NextYearDow >= 4) {
weeknum = 53;
}
return weeknum;
};
and changing spinner script this way
var year = dijit.byId('yearSpinner').get('value');
if (this.value > lastWeekOfTheYear(year,1)) {
dijit.byId('weekSpinner').set('value', 1);
year = parseInt(year) + 1;
dijit.byId('yearSpinner').set('value', year);
} else if (this.value == 0) {
year = parseInt(year) - 1;
dijit.byId('weekSpinner').set('value', lastWeekOfTheYear(year,1));
dijit.byId('yearSpinner').set('value', year);
}
var week = dijit.byId('weekSpinner').get('value') + '';
week = (week.length == 1) ? '0' + week : week;
dojo.byId('periodValue').value = '' + year + week;


?