Page MenuHomePhabricator

phutil_utf8_shorten => PhutilUTF8StringTruncator
ClosedPublic

Authored by btrahan on Aug 29 2014, 9:34 PM.
Tags
None
Referenced Files
F19821761: D10392.id25011.diff
Sat, Mar 7, 12:27 AM
F19791420: D10392.id25011.diff
Wed, Feb 25, 11:09 PM
F19670885: D10392.diff
Sat, Feb 7, 11:55 PM
F19552265: D10392.id25011.diff
Jan 28 2026, 11:48 PM
F19524875: D10392.id.diff
Jan 18 2026, 4:40 AM
F19510622: D10392.id.diff
Jan 11 2026, 12:44 PM
F19510618: D10392.id.diff
Jan 11 2026, 12:41 PM
F19510609: D10392.id.diff
Jan 11 2026, 12:36 PM
Subscribers

Details

Summary

Ref T3307. Only one I thought was tricky was Excel; I went with bytes there like it was email.

Test Plan

played around on a few endpoints but mostly thought carefully

Diff Detail

Repository
rP Phabricator
Branch
T3307
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2396
Build 2400: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to phutil_utf8_shorten => PhutilUTF8StringTruncator.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

Cool, these look good to me. I think bytes is reasonable for Excel.

src/applications/herald/storage/transcript/HeraldObjectTranscript.php
55

Missing "PhutilUTF8StringTruncator has"?

This revision is now accepted and ready to land.Aug 29 2014, 9:58 PM
btrahan updated this revision to Diff 25011.

Closed by commit rPb93bc7e47947 (authored by @btrahan).