revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-30 02:32:42 +01:00
Code Issues Releases Wiki Activity

Avoid HTML escaping in plain text e-mails

Browse source 
Summary: Seen in e-mail from T2016.

Test Plan: None.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T2432

Differential Revision: https://secure.phabricator.com/D5016
This commit is contained in:
vrana 2013-02-19 13:51:55 -08:00
parent 0d38bca11f
commit 8eb404aea7
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/applications/maniphest/view/ManiphestTransactionDetailView.php
View file

@ -624,8 +624,12 @@ final class ManiphestTransactionDetailView extends ManiphestView {
$links[] = $this->handles[$phid]->renderLink(); $links[] = $this->handles[$phid]->renderLink();
} }
} }
if ($this->forEmail) {
return implode(', ', $links);
} else {
return phutil_implode_html(', ', $links); return phutil_implode_html(', ', $links);
} }
}
private function renderString($string) { private function renderString($string) {
if ($this->forEmail) { if ($this->forEmail) {