soundasleep/html2text

Abusive removal of br nodes leads to incorrect output

Aperta

#54 aperta il 17 ott 2017

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)HTML (138 fork)github user discovery
hacktoberfest

Metriche repository

Star
 (485 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Hello !

There is some code doing intentional removal of nodes when they are the last child of a node that also contained text. Here's a very simple example about how this can lead to incorrect results (this is stuff I'm receiving from bad html emails) :

<font size="+1">Vikings: Wolves of Midgard<br></font><font size="+1">Valkyria Chronicles<br>
<br>
World Of Warcraft Battlechest</font>

The expected output would be

Vikings: Wolves of Midgard
Valkyria Chronicles

World Of Warcraft Battlechest

The actual output is:

Vikings: Wolves of MidgardValkyria Chronicles

World Of Warcraft Battlechest

Guida contributor

Direzione di ricerca
Trova la sezione di codice che rimuove intenzionalmente i nodi <br> quando sono l'ultimo figlio di un nodo contenente testo. Esegui il debug della logica per garantire che le interruzioni di riga siano preservate correttamente.
Tech stack
php
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
php
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70