From e29255650cf3d6403e2019970ece45a1eac80dbc Mon Sep 17 00:00:00 2001
From: Chad Little <chad@phacility.com>
Date: Sun, 16 Aug 2015 17:50:23 -0700
Subject: [PATCH] Fix Macro active query

Summary: When using the active query in Macro, all Macros are returned. Properly set the status query.

Test Plan: Review active macros, don't see archived macros.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D13912
---
 src/applications/macro/query/PhabricatorMacroSearchEngine.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/applications/macro/query/PhabricatorMacroSearchEngine.php b/src/applications/macro/query/PhabricatorMacroSearchEngine.php
index 2000b3275a..039f396d5f 100644
--- a/src/applications/macro/query/PhabricatorMacroSearchEngine.php
+++ b/src/applications/macro/query/PhabricatorMacroSearchEngine.php
@@ -111,7 +111,9 @@ final class PhabricatorMacroSearchEngine
 
     switch ($query_key) {
       case 'active':
-        return $query;
+        return $query->setParameter(
+          'status',
+          PhabricatorMacroQuery::STATUS_ACTIVE);
       case 'all':
         return $query->setParameter(
           'status',