#40656 [Opn->Bgs]: DOMDocument::load() urlencodes parts of an URI

ID: 40656 Updated by: rrichards@php.net Reported By: bugs at php dot frankkleine dot de -Status: Open +Status: Bogus Bug Type: DOM XML related Operating System: irrelevant PHP Version: 5.2.1 New Comment: Thank you for taking the time to write to us, but this is not a bug. Please double-check the documentation available at http://www.php.net/manual/ and the instructions on how to report a bug at http://bugs.php.net/how-to-report.php You have to use valid URIs (hence the "\" gets encoded). Previous Comments: ------------------------------------------------------------------------ [2007-02-28 14:23:58] bugs at php dot frankkleine dot de Thanks to your reply I found out what the real problem is. The DOMDocument::load() urlencodes parts of the URI used for the stream wrapper: myStream://C:\master\bla.php?foo.xml becomes myStream://C:%5Cmaster%5Cbla.php?foo.xml" To work properly, the stream wrapper class first has to look if the $path argument of stream_open() does exist, if not it has to do an urldecode() on the $path and then check if the urldecoded $path exists. It is my expectation that the stream wrapper should get the correct $path from DOMDocument::load(), not an urlencoded one. ------------------------------------------------------------------------ [2007-02-27 11:22:35] rrichards@php.net Thank you for taking the time to write to us, but this is not a bug. Please double-check the documentation available at http://www.php.net/manual/ and the instructions on how to report a bug at http://bugs.php.net/how-to-report.php url_stat needs to return an array ------------------------------------------------------------------------ [2007-02-27 10:05:52] bugs at php dot frankkleine dot de Description: ------------ The DOMDocument::load() method does not support working with stream wrappers. It's an inconsistency within PHP, one would expect that this method works with stream wrappers as well just as other methods and functions do. Reproduce code: --------------- class MyStreamWrapper { protected $read = false; protected $stream = '<?xml version="1.0" encoding="iso-8859-1"?><foo><bar id="1">hello world</bar></foo>'; public function stream_open($path, $mode, $options, $opened_path) {return true; } public function stream_close() { } public function stream_read($count) { if (true == $this->read) { return ''; } $this->read = true; return $this->stream; } public function stream_eof() { return $this->read; } public function stream_stat() { return strlen($this->stream); } public function url_stat($path) { return strlen($this->stream); } } stream_wrapper_register('myStream', 'MyStreamWrapper'); $doc = DOMDocument::load('myStream://foo.xml'); var_dump($doc); Expected result: ---------------- object(DOMDocument)#1 (0) { } Actual result: -------------- Warning: DOMDocument::load() [function.DOMDocument-load]: I/O warning : failed to load external entity "myStream://foo" in DomDocument_load-StreamWrapper.php on line 43 ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=40656&edit=1