Fix the get_email_addresses_for_group query with local_sites.
Review Request #6374 — Created Sept. 25, 2014 and submitted
Against all odds, some users actually end up trying to set up local sites. In
this case, a user was a member of a review group on a local site, which broke
the query that chooses e-mail addresses for a normal group. I've added a
conditional to only check the user's local_site access if the group in question
has a local site.
Ran unit tests.
| Description | From | Last Updated |
|---|---|---|
|
No need to save, since adding to a M2M relation will directly affect the database. |
 chipx86
|
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+28 -3) |
-
Tool: Pyflakes Processed Files: reviewboard/notifications/email.py reviewboard/notifications/tests.py Tool: PEP8 Style Checker Processed Files: reviewboard/notifications/email.py reviewboard/notifications/tests.py
-
-
reviewboard/notifications/tests.py (Diff revision 2) No need to save, since adding to a M2M relation will directly affect the database.
Tool: PEP8 Style Checker Processed Files: reviewboard/notifications/email.py Tool: Pyflakes Processed Files: reviewboard/notifications/email.py