Merge pull request #16 from asmecher/fix-nongregorian

#15 Fix fatal error with non-Gregorian locales
alphp · Dec 18, 2023 · f5e1ece · f5e1ece
2 parents 51a4ef7 + 8b72060
commit f5e1ece
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/php-8.1-strftime.php b/src/php-8.1-strftime.php
@@ -95,7 +95,11 @@ function strftime (string $format, $timestamp = null, ?string $locale = null) :
       //  in formatted strings.
       // To adjust for this, a custom calendar can be supplied with a cutover date arbitrarily far in the past.
       $calendar = IntlGregorianCalendar::createInstance();
-      $calendar->setGregorianChange(PHP_INT_MIN);
+      // NOTE: IntlGregorianCalendar::createInstance DOES NOT return an IntlGregorianCalendar instance when
+      // using a non-Gregorian locale (e.g. fa_IR)! In that case, setGregorianChange will not exist.
+      if ($calendar instanceof IntlGregorianCalendar) {
+        $calendar->setGregorianChange(PHP_INT_MIN);
+      }
 
       return (new IntlDateFormatter($locale, $date_type, $time_type, $tz, $calendar, $pattern))->format($timestamp);
     };