Remove errant double HTML entity encoding in API source attribution

This commit is contained in:
Zach Copley 2010-05-06 21:36:13 +00:00
parent ee8c9d1422
commit 869bc32d0d
3 changed files with 13 additions and 6 deletions

View File

@ -349,14 +349,17 @@ class TwitapisearchatomAction extends ApiAction
$ns = $notice->getSource();
if ($ns) {
if (!empty($ns->name) && !empty($ns->url)) {
$source = '<a href="' . $ns->url . '" rel="nofollow">' . $ns->name . '</a>';
$source = '<a href="'
. htmlspecialchars($ns->url)
. '" rel="nofollow">'
. htmlspecialchars($ns->name)
. '</a>';
} else {
$source = $ns->code;
}
}
$this->element("twitter:source", null,
htmlentities($source));
$this->element("twitter:source", null, $source);
$this->elementStart('author');

View File

@ -1221,7 +1221,7 @@ class Notice extends Memcached_DataObject
$ns = $this->getSource();
if ($ns) {
if (!empty($ns->url)) {
$noticeInfoAttr['source_link'] = htmlentities($ns->url);
$noticeInfoAttr['source_link'] = $ns->url;
}
}

View File

@ -319,13 +319,17 @@ class ApiAction extends Action
$ns = $notice->getSource();
if ($ns) {
if (!empty($ns->name) && !empty($ns->url)) {
$source = '<a href="' . $ns->url . '" rel="nofollow">' . $ns->name . '</a>';
$source = '<a href="'
. htmlspecialchars($ns->url)
. '" rel="nofollow">'
. htmlspecialchars($ns->name)
. '</a>';
} else {
$source = $ns->code;
}
}
$twitter_status['source'] = htmlentities($source);
$twitter_status['source'] = $source;
$twitter_status['id'] = intval($notice->id);
$replier_profile = null;