Skip to content

[pull] master from php:master#701

Merged
pull[bot] merged 2 commits intoConnectionMaster:masterfrom
php:master
Apr 25, 2026
Merged

[pull] master from php:master#701
pull[bot] merged 2 commits intoConnectionMaster:masterfrom
php:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Apr 25, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

Fixes an existing TODO by @ndossche.

Notation nodes returned from $doctype->notations were not linked to their owning document or parent DocumentType. This caused several incorrect behaviour; including:

ownerDocument was missing
parentNode was NULL
isConnected returned false
baseURI fell back to "about:blank"
textContent returned an empty string instead of NULL

The last point is a violation of the DOM specification. Since Notation is not an Element, CharacterData, Attr, or DocumentFragment, its textContent must return NULL.

Spec reference: https://dom.spec.whatwg.org/#dom-node-textcontent

close GH-21868
@pull pull Bot locked and limited conversation to collaborators Apr 25, 2026
@pull pull Bot added the ⤵️ pull label Apr 25, 2026
@pull pull Bot merged commit 9498bc3 into ConnectionMaster:master Apr 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant