web commit by http://ethan.betacantrips.com/: update on this patch (it still sucks)
parent
9230487f19
commit
2a34df594e
|
@ -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:
|
A bunch of obvious shortcomings exist:
|
||||||
|
|
||||||
* I'm not sure all the untaints are safe.
|
* 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
|
* No precautions whatsoever are made to protect against race conditions or failures
|
||||||
in the rcs\_move function.
|
in the rcs\_move function.
|
||||||
* movepage.tmpl doesn't exist yet.
|
* 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
|
> issue that occurs if the page is moved using a regular svn commit too, so
|
||||||
> it's not really your concern in a way. :-)
|
> 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
|
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?
|
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
|
(That is, nobody changes W and leaves it uncommitted.) I would guess
|
||||||
|
|
Loading…
Reference in New Issue