iterable type for event properties leads to exception in PhpStan #71

Activity

1. zeleznypa commented on Jan 26, 2023

   

   zeleznypa

   on Jan 26, 2023

   Author

   More actions

   As @ondrejmirtes wrote in Pehapkari Slack:

   Because iterable is an array|Traversable. And o Traversable is not safe to do $foo[] = …
2. dg

   closed this as completedin 3c7aea4on Jan 26, 2023
3. dg

   added 2 commits that reference this issue on Jan 26, 2023

   event handlers: changed iterable -> array [Closes #71]

   This issue will close once commit ed25974 is merged into the 'master' branch.

   ed25974

   event handlers: changed iterable -> array [Closes nette/bootstrap#71]

   This issue will close once commit b582051 is merged into the 'master' branch.

   b582051
4. dg

   added a commit that references this issue on Feb 7, 2023

   [application] event handlers: changed iterable -> array [Closes nette…

   ...

   This issue will close once commit 1879882 is merged into the 'master' branch.

   1879882
5. dg

   added 2 commits that reference this issue on Mar 2, 2023

   [application] event handlers: changed iterable -> array [Closes nette…

   ...

   This issue will close once commit 1877378 is merged into the 'master' branch.

   1877378

   event handlers: changed iterable -> array [Closes nette/bootstrap#71]

   This issue will close once commit e798121 is merged into the 'master' branch.

   e798121
6. dg

   added a commit that references this issue on Apr 15, 2023

   [application] event handlers: changed iterable -> array [Closes nette…

   ...

   This issue will close once commit 18918ac is merged into the 'master' branch.

   18918ac