UnusedMethodCall false positive with foreach

[enumag]

https://psalm.dev/r/dd77299d18

[psalm-github-bot]

I found these snippets:

https://psalm.dev/r/dd77299d18

<?php

/**
 * @psalm-immutable
 */
class A implements IteratorAggregate
{
    /**
     * @return Iterator<string>
     */
	public function getIterator() {
    	yield from [];
    }
}

$a = new A;

foreach ($a as $v) {
    echo $v;
}

Psalm output (using commit b904b1d):

ERROR: UnusedMethodCall - 18:10 - The call to A::getIterator is not used

[orklah]

Seems similar to: #3625

EDIT: except it didn't have an immutable annotation

[enumag]

Yeah it's the same problem. My snippet is just a bit cleaner since it doesn't have other issues.

Immutable annotation is not completely necessary. For instance @psalm-mutation-free has the same effect.

https://psalm.dev/r/ab4bb38b1e

[psalm-github-bot]

I found these snippets:

https://psalm.dev/r/ab4bb38b1e

<?php

class A implements IteratorAggregate
{
    /**
     * @return Iterator<string>
     * @psalm-mutation-free
     */
	public function getIterator() {
    	yield from [];
    }
}

$a = new A;

foreach ($a as $v) {
    echo $v;
}

Psalm output (using commit b904b1d):

ERROR: UnusedMethodCall - 16:10 - The call to A::getIterator is not used

[enumag]

Closing as duplicate of #3625