Modify

Opened 5 years ago

Closed 5 years ago

#6260 closed enhancement (fixed)

[PATCH] Show week numbers in calendar

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

Attachments (2)

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

Download all attachments as: .zip

Change History (7)

Changed 5 years ago by terje

comment:1 Changed 5 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 5 years ago by doki_pen

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 follow-up: Changed 5 years ago by doki_pen

  • Status changed from new to assigned

Changed 5 years ago by terje

Updated with correct fix

comment:4 in reply to: ↑ 3 Changed 5 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 5 years ago by doki_pen

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [7183]) Week numbers.

and jquery-ui fixes related to week numbers

fixes #6260
thanks terje

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.