Paths

Table of Contentst

Differential D10941

Fix an undefined variable
ClosedPublic
Actions

Authored by joshuaspence on Dec 8 2014, 12:10 PM.

Tags

None

Referenced Files

	F15473604: D10941.id26277.diff
	Sun, Apr 6, 12:19 AM

	F15458375: D10941.id.diff
	Sun, Mar 30, 11:22 PM

	F15379819: D10941.diff
	Mar 13 2025, 11:33 PM

	F15343132: D10941.diff
	Mar 9 2025, 10:13 PM

	Unknown Object (File)
	Feb 21 2025, 9:33 AM

	Unknown Object (File)
	Feb 1 2025, 3:41 AM

	Unknown Object (File)
	Jan 24 2025, 4:14 PM

	Unknown Object (File)
	Jan 24 2025, 2:35 PM

Subscribers

Details

Reviewers

hach-que
epriestley
btrahan

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP9e54e6e88661: Fix an undefined variable

Summary

The $timeline variable is undefined. I was seeing the following error in the logs:

EXCEPTION: (RuntimeException) Undefined variable: timeline at [<phutil>/src/error/PhutilErrorHandler.php:210]
   #0 PhutilErrorHandler::handleError(integer, string, string, integer, array) called at [<phabricator>/src/applications/harbormaster/controller/HarbormasterStepEditController.php:205]
   #1 HarbormasterStepEditController::processRequest() called at [<phabricator>/src/aphront/AphrontController.php:33]
   #2 AphrontController::handleRequest(AphrontRequest) called at [<phabricator>/webroot/index.php:103]

Test Plan

Created a build step without a fatal error.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 26277.Dec 8 2014, 12:10 PM

joshuaspence retitled this revision from to Fix an undefined variable.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added reviewers: epriestley, btrahan, hach-que.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptDec 8 2014, 12:10 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Dec 8 2014, 12:10 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 8 2014, 12:10 PM

Closed by commit rP9e54e6e88661: Fix an undefined variable (authored by joshuaspence, committed by epriestley). · Explain WhyDec 8 2014, 12:11 PM

This revision was automatically updated to reflect the committed changes.

Oops, just pulled this myself.

ohwellok

I have so many unearned internet points

Thanks for fixing this and sorry for the slop!

src/applications/harbormaster/controller/HarbormasterStepEditController.php
192	this xaction_view can probably go if someone needs internet points. :D

Revision Contents
Changeset List

Path

Size

src/

applications/

harbormaster/

controller/

HarbormasterStepEditController.php

1 line

Diff 26278

src/applications/harbormaster/controller/HarbormasterStepEditController.php

Loading...