Skip to content

Document DOM BC breaks in 8.3#2984

Merged
Girgias merged 4 commits into
php:masterfrom
ndossche:dom-bc-breaks
Dec 2, 2023
Merged

Document DOM BC breaks in 8.3#2984
Girgias merged 4 commits into
php:masterfrom
ndossche:dom-bc-breaks

Conversation

@ndossche
Copy link
Copy Markdown
Member

@ndossche ndossche commented Dec 2, 2023

Basically these are BC-breaking bugfixes

Comment thread reference/dom/domparentnode/append.xml Outdated
Comment on lines +65 to +66
Calling this method on a node without an owner document now works.
Previously this threw a <classname>DOMException</classname> with code <constant>DOM_HIERARCHY_REQUEST_ERR</constant>.
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.

This text seems to be repeated a lot maybe making this an entity would be helpful?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ait, done that now

@Girgias Girgias merged commit c1349f9 into php:master Dec 2, 2023
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