Rename votingapi_cache to cache_votingapi to make it consistent with core and other modules. This would make it easier for site hosts and maintainers.

Comments

eaton’s picture

eaton commented
Status: Active » Closed (works as designed)

Arguably, the table shouldn't be named _cache() at all, rather 'votingapi_results'. Unlike Drupal's core cache tables, it is NOT just a bin of keys with serialized data. It's an actual meaningful data schema and shouldn't be arbitrarily cleared the way Drupal's default cache tables can be.

Naming them the same way would probably cause more confusion, because they can't be treated the same way.

hefox’s picture

hefox commented
Title: Rename votingapi_cache to cache_votingapi » Rename votingapi_cache to votingapi_results
Status: Closed (works as designed) » Active

Turning this into a feature request

In #382866: provide script and documentation to rebuild the votingapi cache there's a way to rebuilt it, but not in the cache clear, so it's not really acting like a cache.

Also, having it end _cache instead of start cache_ does not clarify that due to other contrib modules using the _cache for their cache tables; ie. biblio_import_cache and views_object_cache.

drclaw’s picture

drclaw commented
Issue summary: View changes
Status: Active » Closed (duplicate)

Duplicate of #1962178: Rename votingapi_cache into votingapi_results