Fix Macro active query
ClosedPublic
Actions

Authored by chad on Aug 15 2015, 9:09 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, May 3, 8:54 AM

	Unknown Object (File)
	Fri, May 3, 2:07 AM

	Unknown Object (File)
	Thu, Apr 25, 2:42 AM

	Unknown Object (File)
	Mon, Apr 22, 12:22 AM

	Unknown Object (File)
	Fri, Apr 12, 8:44 AM

	Unknown Object (File)
	Fri, Apr 12, 8:44 AM

	Unknown Object (File)
	Fri, Apr 12, 8:27 AM

	Unknown Object (File)
	Fri, Apr 12, 7:18 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rPe29255650cf3: 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.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 33578.Aug 15 2015, 9:09 PM

chad retitled this revision from to Fix Macro active query.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptAug 15 2015, 9:09 PM

epriestley accepted this revision.Aug 17 2015, 12:42 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Aug 17 2015, 12:42 AM

Closed by commit rPe29255650cf3: Fix Macro active query (authored by chad, committed by chad). · Explain WhyAug 17 2015, 12:50 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

macro/

query/

PhabricatorMacroSearchEngine.php

4 lines

Diff 33589

View Options

src/applications/macro/query/PhabricatorMacroSearchEngine.php

Fix Macro active queryClosedPublicActions