]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/bugs/goto_with_bad_page_name.mdwn
Added a comment: i sign stuff
[git.ikiwiki.info.git] / doc / bugs / goto_with_bad_page_name.mdwn
1 If goto is passed a page name that 
2 contains spaces or is otherwise not a valid page name,
3 it will display a "page does not exist", with a create link. But,
4 clicking on the link will result in "bad page name".
6 I have found at least two ways it can happen:
8 * If 404 is enabled, and the user goes to "http://wiki/some page with spaces"
9 * If mercurial is used, it pulls the user's full name, with spaces,
10   out for `rcs_recentchanges` and that ends up on RecentChanges.
12 When fixing, need to keep in mind that we can't just run the input through
13 titlepage, since in all other circumstances, the page name is already valid
14 and we don't want to doubly-encode it.
16 Seems like the goto plugin needs to check if the page name is valid and
17 pass it through titlepage if not.
19 (As a side effect of this, 404 will start redirecting "http://wiki/some page
20 with spaces" to "http://wiki/some_page_with_spaces", if the latter exists.
21 That seems like a fairly good thing.)
23 [[done]]
25 --[[Joey]]