2009-09-03 14:11:36 |
Niels Huylebroeck |
bug |
|
|
added bug |
2009-09-03 14:12:07 |
Niels Huylebroeck |
branch linked |
|
lp:~niels-bubbles-it/openobject-addons/small-fixes-5.0 |
|
2009-09-03 14:12:19 |
Niels Huylebroeck |
openobject-addons: assignee |
|
Red15 (niels-bubbles-it) |
|
2009-09-03 14:12:29 |
Niels Huylebroeck |
openobject-addons: status |
New |
Fix Committed |
|
2009-09-03 14:32:09 |
Niels Huylebroeck |
openobject-addons: status |
Fix Committed |
Confirmed |
|
2009-09-03 15:19:18 |
Niels Huylebroeck |
description |
The code assumes that if there is less than 30 days to go it only needs to output one month.
The linked branch has a fix where it really checks the amount of days in the coming month.
There might be a problem in that file as i see the code constantly using lengthmonth(year,month+i) + 1
Not sure why the +1 is there. I have duplicated this behaviour though not 100% sure if that is correct but my test case now comes out perfectly. |
The code assumes that if there is less than 30 days to go it only needs to output one month.
The linked branch has a fix where it really checks the amount of days in the coming month.
There might be a problem in that file as i see the code constantly using lengthmonth(year,month+i) + 1
Not sure why the +1 is there on the other calls to lengthmonth but I had to remove them for it to work when i was checking day_diff1 to the lenth of days in a month. |
|
2009-09-03 15:21:10 |
Niels Huylebroeck |
openobject-addons: status |
Confirmed |
Fix Committed |
|
2009-09-03 15:45:14 |
Niels Huylebroeck |
attachment added |
|
before / after screenshot http://launchpadlibrarian.net/31276880/holidays_proof.png |
|
2009-09-10 08:34:10 |
Niels Huylebroeck |
openobject-addons: status |
Fix Committed |
Fix Released |
|