Changeset 7379 for trunk/src/includes/replies/template.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/replies/template.php (modified) (3 diffs)

trunk/src/includes/replies/template.php

@@ -497 +497 @@
 
     // Include pagination
+    // phpcs:ignore Universal.ControlStructures.DisallowLonelyIf.Found
     } else {
 
@@ -2546 +2547 @@
 
     // We are including the lead topic
+    // phpcs:ignore Universal.ControlStructures.DisallowLonelyIf.Found
     } else {
 
@@ -2866 +2868 @@
             $r['selected'] = sanitize_key( $_POST[ $r['select_id'] ] );
 
-        // No Post value was passed
+        // Edit reply
+        } elseif ( bbp_is_reply_edit() ) {
+            $r['reply_id'] = bbp_get_reply_id( $r['reply_id'] );
+            $r['selected'] = bbp_get_reply_status( $r['reply_id'] );
+
+        // New reply
         } else {
-
-            // Edit reply
-            if ( bbp_is_reply_edit() ) {
-                $r['reply_id'] = bbp_get_reply_id( $r['reply_id'] );
-                $r['selected'] = bbp_get_reply_status( $r['reply_id'] );
-
-            // New reply
-            } else {
-                $r['selected'] = bbp_get_public_status_id();
-            }
+            $r['selected'] = bbp_get_public_status_id();
         }
     }