Replace some hsprintf() with phutil_tag()
ClosedPublic
Actions

Authored by vrana on Nov 8 2013, 5:56 PM.

Tags

None

Referenced Files

	F18463664: D7533.diff
	Tue, Sep 2, 1:48 AM

	F18045406: D7533.id17022.diff
	Aug 3 2025, 8:57 AM

	F17874223: D7533.id.diff
	Jul 28 2025, 5:38 PM

	F17848200: D7533.diff
	Jul 27 2025, 7:05 AM

	Unknown Object (File)
	Jun 19 2025, 11:09 AM

	Unknown Object (File)
	Jun 17 2025, 5:02 PM

	Unknown Object (File)
	Jun 16 2025, 7:03 AM

	Unknown Object (File)
	May 20 2025, 1:15 AM

Subscribers

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rPaca621e21f9c: Replace some hsprintf() with phutil_tag()

Summary

I just want to make sure that this is the style we want.
It seems less readable to me in some cases.

Test Plan

Looked at DarkConsole with errors.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

vrana added inline comments.Nov 8 2013, 5:59 PM

src/aphront/console/plugin/DarkConsoleXHProfPlugin.php
47–60	This is untranslatable with current `pht()`.

I think this code (which isn't using standard UI classes) is some of the least-readable with phutil_tag(), which is partly why it's hsprintf() in the first place. The next time DarkConsole gets some design work done we'll probably simplify some of this (e.g., remove inline style attributes), which should make it a bit more readable.