+1

Though my preference would be without the transaction code, I think
it's good it's just in the Wiki module, to see how it goes.

2008/7/9 Christian Boos <[EMAIL PROTECTED]>:
>
> Hello,
>
> Here's a proposal for the inclusion of the Wiki rename feature (#1106),
> in 0.12. I've sent that proposal a while ago, but 0.12 was not started
> at that time.
>
> You can find a brief summary of the design in the WikiRename page [1]
> and the corresponding set of changes in the wiki-rename sandbox,
> revisions r7333 through r7338 (r7338 for Eli ;-) ).
>
> While this feature works quite fine as it is now, the approach taken is
> compatible with further improvements, like rewriting of the original
> references (once that become possible), or real redirects.
> There are also a few related changes that can be useful for other
> features, like the `reparent_all` method for attachments which can be
> used for resource consolidation (wiki page, ticket merging?).
>
> Note that the first changeset [2] is a little refactoring which was
> simply comfortable to have in place before the actual changes started
> (it introduces a `transaction` db helper function for simplifying the
> `handle_ta` logic which was duplicated in every data model method - but
> that's already discussed in another thread).
>
> The last changeset is my first functional test and it probably needs to
> be split in multiple tests and fixed in various ways. It was quite
> tedious to get it right, so at least this enabled me to appreciate even
> more the hard work of Eli and Tim! :-)
>
> -- Christian
>
> [1] - http://trac.edgewall.org/wiki/WikiRename
> [2] - http://trac.edgewall.org/changeset/7333
>
> >
>



-- 
And the next thing you know, you're at the zoo, shaving a yak,
all so you can wax your car.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Trac 
Development" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/trac-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to