response

[git.ikiwiki.info.git] / doc / todo / need_global_renamepage_hook.mdwn

diff --git a/doc/todo/need_global_renamepage_hook.mdwn b/doc/todo/need_global_renamepage_hook.mdwn
index 906bf12202825e6d3c710ecb31e1b9192fc0f42f..b123340afc513340df09f1b1f7642ac2b8404dd4 100644 (file)
--- a/doc/todo/need_global_renamepage_hook.mdwn
+++ b/doc/todo/need_global_renamepage_hook.mdwn
@@ -87,6 +87,8 @@ hashes is desired, it could return the full set of hashes.
 > though, because one can't build a hash containing an array of hashes
 > as a value, without passing this array as a reference.
 > 
 > though, because one can't build a hash containing an array of hashes
 > as a value, without passing this array as a reference.
 > 

+>> Sure.
+> 

 > I'm not entirely sure about your first concern. Calling the hook
 > before or after the subpages addition both have their own problems.
 > 
 > I'm not entirely sure about your first concern. Calling the hook
 > before or after the subpages addition both have their own problems.
 > 


@@ -95,3 +97,19 @@ hashes is desired, it could return the full set of hashes.
 > a given hook function can choose to act only before or after, or both?
 > 
 > --[[intrigeri]]
 > a given hook function can choose to act only before or after, or both?
 > 
 > --[[intrigeri]]

+>> 
+>> Have you thought about making the hook be run once *per* file that is
+>> selected to be renamed? This would even handle the case where two
+>> plugins use the hook; plugin A would see when plugin B adds a new file
+>> to be renamed. And the subpage renaming stuff could probably be moved
+>> into the rename hook too. --[[Joey]] 
+>>>
+>>> I've implemented this nice solution in my po branch, please review.
+>>> I'm slowly coming back to do the last bits needed to get my po and
+>>> meta branch merged.  --[[intrigeri]]
+
+>>>> It looks good. I made some small changes to it in my own po branch.
+>>>> Nothing significant really. If this were not tied up in the po branch,
+>>>> I've have merged it to master already. --[[Joey]] 
+
+>>>> Thanks, this is great :) --[[intrigeri]]