Modify

Opened 7 years ago

Closed 7 years ago

#6260 closed enhancement (fixed)

[PATCH] Show week numbers in calendar

Reported by: terje Owned by: Robert Corsaro
Priority: normal Component: DateFieldPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Attachments (2)

weeknumbers.diff (90.6 KB) - added by terje 7 years ago.
weeknumbers2.diff (90.3 KB) - added by terje 7 years ago.
Updated with correct fix

Download all attachments as: .zip

Change History (7)

Changed 7 years ago by terje

Attachment: weeknumbers.diff added

comment:1 Changed 7 years ago by anonymous

Where did this change come from and what does it fix?

< if(firstDay>4&&(checkDate.getDay()
7)<firstDay-3){return 1}}}return Math.floor(((checkDate-firstMon)/86400000)/7)+1},

---

if(firstDay>4&&(checkDate.getDay()
7)<firstDay-3){return 1}}}return Math.floor(Math.round((checkDate-firstMon)/86400000)/7)+1},

comment:2 Changed 7 years ago by Robert Corsaro

Sorry, I meant this:

 <
 if(firstDay>4&&(checkDate.getDay()||7)<firstDay-3){return 1}}}return
 Math.floor(((checkDate-firstMon)/86400000)/7)+1},
 ---
 >
 if(firstDay>4&&(checkDate.getDay()||7)<firstDay-3){return 1}}}return
 Math.floor(Math.round((checkDate-firstMon)/86400000)/7)+1},

comment:3 Changed 7 years ago by Robert Corsaro

Status: newassigned

Changed 7 years ago by terje

Attachment: weeknumbers2.diff added

Updated with correct fix

comment:4 in reply to:  3 Changed 7 years ago by terje

Replying to doki_pen: Sorry that was a quick hack to fix an off-by-one error New attachment includes correct fix from http://dev.jqueryui.com/changeset/2423

comment:5 Changed 7 years ago by Robert Corsaro

Resolution: fixed
Status: assignedclosed

(In [7183]) Week numbers.

and jquery-ui fixes related to week numbers

fixes #6260 thanks terje

Modify Ticket

Action
as closed The owner will remain Robert Corsaro.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.