Merge ~twom/launchpad:gdpr-overview-data into launchpad:master

Proposed by Tom Wardill
Status: Merged
Approved by: Tom Wardill
Approved revision: 4c586c98b0a160d3ed424d02035c37b943bbaa62
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~twom/launchpad:gdpr-overview-data
Merge into: launchpad:master
Diff against target: 32 lines (+4/-4)
1 file modified
lib/lp/registry/model/person.py (+4/-4)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+405695@code.launchpad.net

Commit message

Use inbuilt any, not DB any

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lib/lp/registry/model/person.py b/lib/lp/registry/model/person.py
2index 3d8939c..edac8e9 100644
3--- a/lib/lp/registry/model/person.py
4+++ b/lib/lp/registry/model/person.py
5@@ -304,7 +304,7 @@ from lp.services.propertycache import (
6 cachedproperty,
7 get_property_cache,
8 )
9-from lp.services.searchbuilder import any
10+from lp.services.searchbuilder import any as search_any
11 from lp.services.statistics.interfaces.statistic import ILaunchpadStatisticSet
12 from lp.services.verification.interfaces.authtoken import LoginTokenType
13 from lp.services.verification.interfaces.logintoken import ILoginTokenSet
14@@ -1469,7 +1469,7 @@ class Person(
15 from lp.bugs.model.bugtask import BugTask
16 today = datetime.today().date()
17 search_params = BugTaskSearchParams(
18- user, assignee=any(*self.participant_ids),
19+ user, assignee=search_any(*self.participant_ids),
20 milestone_dateexpected_before=date,
21 milestone_dateexpected_after=today)
22
23@@ -4103,8 +4103,8 @@ class PersonSet:
24
25 # Related packages is a general overview, if we have any of them
26 # the related packages exist
27- if any(maintained_packages, uploaded_packages,
28- ppa_packages, synchronised_packages):
29+ if any([maintained_packages, uploaded_packages,
30+ ppa_packages, synchronised_packages]):
31 overview['related-packages'] = canonical_url(
32 person, view_name='+related-packages')
33

Subscribers

People subscribed via source and target branches

to status/vote changes: