Mixing processing instructions and element nodes gives inconsistent results in SimpleXML
Abierto
Nadie ha tomado este issue todavía.
Bug
Extension: simplexml
Status: Verified
- Lenguaje dominante
- C
- Estrellas
- 40.4k
- Forks
- 8.1k
- Merge medio
- 2 d 13 h
- PR fusionados (30 d)
- 96
Descripción
Description
The following code:
<?php
$xml = <<<XML
<?xml version="1.0" encoding="utf-8"?>
<container>
<x/><?hello world?>
</container>
XML;
$sxe = simplexml_load_string($xml);
var_dump($sxe->children());
// x and hello swapped
$xml = <<<XML
<?xml version="1.0" encoding="utf-8"?>
<container>
<?hello world?><x/>
</container>
XML;
$sxe = simplexml_load_string($xml);
var_dump($sxe->children());
Resulted in this output:
object(SimpleXMLElement)#2 (2) {
["x"]=>
object(SimpleXMLElement)#4 (0) {
}
["hello"]=>
object(SimpleXMLElement)#5 (0) {
}
}
object(SimpleXMLElement)#1 (1) {
["x"]=>
object(SimpleXMLElement)#5 (0) {
}
}
But I expected this output instead:
object(SimpleXMLElement)#2 (2) {
["x"]=>
object(SimpleXMLElement)#4 (0) {
}
["hello"]=>
object(SimpleXMLElement)#5 (0) {
}
}
object(SimpleXMLElement)#1 (1) {
["hello"]=>
object(SimpleXMLElement)#%d (0) {
}
["x"]=>
object(SimpleXMLElement)#%d (0) {
}
}
PHP Version
PHP 8.1+
Operating System
Linux
Guía de contribución
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.
Línea de trabajo
Comienza con las dos reproducciones en PHP del issue e inspecciona la implementación de SimpleXML y sus pruebas existentes sobre el orden de children(). Compara el tratamiento de processing-instruction y element-node en ambos órdenes de entrada; después, añade cobertura que muestre el orden esperado y ejecuta las pruebas relevantes de SimpleXML.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- backend
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100