web commit by http://ethan.betacantrips.com/: update on this patch (it still sucks)

1 files changed, 12 insertions, 0 deletions

diff --git a/doc/patchqueue/move_page.mdwn b/doc/patchqueue/move_page.mdwn
index 3af49bbdb..5329a7a3b 100644
--- a/doc/patchqueue/move_page.mdwn
+++ b/doc/patchqueue/move_page.mdwn
@@ -3,6 +3,16 @@ This is my first cut at a feature like that requested in [[todo/Moving_Pages]].
 A bunch of obvious shortcomings exist: 
 
 * I'm not sure all the untaints are safe.
+
+  > I've looked them over. 
+  >
+  > * one is safe because it only allows existing page names through.
+  > * one is safe because it follows the example of editpage in calling 
+  > titlepage to sanitize.
+  > * the last one follows the example of rcs_commit in svn.pm, so I
+  > hope it's safe.. 
+  >
+  > --Ethan
 * No precautions whatsoever are made to protect against race conditions or failures
   in the rcs\_move function.
 * movepage.tmpl doesn't exist yet.
@@ -50,6 +60,8 @@ before it is moved? Bail, or shrug and proceed?
 > issue that occurs if the page is moved using a regular svn commit too, so
 > it's not really your concern in a way. :-)
 
+>> I wrote a patch to address this -- it's in [[disappearing_pages]]. --Ethan
+
 Could you elaborate on [[commit-internals]]? Can I assume that ikiwiki's 
 working copy W will always reflect a revision of the master copy M? 
 (That is, nobody changes W and leaves it uncommitted.) I would guess