Changeset 45632 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-comments-controller.php

Timestamp:

07/14/2019 02:34:02 PM (7 years ago)

Author:

SergeyBiryukov

Message:

REST API: Allow rest_get_avatar_urls() to accept full user, post, or comment objects, rather than just an email address, to provide better flexibility for alternative avatar data.

Since the function uses get_avatar_url() internally, which already supports it, this should not have any backward compatibility concerns.

Props donmhico, dshanske, pputzer, joehoyle, TimothyBlynJacobs.
Fixes #40030.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-comments-controller.php (modified) (1 diff)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-comments-controller.php

@@ -950 +950 @@
 
                 if ( in_array( 'author_avatar_urls', $fields, true ) ) {
-                        $data['author_avatar_urls'] = rest_get_avatar_urls( $comment->comment_author_email );
+                        $data['author_avatar_urls'] = rest_get_avatar_urls( $comment );
                 }