It seems like $this->callable($listener) targeting Mvc5\Event\Generator::callable() can also be of type null; however, Mvc5\Event\Generator::emit() does only seem to accept callable, maybe add an additional type check?
This check looks at variables that are passed out again to other methods.
If the outgoing method call has stricter type requirements than the method itself, an issue is raised.
An additional type check may prevent trouble.
Loading history...
90
}
91
92
/**
93
* @param array|Iterator $queue
94
* @return mixed|null
95
*/
96
40
protected function start(&$queue)
97
{
98
40
if (is_array($queue)) {
99
37
return reset($queue);
100
}
101
102
3
$queue->rewind();
103
104
3
return $queue->current();
105
}
106
107
/**
108
* @param array|Iterator $queue
109
* @return mixed|null
110
*/
111
40
protected function step(&$queue)
112
{
113
40
if (is_array($queue)) {
114
37
return next($queue);
115
}
116
117
3
$queue->next();
118
119
3
return $queue->current();
120
}
121
122
/**
123
* @param $event
124
* @param $queue
125
* @param $args
126
* @param $callback
127
* @return null
128
*/
129
40
protected function traverse($event, $queue, $args, $callback)
mvc5 /
mvc5
devosc
This check looks at variables that are passed out again to other methods.
If the outgoing method call has stricter type requirements than the method itself, an issue is raised.
An additional type check may prevent trouble.