]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
Merge commit 'upstream/master' into pub/master
authorintrigeri <intrigeri@boum.org>
Mon, 24 Nov 2008 21:38:31 +0000 (22:38 +0100)
committerintrigeri <intrigeri@boum.org>
Mon, 24 Nov 2008 21:38:31 +0000 (22:38 +0100)
doc/bugs/links_misparsed_in_CSV_files.mdwn
doc/todo/apache_404_ErrorDocument_handler.mdwn [new file with mode: 0644]
doc/todo/generic___39__do__61__goto__39___for_CGI.mdwn

index 54f099935bf19cfce48100c88f1dd22957b4b5cc..169c070e7e1017baf25fdbe70b7d51ef8789558e 100644 (file)
@@ -6,4 +6,12 @@ Links with only one underscore are OK.
 
 Update 2008-11-24: The problem only occurs if the CSV data is in an external file. If I load it using data="""...""" then it works fine.
 
+The problem appears to be the call to htmlize inside genrow. If the data is inline, then wikilinks get expanded before they get here, and are OK. If the data is from an external file, the wikilinks aren't expanded, and htmlize will expand \[[single\_track\_lines]] into \[[single&lt;em&gt;track&lt;/em&gt;lines]].
+
+Oh, wait, I see the problem. IkiWiki::linkify is only called if the external file doesn't exist. If I remove this check and always call IkiWiki::linkify, then the problem is solved.
+
+(this is inside /usr/share/perl5/IkiWiki/Plugin/table.pm).
+
+I am rather confused what this check does, and the fact the comments are very different for CSV and DSV when the code is the same doesn't seem to help.
+
 -- Brian May
diff --git a/doc/todo/apache_404_ErrorDocument_handler.mdwn b/doc/todo/apache_404_ErrorDocument_handler.mdwn
new file mode 100644 (file)
index 0000000..b200ff3
--- /dev/null
@@ -0,0 +1,18 @@
+Apache's ErrorDocument directive lets you write a CGI script that will be invoked for all 404s.
+IkiWiki could offer one as an optional wrapper; it would do much the same thing that the
+existing recentchanges_link (or [[generic___39__do__61__goto__39___for_CGI]]) does when
+encountering a nonexistent page.
+
+I think it'd probably have to be a separate CGI script because the environment with which
+404 handlers are invoked is somewhat odd, and because it needs to return a 404 status
+(having said that, it might make sense for `recentchanges_link` to return 404 rather than
+200 anyway if the page doesn't exist).
+
+This would give IkiWiki the behaviour of many other wikis, where visiting a page that
+does not yet exist prompts you to create it, without having to invoke the CGI for
+successful requests.
+
+Due to [a well-known MSIE misfeature](http://support.microsoft.com/default.aspx?scid=kb;en-us;Q294807),
+error output needs to be at least 512 bytes long, so some padding might also be required.
+
+I'm happy to write such a script if there is interest. --[[smcv]]
index ba484c877fdf9597520255766e29d261638da724..1828f0a7bbdbec2ee8bf91655603c8db100128f0 100644 (file)
@@ -6,4 +6,4 @@ the same for users. For now I've just cloned the functionality into the comments
 plugin, but perhaps this functionality could be renamed to `do=goto` or
 something, and moved to `IkiWiki/CGI.pm`?
 
-If there's general approval I'm happy to write a patch.
+If there's general approval I'm happy to write a patch. --[[smcv]]