mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-22 23:02:42 +01:00
Use phutil_utf8_hard_wrap() in Phabricator
Summary: See D1433. Test Plan: Created a new diff with a line >80chars, observed it wrapping correctly. Reviewers: btrahan, jungejason Reviewed By: btrahan CC: aran Differential Revision: https://secure.phabricator.com/D1438
This commit is contained in:
parent
e2c75d5dc2
commit
ef768f9694
1 changed files with 5 additions and 51 deletions
|
@ -645,6 +645,9 @@ class DifferentialChangesetParser {
|
|||
}
|
||||
|
||||
protected function applyIntraline(&$render, $intra, $corpus) {
|
||||
|
||||
$line_break = "<span class=\"over-the-line\">\xE2\xAC\x85</span><br />";
|
||||
|
||||
foreach ($render as $key => $text) {
|
||||
if (isset($intra[$key])) {
|
||||
$render[$key] = ArcanistDiffUtils::applyIntralineDiff(
|
||||
|
@ -652,61 +655,12 @@ class DifferentialChangesetParser {
|
|||
$intra[$key]);
|
||||
}
|
||||
if (isset($corpus[$key]) && strlen($corpus[$key]) > $this->lineWidth) {
|
||||
$render[$key] = $this->lineWrap($render[$key]);
|
||||
$lines = phutil_utf8_hard_wrap_html($render[$key], $this->lineWidth);
|
||||
$render[$key] = implode($line_break, $lines);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Hard-wrap a piece of UTF-8 text with embedded HTML tags and entities.
|
||||
*
|
||||
* @param string An HTML string with tags and entities.
|
||||
* @return string Hard-wrapped string.
|
||||
*/
|
||||
protected function lineWrap($line) {
|
||||
$c = 0;
|
||||
$break_here = array();
|
||||
|
||||
// Convert the UTF-8 string into a list of UTF-8 characters.
|
||||
$vector = phutil_utf8v($line);
|
||||
$len = count($vector);
|
||||
$byte_pos = 0;
|
||||
for ($ii = 0; $ii < $len; ++$ii) {
|
||||
// An ampersand indicates an HTML entity; consume the whole thing (until
|
||||
// ";") but treat it all as one character.
|
||||
if ($vector[$ii] == '&') {
|
||||
do {
|
||||
++$ii;
|
||||
} while ($vector[$ii] != ';');
|
||||
++$c;
|
||||
// An "<" indicates an HTML tag, consume the whole thing but don't treat
|
||||
// it as a character.
|
||||
} else if ($vector[$ii] == '<') {
|
||||
do {
|
||||
++$ii;
|
||||
} while ($vector[$ii] != '>');
|
||||
} else {
|
||||
++$c;
|
||||
}
|
||||
|
||||
// Keep track of where we need to break the string later.
|
||||
if ($c == $this->lineWidth) {
|
||||
$break_here[$ii] = true;
|
||||
$c = 0;
|
||||
}
|
||||
}
|
||||
|
||||
$result = array();
|
||||
foreach ($vector as $ii => $char) {
|
||||
$result[] = $char;
|
||||
if (isset($break_here[$ii])) {
|
||||
$result[] = "<span class=\"over-the-line\">\xE2\xAC\x85</span><br />";
|
||||
}
|
||||
}
|
||||
|
||||
return implode('', $result);
|
||||
}
|
||||
|
||||
protected function getHighlightFuture($corpus) {
|
||||
return $this->highlightEngine->getHighlightFuture(
|
||||
$this->highlightEngine->getLanguageFromFilename($this->filename),
|
||||
|
|
Loading…
Reference in a new issue