Skip to content

Commit b6f59d2

Browse files
committed
Merge branch 'PHP-8.4'
* PHP-8.4: Fix GH-16473: dom_import_simplexml stub is wrong
2 parents 7aeafe4 + 55266d4 commit b6f59d2

File tree

3 files changed

+18
-5
lines changed

3 files changed

+18
-5
lines changed

ext/dom/php_dom.stub.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1234,7 +1234,7 @@ public static function quote(string $str): string {}
12341234
}
12351235
#endif
12361236

1237-
function dom_import_simplexml(object $node): DOMElement {}
1237+
function dom_import_simplexml(object $node): DOMAttr|DOMElement {}
12381238
}
12391239

12401240
namespace Dom
@@ -2171,5 +2171,5 @@ public static function quote(string $str): string {}
21712171
}
21722172
#endif
21732173

2174-
function import_simplexml(object $node): Element {}
2174+
function import_simplexml(object $node): Attr|Element {}
21752175
}

ext/dom/php_dom_arginfo.h

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ext/dom/tests/gh16473.phpt

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
--TEST--
2+
GH-16473 (dom_import_simplexml stub is wrong)
3+
--EXTENSIONS--
4+
dom
5+
simplexml
6+
--FILE--
7+
<?php
8+
$root = simplexml_load_string('<root xmlns:x="urn:x" x:attr="foo"/>');
9+
$attr = $root->attributes('urn:x');
10+
var_dump(dom_import_simplexml($attr)->textContent);
11+
?>
12+
--EXPECT--
13+
string(3) "foo"

0 commit comments

Comments
 (0)