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

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
=== modified file 'harvest/services/__init__.py'
--- harvest/services/__init__.py 2012-04-11 16:47:39 +0000
+++ harvest/services/__init__.py 2012-04-13 10:38:18 +0000
@@ -75,6 +75,8 @@
75 if exclude is not None:75 if exclude is not None:
76 search_fields = [f for f in search_fields if f not in exclude]76 search_fields = [f for f in search_fields if f not in exclude]
77 search_values = dict([(str(key), value) for (key, value) in request.items()])77 search_values = dict([(str(key), value) for (key, value) in request.items()])
78 if model._meta.object_name == 'OpportunityList':
79 search_values['active'] = True
7880
79 extra_arguments = {}81 extra_arguments = {}
80 for key in search_values.keys():82 for key in search_values.keys():

Subscribers

People subscribed via source and target branches

to all changes: