Re: [Django] #24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in relation to CACHES[alias]['KEY_PREFIX']

2014-12-23 Thread Django
#24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in 
relation to
CACHES[alias]['KEY_PREFIX']
---+
 Reporter:  frankier   |Owner:  frankier
 Type:  Bug|   Status:  closed
Component:  Documentation  |  Version:  master
 Severity:  Normal |   Resolution:  fixed
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+

Comment (by Tim Graham ):

 In [changeset:"70c965ad4ff06337229c4095a2b2221467781b92"]:
 {{{
 #!CommitTicketReference repository=""
 revision="70c965ad4ff06337229c4095a2b2221467781b92"
 [1.6.x] Fixed #24035 -- Clarified docs on CACHE_MIDDLEWARE_KEY_PREFIX vs
 KEY_PREFIX

 Backport of 446b50b90e9e60760618b236d8b0ea75a3b19d5a from master
 }}}

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.0defb4d5b16aea92792e479f7a7606af%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in relation to CACHES[alias]['KEY_PREFIX']

2014-12-23 Thread Django
#24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in 
relation to
CACHES[alias]['KEY_PREFIX']
---+
 Reporter:  frankier   |Owner:  frankier
 Type:  Bug|   Status:  closed
Component:  Documentation  |  Version:  master
 Severity:  Normal |   Resolution:  fixed
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+

Comment (by Tim Graham ):

 In [changeset:"126eb58abef119b799f81bd3626cb777910847cf"]:
 {{{
 #!CommitTicketReference repository=""
 revision="126eb58abef119b799f81bd3626cb777910847cf"
 [1.7.x] Fixed #24035 -- Clarified docs on CACHE_MIDDLEWARE_KEY_PREFIX vs
 KEY_PREFIX

 Backport of 446b50b90e9e60760618b236d8b0ea75a3b19d5a from master
 }}}

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.0475ae6a165ef76114a5452ed5d31a4c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in relation to CACHES[alias]['KEY_PREFIX']

2014-12-23 Thread Django
#24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in 
relation to
CACHES[alias]['KEY_PREFIX']
---+
 Reporter:  frankier   |Owner:  frankier
 Type:  Bug|   Status:  closed
Component:  Documentation  |  Version:  master
 Severity:  Normal |   Resolution:  fixed
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+
Changes (by Tim Graham ):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"446b50b90e9e60760618b236d8b0ea75a3b19d5a"]:
 {{{
 #!CommitTicketReference repository=""
 revision="446b50b90e9e60760618b236d8b0ea75a3b19d5a"
 Fixed #24035 -- Clarified docs on CACHE_MIDDLEWARE_KEY_PREFIX vs
 KEY_PREFIX
 }}}

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.f16f860ad1c94a0f92fc64e210feca27%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in relation to CACHES[alias]['KEY_PREFIX']

2014-12-22 Thread Django
#24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in 
relation to
CACHES[alias]['KEY_PREFIX']
---+
 Reporter:  frankier   |Owner:  frankier
 Type:  Bug|   Status:  new
Component:  Documentation  |  Version:  master
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+
Changes (by timgraham):

 * needs_better_patch:   => 0
 * needs_docs:   => 0
 * needs_tests:   => 0
 * stage:  Unreviewed => Accepted


Old description:

> The interaction between CACHES[alias]['KEY_PREFIX'],
> CACHE_MIDDLEWARE_KEY_PREFIX and @cache_page(... key_prefix) is poorly
> documented. It should be made explicit that KEY_PREFIX is always used and
> combined with CACHE_MIDDLEWARE_KEY_PREFIX or @cache_page(... key_prefix)
>
> Changes should be made to both ref/settings and topics/cache.
>
> The paragraph at the bottom of
> https://docs.djangoproject.com/en/dev/topics/cache/#the-per-view-cache is
> particularly troublesome:
>
> "The two settings can also be combined. If you specify a cache and a
> key_prefix, you will get all the settings of the requested cache alias,
> but with the key_prefix overridden.
>
> Having looked through the source it appears as though the setting
> CACHE_MIDDLEWARE_KEY_PREFIX OR the argument to @cache_page is combined
> with the rest of the cache key using django.utils.cache.get_cache_key
> which is then combined with the CACHES[alias]['KEY_PREFIX'] using
> key_func.

New description:

 The interaction between `CACHES[alias]['KEY_PREFIX']`,
 `CACHE_MIDDLEWARE_KEY_PREFIX` and `@cache_page(... key_prefix)` is poorly
 documented. It should be made explicit that `KEY_PREFIX` is always used
 and combined with `CACHE_MIDDLEWARE_KEY_PREFIX` or `@cache_page(...
 key_prefix)`

 Changes should be made to both ref/settings and topics/cache.

 The paragraph at the bottom of
 https://docs.djangoproject.com/en/dev/topics/cache/#the-per-view-cache is
 particularly troublesome:

 "The two settings can also be combined. If you specify a cache and a
 key_prefix, you will get all the settings of the requested cache alias,
 but with the key_prefix overridden.

 Having looked through the source it appears as though the setting
 `CACHE_MIDDLEWARE_KEY_PREFIX` OR the argument to `@cache_page` is combined
 with the rest of the cache key using `django.utils.cache.get_cache_key`
 which is then combined with the `CACHES[alias]['KEY_PREFIX']` using
 `key_func`.

--

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.92b4d60cb06177f39b54b83d1d1225aa%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


[Django] #24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in relation to CACHES[alias]['KEY_PREFIX']

2014-12-21 Thread Django
#24035: Poor/incorrect documentation for CACHE_MIDDLEWARE_KEY_PREFIX in 
relation to
CACHES[alias]['KEY_PREFIX']
---+--
 Reporter:  frankier   |  Owner:  frankier
 Type:  Bug| Status:  new
Component:  Documentation  |Version:  master
 Severity:  Normal |   Keywords:
 Triage Stage:  Unreviewed |  Has patch:  0
Easy pickings:  0  |  UI/UX:  0
---+--
 The interaction between CACHES[alias]['KEY_PREFIX'],
 CACHE_MIDDLEWARE_KEY_PREFIX and @cache_page(... key_prefix) is poorly
 documented. It should be made explicit that KEY_PREFIX is always used and
 combined with CACHE_MIDDLEWARE_KEY_PREFIX or @cache_page(... key_prefix)

 Changes should be made to both ref/settings and topics/cache.

 The paragraph at the bottom of
 https://docs.djangoproject.com/en/dev/topics/cache/#the-per-view-cache is
 particularly troublesome:

 "The two settings can also be combined. If you specify a cache and a
 key_prefix, you will get all the settings of the requested cache alias,
 but with the key_prefix overridden.

 Having looked through the source it appears as though the setting
 CACHE_MIDDLEWARE_KEY_PREFIX OR the argument to @cache_page is combined
 with the rest of the cache key using django.utils.cache.get_cache_key
 which is then combined with the CACHES[alias]['KEY_PREFIX'] using
 key_func.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/051.10309e65d0123e75eb3738b8a09823dc%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.