]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/todo/generic___39__do__61__goto__39___for_CGI.mdwn
YAML is YAML
[git.ikiwiki.info.git] / doc / todo / generic___39__do__61__goto__39___for_CGI.mdwn
1 The [[plugins/recentchanges]] plugin has a `do=recentchanges_link` feature that will
2 redirect to a given wiki page, or an error page with a creation link.
4 In the [[plugins/contrib/comments]] plugin I've found that it would be useful to do
5 the same for users. For now I've just cloned the functionality into the comments
6 plugin, but perhaps this functionality could be renamed to `do=goto` or
7 something, and moved to `IkiWiki/CGI.pm`?
9 > Now implemented as the 'goto' branch in my git repository, along with
10 > [[apache_404_ErrorDocument_handler]]. --[[smcv]]
12 >> Looks good, the only things I wonder are:
13 >> * Should it be a separate plugin? In particular `cgi_page_from_404()` is
14 >>   pretty big, and only works if apache is configured so seems somewhat
15 >>   pluginaable.
17 >>> I've split out `goto` and `apache404` plugins in the branch. I think
18 >>> you're right that apache404 should be a plugin. If you think goto is small
19 >>> and general enough to not be a plugin, just don't merge my most recent
20 >>> patch! --[[smcv]]
22 >> * I wish there were some way to generalize the workaround for the stupid
23 >>   MSIE behavior. Actually, I wish we could ignore the MSIE stupidity,
24 >>   as I tend to do, but perhaps it's too stupid in this case for that to
25 >>   fly..
26 >> * Is there any reason to require do=goto before checking for
27 >>   `REDIRECT_STATUS`? Seems that if that code were moved
28 >>   out of the enclosing if block, the apache 404 handler could
29 >>   be set direct to the cgi, which seems simpler to remember.
30 >> --[[Joey]] 
32 >>> No, good point - the `REDIRECT_STATUS` check is sufficiently unambiguous
33 >>> already. Fixed. --[[smcv]]
35 [[done]]