X-Git-Url: https://sipb.mit.edu/gitweb.cgi/ikiwiki.git/blobdiff_plain/65f902df434def7979fac6504cbc5d885d6061ec..c6e75e7397fccb26a0a756cda1fd0d85af7cff38:/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn diff --git a/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn b/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn index b78515596..bb3f92f9c 100644 --- a/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn +++ b/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn @@ -38,19 +38,19 @@ At the moment I see two possible solutions: +++ IkiWiki.pm 2008-07-21 20:41:35.000000000 +0200 @@ -477,13 +477,13 @@ - sub titlepage ($) { #{{{ + sub titlepage ($) { my $title=shift; - $title=~s/([^-[:alnum:]:+\/.])/$1 eq ' ' ? '_' : "__".ord($1)."__"/eg; + $title=~s/([^-[:alnum:]+\/.])/$1 eq ' ' ? '_' : "__".ord($1)."__"/eg; return $title; - } #}}} + } - sub linkpage ($) { #{{{ + sub linkpage ($) { my $link=shift; - $link=~s/([^-[:alnum:]:+\/._])/$1 eq ' ' ? '_' : "__".ord($1)."__"/eg; + $link=~s/([^-[:alnum:]+\/._])/$1 eq ' ' ? '_' : "__".ord($1)."__"/eg; return $link; - } #}}} + } What do you think about that? Does the patch have any side-effects I didn't see? @@ -61,6 +61,9 @@ What do you think about that? Does the patch have any side-effects I didn't see? >> at least I hope so ;-) ): I just created a test post in the sandbox here: [[sandbox/test: with a colon in its name]] >> (btw, why doesn't this get a hyperlink here?). >> +>>> Because wikilinks cannot have spaces, convert to underscores. +>>> --[[Joey]] +>> >> As it is put in the list of blog posts as a relative link, it starts >> with `` -- this makes the browser think that "test" is a protocol specification which is to replace `http`, >> so it complains (at least Opera and Firefox/Iceweasel on my Debian Etch do). What I described above for subpages @@ -69,3 +72,5 @@ What do you think about that? Does the patch have any side-effects I didn't see? >> >> I think the cleanest solution would be to quote colons in page names (like it is currently done for slashes)? >> Starting the links with "`./`", as I proposed above, now seems a bit ugly to me... --Mathias + +>>> No, it's all [[done]] in 2.55. --[[Joey]]