search

JosephMajaseSithole / datejs

Automatically exported from code.google.com/p/datejs
0 stars 0 forks source link

setWeek patch #58

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
Hi

I've run into a small bug with setWeek.

My interpretation of how this function should work is: Move to the given
week in this year.
The current implementation does not take into consideration the case, when
the week is No. 1 but the month is December.

Why are you moving to the start of the week? If I need to move to some
arbitrary week and see what date the same dayofweek is I'll need to reset
the dayofweek.

I've created a patch to fix this issue

Original issue reported on code.google.com by Divine...@gmail.com on 14 Jan 2009 at 10:33

Attachments:

GoogleCodeExporter commented 8 years ago 
updated patch, where this is not altered by .addWeeks(-1)

this patch invalidates the previous patch

Original comment by Divine...@gmail.com on 14 Jan 2009 at 3:21

Attachments:

GoogleCodeExporter commented 8 years ago 
[deleted comment]
GoogleCodeExporter commented 8 years ago 
fixed in the current build on my fork: https://github.com/abritinthebay/datejs/

Original comment by darkcr...@gmail.com on 16 Sep 2013 at 5:43