Stdlib: DateTimeZone invalid id — must throw DateInvalidTimeZoneException not E_WARNING (ext/date/php_date.c, pairs #6048)

[PurHur]

Category

stdlib | php-src-strict | ext/date

Problem

On unknown timezone identifiers, Zend PHP 8.3+ throws DateInvalidTimeZoneException (subclass of DateException, see #6048). This compiler emits only DateTimeZone::__construct(): Unknown or bad timezone (...) as E_WARNING — catch (DateInvalidTimeZoneException $e) never runs.

	Behavior
Zend 8.3+	new DateTimeZone('Not/A/Timezone') → DateInvalidTimeZoneException
VM	warning only; object may still construct

php-src reference

• php/php-src ext/date/php_date.c — throw_date_exception(), timezone ctor
• php/php-src ext/date/php_datetime.c — date_object_new_date_timezone

Repro

test/repro/maintainer-date-timezone-exception-throw.php:

./script/docker-exec.sh -- php bin/vm.php test/repro/maintainer-date-timezone-exception-throw.php
# DateInvalidTimeZoneException: no / warning only

Scope

Path	Work
ext/date/DateTimeZone.php (or handler)	On invalid identifier, throw new DateInvalidTimeZoneException(...) after #6048 registers class
date_default_timezone_set()	Same exception for invalid ids (#4634)
Tests	Compliance: invalid id throws; valid id succeeds

PHP-in-PHP first — throw from ext/date/ handler; no new C paths.

Done when

• Invalid timezone id throws DateInvalidTimeZoneException (requires Stdlib: DateMalformedStringException / DateInvalidTimeZoneException / DateInvalidOperationException — PHP 8.3 date exception hierarchy (ext/date) #6048 class registration)
• No E_WARNING-only path for bad identifiers
• date_default_timezone_set('Bad/Zone') matches Zend failure mode

Related

• Stdlib: DateMalformedStringException / DateInvalidTimeZoneException / DateInvalidOperationException — PHP 8.3 date exception hierarchy (ext/date) #6048 register DateInvalidTimeZoneException
• Stdlib: DateException base class — Exception-branch date hierarchy root missing (ext/date/php_date.h, pairs #6048) #7277 DateException base class
• Stdlib: DateTime::__construct() — malformed string warning-only, no catchable throw (pairs #6048) #7113 malformed DateTime string throw site (same pattern)
• Stdlib: timezone_open() / date_default_timezone_get() / date_default_timezone_set() — procedural ext/date (php_date.c parity) #4634 procedural timezone_open() / date_default_timezone_set()

[PurHur]

claim: worker-lane-c — starting this run

[PurHur]

Merged via #7514.

Fix: Invalid DateTimeZone ids now throw catchable DateInvalidTimeZoneException (with DateException base registered in ThrowableManifest) instead of fatal LogicException.

Verification (green):

• php bin/vm.php test/repro/maintainer_datetimezone_invalid.php → catchable DateInvalidTimeZoneException
• vendor/bin/phpunit test/compliance/VMTest.php --filter datetimezone_invalid_id → OK
• vendor/bin/phpunit test/compliance/VMTest.php --filter datetime → OK (no regressions)

php-src ref: ext/date/php_datetimezone.c
PHP implementation: lib/VM/DateTimeSupport.php, ext/standard/ThrowableManifest.php