Differential D19148

Fix a Harbormaster build log issue where too few header lines were returned when expanding logs in the middle
ClosedPublic
Actions

Authored by epriestley on Feb 28 2018, 1:45 PM.

Tags

None

Referenced Files

	F15492672: D19148.id45879.diff
	Sat, Apr 12, 5:02 PM

	F15491967: D19148.id45873.diff
	Sat, Apr 12, 9:26 AM

	F15490816: D19148.id.diff
	Fri, Apr 11, 8:47 PM

	F15487465: D19148.diff
	Thu, Apr 10, 2:17 PM

	F15473625: D19148.id45879.diff
	Sun, Apr 6, 12:23 AM

	F15471783: D19148.id45873.diff
	Sat, Apr 5, 8:06 AM

	F15442380: D19148.id.diff
	Wed, Mar 26, 11:02 PM

	F15336995: D19148.id45879.diff
	Mar 9 2025, 3:38 AM

View All Files

Subscribers

Restricted Owners Package

Details

Reviewers: None
Maniphest Tasks: T13088: Plans: Harbormaster UI usability and interconnectedness
Commits: rP985d499f5099: Fix a Harbormaster build log issue where too few header lines were returned…

Summary

Ref T13088. This variable bled through from an earlier loop and caused us to drop some of the lines in the middle.

Test Plan

Clicked "Show More", got an equal number of header and footer lines.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 28 2018, 1:45 PM

Owners added a subscriber: Restricted Owners Package.Feb 28 2018, 1:45 PM

Harbormaster completed remote builds in B19684: Diff 45873.Feb 28 2018, 1:46 PM

epriestley requested review of this revision.Feb 28 2018, 1:46 PM

epriestley added a child revision: D19149: When showing a small piece of a Harbormaster build log, load a small piece of data instead of the entire log.Feb 28 2018, 2:22 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 28 2018, 8:30 PM

Closed by commit rP985d499f5099: Fix a Harbormaster build log issue where too few header lines were returned… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

harbormaster/

controller/

HarbormasterBuildLogRenderController.php

4 lines

Diff 45879

View Options

src/applications/harbormaster/controller/HarbormasterBuildLogRenderController.php

Fix a Harbormaster build log issue where too few header lines were returned when expanding logs in the middleClosedPublicActions