Changeset 43982 for trunk/src/wp-includes/comment.php

Timestamp:

12/12/2018 03:02:00 AM (8 years ago)

Author:

jeremyfelt

Message:

REST API: Move object type-specific metadata integrations from the wrapper functions to the low-level Meta API functions.

Object type-specific actions that should happen before or after modification of metadata have so far been part of the respective wrapper functions. By using action and filter hooks, this changeset ensures they are always executed, even when calling the lower-level Meta API functions directly, which the REST API does as a prime example.

Merges [43729] to trunk.

Props flixos90, spacedmonkey.
Fixes #44467.

Location:

trunk

Files:

• . (modified) (1 prop)
• src/wp-includes/comment.php (modified) (4 diffs)

trunk/src/wp-includes/comment.php

@@ -443 +443 @@
  */
 function add_comment_meta( $comment_id, $meta_key, $meta_value, $unique = false ) {
-        $added = add_metadata( 'comment', $comment_id, $meta_key, $meta_value, $unique );
-        if ( $added ) {
-                wp_cache_set( 'last_changed', microtime(), 'comment' );
-        }
-        return $added;
+        return add_metadata( 'comment', $comment_id, $meta_key, $meta_value, $unique );
 }
 
@@ -466 +462 @@
  */
 function delete_comment_meta( $comment_id, $meta_key, $meta_value = '' ) {
-        $deleted = delete_metadata( 'comment', $comment_id, $meta_key, $meta_value );
-        if ( $deleted ) {
-                wp_cache_set( 'last_changed', microtime(), 'comment' );
-        }
-        return $deleted;
+        return delete_metadata( 'comment', $comment_id, $meta_key, $meta_value );
 }
 
@@ -507 +499 @@
  */
 function update_comment_meta( $comment_id, $meta_key, $meta_value, $prev_value = '' ) {
-        $updated = update_metadata( 'comment', $comment_id, $meta_key, $meta_value, $prev_value );
-        if ( $updated ) {
-                wp_cache_set( 'last_changed', microtime(), 'comment' );
-        }
-        return $updated;
+        return update_metadata( 'comment', $comment_id, $meta_key, $meta_value, $prev_value );
 }
 
@@ -3513 +3501 @@
         );
 }
+
+/**
+ * Sets the last changed time for the 'comment' cache group.
+ *
+ * @since 5.0.0
+ */
+function wp_cache_set_comments_last_changed() {
+        wp_cache_set( 'last_changed', microtime(), 'comment' );
+}