Code

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#16280 closed Cleanup/optimization (fixed)

documentation for reverse() should include positional/keyword conflict

Reported by: bhayes@… Owned by: paulcwatts
Component: Documentation Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

Ticket #12946 was resolved in [12652] by documenting the behavior of {% url %} tags containing both positional and keyword arguments, and calling out in the documentation that this is not allowed.

It would be nice if this same note were added to the documentation of reverse() in https://docs.djangoproject.com/en/dev/topics/http/urls/#reverse as well.

Attachments (1)

16280.diff (617 bytes) - added by paulcwatts 3 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 3 years ago by aaugustin

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 3 years ago by paulcwatts

  • Owner changed from nobody to paulcwatts

Changed 3 years ago by paulcwatts

comment:3 Changed 3 years ago by paulcwatts

  • Has patch set

comment:4 Changed 3 years ago by adamnelson

  • Triage Stage changed from Accepted to Ready for checkin

Looks good.

comment:5 Changed 3 years ago by Alex

  • Resolution set to fixed
  • Status changed from new to closed

In [16780]:

Fixed #16280. Document that both args and kwargs cannot be passed to reverse at the same time.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.