ant-druha / intellij-powershell

Adds PowerShell language support to IntelliJ-based IDEs.
https://plugins.jetbrains.com/plugin/10249-powershell
Apache License 2.0
71 stars 20 forks source link

TODO comments on default constructor resolve #195

Open ForNeVeR opened 6 months ago

ForNeVeR commented 6 months ago

See this comment: https://github.com/ant-druha/intellij-powershell/blob/baccc65fb41e834e8eb66642911f1a1259926856/src/main/kotlin/com/intellij/plugin/powershell/ide/resolve/PowerShellResolveUtil.kt#L126

Also this one: https://github.com/ant-druha/intellij-powershell/blob/012fa5c741188e761aab38f7dfe4dd5ea44e1bec/src/main/kotlin/com/intellij/plugin/powershell/psi/impl/PowerShellInvocationExpressionImpl.kt#L28

Apparently, there's something for us to improve with the "default constructor" handling (I suppose that's about classes in PowerShell): perhaps the current navigation to these constructors doesn't work well enough? Either they aren't properly considered references from the places that call them (if a constructor is written explicitly), or the navigation doesn't properly work from the places where the constructors are used implicitly (e.g. when casting an object to a class or whatnot)?