Skip to content

Fix GH-12171: resolve SimpleXML nodes before equality compare - #222

Closed
iliaal wants to merge 1 commit into
PHP-8.4from
fix/gh-12171-sxe-property-compare
Closed

Fix GH-12171: resolve SimpleXML nodes before equality compare#222
iliaal wants to merge 1 commit into
PHP-8.4from
fix/gh-12171-sxe-property-compare

Conversation

@iliaal

@iliaal iliaal commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Property proxies like $xml->first store the parent node plus an element iterator name. sxe_objects_compare compared the parent wrappers, so sibling proxies always compared equal. Resolve each side through php_sxe_get_first_node_non_destructive before comparing.

Property proxies like $xml->first store the parent node plus an element
iterator name. sxe_objects_compare compared the parent wrappers, so
sibling proxies always compared equal. Resolve each side through
php_sxe_get_first_node_non_destructive before comparing.

Fixes phpGH-12171
Closes phpGH-23187
@iliaal
iliaal force-pushed the fix/gh-12171-sxe-property-compare branch from 4d4fd48 to 5a01c95 Compare August 9, 2026 14:33
@iliaal

iliaal commented Aug 9, 2026

Copy link
Copy Markdown
Owner Author

Promoted upstream: php#23187

@iliaal iliaal closed this Aug 9, 2026
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.

1 participant