Issue 8299: Don't manipulate DFRN content upon receiving
This commit is contained in:
parent
8a4a65f418
commit
01d6bfd2dd
|
@ -2369,32 +2369,11 @@ class DFRN
|
||||||
|
|
||||||
$item["body"] = XML::getFirstNodeValue($xpath, "dfrn:env/text()", $entry);
|
$item["body"] = XML::getFirstNodeValue($xpath, "dfrn:env/text()", $entry);
|
||||||
$item["body"] = str_replace([' ',"\t","\r","\n"], ['','','',''], $item["body"]);
|
$item["body"] = str_replace([' ',"\t","\r","\n"], ['','','',''], $item["body"]);
|
||||||
// make sure nobody is trying to sneak some html tags by us
|
|
||||||
$item["body"] = Strings::base64UrlDecode($item["body"]);
|
$item["body"] = Strings::base64UrlDecode($item["body"]);
|
||||||
|
|
||||||
$item["body"] = BBCode::limitBodySize($item["body"]);
|
$item["body"] = BBCode::limitBodySize($item["body"]);
|
||||||
|
|
||||||
/// @todo Do we really need this check for HTML elements? (It was copied from the old function)
|
|
||||||
if ((strpos($item['body'], '<') !== false) && (strpos($item['body'], '>') !== false)) {
|
|
||||||
$base_url = DI::baseUrl()->get();
|
|
||||||
$item['body'] = HTML::relToAbs($item['body'], $base_url);
|
|
||||||
|
|
||||||
$item['body'] = HTML::toBBCodeVideo($item['body']);
|
|
||||||
|
|
||||||
$item['body'] = OEmbed::HTML2BBCode($item['body']);
|
|
||||||
|
|
||||||
$config = HTMLPurifier_Config::createDefault();
|
|
||||||
$config->set('Cache.DefinitionImpl', null);
|
|
||||||
|
|
||||||
// we shouldn't need a whitelist, because the bbcode converter
|
|
||||||
// will strip out any unsupported tags.
|
|
||||||
|
|
||||||
$purifier = new HTMLPurifier($config);
|
|
||||||
$item['body'] = $purifier->purify($item['body']);
|
|
||||||
|
|
||||||
$item['body'] = @HTML::toBBCode($item['body']);
|
|
||||||
}
|
|
||||||
|
|
||||||
/// @todo We should check for a repeated post and if we know the repeated author.
|
/// @todo We should check for a repeated post and if we know the repeated author.
|
||||||
|
|
||||||
// We don't need the content element since "dfrn:env" is always present
|
// We don't need the content element since "dfrn:env" is always present
|
||||||
|
|
Loading…
Reference in New Issue
Block a user