Modify ↓

Opened 18 years ago

Closed 18 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

Description

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 18 years ago by osimons

Type: enhancement → defect

comment:2 Changed 18 years ago by Alec Thomas

Owner: changed from Alec Thomas to Noah Kantrowitz

comment:3 Changed 18 years ago by Christian Boos

Owner: changed from Noah Kantrowitz to Christian Boos
Status: new → assigned
Trac Release: 0.9 → 0.11

Sounds good.

comment:4 Changed 18 years ago by Christian Boos

Resolution: → fixed
Status: assigned → closed

(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
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

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

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.