diff options
author | Joey Hess <joey@gnu.kitenet.net> | 2009-06-06 02:25:25 -0400 |
---|---|---|
committer | Joey Hess <joey@gnu.kitenet.net> | 2009-06-06 02:25:25 -0400 |
commit | 07b61ec8b6994fac49ba5f0e22397e8fb143bf87 (patch) | |
tree | cf219ad9fbbd39969a09cb9ae0d8817d700cc292 /doc/bugs | |
parent | e8d8386e5258c055f253b20ad717461972b11c7d (diff) |
add
Diffstat (limited to 'doc/bugs')
-rw-r--r-- | doc/bugs/goto_with_bad_page_name.mdwn | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/doc/bugs/goto_with_bad_page_name.mdwn b/doc/bugs/goto_with_bad_page_name.mdwn new file mode 100644 index 000000000..722c4e266 --- /dev/null +++ b/doc/bugs/goto_with_bad_page_name.mdwn @@ -0,0 +1,23 @@ +If goto is passed a page name that +contains spaces or is otherwise not a valid page name, +it will display a "page does not exist", with a create link. But, +clicking on the link will result in "bad page name". + +I have found at least two ways it can happen: + +* If 404 is enabled, and the user goes to "http://wiki/some page with spaces" +* If mercurial is used, it pulls the user's full name, with spaces, + out for `rcs_recentchanges` and that ends up on RecentChanges. + +When fixing, need to keep in mind that we can't just run the input through +titlepage, since in all other circumstances, the page name is already valid +and we don't want to doubly-encode it. + +Seems like the goto plugin needs to check if the page name is valid and +pass it through titlepage if not. + +(As a side effect of this, 404 will start redirecting "http://wiki/some page +with spaces" to "http://wiki/some_page_with_spaces", if the latter exists. +That seems like a fairly good thing.) + +--[[Joey]] |