Changeset 55634

Timestamp:

04/08/2023 09:07:18 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Comments: Add missing arguments for get_comment_time() in comment_time().

In a recent change, comment_time() was updated to accept a $comment_id parameter for consistency with comment_date(), following a similar change for get_comment_time().

However, the new parameter was not correctly passed to get_comment_time() inside the function. It should be passed as the fourth parameter after $format, $gmt and $translate, not the second.

This commit adds the missing arguments and a few unit tests to confirm the correct behavior.

Follow-up to [55284], [55287], [55308].

Props costdev, tmatsuur, ugyensupport, johnbillion.
Merges [55632] to the 6.2 branch.
Fixes #58064.

Location:

branches/6.2

Files:

• . (modified) (1 prop)
• src/wp-includes/comment-template.php (modified) (1 diff)
• tests/phpunit/tests/comment/commentTime.php (copied) (copied from trunk/tests/phpunit/tests/comment/commentTime.php)

branches/6.2/src/wp-includes/comment-template.php

@@ -1110 +1110 @@
  */
 function comment_time( $format = '', $comment_id = 0 ) {
-    echo get_comment_time( $format, $comment_id );
+    echo get_comment_time( $format, false, true, $comment_id );
 }