Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature Request: Support Wirebox Injection Definition #21

Open
homestar9 opened this issue Apr 23, 2021 · 1 comment
Open

Feature Request: Support Wirebox Injection Definition #21

homestar9 opened this issue Apr 23, 2021 · 1 comment

Comments

@homestar9
Copy link

homestar9 commented Apr 23, 2021

This VS code plugin makes developing Coldbox apps so much better! Thank you for all the time you've spent working on this. One thing that I feel would be a great addition would be if we could "CTRL + Click" directly to a method in an injected property.

For example, let's say you have the following ColdBox handler:

component {
  property name="userService" inject="UserService";

  function index( event, rc, prc ) {
    prc.user = userService.getById( 1 );
  }

}

The idea would be if VS Code could understand that userService.getById() belongs to the UserService model and could take the user right to the definition of getById() in models/UserService.cfc. Unfortunately, this probably adds quite a bit of complexity because there's no way to know from the injection DSL where UserService lives unless the extension would also understand all of Coldbox's conventions. However, this would be pretty slick if it were possible.

@lmajano
Copy link
Member

lmajano commented Apr 23, 2021

I would love this @homestar9 But I think this might be under the realm of @KamasamaK and the cfml plugin, where it can somehow map variables (that ;are objects) back to a mapping. Not sure what the best approach is, or if there are such things in other dynamic language editors like if you are using php or groovy? Thoughts @KamasamaK ??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants