pureMethod.redundantUnlessCallable

Code example

<?php declare(strict_types = 1);

class Mapper
{

	/**
	 * @param pure-callable(int): int $f
	 * @param array<int> $arr
	 * @return array<int>
	 * @pure-unless-callable-is-impure $f
	 */
	public function map(callable $f, array $arr): array
	{
		$result = [];
		foreach ($arr as $i => $v) {
			$result[$i] = $f($v);
		}

		return $result;
	}

}

Why is it reported?

The @pure-unless-callable-is-impure tag marks a method as pure except when the named callable parameter is impure — its purity depends on the callable passed in at each call site.

Here the parameter $f is typed as pure-callable, which guarantees it is always pure. Because the only condition that could make the method impure can never happen, the tag has no effect: the method is unconditionally pure. Marking it @pure-unless-callable-is-impure is misleading and hides the fact that the method can simply be declared @phpstan-pure.

How to fix it

Replace @pure-unless-callable-is-impure with @phpstan-pure:

 	/**
 	 * @param pure-callable(int): int $f
 	 * @param array<int> $arr
 	 * @return array<int>
-	 * @pure-unless-callable-is-impure $f
+	 * @phpstan-pure
 	 */
 	public function map(callable $f, array $arr): array

If the method is actually meant to accept impure callables, widen the parameter type from pure-callable to callable so that the @pure-unless-callable-is-impure tag becomes meaningful again:

 	/**
-	 * @param pure-callable(int): int $f
+	 * @param callable(int): int $f
 	 * @param array<int> $arr
 	 * @return array<int>
 	 * @pure-unless-callable-is-impure $f
 	 */
 	public function map(callable $f, array $arr): array