- Notifications
You must be signed in to change notification settings - Fork 14
Labels
Description
Bug Report
- Yes, I reviewed the contribution guidelines.
- Yes, more specifically, I reviewed the guidelines on how to write clear bug reports.
Describe the current, buggy behavior
Type something that starts with -, i.e. -1:
wp> -1 /bin/bash: line 1: history: -1: invalid option history: usage: history [-c] [-d offset] [n] or history -anrw [filename] or history -ps arg [arg...] => int(-1) Describe how other contributors can replicate this bug
$ wp shell wp> -1 /bin/bash: line 1: history: -1: invalid option history: usage: history [-c] [-d offset] [n] or history -anrw [filename] or history -ps arg [arg...] => int(-1) Describe what you would expect as the correct outcome
If it's negative number in PHP, it should evaluate to negative number:
wp> -1 => int(-1) Let us know what environment you are running this on
OS: Linux 5.10.0-31-amd64 #1 SMP Debian 5.10.221-1 (2024-07-14) x86_64 Shell: /bin/bash PHP binary: /usr/local/php8.3/bin/php PHP version: 8.3.24 php.ini used: /usr/local/php8.3/conf/php.ini MySQL binary: /usr/bin/mariadb MySQL version: mariadb Ver 15.1 Distrib 10.5.29-MariaDB, for debian-linux-gnu (x86_64) using EditLine wrapper SQL modes: ERROR_FOR_DIVISION_BY_ZERO,NO_ENGINE_SUBSTITUTION WP-CLI root dir: phar://wp-cli.phar/vendor/wp-cli/wp-cli WP-CLI vendor dir: phar://wp-cli.phar/vendor WP_CLI phar path: phar:///usr/local/bin/wp WP-CLI packages dir: WP-CLI cache dir: /root/.wp-cli/cache WP-CLI global config: /chroot/var/www/config/wp-cli.yml WP-CLI project config: WP-CLI version: 2.12.0 Provide a possible solution
Likely the input prompt in https://github.com/wp-cli/shell-command/blob/main/src/WP_CLI/Shell/REPL.php#L135-L144 need to be escaped?
Just leave this section out if you don't know how to fix it.
Provide additional context/Screenshots
Add any other context about the problem here.
If applicable, add screenshots to help explain (you can just drag&drop images into the Github issue).
Copilot