harvest-dev team mailing list archive

[Daniel Holbach <daniel.holbach@xxxxxxxxxx>]

Daniel Holbach has proposed merging lp:~dholbach/harvest/980654 into lp:harvest.

Requested reviews:
  harvest-dev (harvest-dev)
Related bugs:
  Bug #980654 in harvest: "API should only return active opportunitylists"
  https://bugs.launchpad.net/harvest/+bug/980654

For more details, see:
https://code.launchpad.net/~dholbach/harvest/980654/+merge/101888

Only return active opportunity lists via the API.
-- 
https://code.launchpad.net/~dholbach/harvest/980654/+merge/101888
Your team harvest-dev is requested to review the proposed merge of lp:~dholbach/harvest/980654 into lp:harvest.

=== 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:34:19 +0000
@@ -75,6 +75,8 @@
     if exclude is not None:
         search_fields = [f for f in search_fields if f not in exclude]
     search_values = dict([(str(key), value) for (key, value) in request.items()])
+    if model._meta.object_name == 'OpportunityList':
+        search_values['active'] = True
 
     extra_arguments = {}
     for key in search_values.keys():