(Maybe this should be moved to planned features?) Feels more like a bug to me —tamc2
While trying out the multipage index, I noticed that the Site Index S page includes all the other Site Index pages (due to the /^S.*/i match), but seems a bit too recursive.
On a related note, the “links to this page” section will always list a site index page.
For both, pruning the site index pages from these lists would be cleaner?
—Bil
Agreed. Currently does what it does because that is the easiest way to make use of the AutomaticSummary class. It either requires the AutomaticSummary class to be extended to include an ability to specify pages to exclude or rewritting a separate MultiPageIndex class that didn’t use the AutomaticSummary class. I’ll add it to my to do list—tamc2
Also, text like site index and a search will have site index a made into a link. —Bil
Ah. The easy solution to this would be to give the index pages a more unique name. Perhaps by having a trailing fullstop, e.g: Site Index A.. The wider solution might be to change the way the automatic links match, so that there must be a non-word character following the match. I’ll have a think. —tamc2
What did you end-up doing?—Bil
Fixed in v0-0-7:
The multipage site index pages now have a trailing fullstop, and all pages looking like a site index should be excluded from the index and the pages that link here on the sidebar.
Note, this means you should manually delete all your old site index pages: rm content/Site%20Index*, the new pages will be created automatically.
I just snagged the latest CVS, removed all the old Site Index content, and restarted. Now, the links like “Site Index A.” on the multipage site index page are not hot, only the “Site Index” part of each is a link. Could this be a side effect of Fixed Bug: Matching page titles in middle of words? Note: valid links for all the individual site index pages do appear in the “pages that link here” section of navigation menu. —Bil
You are right (and about it being a side effect of the other fix I think, hopefully now I’m getting more ‘test first’ these bugs will become rarer). Opening a new bug: automatic linking does not work with trailing punctuation to cover this. Thanks—tamc2
Hopefully now fixed Bug: automatic linking does not work with trailing punctuation, see the cvs. —tamc2
It is working now. Thanks for the prompt fix. However, now the multipage site index page page is a bit confusing because subpages that do not exist still show up in the list and have only the “Site Index” part linked in, say, “Site Index A.” (when there are no pages that begin with “A”.) Can the Site Index summary page only include those subpage listings that actually have pages under them?—Bil
