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

	Unknown Object (File)
	Tue, Jan 21, 11:20 AM

	Unknown Object (File)
	Sun, Jan 19, 2:48 AM

	Unknown Object (File)
	Wed, Jan 8, 2:32 PM

	Unknown Object (File)
	Fri, Dec 27, 6:38 PM

	Unknown Object (File)
	Dec 20 2024, 9:24 PM

	Unknown Object (File)
	Dec 16 2024, 5:54 AM

	Unknown Object (File)
	Nov 26 2024, 10:55 PM

	Unknown Object (File)
	Nov 25 2024, 12:48 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...