Skip to content

Fix #181: Fix crash for unsupported field types #182

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

Merged
merged 1 commit into from
Mar 8, 2016

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Mar 8, 2016

This change adds proper error handling for input prompt requests for field
types that we don't yet support like SecureString and PSCredential.
Previously any Read-Host or Get-Credential call which prompts for either
of these types would crash the language and debugging services. The fix
causes errors to be written to the host stating that these commands and
field types are not yet supported.

This change adds proper error handling for input prompt requests for field
types that we don't yet support like SecureString and PSCredential.
Previously any Read-Host or Get-Credential call which prompts for either
of these types would crash the language and debugging services.  The fix
causes errors to be written to the host stating that these commands and
field types are not yet supported.
daviwil added a commit that referenced this pull request Mar 8, 2016
@daviwil daviwil merged commit 5e88358 into master Mar 8, 2016
@daviwil daviwil deleted the daviwil/fix-181-read-host-crash branch March 26, 2016 15:15
TylerLeonhardt pushed a commit to TylerLeonhardt/PowerShellEditorServices that referenced this pull request Feb 26, 2019
This change fixes an issue with the "Run Selection" command when it is
executed on a line without any text being selected.  In this case the
expected behavior is to execute the entire line but only the line contents
up to the cursor position were being executed.  The fix is to change the
content selection code to ensure that the line's full content is executed.
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

Successfully merging this pull request may close these issues.

2 participants