Page MenuHomePhabricator
Differential D17545

Restrict Audit buckets to just ApplicationSearch views
ClosedPublic
Actions

Authored by chad on Mar 23 2017, 7:19 PM.
Tags
None
Referenced Files
F18825672: D17545.id.diff
Thu, Oct 23, 11:27 PM
F18814592: D17545.diff
Mon, Oct 20, 11:33 PM
F18812182: D17545.id42205.diff
Mon, Oct 20, 7:32 AM
F18745733: D17545.id42202.diff
Fri, Oct 3, 9:02 AM
F18592198: D17545.diff
Sep 12 2025, 11:35 AM
F18484781: D17545.id.diff
Sep 3 2025, 5:06 PM
F18472514: D17545.diff
Sep 2 2025, 8:33 PM
F18094501: D17545.id.diff
Aug 7 2025, 10:10 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9363: improve handling of panels that do not always return results
Commits
rP2707681b48a2: Restrict Audit buckets to just ApplicationSearch views
Summary

Fixes T9363. This drops empty buckets from dashboard panel context. Still see full results in Audit.

Test Plan

Create an "Active Audits" panel, add to Dashboard. See no commits found. Check Audit, see all buckets.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Mar 23 2017, 7:19 PM
Herald added a subscriber: Korvin. · View Herald TranscriptMar 23 2017, 7:19 PM
Harbormaster completed remote builds in B16116: Diff 42202.Mar 23 2017, 7:20 PM
epriestley accepted this revision.Mar 23 2017, 7:30 PM
epriestley added inline comments.
src/applications/audit/query/PhabricatorCommitSearchEngine.php
195–201

"commit" -> "commits"?

This revision is now accepted and ready to land.Mar 23 2017, 7:30 PM
Closed by commit rP2707681b48a2: Restrict Audit buckets to just ApplicationSearch views (authored by chad, committed by chad). · Explain WhyMar 23 2017, 7:46 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
query/
19 lines

Diff 42205

View Options

src/applications/audit/query/PhabricatorCommitSearchEngine.php

Loading...
Phabricator · Built by Phacility