Skip to content

Document DOM node collection cloning failure for PHP 8.5 - #5877

Merged
lacatoire merged 1 commit into
php:masterfrom
lacatoire:fix/dom-clone-namednodemap-nodelist-85
Sep 24, 2026
Merged

lacatoire merged 1 commit into
php:masterfrom
lacatoire:fix/dom-clone-namednodemap-nodelist-85

Conversation

@lacatoire

@lacatoire lacatoire commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Add changelog entries noting that cloning DOMNamedNodeMap, DOMNodeList,
Dom\NamedNodeMap, Dom\NodeList, Dom\HTMLCollection and
Dom\DtdNamedNodeMap objects now fails.
@lacatoire
lacatoire requested a review from ndossche as a code owner September 24, 2026 15:55

@ndossche ndossche left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@lacatoire
lacatoire merged commit 38fcf62 into php:master Sep 24, 2026
3 checks passed
@lacatoire
lacatoire deleted the fix/dom-clone-namednodemap-nodelist-85 branch September 24, 2026 20:57
KentarouTakeda added a commit to php/doc-ja that referenced this pull request Sep 25, 2026
dom の 8 ファイルを原文の変更に追従しました。これで dom モジュールの未訳・要更新はなくなります。

- PHP 8.5 で `DOMNamedNodeMap`・`DOMNodeList`・`Dom\NamedNodeMap`・`Dom\NodeList`・`Dom\HTMLCollection`・`Dom\DtdNamedNodeMap` のオブジェクトをクローンすると失敗するようになったことが、各クラスの変更履歴に追加されました。あわせて `Dom\DtdNamedNodeMap`・`Dom\HTMLCollection`・`Dom\NamedNodeMap` の説明に、似たクラスとの関係が追記されました(php/doc-en@38fcf62、php/doc-en#5877)
- PHP 8.5 で `Dom\Element::getElementsByClassName()` が追加されたことが、`Dom\Element` の変更履歴に追加されました(php/doc-en@d4f96da、php/doc-en#5879)
- `DOMDocument::xinclude()` に、信頼できない XML に対して使わないよう求める警告が追加されました(php/doc-en@a809316、php/doc-en#5862)

## 対象ファイル

reference/dom/ (8) — domdocument/xinclude.xml, domnamednodemap.xml, domnodelist.xml, dom/dom-dtdnamednodemap.xml, dom/dom-element.xml, dom/dom-htmlcollection.xml, dom/dom-namednodemap.xml, dom/dom-nodelist.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants