X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/08d4a3bd03835bada59d60a26247002db21581d2..2cb0bb432daa612d928de543e74927755094450d:/doc/todo/preview_changes.mdwn?ds=inline diff --git a/doc/todo/preview_changes.mdwn b/doc/todo/preview_changes.mdwn index 9533b993c..28797b452 100644 --- a/doc/todo/preview_changes.mdwn +++ b/doc/todo/preview_changes.mdwn @@ -7,65 +7,8 @@ Some discussion from the main [[/index/discussion]] page: > >> It's doable, it could even be done by a [[todo/plugin]], I think. >> --[[Joey]] +>> ---- +See the [[plugins/editdiff]] plugin --[[JeremieKoenig]] -I need help with this. Supposedly, such a plugin would provide a `formbuilder_setup` -hook which would add the button, and a cgi hook which would interecept -"Show Diff" clicks. This would show a diff between -`titlepage(possibly_foolish_untaint($form->field("page")))` -and the provided `$form->field("editcontent")`. - -But: - - * How could the case of concurrent editing be handled ? - Especially as (I think) only the RCS backend can know that - this has happened. - * Doing the formbuilder trick for adding a button needs the following - patch. (note that this is not the only template which has its submit - buttons hardcoded; is this supposed to work around something?) - * Coercing `editpage()` into showing the edit form again rather that - saving the page would be tricky and hackish to do from a module; - maybe this would better be implemented directly in `showpage`? - -
-Index: templates/editpage.tmpl -=================================================================== ---- templates/editpage.tmpl (révision 4130) -+++ templates/editpage.tmpl (copie de travail) -@@ -57,9 +57,7 @@ - Optional comment about this change:- ---[[JeremieKoenig]] \ No newline at end of file +[[done]]
-
- -- -- -- -+- - - -Index: IkiWiki/CGI.pm -=================================================================== ---- IkiWiki/CGI.pm (révision 4130) -+++ IkiWiki/CGI.pm (copie de travail) -@@ -304,6 +304,7 @@ - eval q{use CGI::FormBuilder}; - error($@) if $@; - my $form = CGI::FormBuilder->new( -+ title => "editpage", - fields => \@fields, - charset => "utf-8", - method => 'POST', -@@ -321,7 +322,8 @@ - ); - - run_hooks(formbuilder_setup => sub { -- shift->(form => $form, cgi => $q, session => $session); -+ shift->(form => $form, cgi => $q, session => $session, -+ buttons => \@buttons); - }); - - decode_form_utf8($form); -