Coding Standards: Remove unnecessary isset() from xmlrpc.php.

SergeyBiryukov · SergeyBiryukov · commit ca0435cea00f · 2024-11-08T15:50:12.000Z
This removes a redundant `isset( $HTTP_RAW_POST_DATA )` from `xmlrpc.php`, as the variable is already set in the code block immediately preceding the affected line. Follow-up to [3498], [5445], [47926]. Props justlevine. See #52217. git-svn-id: https://develop.svn.wordpress.org/trunk@59376 602fd350-edb4-49c9-b593-d223f7449a82
diff --git a/src/xmlrpc.php b/src/xmlrpc.php
@@ -22,9 +22,7 @@
 }
 
 // Fix for mozBlog and other cases where '<?xml' isn't on the very first line.
-if ( isset( $HTTP_RAW_POST_DATA ) ) {
-	$HTTP_RAW_POST_DATA = trim( $HTTP_RAW_POST_DATA );
-}
+$HTTP_RAW_POST_DATA = trim( $HTTP_RAW_POST_DATA );
 // phpcs:enable
 
 /** Include the bootstrap for setting up WordPress environment */

Original file line number	Diff line number	Diff line change
`@@ -22,9 +22,7 @@`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`// Fix for mozBlog and other cases where '<?xml' isn't on the very first line.`
`25`		`-if ( isset( $HTTP_RAW_POST_DATA ) ) {`
`26`		`- $HTTP_RAW_POST_DATA = trim( $HTTP_RAW_POST_DATA );`
`27`		`-}`
	`25`	`+$HTTP_RAW_POST_DATA = trim( $HTTP_RAW_POST_DATA );`
`28`	`26`	`// phpcs:enable`
`29`	`27`
`30`	`28`	`/** Include the bootstrap for setting up WordPress environment */`