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

Reorganize escaping in DifferentialRevisionUpdateHistoryView

Browse source 
Summary:
Escaped $id is compared with non-escaped $max_id.
Escaped $id is escaped again in phutil_render_tag().

Note: $id is numeric :-).

Test Plan: Display diff.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1580
This commit is contained in:
vrana 2012-02-06 10:55:55 -08:00
parent 4ee714d404
commit 1ab2a88605
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/applications/differential/view/revisionupdatehistory/DifferentialRevisionUpdateHistoryView.php
View file

@ -89,8 +89,8 @@ final class DifferentialRevisionUpdateHistoryView extends AphrontView {
foreach ($data as $row) {
$diff = $row['obj'];
$name = phutil_escape_html($row['name']);
$id = phutil_escape_html($row['id']);
$name = $row['name'];
$id = $row['id'];
$old_class = null;
$new_class = null;
@ -172,8 +172,8 @@ final class DifferentialRevisionUpdateHistoryView extends AphrontView {
$rows[] =
'<tr'.$class.'>'.
'<td class="revhistory-name">'.$name.'</td>'.
'<td class="revhistory-id">'.$id.'</td>'.
'<td class="revhistory-name">'.phutil_escape_html($name).'</td>'.
'<td class="revhistory-id">'.phutil_escape_html($id).'</td>'.
'<td class="revhistory-base">'.phutil_escape_html($base).'</td>'.
'<td class="revhistory-desc">'.phutil_escape_html($desc).'</td>'.
'<td class="revhistory-age">'.$age.'</td>'.