When instance was created: __construct is suggested

[jens1o]

Maybe exclude all magic methods from being suggested?

[felixfbecker]

Well, technically you can actually call them. Just like you can call __toString(). Whether that is a good practice is a different story. It is of course annoying that these are always on the top because underscore sorts before letters

[jens1o]

Well, could we introduce a setting that checks for methods that won't be suggested, just like this:

{
     php.intellisense.excludeMethods: [
         '__construct',
         '__call',
         '__clone',
         '__sleep',
         '__wakeup'
    ]
}

[felixfbecker]

Do you think it's beneficial for a user to configure these?

[jens1o]

Hmm, you said (in a indirect way) that there are users who call them directly, so here we are. Didn't you?

[felixfbecker]

No, I said that you can technically call them. I think it would be fine to filter out the magic methods as they are not intended to be called directly by their nature. A simple filter for a __ prefix would do.

[jens1o]

Hmm, I don't think so(regarding the filter). Because there are open-source libs that calls a method with __ in front of it. See: https://github.com/WoltLab/WCF/blob/master/wcfsetup/install/files/lib/page/IPage.class.php#L16

[felixfbecker]

Then these libs are wrong, methods with double underscores are reserved. http://php.net/manual/en/language.oop5.magic.php

PHP reserves all function names starting with __ as magical. It is recommended that you do not use function names with __ in PHP unless you want some documented magic functionality.