]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn
f000878b751069f027c7300a38e01d8e338191ac
[git.ikiwiki.info.git] / 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`.
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.
5 Diff follows. --[[Daniel Andersson]]
7 [[!tag patch]]
9 ---
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();