jakob/friendica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Tests added

Browse source
This commit is contained in:
Michael 2023-03-21 04:20:03 +00:00
parent 0f9099dec6
commit 7e6ba0b4d2
2 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/Content/Text/BBCode.php
View file

@ -1568,7 +1568,7 @@ class BBCode
function ($matches) use ($simple_html, $uriid) { function ($matches) use ($simple_html, $uriid) {
$matches[1] = self::proxyUrl($matches[1], $simple_html, $uriid); $matches[1] = self::proxyUrl($matches[1], $simple_html, $uriid);
$alt = htmlspecialchars($matches[2], ENT_COMPAT); $alt = htmlspecialchars($matches[2], ENT_COMPAT);
// Fix for Markdown problems wirh Diaspora, see issue #12701 // Fix for Markdown problems with Diaspora, see issue #12701
if (($simple_html != self::DIASPORA) || strpos($matches[2], '"') === false) { if (($simple_html != self::DIASPORA) || strpos($matches[2], '"') === false) {
return '<img src="' . $matches[1] . '" alt="' . $alt . '" title="' . $alt . '">'; return '<img src="' . $matches[1] . '" alt="' . $alt . '" title="' . $alt . '">';
} else { } else {

10
tests/src/Content/Text/BBCodeTest.php
View file

@ -317,6 +317,14 @@ Karl Marx - Die ursprüngliche Akkumulation
'expected' => '&amp;`&`', 'expected' => '&amp;`&`',
'text' => '&[code]&[/code]', 'text' => '&[code]&[/code]',
], ],
'bug-12701-quotes' => [
'expected' => '[![abc"fgh](https://domain.tld/photo/86912721086415cdc8e0a03226831581-1.png)](https://domain.tld/photos/user/image/86912721086415cdc8e0a03226831581)',
'text' => '[url=https://domain.tld/photos/user/image/86912721086415cdc8e0a03226831581][img=https://domain.tld/photo/86912721086415cdc8e0a03226831581-1.png]abc"fgh[/img][/url]'
],
'bug-12701-no-quotes' => [
'expected' => '[![abcfgh](https://domain.tld/photo/86912721086415cdc8e0a03226831581-1.png "abcfgh")](https://domain.tld/photos/user/image/86912721086415cdc8e0a03226831581)',
'text' => '[url=https://domain.tld/photos/user/image/86912721086415cdc8e0a03226831581][img=https://domain.tld/photo/86912721086415cdc8e0a03226831581-1.png]abcfgh[/img][/url]'
],
]; ];
} }
@ -331,7 +339,7 @@ Karl Marx - Die ursprüngliche Akkumulation
* *
* @throws InternalServerErrorException * @throws InternalServerErrorException
*/ */
public function testToMarkdown(string $expected, string $text, $for_diaspora = false) public function testToMarkdown(string $expected, string $text, $for_diaspora = true)
{ {
$actual = BBCode::toMarkdown($text, $for_diaspora); $actual = BBCode::toMarkdown($text, $for_diaspora);