human_time_diff(): Do not use abbreviations in minutes

[wildworks]

I discovered this problem while researching an issue with Gutenberg: ​https://github.com/WordPress/gutenberg/issues/62976

As I understand it, when relative times are displayed in the block editor, this format is referenced. No abbreviations are used for minutes:

​https://github.com/WordPress/wordpress-develop/blob/d3ac93a93839fbe5e441309607786bacdad755cd/src/wp-includes/script-loader.php#L444-L447

On the other hand, in PHP, i.e. the human_time_diff() function, abbreviations (min/mins) are only used for minutes:

​https://github.com/WordPress/wordpress-develop/blob/d3ac93a93839fbe5e441309607786bacdad755cd/src/wp-includes/formatting.php#L3887-L3888

As a result, relative times can be formatted differently depending on whether they're expressed in JS or PHP.

One example is the Post Date block. If the post date is within an hour of the current time and relative format is enabled, it will be displayed as 1 minute/10 minutes in the block editor and represented as 1 min / 10 mins on the frontend.

I suggest not using abbreviations in human_time_diff() functions.

[wildworks]

Post Date Block: Editor

[wildworks]

Post Date Block: Frontend

Trac ticket: https://core.trac.wordpress.org/ticket/61535

[SergeyBiryukov]

In 58600:

Date/Time: Replace abbreviations for minutes in human_time_diff().

This ensures that relative times are formatted the same way in both JS and PHP.

Follow-up to [1976], [2124], [4658], [41018], [56496].

Props wildworks.
Fixes #61535.

Thanks for the PR! Merged in r58600.

[wildworks]

Hi @SergeyBiryukov,

Thank you for the commit.

Looking at r58600, it seems that the singular form is now minutes.

Also, I missed it, but maybe we should remove the abbreviation ((min=minute)) from the comment here as well:

​https://github.com/WordPress/wordpress-develop/blob/57910f67dd4ba8b1db1a5699be916ce0c805a69b/src/wp-includes/formatting.php#L3887

[SergeyBiryukov]

Replying to wildworks:

Looking at r58600, it seems that the singular form is now minutes.

Indeed, that was a typo on my part :)

Also, I missed it, but maybe we should remove the abbreviation ((min=minute)) from the comment here as well

Good catch, thanks!

[SergeyBiryukov]

In 58601:

Date/Time: Correct singular form for minutes in human_time_diff().

Follow-up to [58600].

Props wildworks.
See #61535.