-
Notifications
You must be signed in to change notification settings - Fork 226
bugfix: fix handling of special symbols from user input dialog #966
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
base: master
Are you sure you want to change the base?
bugfix: fix handling of special symbols from user input dialog #966
Conversation
Special symbols from user input dialog not handled properly. how to reproduce: create menu item with user input dialog: git !?$GitCommand$Command:?! !?$GitCommandAddOptions$Parameters? ! !?$GitCommandAddOptions2$&2 Additional options?! call this menu item, enter input string with special symbols, for example: checkout !& resulting string is: git checkout !& This patch fixes it.
|
The current behavior is intentional, for obvious reasons. |
No. With this patch, |
Exactly, but not only Guess what will happen when you paste names like |
…s-from-user-input-dialog
ok. created file Readme!.txt, selected it, called menu item: It works as intended, nothing wrong is happened. |
|
|
Good to know, but it's not what I asked. |
|
Just in case if it's still not obvious what is wrong with this approach:
I'm not against expanding special symbols in user inputs in principle, but we cannot break the processing of random files that just happen to have |



Special symbols from user input dialog not handled properly.
how to reproduce:
create menu item with user input dialog:
git !?$GitCommand$Command:?! !?$GitCommandAddOptions$Parameters? ! !?$GitCommandAddOptions2$&2 Additional options?!
call this menu item, enter input string with special symbols, for example: checkout !&
resulting string is: git checkout !&
This patch fixes it.