Allow form feed characters in attributes
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
Línea de trabajo
Comienza en Tokenizer::quotedAttributeValue() y reproduce el problema con un atributo alt de img que contiene un salto de página, comparando la salida de HTML5 con DOMDocument. Rastrea por qué se descarta el texto posterior al salto de página y añade cobertura que demuestre que el carácter se conserva como en la salida del DOM nativo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Tokenizer::quotedAttributeValue() has special handling for form feed characters.
(This used to also include newlines and tabs but was removed in https://github.com/Masterminds/html5-php/commit/8785fc5775df52dd4a6b63677ebd08f8301ffdcc.)
Because of this, if an attribute contains a form feed, everything after this character is ignored.
This creates a discrepancy between masterminds/html5 and the native DOM extension:
$html = "<img src='foo.png' alt='before \f after'>";
$html5 = new HTML5();
$dom = $html5->loadHTML($html);
echo $html5->saveHTML($dom);
// <img src="foo.png" alt="before ">
$dom = new DOMDocument();
$dom->loadHTML($html);
echo $dom->saveHTML();
// <img src="foo.png" alt="before after">
This HTML containing a form feed character in the alt attribute is valid according to https://validator.w3.org/:
<!doctype html>
<html lang="en">
<head>
<meta charset="UTF-8">
<title>Document</title>
</head>
<body>
<img src="foo.png" alt=" foo">
</body>
</html>
- Lenguaje dominante
- HTML
- Estrellas
- 1.8k
- Forks
- 123
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Masterminds/html5-php
-
Template contents participate in the DOM treePosiblemente ocupada @eyupcanakman la tomó hace 204 días. Abiertohelp-wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Masterminds/html5-php#252 · 1 comentario · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Masterminds/html5-php#251 ·
-
Parser remove the single < (less than) character from given html stringPosiblemente ocupada @bsweeney la tomó hace 790 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Masterminds/html5-php#250 · 5 comentarios · 2 reacciones ·
-
multiple <html> <body>Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 28/100
Masterminds/html5-php#246 · 1 comentario ·
-
<noscript> with html insideAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Masterminds/html5-php#245 · 1 comentario ·
Todos los issues de Masterminds/html5-php
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
FuRongJun-1999/dsh-memory#65 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
hexlet-codebattle/codebattle#2361 ·
-
`FakeBackendV2.run` fails with `NoiseError` on circuits with delays on qubits where T2 > 2·T1Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Qiskit/qiskit-aer#2466 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día