feature detect

Author: staabm

src/Php/PhpVersion.php

@@ -394,6 +394,19 @@ public function isCurloptUrlCheckingFileSchemeWithOpenBasedir(): bool
 		return $this->versionId < 80000;
 	}
 
+	/**
+	 * whether curl handles are represented as 'resource' or CurlShareHandle
+	 */
+	public function supportsCurlShareHandle(): bool
+	{
+		return $this->versionId >= 80000;
+	}
+
+	public function supportsCurlSharePersistentHandle(): bool
+	{
+		return $this->versionId >= 80500;
+	}
+
 	public function highlightStringDoesNotReturnFalse(): bool
 	{
 		return $this->versionId >= 80400;

src/Reflection/ParametersAcceptorSelector.php

@@ -1215,11 +1215,18 @@ private static function getCurlOptValueType(int $curlOpt): ?Type
 		}
 
 		if ($curlOpt === CURLOPT_SHARE) {
-			return new UnionType([
-				new ResourceType(), // PHP 7.x
-				new ObjectType('CurlShareHandle'), // since PHP 8.0
-				new ObjectType('CurlSharePersistentHandle'), // since PHP 8.5
-			]);
+			$phpversion = PhpVersionStaticAccessor::getInstance();
+
+			if ($phpversion->supportsCurlShareHandle()) {
+				$shareType = new ObjectType('CurlShareHandle');
+			} else {
+				$shareType = new ResourceType();
+			}
+			if ($phpversion->supportsCurlSharePersistentHandle()) {
+				$shareType = TypeCombinator::union($shareType, new ObjectType('CurlSharePersistentHandle'));
+			}
+
+			return $shareType;
 		}
 
 		// unknown constant

tests/PHPStan/Rules/Functions/CallToFunctionParametersRuleTest.php

@@ -1398,13 +1398,28 @@ public function testCurlSetOpt(): void
 				'Parameter #3 $value of function curl_setopt expects bool|int, int|string given.',
 				96,
 			],
-			[
-				'Parameter #3 $value of function curl_setopt expects CurlShareHandle|CurlSharePersistentHandle|resource, string given.',
-				101,
-			],
 		]);
 	}
 
+	public function testCurlSetOptInvalidShare(): void
+	{
+		if (PHP_VERSION_ID < 80000) {
+			$errors = [
+				['Parameter #3 $value of function curl_setopt expects resource, string given.', 8],
+			];
+		} elseif (PHP_VERSION_ID < 80500) {
+			$errors = [
+				['Parameter #3 $value of function curl_setopt expects CurlShareHandle, string given.', 8],
+			];
+		} else {
+			$errors = [
+				['Parameter #3 $value of function curl_setopt expects CurlShareHandle|CurlSharePersistentHandle, string given.', 8],
+			];
+		}
+
+		$this->analyse([__DIR__ . '/data/curl_setopt_share.php'], $errors);
+	}
+
 	#[RequiresPhp('>= 8.1')]
 	public function testCurlSetOptArray(): void
 	{

tests/PHPStan/Rules/Functions/data/curl_setopt.php

@@ -98,7 +98,6 @@ public function unionType() {
 
 	public function curlShare() {
 		$curl = curl_init();
-		curl_setopt($curl, CURLOPT_SHARE, 'this is wrong');
 
 		$share = curl_share_init();
 		curl_share_setopt($share, CURLSHOPT_SHARE, CURL_LOCK_DATA_DNS);

tests/PHPStan/Rules/Functions/data/curl_setopt_share.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace CurlSetOptShare;
+
+function curlShare()
+{
+	$curl = curl_init();
+	curl_setopt($curl, CURLOPT_SHARE, 'this is wrong');
+}