Changeset 7379 for trunk/src/includes/admin/classes/class-bbp-converter.php

Timestamp:

11/24/2025 07:23:06 PM (6 months ago)

Author:

johnjamesjacoby

Message:

Tools - Code Improvement: stop ignoring these PHPCS sniffs:

• PEAR.Files.IncludingFile.UseRequire
• PSR2.ControlStructures.SwitchDeclaration.BodyOnNextLineDEFAULT
• PSR2.Methods.FunctionClosingBrace.SpacingBeforeClose
• Squiz.ControlStructures.ControlSignature.NewlineAfterOpenBrace
• Squiz.PHP.DisallowMultipleAssignments.FoundInControlStructure
• Squiz.PHP.EmbeddedPhp.MultipleStatements
• Squiz.PHP.NonExecutableCode.Unreachable
• Universal.Arrays.DuplicateArrayKey.Found
• Universal.ControlStructures.DisallowLonelyIf.Found
• WordPress.PHP.NoSilencedErrors.Discouraged

This commit includes code formatting changes for the above sniffs.

See #3658.

File:

• trunk/src/includes/admin/classes/class-bbp-converter.php (modified) (2 diffs)

trunk/src/includes/admin/classes/class-bbp-converter.php

@@ -270 +270 @@
 
         // Get disabled PHP functions (to avoid using them)
-        $disabled = explode( ',', @ini_get( 'disable_functions' ) );
+        $disabled = explode( ',', @ini_get( 'disable_functions' ) ); // phpcs:ignore
 
         // Maybe avoid terminating when the client goes away (if function is not disabled)
         if ( ! in_array( 'ignore_user_abort', $disabled, true ) ) {
-            @ignore_user_abort( true );
+            @ignore_user_abort( true ); // phpcs:ignore
         }
 
@@ -280 +280 @@
         if ( ! in_array( 'ini_set', $disabled, true ) ) {
             foreach ( $r as $key => $value ) {
-                // phpcs:ignore WordPress.PHP.IniSet.Risky
-                @ini_set( $key, $value );
+                @ini_set( $key, $value ); // phpcs:ignore
             }
         }