Re: [Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
 Reporter:  Nick Pope|Owner:  Nick Pope
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  master
 Severity:  Normal   |   Resolution:  fixed
 Keywords:  paginator,   | Triage Stage:  Ready for
  queryset, deprecation  |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Mariusz Felisiak ):

 In [changeset:"81993b47eaac3cea1ebbc610a3a6b824f5195523" 81993b47]:
 {{{
 #!CommitTicketReference repository=""
 revision="81993b47eaac3cea1ebbc610a3a6b824f5195523"
 Refs #29703 -- Removed QuerySetPaginator alias per deprecation timeline.
 }}}

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.a801ba4f80f016b4c14bb1153111fcc7%40djangoproject.com.

Re: [Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
 Reporter:  Nick Pope|Owner:  Nick Pope
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  master
 Severity:  Normal   |   Resolution:  fixed
 Keywords:  paginator,   | Triage Stage:  Ready for
  queryset, deprecation  |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham ):

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


Comment:

 In [changeset:"ed4bfacb3c942d0d32795e1a733b1b9367afd2e9" ed4bfac]:
 {{{
 #!CommitTicketReference repository=""
 revision="ed4bfacb3c942d0d32795e1a733b1b9367afd2e9"
 Fixed #29703 -- Deprecated QuerySetPaginator alias.

 Unused since 4406d283e13819b04556df21044089b7d119edb0.
 }}}

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

Re: [Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
 Reporter:  Nick Pope|Owner:  Nick Pope
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  paginator,   | Triage Stage:  Ready for
  queryset, deprecation  |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham):

 * stage:  Unreviewed => Ready for checkin


Comment:

 Django has been criticized for "needless deprecations" in the past, that's
 why I'm skeptical. I don't care much though.

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

Re: [Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
 Reporter:  Nick Pope|Owner:  Nick Pope
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  paginator,   | Triage Stage:
  queryset, deprecation  |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Carlton Gibson):

 > We should formalize the deprecation.

 Is there any way of estimating of those search results how many are still
 being updated? (I suspect a lot are not...)

 **Question**: given that it's 10 years old and the fix would just be a
 search and replace operation, could we not just remove it and mark it as a
 breaking change?

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

Re: [Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
 Reporter:  Nick Pope|Owner:  Nick Pope
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  paginator,   | Triage Stage:
  queryset, deprecation  |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Jon Dufresne):

 I think having an undocumented alias would only lead to confusion if it
 were used. Removing it better follows the zen of Python "There should be
 one-- and preferably only one --obvious way to do it.".

 +1 for deprecating and removing.

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

Re: [Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
 Reporter:  Nick Pope|Owner:  Nick Pope
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  paginator,   | Triage Stage:
  queryset, deprecation  |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Tim Graham):

 I don't think it's worth the churn. A one line alias isn't adding any
 maintenance overhead or clutter.

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

Re: [Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
 Reporter:  Nick Pope|Owner:  Nick Pope
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  paginator,   | Triage Stage:
  queryset, deprecation  |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Nick Pope):

 * has_patch:  0 => 1


Comment:

 [https://github.com/django/django/pull/10330 PR]

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

[Django] #29703: Deprecate undocumented QuerySetPaginator

[Django]

#29703: Deprecate undocumented QuerySetPaginator
-+-
   Reporter:  Nick Pope  |  Owner:  Nick Pope
   Type: | Status:  assigned
  Cleanup/optimization   |
  Component:  Core   |Version:  master
  (Other)|   Keywords:  paginator,
   Severity:  Normal |  queryset, deprecation
   Triage Stage: |  Has patch:  0
  Unreviewed |
Needs documentation:  0  |Needs tests:  0
Patch needs improvement:  0  |  Easy pickings:  0
  UI/UX:  0  |
-+-
 {{{QuerySetPaginator}}} was
 
[https://github.com/django/django/commit/4406d283e13819b04556df21044089b7d119edb0
 merged] into {{{Paginator}}} over 10 years ago before the release of
 Django 1.0, at which point it was undocumented but aliased for backward
 compatibility.

 I would have suggested just removing it, but curiously it seems that
 people are
 
[https://github.com/search?utf8=%E2%9C%93&q=QuerySetPaginator+extension%3Apy&type=Code
 still using it quite a lot]. We should formalize the deprecation.

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