Opened 12 years ago

Closed 11 years ago

#661 closed defect (fixed)

TocMacro titleindex argument - empty page list should list all pages.

Reported by: osimons Owned by: Christian Boos
Priority: normal Component: TocMacro
Severity: normal Keywords:
Cc: Trac Release: 0.11


Have had Trac projects running for close to a year, and earlier the [[TOC(titleindex) used to list all pages if no pages where supplied.

Now, it returns a titleindex only including the current page. Not a very useful titleindex.

Looked at the code, and suppose it has to do with how the list of pages is determined - where empty page list sets the list to current page. Correct in all other circumstances, but for title index it should fetch 'all' when list is empty.

Attachments (0)

Change History (4)

comment:1 Changed 12 years ago by osimons

Type: enhancementdefect

comment:2 Changed 12 years ago by Alec Thomas

Owner: changed from Alec Thomas to Noah Kantrowitz

comment:3 Changed 11 years ago by Christian Boos

Owner: changed from Noah Kantrowitz to Christian Boos
Status: newassigned
Trac Release: 0.90.11

Sounds good.

comment:4 Changed 11 years ago by Christian Boos

Resolution: fixed
Status: assignedclosed

(In [1903]) Calling [[TOC(titleindex)]] now shows the table of content for all the pages. Fixes #661.

Refactored the expand_macro method to make it clearer and to be able to reuse the _render_title_index part.

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Christian Boos.
The resolution will be deleted.

Add Comment

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

Note: See TracTickets for help on using tickets.