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

Debugger completion inserted wrong #13655

Closed
felixfbecker opened this issue Oct 13, 2016 · 5 comments
Closed

Debugger completion inserted wrong #13655

felixfbecker opened this issue Oct 13, 2016 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues info-needed Issue requires more information from poster

Comments

@felixfbecker
Copy link
Contributor

felixfbecker commented Oct 13, 2016

  • VSCode Version: 1.7.0-insider

Assuming we have these variables:

image

I'm typing $aBoo into the debug console and want to get completion to $aBoolean:

image

Notice how the $ is not highlighted.

The debug adapter responds correctly with:

targets: [
  { label: '$aBoolean', length: 9, start: 0, type: 'variable' }
]

As you can see it even specifies that the completion should be inserted at the beginning of the console.

After hitting enter:

image

$ is duplicated.

@isidorn isidorn self-assigned this Oct 13, 2016
@isidorn
Copy link
Contributor

isidorn commented Oct 13, 2016

October to investigate

@isidorn isidorn added this to the October 2016 milestone Oct 13, 2016
@isidorn isidorn added bug Issue identified by VS Code Team member as probable bug debug-console labels Oct 13, 2016
@isidorn
Copy link
Contributor

isidorn commented Oct 14, 2016

This bottomline issue is the definition of a word in php. Since the repl input is asking the editor model what is the word at a position, and the returned word does not contain the character $ which is wrong.

I believe the definition of a word can be changed in the php extension. Assigning to @mousetraps to investigate further. I think @aeschli can help if needed

@isidorn isidorn removed this from the October 2016 milestone Oct 14, 2016
@isidorn isidorn assigned mousetraps and unassigned isidorn Oct 14, 2016
@kieferrm kieferrm assigned roblourens and unassigned mousetraps May 1, 2017
@roblourens
Copy link
Member

@felixfbecker Do you know if this is still an issue? I don't get completions from the PHP Debug extension, should I?

@roblourens roblourens added the info-needed Issue requires more information from poster label Sep 4, 2017
@felixfbecker
Copy link
Contributor Author

Haven't tried in a long time, here's the PR: xdebug/vscode-php-debug#71

@roblourens
Copy link
Member

Thanks. Now I see the correct highlighting. When I accept the completion, it doesn't insert the $. But in the debug adapter I see
image

I think 'text' shouldn't omit the $.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
@weinand weinand added the debug Debug viewlet, configurations, breakpoints, adapter issues label May 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues info-needed Issue requires more information from poster
Projects
None yet
Development

No branches or pull requests

5 participants