From c2f2f68692a1fc80c22660c97b83c84332c44905 Mon Sep 17 00:00:00 2001 From: Adam Harvey Date: Thu, 9 Sep 2010 06:41:23 +0000 Subject: [PATCH] Fixed bug #52744 (cal_days_in_month incorrect for December 1 BCE). Original patch by gpap at internet dot gr. --- ext/calendar/calendar.c | 11 +++++++++-- ext/calendar/tests/bug52744.phpt | 12 ++++++++++++ 2 files changed, 21 insertions(+), 2 deletions(-) create mode 100644 ext/calendar/tests/bug52744.phpt diff --git a/ext/calendar/calendar.c b/ext/calendar/calendar.c index 54f2658eea..6eb2d31bef 100644 --- a/ext/calendar/calendar.c +++ b/ext/calendar/calendar.c @@ -348,8 +348,15 @@ PHP_FUNCTION(cal_days_in_month) sdn_next = calendar->to_jd(year, 1 + month, 1); if (sdn_next == 0) { -/* if invalid, try first month of the next year... */ - sdn_next = calendar->to_jd(year + 1, 1, 1); + /* If the next month is invalid, then we need to try the first month of + * the next year, bearing in mind that the next year after 1 BCE is + * actually 1 AD and not 0. */ + if (year == -1) { + sdn_next = calendar->to_jd(1, 1, 1); + } + else { + sdn_next = calendar->to_jd(year + 1, 1, 1); + } } RETURN_LONG(sdn_next - sdn_start); diff --git a/ext/calendar/tests/bug52744.phpt b/ext/calendar/tests/bug52744.phpt new file mode 100644 index 0000000000..886086a294 --- /dev/null +++ b/ext/calendar/tests/bug52744.phpt @@ -0,0 +1,12 @@ +--TEST-- +Bug #52744 (cal_days_in_month incorrect for December 1 BCE) +--SKIPIF-- + +--FILE-- + +--EXPECT-- +int(31) +int(31) -- 2.40.0