!array_key_exists() should imply array for PHP8+

src/Type/Php/ArrayKeyExistsFunctionTypeSpecifyingExtension.php

@@ -13,6 +13,7 @@
 use PHPStan\Analyser\TypeSpecifierAwareExtension;
 use PHPStan\Analyser\TypeSpecifierContext;
 use PHPStan\DependencyInjection\AutowiredService;
+use PHPStan\Php\PhpVersion;
 use PHPStan\Reflection\FunctionReflection;
 use PHPStan\Type\Accessory\HasOffsetType;
 use PHPStan\Type\Accessory\NonEmptyArrayType;
@@ -31,6 +32,12 @@ final class ArrayKeyExistsFunctionTypeSpecifyingExtension implements FunctionTyp
 
 	private TypeSpecifier $typeSpecifier;
 
+	public function __construct(
+		private PhpVersion $phpVersion,
+	)
+	{
+	}
+
 	public function setTypeSpecifier(TypeSpecifier $typeSpecifier): void
 	{
 		$this->typeSpecifier = $typeSpecifier;
@@ -110,6 +117,12 @@ public function specifyTypes(
 				new ArrayType(new MixedType(), new MixedType()),
 				new HasOffsetType($keyType),
 			);
+		} elseif (
+			$this->phpVersion->throwsValueErrorForInternalFunctions()
+			&& !$arrayType->isArray()->yes()
+		) {
+			$type = new ArrayType(new MixedType(), new MixedType());
+			$context = $context->negate();
 		} else {
 			$type = new HasOffsetType($keyType);
 		}

tests/PHPStan/Analyser/nsrt/bug-13270b-php8.php

@@ -0,0 +1,30 @@
+<?php // lint >= 8.0
+
+declare(strict_types=1);
+
+namespace Bug13270bPhp8;
+
+use function PHPStan\Testing\assertType;
+
+class Test
+{
+	/**
+	 * @param mixed[] $data
+	 * @return mixed[]
+	 */
+	public function parseData(array $data): array
+	{
+		if (isset($data['price'])) {
+			assertType('mixed~null', $data['price']);
+			if (!array_key_exists('priceWithVat', $data['price'])) {
+				$data['price']['priceWithVat'] = null;
+			}
+			assertType("non-empty-array&hasOffsetValue('priceWithVat', mixed)", $data['price']);
+			if (!array_key_exists('priceWithoutVat', $data['price'])) {
+				$data['price']['priceWithoutVat'] = null;
+			}
+			assertType("non-empty-array&hasOffsetValue('priceWithoutVat', mixed)&hasOffsetValue('priceWithVat', mixed)", $data['price']);
+		}
+		return $data;
+	}
+}

tests/PHPStan/Analyser/nsrt/bug-13270b.php

@@ -1,4 +1,6 @@
-<?php declare(strict_types=1);
+<?php // lint < 8.0
+
+declare(strict_types=1);
 
 namespace Bug13270b;
 

tests/PHPStan/Analyser/nsrt/bug-13301-php8.php

@@ -0,0 +1,52 @@
+<?php // lint >= 8.0
+
+namespace Bug13301Php8;
+
+use function PHPStan\Testing\assertType;
+
+function doFoo($mixed) {
+	if (array_key_exists('a', $mixed)) {
+		assertType("non-empty-array&hasOffset('a')", $mixed);
+		echo "has-a";
+	} else {
+		assertType('array', $mixed); // could be array~hasOffset('a') after arrays got subtractable
+		echo "NO-a";
+	}
+	assertType('array', $mixed);
+}
+
+function doFooTrue($mixed) {
+	if (array_key_exists('a', $mixed) === true) {
+		assertType("non-empty-array&hasOffset('a')", $mixed);
+	} else {
+		assertType('array', $mixed); // could be array~hasOffset('a') after arrays got subtractable
+	}
+	assertType('array', $mixed);
+}
+
+function doFooTruethy($mixed) {
+	if (array_key_exists('a', $mixed) == true) {
+		assertType("non-empty-array&hasOffset('a')", $mixed);
+	} else {
+		assertType('array', $mixed); // could be array~hasOffset('a') after arrays got subtractable
+	}
+	assertType('array', $mixed);
+}
+
+function doFooFalsey($mixed) {
+	if (array_key_exists('a', $mixed) == 0) {
+		assertType("array", $mixed);
+	} else {
+		assertType("non-empty-array&hasOffset('a')", $mixed);
+	}
+	assertType('array', $mixed);
+}
+
+function doArray(array $arr) {
+	if (array_key_exists('a', $arr)) {
+		assertType("non-empty-array&hasOffset('a')", $arr);
+	} else {
+		assertType('array', $arr);
+	}
+	assertType('array', $arr);
+}

tests/PHPStan/Analyser/nsrt/bug-13301.php

@@ -0,0 +1,15 @@
+<?php // lint < 8.0
+
+namespace Bug13301Php7;
+
+use function PHPStan\Testing\assertType;
+
+function doFoo($mixed) {
+	if (array_key_exists('a', $mixed)) {
+		assertType("non-empty-array&hasOffset('a')", $mixed);
+		echo "has-a";
+	} else {
+		assertType("mixed~hasOffset('a')", $mixed);
+		echo "NO-a";
+	}
+}

tests/PHPStan/Analyser/nsrt/bug-2001-php8.php

@@ -0,0 +1,51 @@
+<?php // lint >= 8.0
+
+namespace Bug2001Php8;
+
+use function PHPStan\Testing\assertType;
+
+class HelloWorld
+{
+	public function parseUrl(string $url): string
+	{
+		$parsedUrl = parse_url(urldecode($url));
+		assertType('array{scheme?: string, host?: string, port?: int<0, 65535>, user?: string, pass?: string, path?: string, query?: string, fragment?: string}|false', $parsedUrl);
+
+		if (array_key_exists('host', $parsedUrl)) {
+			assertType('array{scheme?: string, host: string, port?: int<0, 65535>, user?: string, pass?: string, path?: string, query?: string, fragment?: string}', $parsedUrl);
+			throw new \RuntimeException('Absolute URLs are prohibited for the redirectTo parameter.');
+		}
+
+		assertType('array{scheme?: string, host?: string, port?: int<0, 65535>, user?: string, pass?: string, path?: string, query?: string, fragment?: string}', $parsedUrl);
+
+		$redirectUrl = $parsedUrl['path'];
+
+		if (array_key_exists('query', $parsedUrl)) {
+			assertType('array{scheme?: string, host?: string, port?: int<0, 65535>, user?: string, pass?: string, path?: string, query: string, fragment?: string}', $parsedUrl);
+			$redirectUrl .= '?' . $parsedUrl['query'];
+		}
+
+		if (array_key_exists('fragment', $parsedUrl)) {
+			assertType('array{scheme?: string, host?: string, port?: int<0, 65535>, user?: string, pass?: string, path?: string, query?: string, fragment: string}', $parsedUrl);
+			$redirectUrl .= '#' . $parsedUrl['query'];
+		}
+
+		assertType('array{scheme?: string, host?: string, port?: int<0, 65535>, user?: string, pass?: string, path?: string, query?: string, fragment?: string}', $parsedUrl);
+
+		return $redirectUrl;
+	}
+
+	public function doFoo(int $i)
+	{
+		$a = ['a' => $i];
+		if (rand(0, 1)) {
+			$a['b'] = $i;
+		}
+
+		if (rand(0,1)) {
+			$a = ['d' => $i];
+		}
+
+		assertType('array{a: int, b?: int}|array{d: int}', $a);
+	}
+}

tests/PHPStan/Analyser/nsrt/bug-2001.php

@@ -1,4 +1,4 @@
-<?php
+<?php // lint < 8.0
 
 namespace Bug2001;