forcedotcom / salesforcedx-vscode

Salesforce Extensions for VS Code
https://developer.salesforce.com/tools/vscode
BSD 3-Clause "New" or "Revised" License
953 stars 406 forks source link

fix: fix handling of abstract methods in outline #5555

Closed peternhale closed 5 months ago

peternhale commented 5 months ago

When document symbols are requested from the apex LS for an Apex class with abstract methods, it did not take into account an empty method body.

The LS update now handles this condition properly

@W-15593844@, #5553

Given the Apex class

public abstract class my {
    public my() {

    }
    public String s1() {
        return 's1';
    }

    public abstract String s2();
}

Functionality Before

Screenshot 2024-04-25 at 8 02 39 AM

Functionality After

Screenshot 2024-04-25 at 8 07 47 AM
diyer commented 5 months ago

Hi @peternhale , can we create a work item in our automation epic to have an end to end test for outline use case?