Pass $comment_date to get_comment_time filter

[lexhair]

I was writing a function to customize the formatted time to include the timezone in the comment meta data using the get_comment_time filter. In order for me to properly indicate whether daylight savings time was in effect at the time of the comment, I needed to globalize $comment to get the comment_time property. The filter passed the $date variable which is the formatted time part of the timestamp but not the $comment_date variable which contained the entire timestamp from $comment. In the function I developed, the $date string wasn't used at all. I returned the formatted time string from $comment->comment_date directly.

Since get_comment_time() already establishes $comment_date, why not pass it to the filter as well?

[MattyRob]

Hi there, apologies that this ticket has remained unanswered for so long.

There is a ​get_comment_date() function that mirrors the ​get_comment_time() function with the former returning a formatted date while the latter returns a formatted time.

I think the confusion here is the use of a $date variable in the get_comment_time() function but the core code seems to consider that a 'time' is simply a small part of the overall timestamp which also includes the 'date'.

I think including 'date' information in the get_comment_time() function might confuse further when these 2 functions are available and documented in the codex.

[SergeyBiryukov]

I think we can pass the comment object to the filter in both get_comment_date() and get_comment_time().

[lexhair]

Replying to SergeyBiryukov:

I think we can pass the comment object to the filter in both get_comment_date() and get_comment_time().

Thank you. That's the solution I was looking for.

[nacin]

In 27283:

Pass the comment object to the get_comment_date and get_comment_time.

props SergeyBiryukov.
fixes #19849.

[jorbin]

related #22380