Page MenuHomePhabricator
Create Task
Maniphest T4105

Undefined index in DifferentialHunkParser
Closed, ResolvedPublic
Actions

Description

We started seeing these recently.

ERROR 8: Undefined index: 50 at [/var/devtools/phabricator/src/applications/differential/parser/DifferentialHunkParser.php:120]

#0 DifferentialHunkParser::setNewLineTypeMap(Array of size 49 starting with: { 1 => null }) called at [/var/devtools/phabricator/src/applications/differential/parser/DifferentialChangesetParser.php:543]

ERROR 8: Undefined index: 51 at [/var/devtools/phabricator/src/applications/differential/parser/DifferentialHunkParser.php:99]

#0 DifferentialHunkParser::setOldLineTypeMap(Array of size 50 starting with: { 1 => null }) called at [/var/devtools/phabricator/src/applications/differential/parser/DifferentialChangesetParser.php:542]

Revisions and Commits

Restricted Differential RevisionRestricted Diffusion Commit

Related Objects

Event Timeline

lifeihuang created this task.Nov 15 2013, 1:24 AM
lifeihuang assigned this task to epriestley.
lifeihuang raised the priority of this task from to Needs Triage.
lifeihuang updated the task description. (Show Details)
lifeihuang added a project: Facebook.
lifeihuang changed the visibility from "Public (No Login Required)" to "Custom Policy".
lifeihuang added a subscriber: lifeihuang.
epriestley added a subscriber: Unknown Object (MLST).Nov 15 2013, 2:28 PM

Those lines don't correspond to anything index-ish at HEAD. What commit are those lines against?

epriestley changed the visibility from "Custom Policy" to "Public (No Login Required)".Nov 15 2013, 2:29 PM

(I set the policy on this back to "Public" since I think you locked yourself out of it.)

epriestley closed this task as Invalid.Nov 26 2013, 2:48 AM

Closing for lack of feedback.

epriestley reopened this task as Open.Jul 9 2014, 11:53 PM
epriestley added a subscriber: hach-que.

@hach-que hit this near HEAD, P1198 has a trace.

chad edited projects, added Differential; removed Facebook.Nov 2 2014, 11:03 PM
epriestley moved this task from Backlog to v2 (UI/Mobile) on the Differential board.Mar 4 2015, 5:11 PM
epriestley added a revision: Restricted Differential Revision.Mar 5 2015, 2:06 AM
epriestley merged a task: T5163: User hitting undefined index in changeset rendering.
epriestley added subscribers: staticshock, turadg, cburroughs and 2 others.
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Mar 5 2015, 10:00 PM
epriestley added a commit: Restricted Diffusion Commit.
Phabricator · Built by Phacility