eclipse-pdt / pdt

PHP Development Tools project (PDT)
https://eclipse.org/pdt
Eclipse Public License 2.0
188 stars 46 forks source link

PDT doesn't correctly parse `Symfony\Component\Console\Application` 6.3 #243

Closed flack closed 1 year ago

flack commented 1 year ago

Describe the bug

Symfony\Component\Console\Application in version 6.3 has 49 methods:

https://github.com/symfony/symfony/blob/v6.3.0/src/Symfony/Component/Console/Application.php

However, in the PDT outline, only 7 methods are shown:

outline

That also has the effect that when you extend this Application likes this:

class testapp extends Symfony\Component\Console\Application
{
}

PDT displays the following error: The type testapp must implement the inherited abstract method ResetInterface::reset() (because reset is one of the methods in the baseclass it doesn't see)

Describe the eclipse environment

Describe your system

To Reproduce see above

Expected behavior PDT should be able to parse all valid PHP files

zulus commented 1 year ago

Good catch! Fix will be available in upcoming build

flack commented 1 year ago

I just updated and cleaned the project, it's recognizing the functions now, but somehow it thinks they don't belong to the class:

Bildschirmfoto 2023-07-04 um 22 53 36

the error when extending the class about the missing reset implementation is still shown:

Bildschirmfoto 2023-07-04 um 22 55 35
zulus commented 1 year ago

I pushed extra patch. Now have to be ok :P I tested with recent 6.3, build will be available after couple minutes

flack commented 1 year ago

just tested again, looks great now, thx!