#17159: Paginator.Page() should throw an exception for invalid
[next|previous]_page_number()
--------------------------------+------------------------------------
     Reporter:  mehta.apurva@…  |                    Owner:  neaf
         Type:  New feature     |                   Status:  new
    Component:  Core (Other)    |                  Version:  1.3
     Severity:  Normal          |               Resolution:
     Keywords:  paginator core  |             Triage Stage:  Accepted
    Has patch:  1               |      Needs documentation:  1
  Needs tests:  1               |  Patch needs improvement:  0
Easy pickings:  1               |                    UI/UX:  0
--------------------------------+------------------------------------

Comment (by neaf):

 Attached another patch with tests and documentation.

 I'm not sure if it isn't compatibility breaking change. I'm pretty sure
 no-one wants to display 0 and out-of-range pages but I don't know if we
 should throw exceptions where we haven't in the ast.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/17159#comment:8>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to