Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Dom\HTMLDocument: Impossible to add text node on document level

Offen
  • Stimmt etwas nicht?
#23,562 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
48/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
c, php
Bereich
api, backend

Rechercherichtung

Beginne damit, das PHP-Snippet mit Dom\HTMLDocument::createFromString, LIBXML_HTML_NOIMPLIED und CharacterData::replaceWith() nachzustellen. Verfolge, wie replaceWith(), insertAdjacentHTML() und appendChild() mit Textknoten auf Dokumentebene umgehen, und vergleiche anschließend die im Issue erwähnten spezifikationskonformen und nicht spezifikationskonformen Lexbor-Einstiegspunkte. Erledigt ist die Aufgabe, wenn eine klare Entscheidung erreicht und ein konsistentes Verhalten für NOIMPLIED-Dokumente implementiert oder dokumentiert wurde, mit Tests für den gezeigten Fall.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Feature Status: Needs Triage
Description

The following code...

$doc = \Dom\HTMLDocument::createFromString('Hello <b>world</b>!',LIBXML_HTML_NOIMPLIED);
$greeting = $doc->firstChild;
echo get_class($greeting)."<br />";
$new_greeting = $doc->createTextNode('Welcome');
$greeting->replaceWith($new_greeting);
echo htmlentities($doc->saveHtml());

results in this error:

DOMException: Cannot insert text as a child of a document in Dom\CharacterData->replaceWith()

This is different behavior from the old DOM classes which allowed this. I understand, this spec-compliance is supposed to be a feature. If I understand correctly, the new Lexbor-based parser offers validity checks for some methods and not for others:

https://lexbor.com/modules/dom/

E.g. lxb_dom_node_insert_before_spec vs lxb_dom_node_insert_before etc.

I'm unaware of the current PHP implementation but I assume it is using the _spec versions throughout without any option to turn it off.

I think this is problematic. The NOIMPLIED option is very useful indeed if you want to handle document fragments etc. without unnecessary wrapper tags.

The above code leads to the peculiar situation that you can indeed create a Document with textNodes on the document level, but if you just want to replace that textNode with a different textNode it won't work. This complicates things a lot unnecessarily. Other methods like insertAdjacentHTML, appendChild etc are equally stubborn.

There are different solutions for this: IMHO the easiest one would be to use the non-spec versions of these functions if NOIMPLIED was specified. NOIMPLIED explicitly allows a non-spec document to be imported. Any manipulation should also be allowed then.

Another alternative would be to create a new general flag LIBXML_HTML_RELAX_SPEC.

A third alternative would be to specifically add option flags to the individual methods to allow this for a single call.

I'm unsure what's really the best solution but at the moment, porting over existing code is unnecessarily difficult due to this restriction and I really think this feature would make a lot of sense anyway. Thanks for considering.

Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.2k
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
151

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus php/php-src

Alle Issues in php/php-src

Ähnliche Issues

Weitere Issues zu C

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.