Opened 9 years ago

Closed 9 years ago

#21125 closed Cleanup/optimization (fixed)

Deprecate old URI syntax for caches

Reported by: FunkyBob Owned by: FunkyBob
Component: Core (Cache system) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Given it was removed from the docs 3 years ago, when multi-cache support was added:

it's about time to remove it.

This cleanup will also help the work to add a thread-safe, non-duplicating cache API.

PR -

Change History (5)

comment:1 Changed 9 years ago by FunkyBob

Owner: changed from nobody to FunkyBob
Status: newassigned

comment:2 Changed 9 years ago by Tim Graham

Easy pickings: unset

It seems likely to me that people migrated to using the new syntax as part of the changes from the CACHE_BACKEND to CACHES setting and thus a justification for making this change without a deprecation. Objections?

comment:3 Changed 9 years ago by Aymeric Augustin

Yes, since the old syntax hasn't been supported for years, and since it's very obvious and easy to fix for people still using it, we can remove it without going through a deprecation path.

I would mention it as a backward-incompatible change in the release notes. "The URL syntax still worked, although that wasn't documented nor officially supported. If you're still using it..."

comment:4 Changed 9 years ago by FunkyBob

I've updated the docs.. please comment on working, etc.

comment:5 Changed 9 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: assignedclosed

In 4e9f800742c3048402acbaea67ec3c6bc3bd0935:

Fixed #21125 -- Removed support for cache URI syntax

Note: See TracTickets for help on using tickets.
Back to Top