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

	Unknown Object (File)
	Sat, Dec 21, 11:44 AM

	Unknown Object (File)
	Fri, Dec 20, 6:33 PM

	Unknown Object (File)
	Thu, Dec 19, 1:21 PM

	Unknown Object (File)
	Thu, Dec 19, 1:21 PM

	Unknown Object (File)
	Thu, Dec 19, 1:21 PM

	Unknown Object (File)
	Thu, Dec 5, 8:03 AM

	Unknown Object (File)
	Nov 23 2024, 1:36 PM

	Unknown Object (File)
	Nov 23 2024, 1:36 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

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3193
Build 3199: [Placeholder Plan] Wait for 30 Seconds

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–193	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 26277

src/applications/harbormaster/controller/HarbormasterStepEditController.php

Loading...