HomePhorge

Fix error in `PhabricatorSetupIssueView`

Description

Fix error in PhabricatorSetupIssueView

Summary:

  • Move prettyPrintJSON() and make it static.
  • Use it from PhabricatorSetupIssueView
  • Update other config/ places that use it to call it from the new class.

This fixes a bug in PhabricatorSetupIssueView which showed up if the value
was an array and couldn't be rendered by phutil_escape_html().

Test Plan:

  • Rendered some config options.
  • Went to /config/issue/config.unknown.phame.skins/ without error.

Reviewers: epriestley, btrahan, chad

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T2255

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

Details

Provenance
Ricky ElrodAuthored on Jan 11 2013, 3:28 PM
epriestleyCommitted on Jan 11 2013, 3:28 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP08687c0b172d: Search options.
Branches
Unknown
Tags
Unknown

Event Timeline