From 246894f233c41ff8c7e75668d06a32f853364e96 Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Tue, 6 Aug 2019 21:22:19 -0400 Subject: [PATCH 1/3] Ensure trimmed HTML nodes are text nodes in HTML::toBBCode --- src/Content/Text/HTML.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Content/Text/HTML.php b/src/Content/Text/HTML.php index 688895da92..c8bed0b9d6 100644 --- a/src/Content/Text/HTML.php +++ b/src/Content/Text/HTML.php @@ -102,7 +102,7 @@ class HTML /** @var \DOMNode $child */ foreach ($node->childNodes as $key => $child) { /* Remove empty text nodes at the start or at the end of the children list */ - if ($key > 0 && $key < $node->childNodes->length - 1 || trim($child->nodeValue)) { + if ($key > 0 && $key < $node->childNodes->length - 1 || $child->nodeName != '#text' || trim($child->nodeValue)) { $newNode = $child->cloneNode(true); $node->parentNode->insertBefore($newNode, $node); } From d32de58e340d2a096feefe510315a93086097065 Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Tue, 6 Aug 2019 21:22:46 -0400 Subject: [PATCH 2/3] Add test case to avoid trimming empty image tags --- .../datasets/content/text/html/bug-7488.html | 14 ++++++++++++++ tests/datasets/content/text/html/bug-7488.txt | 19 +++++++++++++++++++ 2 files changed, 33 insertions(+) create mode 100644 tests/datasets/content/text/html/bug-7488.html create mode 100644 tests/datasets/content/text/html/bug-7488.txt diff --git a/tests/datasets/content/text/html/bug-7488.html b/tests/datasets/content/text/html/bug-7488.html new file mode 100644 index 0000000000..a994260363 --- /dev/null +++ b/tests/datasets/content/text/html/bug-7488.html @@ -0,0 +1,14 @@ +

Je n'arrive pas à être sûre et certaine que c'est bien une petite guêpe.
+ Tous les matins, avant 9 heures, cette bestiolette vient prendre sa part de petit-déjeuner. Elle a repéré que c'était le bon moment, je ne la vois jamais plus tard. Comme la porte est ouverte et seulement masquée d'un rideau, elle entre et va faire sa danse zigzaguante sur le comptoir, où elle sait qu'il y aura des choses sucrées dont elle est friande.
+ Du coup, j'ai pris l'habitude de laisser quelques gouttes de miel ou de confiture sur l'envers des couvercles.
+ Un matin, j'ai oublié... Elle est venue tourner autour de moi, comme pour dire "Alors, quoi ? Le p'tit-dèje n'est pas encore servi ?"
+ Elle n'est pas agressive du tout, elle est même venue sur ma main se lisser les antennes.
+ J'ai pensé que c'était un syrphe, mais non, après vérification, ce ne sont pas du tout les mêmes yeux, ni le même abdomen. Il semble bien que ce soit une sorte de petite guêpe, mais elles ne ressemble pas aux autres qui nichent derrière la lampe de jardin, et qui sont plus grandes.

+ +

Quelqu'un.e peut confirmer, d'après les photos ?... :)

+ +

+ +

+ +

#faune-sauvage #insectes #guêpes #Vespoidea

\ No newline at end of file diff --git a/tests/datasets/content/text/html/bug-7488.txt b/tests/datasets/content/text/html/bug-7488.txt new file mode 100644 index 0000000000..1e1b15cb3d --- /dev/null +++ b/tests/datasets/content/text/html/bug-7488.txt @@ -0,0 +1,19 @@ +Je n'arrive pas à être sûre et certaine que c'est bien une petite guêpe. + +Tous les matins, avant 9 heures, cette bestiolette vient prendre sa part de petit-déjeuner. Elle a repéré que c'était le bon moment, je ne la vois jamais plus tard. Comme la porte est ouverte et seulement masquée d'un rideau, elle entre et va faire sa danse zigzaguante sur le comptoir, où elle sait qu'il y aura des choses sucrées dont elle est friande. + +Du coup, j'ai pris l'habitude de laisser quelques gouttes de miel ou de confiture sur l'envers des couvercles. + +Un matin, j'ai oublié... Elle est venue tourner autour de moi, comme pour dire "Alors, quoi ? Le p'tit-dèje n'est pas encore servi ?" + +Elle n'est pas agressive du tout, elle est même venue sur ma main se lisser les antennes. + +J'ai pensé que c'était un syrphe, mais non, après vérification, ce ne sont pas du tout les mêmes yeux, ni le même abdomen. Il semble bien que ce soit une sorte de petite guêpe, mais elles ne ressemble pas aux autres qui nichent derrière la lampe de jardin, et qui sont plus grandes. + +Quelqu'un.e peut confirmer, d'après les photos ?... :) + +https://framapic.org/Nh4cZUUfzsZr/3TMZ1ZZYldGS + +https://framapic.org/yLMbCcaX3s0U/PYWzYdLOzy7t + +#faune-sauvage #insectes #guêpes #Vespoidea \ No newline at end of file From 66d4e5ce5fe6c0dc5aedd9dcc2233b76218ac18c Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Tue, 6 Aug 2019 21:23:09 -0400 Subject: [PATCH 3/3] Remove wrap length in Babel HTML::toPlaintext display --- src/Module/Debug/Babel.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Module/Debug/Babel.php b/src/Module/Debug/Babel.php index b9b629f073..19e3ec2131 100644 --- a/src/Module/Debug/Babel.php +++ b/src/Module/Debug/Babel.php @@ -160,7 +160,7 @@ class Babel extends BaseModule 'content' => visible_whitespace($markdown) ]; - $text = Text\HTML::toPlaintext($html); + $text = Text\HTML::toPlaintext($html, 0); $results[] = [ 'title' => L10n::t('HTML::toPlaintext'), 'content' => '
' . $text . '
'