Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
-+-
 Reporter:  giggaflop@…  |Owner:  dhoffman
 Type:  New feature  |   Status:  closed
Component:  Documentation|  Version:  1.6
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
Has patch:  1|  checkin
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  1|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by Tim Graham ):

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


Comment:

 In [changeset:"6d5daa30cf29d0bada0586213ecb2959152566b4"]:
 {{{
 #!CommitTicketReference repository=""
 revision="6d5daa30cf29d0bada0586213ecb2959152566b4"
 Fixed #22553 -- Added refreshing queryset info to docs.
 }}}

-- 
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/077.63318e5fdf8c71f87f3a99a44a430dec%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
-+-
 Reporter:  giggaflop@…  |Owner:  dhoffman
 Type:  New feature  |   Status:  closed
Component:  Documentation|  Version:  1.6
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
Has patch:  1|  checkin
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  1|  Patch needs improvement:  0
 |UI/UX:  0
-+-

Comment (by Tim Graham ):

 In [changeset:"e3a99357cba5085f5ae937920529292edafc5c7a"]:
 {{{
 #!CommitTicketReference repository=""
 revision="e3a99357cba5085f5ae937920529292edafc5c7a"
 [1.6.x] Fixed #22553 -- Added refreshing queryset info to docs.

 Backport of 6d5daa30cf 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/077.46910ba40f3a6fb13737b080f68127a7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
-+-
 Reporter:  giggaflop@…  |Owner:  dhoffman
 Type:  New feature  |   Status:  closed
Component:  Documentation|  Version:  1.6
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
Has patch:  1|  checkin
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  1|  Patch needs improvement:  0
 |UI/UX:  0
-+-

Comment (by Tim Graham ):

 In [changeset:"1ef544f91f0d4be3f44ae7fd0498530e57a9b160"]:
 {{{
 #!CommitTicketReference repository=""
 revision="1ef544f91f0d4be3f44ae7fd0498530e57a9b160"
 [1.7.x] Fixed #22553 -- Added refreshing queryset info to docs.

 Backport of 6d5daa30cf 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/077.2e1498156bdbdc5f30972953694f1427%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
-+-
 Reporter:  giggaflop@…  |Owner:  dhoffman
 Type:  New feature  |   Status:  assigned
Component:  Documentation|  Version:  1.6
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
Has patch:  1|  checkin
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  1|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by timo):

 * has_patch:  0 => 1
 * stage:  Accepted => Ready for checkin


Comment:

 Looks good. Don't forget to check "Has patch" so the ticket appears in the
 review queue. Thanks!

-- 
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/077.32cb0b6ece48683e15a0272390721863%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
---+
 Reporter:  giggaflop@…|Owner:  dhoffman
 Type:  New feature|   Status:  assigned
Component:  Documentation  |  Version:  1.6
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+

Comment (by dhoffman):

 Completed: https://github.com/django/django/pull/2974

-- 
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/077.7d4a101fdd9c5cc8b1db3d8b11d8257a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
---+
 Reporter:  giggaflop@…|Owner:  dhoffman
 Type:  New feature|   Status:  assigned
Component:  Documentation  |  Version:  1.6
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+
Changes (by dhoffman):

 * status:  new => assigned
 * owner:  nobody => dhoffman


-- 
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/077.e0e3d6d1a652a5aa2236824bb82d6c3c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
---+
 Reporter:  giggaflop@…|Owner:  nobody
 Type:  New feature|   Status:  new
Component:  Documentation  |  Version:  1.6
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+

Comment (by timo):

 Fortunately, we have
 [https://docs.djangoproject.com/en/dev/internals/contributing/ extensive
 docs] that explains how to contribute. :-)

-- 
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/077.9b0c85f42c8eff87a389f7ae485b0379%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
---+
 Reporter:  giggaflop@…|Owner:  nobody
 Type:  New feature|   Status:  new
Component:  Documentation  |  Version:  1.6
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+

Comment (by nikisweeting@…):

 I'd be happy to write a little blurb in the docs showing how to use .all()
 for refreshing a QuerySet.  I'm new to contributing to here though, how do
 I go about submitting my proposed documentation amendment?

-- 
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/077.d2394dd75c00433d67d549a896807735%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
---+
 Reporter:  giggaflop@…|Owner:  nobody
 Type:  New feature|   Status:  new
Component:  Documentation  |  Version:  1.6
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+
Changes (by akaariai):

 * component:  Database layer (models, ORM) => Documentation
 * easy:  0 => 1


Comment:

 .all() can be used for this. Seems like this isn't documented, so lets do
 that. The right place seems to be
 
https://docs.djangoproject.com/en/dev/ref/models/querysets/#django.db.models.query.QuerySet.all.

-- 
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/077.dc69e19d2d8cbe7a13ddda0861198752%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
-+-
 Reporter:  giggaflop@…  |Owner:  nobody
 Type:  New feature  |   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  0|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by timo):

 * stage:  Unreviewed => Accepted


Comment:

 I guess we probably don't want to document an underscore method. I wonder
 if there is any reason not to rename it (with deprecation) to `clone()` or
 something similar?

-- 
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/077.dd06056b652d076d1a74d42db77dbaec%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
-+-
 Reporter:  giggaflop@…  |Owner:  nobody
 Type:  New feature  |   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:
 Keywords:   |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by aaugustin):

 * needs_better_patch:   => 0
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 Maybe `_clone()` already does what you need.

-- 
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/077.507b5a7c11b6ac51b946290fe4e8bb5d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #22553: Add ability to refresh queryset

[Django]

#22553: Add ability to refresh queryset
--+
 Reporter:  giggaflop@…   |  Owner:  nobody
 Type:  New feature   | Status:  new
Component:  Database layer (models, ORM)  |Version:  1.6
 Severity:  Normal|   Keywords:
 Triage Stage:  Unreviewed|  Has patch:  0
Easy pickings:  0 |  UI/UX:  0
--+
 I propose adding a Queryset method '.refresh()' which will clear the
 built-in cache on any queryset passed to it.

 I suggest an initial implementation of:


 {{{
 def refresh(self, *args, **kwargs):
 """
 Returns a new QuerySet instance with the args ANDed to the
 existing
 set.
 """
 self._result_cache = None
 return self
 }}}

 Reason this is requested:

 When I'm using modelformsets I typically find the need to refresh the
 queryset that is used by the ModelFormset.

 To do this I perform the following action:


 {{{
 updated_formset = ConciergeUserPoolFormset(request.POST)
 if updated_formset.is_valid():
 updated_formset.save()
 queryset = updated_formset.get_queryset()
 # we need an updated queryset, not the cached version. set the
 results cache to None to force re-fetch
 queryset._result_cache = None
 updated_formset = ConciergeUserPoolFormset(queryset=queryset)
 }}}

 This is not an obvious solution to the issue of updating a ModelFormset or
 forcing a queryset update in general, it also uses a private method of the
 Queryset object which I would prefer to avoid.

-- 
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/062.1b765b650489b58748c13df1bf35c2d2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.