Skip to content

Commit a220b95

Browse files
committed
Merge branch 'main' into copilot/fix-password-escaping-issue
2 parents a4d520d + 71a1d29 commit a220b95

2 files changed

Lines changed: 4 additions & 45 deletions

File tree

features/core-install.feature

Lines changed: 0 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -193,51 +193,6 @@ Feature: Install WordPress core
193193
When I run `wp user check-password wpcli 'R^^CzY;G"iZ@]H9b,' --escape-chars`
194194
Then the return code should be 0
195195
196-
@less-than-php-7
197-
Scenario: Install WordPress with locale set to de_DE on WP < 4.0
198-
Given an empty directory
199-
And an empty cache
200-
And a database
201-
202-
When I run `wp core download --version=3.7 --locale=de_DE`
203-
And save STDOUT 'Downloading WordPress ([\d\.]+)' as {VERSION}
204-
And I run `echo {VERSION}`
205-
Then STDOUT should contain:
206-
"""
207-
3.7
208-
"""
209-
And the wp-settings.php file should exist
210-
And the {SUITE_CACHE_DIR}/core/wordpress-{VERSION}-de_DE.tar.gz file should exist
211-
212-
When I run `wp config create --dbname={DB_NAME} --dbuser={DB_USER} --dbpass={DB_PASSWORD} --dbhost={DB_HOST} --locale=de_DE --skip-check`
213-
Then STDOUT should be:
214-
"""
215-
Success: Generated 'wp-config.php' file.
216-
"""
217-
218-
# Old versions of WP can generate wpdb database errors if the WP tables don't exist, so STDERR may or may not be empty
219-
When I try `wp core install --url=example.org --title=Test --admin_user=testadmin --admin_email=testadmin@example.com --admin_password=newpassword --locale=de_DE --skip-email`
220-
Then STDERR should contain:
221-
"""
222-
Warning: The flag --locale=de_DE is being ignored as it requires WordPress 4.0+.
223-
"""
224-
And STDOUT should contain:
225-
"""
226-
Success: WordPress installed successfully.
227-
"""
228-
229-
When I run `wp core version`
230-
Then STDOUT should contain:
231-
"""
232-
3.7
233-
"""
234-
235-
When I run `wp taxonomy list`
236-
Then STDOUT should contain:
237-
"""
238-
Kategorien
239-
"""
240-
241196
# This test downgrades to an older WordPress version, but the SQLite plugin requires 6.0+
242197
@require-mysql
243198
Scenario: Install WordPress with locale set to de_DE on WP >= 4.0

src/Core_Command.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,7 @@ public function check_update( $args, $assoc_args ) {
156156
* md5 hash verified: c5366d05b521831dd0b29dfc386e56a5
157157
* Success: WordPress downloaded.
158158
*
159+
* @skipglobalargcheck Reusing `--path` on purpose.
159160
* @when before_wp_load
160161
*
161162
* @param array{0?: string} $args Positional arguments.
@@ -467,6 +468,8 @@ public function is_installed( $args, $assoc_args ) {
467468
* # Install WordPress without disclosing admin_password to bash history
468469
* $ wp core install --url=example.com --title=Example --admin_user=supervisor --admin_email=info@example.com --prompt=admin_password < admin_password.txt
469470
*
471+
* @skipglobalargcheck Reusing `--url` on purpose.
472+
*
470473
* @param string[] $args Positional arguments. Unused.
471474
* @param array{url: string, title: string, admin_user: string, admin_password?: string, admin_email: string, locale?: string, 'skip-email'?: bool} $assoc_args Associative arguments.
472475
*/
@@ -596,6 +599,7 @@ public function multisite_convert( $args, $assoc_args ) {
596599
* Added multisite constants to wp-config.php.
597600
* Success: Network installed. Don't forget to set up rewrite rules.
598601
*
602+
* @skipglobalargcheck Reusing `--url` on purpose.
599603
* @subcommand multisite-install
600604
*
601605
* @param string[] $args Positional arguments. Unused.

0 commit comments

Comments
 (0)