Paths

Table of Contentst

Differential D21269

Fix an issue where builds with no initiator failed to render in build plans
ClosedPublic
Actions

Authored by epriestley on May 19 2020, 4:44 PM.

Tags

None

Referenced Files

	F15463674: D21269.diff
	Wed, Apr 2, 2:09 AM

	F15340470: D21269.diff
	Sun, Mar 9, 3:17 PM

	F15316177: D21269.diff
	Mar 6 2025, 6:25 PM

	Unknown Object (File)
	Feb 25 2025, 7:49 PM

	Unknown Object (File)
	Feb 23 2025, 11:29 PM

	Unknown Object (File)
	Feb 22 2025, 6:32 AM

	Unknown Object (File)
	Feb 22 2025, 4:13 AM

	Unknown Object (File)
	Feb 1 2025, 12:40 PM

Subscribers

Restricted Owners Package

Details

Reviewers: None
Commits: rP4fd7c0f7b3ca: (stable) Fix an issue where builds with no initiator failed to render in build…
rP86d6abe9db26: Fix an issue where builds with no initiator failed to render in build plans

Summary

See PHI1743. If a build has no initiator PHID, the rendering pathway incorrectly tries to access a handle for it anyway.

Test Plan

Set a build to have no initiator PHID.
Viewed the build plan for the build.
Before: fatal when trying to access the null handle.
After: clean build plan rendering.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 19 2020, 4:44 PM

Owners added a subscriber: Restricted Owners Package.May 19 2020, 4:44 PM

Harbormaster completed remote builds in B24466: Diff 50647.May 19 2020, 4:46 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 19 2020, 4:46 PM

epriestley requested review of this revision.

Closed by commit rP86d6abe9db26: Fix an issue where builds with no initiator failed to render in build plans (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP86d6abe9db26: Fix an issue where builds with no initiator failed to render in build plans.

epriestley added a commit: rP4fd7c0f7b3ca: (stable) Fix an issue where builds with no initiator failed to render in build….

Revision Contents
Changeset List

Path

Size

src/

applications/

harbormaster/

view/

HarbormasterBuildView.php

6 lines

Diff 50648

src/applications/harbormaster/view/HarbormasterBuildView.php

Loading...