minor symfony#44626 [Intl] Leverage array_is_list (fancyweb)

fabpot · fabpot · commit d8fd632134d5 · 2021-12-14T19:19:10.000-08:00
This PR was merged into the 6.1 branch. Discussion ---------- [Intl] Leverage array_is_list | Q | A | ------------- | --- | Branch? | 6.1 | Bug fix? | no | New feature? | no | Deprecations? | - | Tickets | symfony#44167 (review) | License | MIT | Doc PR | - It was forgotten in symfony#44167 and this is the last occurrence in the code base 🙏 Commits ------- 575ebf2 [Intl] Leverage array_is_list
diff --git a/src/Symfony/Component/Intl/Data/Bundle/Writer/TextBundleWriter.php b/src/Symfony/Component/Intl/Data/Bundle/Writer/TextBundleWriter.php
@@ -78,10 +78,8 @@ private function writeResource($file, mixed $value, int $indentation, bool $requ
         if (\is_array($value)) {
             $intValues = \count($value) === \count(array_filter($value, 'is_int'));
 
-            $keys = array_keys($value);
-
             // check that the keys are 0-indexed and ascending
-            $intKeys = $keys === range(0, \count($keys) - 1);
+            $intKeys = array_is_list($value);
 
             if ($intValues && $intKeys) {
                 $this->writeIntVector($file, $value, $indentation);
diff --git a/src/Symfony/Component/Intl/composer.json b/src/Symfony/Component/Intl/composer.json
@@ -24,7 +24,8 @@
         }
     ],
     "require": {
-        "php": ">=8.0.2"
+        "php": ">=8.0.2",
+        "symfony/polyfill-php81": "^1.22"
     },
     "require-dev": {
         "symfony/filesystem": "^5.4|^6.0"
