Changeset 56395 for trunk/src/wp-includes/class-wp.php

Timestamp:

08/13/2023 10:29:04 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Bring more consistency to Last-Modified and ETag checks.

This updates two fragments for sending a 304 Not Modified header to better align with each other by using consistent variable names and formatting.

Follow-up to [1036], [1037], [1043], [2534], [2584], [2627], [12603], [12936], [56362].

See #58831.

File:

• trunk/src/wp-includes/class-wp.php (modified) (2 diffs)

trunk/src/wp-includes/class-wp.php

@@ -506 +506 @@
 
             $wp_last_modified .= ' GMT';
-
-            $wp_etag                  = '"' . md5( $wp_last_modified ) . '"';
+            $wp_etag           = '"' . md5( $wp_last_modified ) . '"';
+
             $headers['Last-Modified'] = $wp_last_modified;
             $headers['ETag']          = $wp_etag;
@@ -515 +515 @@
                 $client_etag = wp_unslash( $_SERVER['HTTP_IF_NONE_MATCH'] );
             } else {
-                $client_etag = false;
-            }
-
-            $client_last_modified = empty( $_SERVER['HTTP_IF_MODIFIED_SINCE'] ) ? '' : trim( $_SERVER['HTTP_IF_MODIFIED_SINCE'] );
+                $client_etag = '';
+            }
+
+            if ( isset( $_SERVER['HTTP_IF_MODIFIED_SINCE'] ) ) {
+                $client_last_modified = trim( $_SERVER['HTTP_IF_MODIFIED_SINCE'] );
+            } else {
+                $client_last_modified = '';
+            }
+
             // If string is empty, return 0. If not, attempt to parse into a timestamp.
             $client_modified_timestamp = $client_last_modified ? strtotime( $client_last_modified ) : 0;
 
-            // Make a timestamp for our most recent modification..
+            // Make a timestamp for our most recent modification.
             $wp_modified_timestamp = strtotime( $wp_last_modified );
 
-            if ( ( $client_last_modified && $client_etag ) ?
-                    ( ( $client_modified_timestamp >= $wp_modified_timestamp ) && ( $client_etag === $wp_etag ) ) :
-                    ( ( $client_modified_timestamp >= $wp_modified_timestamp ) || ( $client_etag === $wp_etag ) )
+            if ( ( $client_last_modified && $client_etag )
+                ? ( ( $client_modified_timestamp >= $wp_modified_timestamp ) && ( $client_etag === $wp_etag ) )
+                : ( ( $client_modified_timestamp >= $wp_modified_timestamp ) || ( $client_etag === $wp_etag ) )
             ) {
                 $status        = 304;