Opened 20 months ago

Closed 19 months ago

Last modified 19 months ago

#18933 closed Cleanup/optimization (fixed)

docstring for cycle templatetag method differs from documentation

Reported by: martey Owned by: nobody
Component: Documentation Version: 1.4
Severity: Normal Keywords: cycle templatetag
Cc: marc.tamlyn@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


A code example in the docstring in the cycle method in django.template.defaulttags for the silent option of the cycle templatetag differs from the example in the actual documentation.

Line 536 of contains "{% cycle rowcolors %}" when it should be "{{ rowcolors }}". The documentation was changed in a commit fixing ticket #15570.

Attachments (0)

Change History (4)

comment:1 Changed 20 months ago by Martey Dodoo <…>

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

I have added a fix for this on a branch on Github and submitted a pull request.

comment:2 Changed 20 months ago by mjtamlyn

  • Cc marc.tamlyn@… added
  • Has patch set
  • Triage Stage changed from Unreviewed to Ready for checkin

comment:3 Changed 19 months ago by Martey Dodoo <martey@…>

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

In 3fb2662edcdfb03ebb4b2ab60ffe309104809246:

Fixes #18933. Fixes code example in docstring.

Makes code example of silent keyword docstring in cycle templatetag
method the same as in the documentation.

comment:4 Changed 19 months ago by Tim Graham <timograham@…>

In 87c06fa4f4b908b940bea7cde95251d4daf78f3c:

Merge pull request #364 from martey/ticket_18933

Fixed #18933 - Fixes code example for cycle in docstring.

Add Comment

Modify Ticket

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

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

Note: See TracTickets for help on using tickets.