Respond with updated goto branch
authorSimon McVittie <smcv@ http://smcv.pseudorandom.co.uk/>
Sat, 31 Jan 2009 23:08:48 +0000 (23:08 +0000)
committerSimon McVittie <smcv@ http://smcv.pseudorandom.co.uk/>
Sat, 31 Jan 2009 23:08:48 +0000 (23:08 +0000)
doc/todo/generic___39__do__61__goto__39___for_CGI.mdwn

index f48702bfa594e24131c69424a171966b2ae9e611..4bf3fb449f0769750dec1fd0ddb68a2321800ec2 100644 (file)
@@ -13,6 +13,12 @@ something, and moved to `IkiWiki/CGI.pm`?
 >> * Should it be a separate plugin? In particular `cgi_page_from_404()` is
 >>   pretty big, and only works if apache is configured so seems somewhat
 >>   pluginaable.
+
+>>> I've split out `goto` and `apache404` plugins in the branch. I think
+>>> you're right that apache404 should be a plugin. If you think goto is small
+>>> and general enough to not be a plugin, just don't merge my most recent
+>>> patch! --[[smcv]]
+
 >> * I wish there were some way to generalize the workaround for the stupid
 >>   MSIE behavior. Actually, I wish we could ignore the MSIE stupidity,
 >>   as I tend to do, but perhaps it's too stupid in this case for that to
@@ -22,3 +28,6 @@ something, and moved to `IkiWiki/CGI.pm`?
 >>   out of the enclosing if block, the apache 404 handler could
 >>   be set direct to the cgi, which seems simpler to remember.
 >> --[[Joey]] 
+
+>>> No, good point - the `REDIRECT_STATUS` check is sufficiently unambiguous
+>>> already. Fixed. --[[smcv]]