Last changed helper function

[spacedmonkey]

The last changed value is used to invalidate many caches in core. Getting the code to last changed value is repeated through the core.

[DrewAPicture]

@spacedmonkey: Thanks for the patch. Looks like 37464.diff​ is missing inline documentation. Also, we should follow current naming conventions and add wp_get_ to the front of the function name.

[SergeyBiryukov]

Replying to DrewAPicture:

Also, we should follow current naming conventions and add wp_get_ to the front of the function name.

Would wp_cache_get_last_changed() be a more descriptive name? 37464.1.diff​ seems good otherwise.

[boonebgorges]

The fact that the incrementor is tied to the "last change" seems like an implementation detail that shouldn't matter to people building cache keys. The important fact is that it's an incrementor. What do people think about wp_get_cache_incrementor()?

[jorbin]

I think wp_cache_get_last_changed() sounds like the best name. It is an implementation detail, but I think this function is clearly tied to implementation and is not overly generalized.

[chriscct7]

Docbloc for new function is also incomplete

[desrosj]

Updated patch changes the function name to wp_cache_get_last_changed() and updates the Docblock.

Would this function be something that needs to be pluggable for when other caching mechanisms are in use? If so, then maybe this should go in cache.php instead of function.php.

[jorbin]

In 38849:

Cache API: introduce wp_cache_get_last_changed to improve DRY

One thing fairly common to the cache groups is a block of code to look to see when the cache was last changed, and if there isn't one, to set it for the current microtime(). It appears in 8 different places in core. This adds a new helper wp_cache_get_last_changed to DRY things up a bit.

Since wp-includes/cache.php isn't guaranteed to be loaded, this new function is in wp-includes/functions.php

Props spacedmonkey, desrosj.
Fixes #37464.

[johnjamesjacoby]

Is it too late in 4.7 to consider a wp_cache_set_last_changed()?

wp_cache_set( 'last_changed', microtime(), $foo ); is also used 11 times in core.

[tillkruess]

I’ve opened a pull request for this: https://core.trac.wordpress.org/ticket/37464