From be9edd83c2b3119018611e178da0ee55f7b71c28 Mon Sep 17 00:00:00 2001 From: Thomas Punt Date: Wed, 23 Aug 2017 23:35:18 +0100 Subject: [PATCH] Fixed bug #75090 --- NEWS | 4 ++++ ext/intl/calendar/calendar_class.cpp | 22 +++++++++++----------- ext/intl/tests/bug75090.phpt | 22 ++++++++++++++++++++++ 3 files changed, 37 insertions(+), 11 deletions(-) create mode 100644 ext/intl/tests/bug75090.phpt diff --git a/NEWS b/NEWS index f0cdd2b8b1..edd4e29e54 100644 --- a/NEWS +++ b/NEWS @@ -8,6 +8,10 @@ PHP NEWS - CURL: . Fixed bug #75093 (OpenSSL support not detected). (Remi) +- Intl: + . Fixed bug #75090 (IntlGregorianCalendar doesn't have constants from parent + class). (tpunt) + - PDO_OCI: . Fixed bug #74631 (PDO_PCO with PHP-FPM: OCI environment initialized before PHP-FPM sets it up). (Ingmar Runge) diff --git a/ext/intl/calendar/calendar_class.cpp b/ext/intl/calendar/calendar_class.cpp index 4414a7a092..94a2d5a199 100644 --- a/ext/intl/calendar/calendar_class.cpp +++ b/ext/intl/calendar/calendar_class.cpp @@ -477,17 +477,6 @@ void calendar_register_IntlCalendar_class(void) Calendar_handlers.dtor_obj = Calendar_objects_dtor; Calendar_handlers.free_obj = Calendar_objects_free; - /* Create and register 'IntlGregorianCalendar' class. */ - INIT_CLASS_ENTRY(ce, "IntlGregorianCalendar", GregorianCalendar_class_functions); - GregorianCalendar_ce_ptr = zend_register_internal_class_ex(&ce, - Calendar_ce_ptr); - if (!GregorianCalendar_ce_ptr) { - //can't happen know without bigger problems before - php_error_docref0(NULL, E_ERROR, - "IntlGregorianCalendar: class registration has failed."); - return; - } - /* Declare 'IntlCalendar' class constants */ #define CALENDAR_DECL_LONG_CONST(name, val) \ zend_declare_class_constant_long(Calendar_ce_ptr, name, sizeof(name) - 1, \ @@ -541,5 +530,16 @@ void calendar_register_IntlCalendar_class(void) CALENDAR_DECL_LONG_CONST("WALLTIME_LAST", UCAL_WALLTIME_LAST); CALENDAR_DECL_LONG_CONST("WALLTIME_NEXT_VALID", UCAL_WALLTIME_NEXT_VALID); #endif + + /* Create and register 'IntlGregorianCalendar' class. */ + INIT_CLASS_ENTRY(ce, "IntlGregorianCalendar", GregorianCalendar_class_functions); + GregorianCalendar_ce_ptr = zend_register_internal_class_ex(&ce, + Calendar_ce_ptr); + if (!GregorianCalendar_ce_ptr) { + //can't happen know without bigger problems before + php_error_docref0(NULL, E_ERROR, + "IntlGregorianCalendar: class registration has failed."); + return; + } } /* }}} */ diff --git a/ext/intl/tests/bug75090.phpt b/ext/intl/tests/bug75090.phpt new file mode 100644 index 0000000000..ff5e0bd584 --- /dev/null +++ b/ext/intl/tests/bug75090.phpt @@ -0,0 +1,22 @@ +--TEST-- +Bug #75090 Constants of parent IntlCalendar class not inherited +--SKIPIF-- + +--FILE-- +getConstants()) === count($intlCalendarRef->getConstants()), + count($intlGregorianCalendarRef->getConstants()) === count($intlCalendarRef->getConstants()) +); +?> +===DONE=== +--EXPECT-- +bool(true) +bool(true) +===DONE=== -- 2.50.1