Opened 11 years ago
Closed 11 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 |
Description
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.
Change History (5)
comment:1 by , 11 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 11 years ago
Easy pickings: | unset |
---|
comment:3 by , 11 years ago
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:5 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
It seems likely to me that people migrated to using the new syntax as part of the changes from the
CACHE_BACKEND
toCACHES
setting and thus a justification for making this change without a deprecation. Objections?