Bug Fix - #329 task shell convert all characters to lowercase
- Fixed bug that inadvertently converted the entire command line to lower case in the shell, rather than just the command, for testing against the "quit" string (thanks to Juergen Daubert).
This commit is contained in:
2
AUTHORS
2
AUTHORS
@@ -41,3 +41,5 @@ Thanks to the following, who submitted detailed bug reports and excellent sugges
|
||||
Ian Mortimer
|
||||
Zach Frazier
|
||||
Joe Pulliam
|
||||
Juergen Daubert
|
||||
|
||||
|
||||
@@ -5,6 +5,8 @@
|
||||
+ Fixed bug that was causing the 'completed' report to sort incorrectly.
|
||||
+ Added feature to allow the user to quit when asked to confirm multiple
|
||||
changes. Now task asks "Proceed with change? (Yes/no/all/quit)".
|
||||
+ Fixed bug #321 where all shell input was converted to lower case (thanks
|
||||
to Juergen Daubert).
|
||||
|
||||
------ old releases ------------------------------
|
||||
|
||||
|
||||
@@ -1178,11 +1178,11 @@ void handleShell ()
|
||||
|
||||
command = "";
|
||||
std::getline (std::cin, command);
|
||||
command = lowerCase (trim (command));
|
||||
command = trim (command);
|
||||
|
||||
if (command.length () > 0 &&
|
||||
command.length () <= quit.length () &&
|
||||
command == quit.substr (0, command.length ()))
|
||||
if (command.length () > 0 &&
|
||||
command.length () <= quit.length () &&
|
||||
lowerCase (command) == quit.substr (0, command.length ()))
|
||||
{
|
||||
keepGoing = false;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user