teraßyte Posted September 15 Posted September 15 I've seen this error in my IDE for a while now but forgot to report it earlier. It is caused by the newly added \IPS\Node\DelayedCount trait. I see this error even in several classes, even inside Application.php for applications that don't implement the trait. 🤷♂️ When I tell the IDE to fix the issue automatically this function gets added to the file (Application.php in this case): /** * Count all comments, items, etc * * @return mixed */ protected function recount() { // TODO: Implement recount() method. } I have no comments, items, or anything else that needs recounting in this specific application or others, though. Afrodude and SeNioR- 2
Marc Posted September 16 Posted September 16 Thank you for bringing this issue to our attention! I can confirm this should be further reviewed and I have logged an internal bug report for our development team to investigate and address as necessary, in a future maintenance release.
Daniel F Posted September 16 Posted September 16 I can't reproduce this in my IDE ( phpStorm 2024.2.1 Build #PS-242.21829.154, built on August 29, 2024 ) What are the exact steps to reproduce this? Which Application.php class will show this?
teraßyte Posted September 16 Author Posted September 16 Just creating a simple application is enough. I can reproduce it even on v5. I create a new application: Name: Hello World Directory: hello Here's what I see in my IDE, that error is on line 19 (red arrow):
Daniel F Posted September 16 Posted September 16 Weird, nothing in my IDE and I really don't see how this could happen because Application isn't using it at all. Same with v4
All Astronauts Posted September 16 Posted September 16 Not seeing it here, latest alpha, PHPStorm 2022.2.5, language level PHP 8.1 What language levels are you others at?
teraßyte Posted September 16 Author Posted September 16 I'm using phpStorm 2021.2.3, language level PHP 8.1. Maybe it's a bug in this old version of the IDE then. 🙄
Solution Daniel F Posted Monday at 07:04 PM Solution Posted Monday at 07:04 PM I've run into this too few times in the last weeks and have finally found the reason So, @Matt created a fancy tool to insert all the traits as @mixins into the Model class, The weird side effect in phpStorm is, that it expects now the abstract method from DelayedCount too. And when you use phpStorms "Add Method Stubs/fix it" feature, it will also add these methods / show all the not implemented abstract methods from the used mixins and at least I just selected them all, because why not.. I expected they were required in my class So to avoid this, we have just to be more careful with using all the automatically suggested methods by phpStorm. All Astronauts 1
teraßyte Posted Monday at 10:14 PM Author Posted Monday at 10:14 PM (edited) Good to know. I've removed that mixin line from the Model class for now. 🙂 Edited Monday at 10:14 PM by teraßyte
Recommended Posts