Skip to content

Commit 1441a38

Browse files
authored
Fixes
1 parent 11b91c5 commit 1441a38

File tree

3 files changed

+7
-9
lines changed

3 files changed

+7
-9
lines changed

.github/workflows/wp-compatibility-test.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,21 +139,21 @@ jobs:
139139
if: matrix.php-version == '8.0' || matrix.php-version == '8.1'
140140
run: |
141141
echo "🔧 Upgrading polyfills to v4.* for PHP 8.0-8.1..."
142-
composer require-dev yoast/phpunit-polyfills:^4 --no-update
142+
composer require --dev yoast/phpunit-polyfills:^4 --no-update
143143
144144
- name: Upgrade PHPUnit & polyfills for PHP 8.2
145145
if: matrix.php-version == '8.2'
146146
run: |
147147
echo "🔧 Upgrading to PHPUnit 10.* and polyfills 4.* for PHP 8.2..."
148-
composer require-dev phpunit/phpunit:^10 --no-update
149-
composer require-dev yoast/phpunit-polyfills:^4 --no-update
148+
composer require --dev phpunit/phpunit:^10 --no-update
149+
composer require --dev yoast/phpunit-polyfills:^4 --no-update
150150
151151
- name: Upgrade PHPUnit & polyfills for PHP 8.3
152152
if: matrix.php-version == '8.3'
153153
run: |
154154
echo "🔧 Upgrading to PHPUnit 12.* and polyfills 4.* for PHP 8.3..."
155-
composer require-dev phpunit/phpunit:^12 --no-update
156-
composer require-dev yoast/phpunit-polyfills:^4 --no-update
155+
composer require --dev phpunit/phpunit:^12 --no-update
156+
composer require --dev yoast/phpunit-polyfills:^4 --no-update
157157
158158
- name: Install Composer dependencies
159159
uses: ramsey/composer-install@v3

includes/export.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -256,8 +256,7 @@ function sse_export_database( string $export_dir ) {
256256
// Only append --allow-root if we are actually running as root (hardening).
257257
$allow_root_flag = '';
258258
if ( function_exists( 'posix_geteuid' ) ) {
259-
$uid = posix_geteuid();
260-
if ( false !== $uid && 0 === $uid ) {
259+
if ( 0 === posix_geteuid() ) {
261260
$allow_root_flag = ' --allow-root';
262261
}
263262
}

includes/security.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,8 @@ function sse_check_path_traversal( string $normalized_file_path ): bool {
3636
* @since 2.0.0
3737
* @param string $normalized_file_path The normalized file path.
3838
* @return string|false Real file path on success, false on failure.
39-
*
40-
* @SuppressWarnings(PHPMD.CyclomaticComplexity)
4139
*/
40+
// phpmd:suppress CyclomaticComplexity -- Consolidated from 7 single-use functions for readability.
4241
function sse_resolve_file_path( string $normalized_file_path ) { // phpcs:ignore Generic.Metrics.CyclomaticComplexity.TooHigh -- Consolidated from 7 single-use functions for readability.
4342
// Security: Only allow files with safe extensions.
4443
if ( ! sse_validate_file_extension( $normalized_file_path ) ) {

0 commit comments

Comments
 (0)