Differential D20902

Improve recovery from panel action rendering exceptions, and mark "Changeset" queries as not suitable for panel generation
ClosedPublic
Actions

Authored by epriestley on Nov 9 2019, 1:12 AM.

Tags

None

Referenced Files

	F19041986: D20902.id.diff
	Wed, Nov 26, 5:28 PM

	F19034017: D20902.diff
	Tue, Nov 25, 12:25 PM

	F18985274: D20902.id49820.diff
	Mon, Nov 17, 12:57 PM

	F18876305: D20902.diff
	Wed, Nov 5, 11:17 PM

	F18828708: D20902.id.diff
	Oct 24 2025, 5:45 PM

	F18823510: D20902.diff
	Oct 23 2025, 12:22 PM

	F18805946: D20902.diff
	Oct 18 2025, 5:06 PM

	F18745499: D20902.id49821.diff
	Oct 3 2025, 7:26 AM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13443: Creating a dashboard query panel with the type "Differential Changesets" breaks the dashboard editor
Commits: rP72f82abe0723: Improve recovery from panel action rendering exceptions, and mark "Changeset"…

Summary

Fixes T13443. When a panel raises an exception during edit action generation, it currently escapes to top level. Instead, catch it more narrowly.

Additionally, mark "ChangesetSearchEngine" as not being a suitable search engine for use in query panels. There's no list view or search URI so it can't generate a sensible panel.

Test Plan

Added a changeset panel to a dashboard.
Before: entire dashboard fataled.
After: panel fataled narrowly, menu fatals narrowly, dashboard no longer permits creation of another Changeset query panel.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 9 2019, 1:12 AM

Harbormaster completed remote builds in B23648: Diff 49820.Nov 9 2019, 1:13 AM

This revision was not accepted when it landed; it landed in state Needs Review.Nov 9 2019, 1:13 AM

epriestley requested review of this revision.

Closed by commit rP72f82abe0723: Improve recovery from panel action rendering exceptions, and mark "Changeset"… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP72f82abe0723: Improve recovery from panel action rendering exceptions, and mark "Changeset"….

Revision Contents
Changeset List

Path

Size

src/

applications/

dashboard/

engine/

PhabricatorDashboardPanelRenderingEngine.php

13 lines

differential/

query/

DifferentialChangesetSearchEngine.php

4 lines

Diff 49821

src/applications/dashboard/engine/PhabricatorDashboardPanelRenderingEngine.php

Loading...

src/applications/differential/query/DifferentialChangesetSearchEngine.php

Loading...