artur-shaik / vim-javacomplete2

DEPRECATED in favor of jc.nvim
Vim License
973 stars 83 forks source link

Variables created using `var` are not completable. #463

Closed InfoSec812 closed 2 years ago

InfoSec812 commented 2 years ago

Actual behavior (Required!)

As shown in THIS EXAMPLE

The variable router does not auto-complete correctly when initiating the omni-complete

Expected behavior (Required!)

That I would be presented the correct options for the Router class type

The steps to reproduce actual behavior (Required!)

  1. Clone THIS REPO
  2. Open the project using vim with this plugin enabled
  3. Go to line 22, type router. and initiate autocompletion

Environment (Required!)

Q&A

Screenshot (Optional)

image

The output of :redir and :message (Optional)

artur-shaik commented 2 years ago

Actually yes, not yet implemented.

InfoSec812 commented 2 years ago

If you give me some hints, I might be able to submit a PR. I'll try to take a look later today.

artur-shaik commented 2 years ago

I guess it requires some hacks in vim java parser class, that I still wish to replace with javaparser library.

But may be there is enough information returned by parser.

InfoSec812 commented 2 years ago

Thanks for the hint! I'll try to look into it.

DEVEN PHILLIPS

ARCHITECT, RED HAT CLOUD NATIVE RUNTIMES PRACTICE [image: https://www.redhat.com/] https://www.redhat.com/

On Mon, Nov 8, 2021 at 8:41 AM Artur Shaik @.***> wrote:

I guess it requires some hacks in vim java parser class, that I still wish to replace with javaparser library.

But may be there is enough information returned by parser.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/artur-shaik/vim-javacomplete2/issues/463#issuecomment-963161052, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAYK24A2ZWPQZEIAYW5GSTUK7HQZANCNFSM5HQX5MPA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

artur-shaik commented 2 years ago

Consider switch to jc.nvim