]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/todo/generic___39__do__61__goto__39___for_CGI.mdwn
pagestate: switch to use_pagespec
[git.ikiwiki.info.git] / doc / todo / generic___39__do__61__goto__39___for_CGI.mdwn
index 1276c8e6a4b886f1c6d111ffde59f374c34d4500..26c5202d01b7a84554f620214039b25f2a48cb30 100644 (file)
@@ -8,3 +8,28 @@ something, and moved to `IkiWiki/CGI.pm`?
 
 > Now implemented as the 'goto' branch in my git repository, along with
 > [[apache_404_ErrorDocument_handler]]. --[[smcv]]
+
+>> Looks good, the only things I wonder are:
+>> * 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
+>>   fly..
+>> * Is there any reason to require do=goto before checking for
+>>   `REDIRECT_STATUS`? Seems that if that code were moved
+>>   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]]
+
+[[done]]