Changeset 7379 for trunk/src/includes/admin/converters/e107v1.php

Timestamp:

11/24/2025 07:23:06 PM (4 weeks 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/converters/e107v1.php (modified) (1 diff)

trunk/src/includes/admin/converters/e107v1.php

@@ -586 +586 @@
             if ( ! is_null( $row ) ) {
                 $this->map_userid[ $field ] = $row->value_id;
+            } elseif ( true === $this->convert_users ) {
+                $this->map_userid[ $field ] = 0;
             } else {
-                if ( true === $this->convert_users ) {
-                    $this->map_userid[ $field ] = 0;
-                } else {
-                    $this->map_userid[ $field ] = $field;
-                }
+                $this->map_userid[ $field ] = $field;
             }
         }