aboutsummaryrefslogtreecommitdiff
path: root/doc/todo
diff options
context:
space:
mode:
Diffstat (limited to 'doc/todo')
-rw-r--r--doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn22
1 files changed, 22 insertions, 0 deletions
diff --git a/doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn b/doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn
new file mode 100644
index 000000000..f000878b7
--- /dev/null
+++ b/doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn
@@ -0,0 +1,22 @@
+As [[Add instructive commit messages for add _47_ edit pages]], but for `remove.pm`.
+
+I use a `join()` since it at least looks like the plugin is able to remove several pages at once (`foreach` looping over file parameters), thus holding multiple entries in `@pages`. I haven't seen this happen, though.
+
+Diff follows. --[[Daniel Andersson]]
+
+[[!tag patch]]
+
+---
+
+ diff -r 4f2ad3a5377e Plugin/remove.pm
+ --- a/Plugin/remove.pm Fri Jul 15 17:39:04 2011 +0200
+ +++ b/Plugin/remove.pm Sat Jul 16 03:20:35 2011 +0200
+ @@ -228,7 +228,7 @@
+ IkiWiki::rcs_remove($file);
+ }
+ IkiWiki::rcs_commit_staged(
+ - message => gettext("removed"),
+ + message => sprintf(gettext("remove %s"), join(', ', @files)),
+ session => $session,
+ );
+ IkiWiki::enable_commit_hook();