harvest

Merge lp:~dholbach/harvest/980654 into lp:harvest

  1. 980654
  2. Merge into trunk
Proposed by Daniel Holbach
Status: Merged
Approved by: James Westby
Approved revision: 334
Merged at revision: 334
Proposed branch: lp:~dholbach/harvest/980654
Merge into: lp:harvest
Diff against target: 12 lines (+2/-0)
1 file modified
harvest/services/__init__.py (+2/-0)
To merge this branch: bzr merge lp:~dholbach/harvest/980654
Reviewer Review Type Date Requested Status
James Westby Approve
Review via email: mp+101888@code.launchpad.net

Description of the change

Only return active opportunity lists via the API.

To post a comment you must log in.
Revision history for this message
Daniel Holbach (dholbach) wrote :

Can anyone help with reviewing these? :)

Revision history for this message
James Westby (james-w) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'harvest/services/__init__.py'
2--- harvest/services/__init__.py 2012-04-11 16:47:39 +0000
3+++ harvest/services/__init__.py 2012-04-13 10:38:18 +0000
4@@ -75,6 +75,8 @@
5 if exclude is not None:
6 search_fields = [f for f in search_fields if f not in exclude]
7 search_values = dict([(str(key), value) for (key, value) in request.items()])
8+ if model._meta.object_name == 'OpportunityList':
9+ search_values['active'] = True
10
11 extra_arguments = {}
12 for key in search_values.keys():

Subscribers

People subscribed via source and target branches

to all changes: