Page MenuHomePhabricator

Replace some hsprintf() with phutil_tag() and phutil_tag_div() Depends on D7545.
ClosedPublic

Authored by vrana on Nov 9 2013, 6:41 AM.
Tags
None
Referenced Files
F19152675: D7547.diff
Thu, Dec 11, 4:21 AM
F18980885: D7547.diff
Mon, Nov 17, 1:27 AM
F18763838: D7547.id17027.diff
Oct 7 2025, 2:34 AM
F18743930: D7547.id17032.diff
Oct 3 2025, 1:20 AM
F18677135: D7547.diff
Sep 25 2025, 8:45 PM
F18620501: D7547.diff
Sep 15 2025, 5:26 AM
F18586759: D7547.id17032.diff
Sep 11 2025, 7:35 PM
F18580634: D7547.id.diff
Sep 11 2025, 4:10 AM

Details

Test Plan

This is one of the rare moments where unit tests for views would be useful.

Diff Detail

Branch
hsprintf
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

These all look correct to me. Thanks for doing this!

src/infrastructure/diff/PhabricatorInlineCommentController.php
74–75

The <p> can be omitted from single paragraphs in dialog and error bodies.