Homestyx hydra
Diffusion hydra aa20faeaa5d1

Fix an invalid index access for synthetic lint inline comments from Harbormaster

Description

Fix an invalid index access for synthetic lint inline comments from Harbormaster

Summary:
Ref T13524. If a Harbormaster lint message has no line number (which is permitted), we try to access an invalid index here. This is an exception after D21044.

Treat comments with no line number as unchanged. These comments do not have "ghost" behavior and do not port across diffs.

Test Plan:

  • Used "harbormaster.sendmessage" to submit lint with no line number on a changeset.
  • Viewed changeset.
    • Before patch: "Undefined index: <null>" error.
    • After patch: Clean changeset with lint message.

{F7400072}

Maniphest Tasks: T13524

Differential Revision: https://secure.phabricator.com/D21178

Details

Provenance
epriestleyAuthored on Apr 27 2020, 5:15 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:661700536579: Make Conduit "www-form-urlencoded" parsing of "true" and "false" case…
Branches
Unknown
Tags
Unknown

Event Timeline