revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-27 09:12:41 +01:00
Code Issues Releases Wiki Activity

Fix double escaping in Pholio subscribers

Browse source 
Summary:
Fallout from D4822. We now escape text by default, and imploding links causes double-escaping. See here for details:

http://www.phabricator.com/docs/phabricator/article/Rendering_HTML.html

Test Plan: {F32100}

Reviewers: ljalonen, chad, btrahan, vrana

Reviewed By: ljalonen

CC: aran

Differential Revision: https://secure.phabricator.com/D4878
This commit is contained in:
epriestley 2013-02-09 06:51:47 -08:00
parent 3706ec4879
commit 3dd5d693c8
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/pholio/controller/PholioMockViewController.php
View file

@ -145,7 +145,7 @@ final class PholioMockViewController extends PholioController {
foreach ($subscribers as $subscriber) { foreach ($subscribers as $subscriber) {
$sub_view[] = $this->getHandle($subscriber)->renderLink(); $sub_view[] = $this->getHandle($subscriber)->renderLink();
} }
$sub_view = implode(', ', $sub_view); $sub_view = array_interleave(', ', $sub_view);
} else { } else {
$sub_view = phutil_tag('em', array(), pht('None')); $sub_view = phutil_tag('em', array(), pht('None'));
} }