From 01f41560e37c0a1182c70c2da23828a1ce06fd61 Mon Sep 17 00:00:00 2001 From: joey Date: Thu, 26 Apr 2007 23:29:19 +0000 Subject: [PATCH 1/1] * Fix sending of commit mails when new pages are added via the web. --- IkiWiki/UserInfo.pm | 11 ++++------- debian/changelog | 3 ++- doc/bugs/no_commit_mails_for_new_pages.mdwn | 4 ++++ 3 files changed, 10 insertions(+), 8 deletions(-) diff --git a/IkiWiki/UserInfo.pm b/IkiWiki/UserInfo.pm index ba6f3d3a9..d32f5c1bf 100644 --- a/IkiWiki/UserInfo.pm +++ b/IkiWiki/UserInfo.pm @@ -94,12 +94,8 @@ sub set_banned_users (@) { #{{{ sub commit_notify_list ($@) { #{{{ my $committer=shift; - - my @pages; - foreach my $file (@_) { - push @pages, grep { $pagesources{$_} eq $file } keys %pagesources; - } - + my @pages = map pagename($_), @_; + my @ret; my $userinfo=userinfo_retrieve(); foreach my $user (keys %{$userinfo}) { @@ -108,7 +104,8 @@ sub commit_notify_list ($@) { #{{{ length $userinfo->{$user}->{subscriptions} && exists $userinfo->{$user}->{email} && length $userinfo->{$user}->{email} && - grep { pagespec_match($_, $userinfo->{$user}->{subscriptions}, "") } @pages) { + grep { pagespec_match($_, $userinfo->{$user}->{subscriptions}, "") } + map pagename($_), @_) { push @ret, $userinfo->{$user}->{email}; } } diff --git a/debian/changelog b/debian/changelog index d0b481a34..b9dc7fae4 100644 --- a/debian/changelog +++ b/debian/changelog @@ -20,8 +20,9 @@ ikiwiki (1.51) UNRELEASED; urgency=low * Due to template changes and style sheet changes that depend on them, wiki rebuilds are recommended on upgrade to this version. * Avoid .svn directories when installing from svn checkout. + * Fix sending of commit mails when new pages are added via the web. - -- Joey Hess Thu, 26 Apr 2007 18:47:38 -0400 + -- Joey Hess Thu, 26 Apr 2007 19:27:42 -0400 ikiwiki (1.50) unstable; urgency=low diff --git a/doc/bugs/no_commit_mails_for_new_pages.mdwn b/doc/bugs/no_commit_mails_for_new_pages.mdwn index b65c8393a..3773a9455 100644 --- a/doc/bugs/no_commit_mails_for_new_pages.mdwn +++ b/doc/bugs/no_commit_mails_for_new_pages.mdwn @@ -4,3 +4,7 @@ the same page do result in commit mails. (Granted, this could be almost considered a feature, if the new page is in an rss feed..) + +Turned out to occur only for web commits that added a new file, since now +the wiki does not rebuild during the commit hook in a web commit, it could +not rely on `%pagesources` having the file. [[done]] --[[Joey]] -- 2.45.0