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

[enhancement] Laravel container IoC intellisense by hardcode #143

Closed
xiaohuilam opened this issue Nov 21, 2018 · 5 comments
Closed

[enhancement] Laravel container IoC intellisense by hardcode #143

xiaohuilam opened this issue Nov 21, 2018 · 5 comments

Comments

@xiaohuilam
Copy link

cd ~/.vscode/extensions/bmewburn.vscode-intelephense-client-*
wget https://gist.github.com/xiaohuilam/8090d181eb3f7ef4778ab2c871352544/raw/926b39a01cbf704587a11bb420c0b7bd49dfba13/completionProvider.patch
git apply completionProvider.patch
cd node_modules/intelephense
npm require tsc
npm install
tsc -p ./
# after done, command+shift+p, reload window, and enjoy your laravel better dev.

demo:

image

image

@xiaohuilam
Copy link
Author

Related Issue https://github.com/bmewburn/intelephense/issues/89

@tiansin
Copy link

tiansin commented Dec 13, 2018

The integration compiled a usable version: vscode-intelephense-client-0.8.8a.vsix

@xiaohuilam
Copy link
Author

xiaohuilam commented Dec 13, 2018

@tiansin

vscode-intelephense-client-0.8.8a.vsix

after run

code --install-extension vscode-intelephense-client-0.8.8a.vsix

it seems do not work, is it working for .phpstorm.meta.php?

usage

add .vscode/settings.json to your workspace

{
    "intelephense.completionProvider.mapMeta": {
        "[al][pkv][pe]\\([\"'](.*)[\"']\\)": {
            "request": "Illuminate\\Http\\Request",
             "...": "more meta maps here"
        }
    }
}

but

  • run through support?
{
    "intelephense.completionProvider.mapMeta": {
        "[al][pkv][pe]\\([\"'](.*)[\"']\\)": {
            ".*": "$1",
        }
    }
}
  • method/property call by chain bug
    image

@antoniopaisfernandes
Copy link

@bmewburn This would be a nice feature. Would you consider adding it?

@xiaohuilam xiaohuilam changed the title [enhancement] Laravel container IoC intelsense by hardcode [enhancement] Laravel container IoC intellisense by hardcode Feb 26, 2019
@bmewburn
Copy link
Owner

tracking in #246

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

4 participants