Merge lp:~danilo/launchpad/bug-790088 into lp:launchpad

[Данило Шеган]

[r=gmb][bug=790088] Add SELECT permission on account table for personnotification user so it could send emails to teams owners who are teams themselves.

= Bug 790088 =

send-person-notification.py fails for teams owned by other teams because access to "account" table for the DB user a script runs under ("personnotification") is impossible.

We add the "SELECT" privilege on "account" table for that user.

== Tests ==

bin/test -cvvt person-notification.txt

== Demo and Q/A ==

Merge a team owned by a team into another team and try out cronscripts/send-person-notifications.py.

= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  database/schema/security.cfg
  lib/lp/registry/doc/person-notification.txt

[Graham Binns (gmb)]

review: Approve (code)

=== modified file 'database/schema/security.cfg'
--- database/schema/security.cfg	2011-07-08 18:46:51 +0000
+++ database/schema/security.cfg	2011-07-14 10:11:40 +0000
@@ -1538,6 +1538,7 @@
 
 [personnotification]
 groups=script
+public.account                          = SELECT
 public.emailaddress                     = SELECT
 public.libraryfilealias                 = SELECT
 public.libraryfilecontent               = SELECT

=== modified file 'lib/lp/registry/doc/person-notification.txt'
--- lib/lp/registry/doc/person-notification.txt	2011-05-19 04:18:07 +0000
+++ lib/lp/registry/doc/person-notification.txt	2011-07-14 10:11:40 +0000
@@ -66,9 +66,15 @@
 
     >>> notification = notification_set.addNotification(
     ...     mark, 'subject2', 'body2')
+
+This includes notifications to teams owned by other teams.
+
+    >>> owning_team = factory.makeTeam()
+    >>> team = factory.makeTeam(owner=owning_team)
+    >>> team_notification = notification_set.addNotification(
+    ...     team, 'subject3', 'body3')
     >>> [n.subject for n in notification_set.getNotificationsToSend()]
-    [u'subject2']
-
+    [u'subject2', u'subject3']
     >>> transaction.commit()
 
     >>> import subprocess