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

	Unknown Object (File)
	Mon, Dec 16, 5:49 PM

	Unknown Object (File)
	Fri, Dec 6, 5:16 PM

	Unknown Object (File)
	Sat, Nov 30, 1:53 AM

	Unknown Object (File)
	Nov 20 2024, 11:41 PM

	Unknown Object (File)
	Nov 9 2024, 3:58 PM

	Unknown Object (File)
	Oct 18 2024, 7:02 PM

	Unknown Object (File)
	Oct 15 2024, 1:58 PM

	Unknown Object (File)
	Oct 14 2024, 5:02 PM

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...