From c0f8df3d3f033a2ee7e0e3ad1c2ec6972576f023 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jun 08 2015 11:42:01 +0000 Subject: Adjust search_pull_requests to support searching for new and old status type The old status type was a boolean: True = Open, False = Closed/Merged. So we now support searching for the provided status, or with a boolean --- diff --git a/pagure/lib/__init__.py b/pagure/lib/__init__.py index 1272235..72206f8 100644 --- a/pagure/lib/__init__.py +++ b/pagure/lib/__init__.py @@ -1414,9 +1414,19 @@ def search_pull_requests( ) if status is not None: - query = query.filter( - model.PullRequest.status == status - ) + if isinstance(status, bool): + if status: + query = query.filter( + model.PullRequest.status == 'Open' + ) + else: + query = query.filter( + model.PullRequest.status != 'Open' + ) + else: + query = query.filter( + model.PullRequest.status == status + ) if assignee is not None: if str(assignee).lower() not in ['false', '0', 'true', '1']: