Since the temporary backward compatibility layer in cache:

function cache($bin = 'cache') {
  // Temporary backwards compatibiltiy layer, allow old style prefixed cache
  // bin names to be passed as arguments.
  $bin = str_replace('cache_', '', $bin);
  // ...

will eventually be removed, we should go through core and remove the last instances of 'cache_FOO' and replace them with 'FOO'.

One place where these type of strings are still present is in hook_flush_caches() and its implementations.

I suggest that we switch from the old style naming to the new style cache bin naming.

CommentFileSizeAuthor
#1 flush_caches-1275808-1.patch4.1 KBbfroehle

Comments

bfroehle’s picture

Status: Active » Needs review
StatusFileSize
new4.1 KB
berdir’s picture

Status: Needs review » Reviewed & tested by the community

Changes look good and are required for #1272706: Remove backwards compatibility layer for cache API. I verified that with this patch, the cache BC layer can be removed and the installer works, see the other issue for details.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.