(no commit message)
[ikiwiki] / doc / todo / Add_instructive_commit_messages_for_removing_pages.mdwn
1 As [[Add instructive commit messages for add _47_ edit pages]], but for `remove.pm`.
2
3 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.
4
5 Diff follows. --[[Daniel Andersson]]
6
7 [[!tag patch]]
8
9 ---
10
11         diff -r 4f2ad3a5377e Plugin/remove.pm
12         --- a/Plugin/remove.pm  Fri Jul 15 17:39:04 2011 +0200
13         +++ b/Plugin/remove.pm  Sat Jul 16 03:20:35 2011 +0200
14         @@ -228,7 +228,7 @@
15                                                 IkiWiki::rcs_remove($file);
16                                         }
17                                         IkiWiki::rcs_commit_staged(
18         -                                       message => gettext("removed"),
19         +                                       message => sprintf(gettext("remove %s"), join(', ', @files)),
20                                                 session => $session,
21                                         );
22                                         IkiWiki::enable_commit_hook();