[BCB] Remove ConstantArrayType::chunk()

Author: ondrejmirtes

UPGRADING.md

@@ -255,6 +255,7 @@ Instead of `PHPStanTestCase::createBroker()`, call `PHPStanTestCase::createRefle
 * Remove `ConstantArrayType::removeLast()`, use [`Type::popArray()`](https://apiref.phpstan.org/2.0.x/PHPStan.Type.Type.html#_popArray) instead
 * Remove `ConstantArrayType::removeFirst()`, use [`Type::shiftArray()`](https://apiref.phpstan.org/2.0.x/PHPStan.Type.Type.html#_shiftArray) instead
 * Remove `ConstantArrayType::reverse()`, use [`Type::reverseArray()`](https://apiref.phpstan.org/2.0.x/PHPStan.Type.Type.html#_reverseArray) instead
+* Remove `ConstantArrayType::chunk()`, use [`Type::chunkArray()`](https://apiref.phpstan.org/2.0.x/PHPStan.Type.Type.html#_chunkArray) instead
 * Made `TypeUtils` thinner by removing methods:
   * Remove `TypeUtils::getArrays()` and `getAnyArrays()`, use [`Type::getArrays()`](https://apiref.phpstan.org/2.0.x/PHPStan.Type.Type.html#_getArrays) instead
   * Remove `TypeUtils::getConstantArrays()` and `getOldConstantArrays()`, use [`Type::getConstantArrays()`](https://apiref.phpstan.org/2.0.x/PHPStan.Type.Type.html#_getConstantArrays) instead

src/Type/Constant/ConstantArrayType.php

@@ -1147,28 +1147,6 @@ public function slice(int $offset, ?int $limit, bool $preserveKeys = false): sel
 		return $preserveKeys ? $slice : $slice->reindex();
 	}
 
-	/**
-	 * @deprecated Use chunkArray() instead
-	 * @param positive-int $length
-	 */
-	public function chunk(int $length, bool $preserveKeys = false): self
-	{
-		$builder = ConstantArrayTypeBuilder::createEmpty();
-
-		$keyTypesCount = count($this->keyTypes);
-		for ($i = 0; $i < $keyTypesCount; $i += $length) {
-			$chunk = $this->slice($i, $length, true);
-			$builder->setOffsetValueType(null, $preserveKeys ? $chunk : $chunk->getValuesArray());
-		}
-
-		$chunks = $builder->getArray();
-		if (!$chunks instanceof self) {
-			throw new ShouldNotHappenException();
-		}
-
-		return $chunks;
-	}
-
 	private function reindex(): self
 	{
 		$keyTypes = [];