#40656 [NEW]: DOMDocument::load() does not support stream wrappers

[bugs at php dot frankkleine dot de]

From: bugs at php dot frankkleine dot de
Operating system: irrelevant
PHP version: 5.2.1
PHP Bug Type: Feature/Change Request
Bug description: DOMDocument::load() does not support stream wrappers

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 bug report at [url]http://bugs.php.net/?id=40656&edit=1[/url]
--
Try a CVS snapshot (PHP 4.4): [url]http://bugs.php.net/fix.php?id=40656&r=trysnapshot44[/url]
Try a CVS snapshot (PHP 5.2): [url]http://bugs.php.net/fix.php?id=40656&r=trysnapshot52[/url]
Try a CVS snapshot (PHP 6.0): [url]http://bugs.php.net/fix.php?id=40656&r=trysnapshot60[/url]
Fixed in CVS: [url]http://bugs.php.net/fix.php?id=40656&r=fixedcvs[/url]
Fixed in release: [url]http://bugs.php.net/fix.php?id=40656&r=alreadyfixed[/url]
Need backtrace: [url]http://bugs.php.net/fix.php?id=40656&r=needtrace[/url]
Need Reproduce Script: [url]http://bugs.php.net/fix.php?id=40656&r=needscript[/url]
Try newer version: [url]http://bugs.php.net/fix.php?id=40656&r=oldversion[/url]
Not developer issue: [url]http://bugs.php.net/fix.php?id=40656&r=support[/url]
Expected behavior: [url]http://bugs.php.net/fix.php?id=40656&r=notwrong[/url]
Not enough info: [url]http://bugs.php.net/fix.php?id=40656&r=notenoughinfo[/url]
Submitted twice: [url]http://bugs.php.net/fix.php?id=40656&r=submittedtwice[/url]
register_globals: [url]http://bugs.php.net/fix.php?id=40656&r=globals[/url]
PHP 3 support discontinued: [url]http://bugs.php.net/fix.php?id=40656&r=php3[/url]
Daylight Savings: [url]http://bugs.php.net/fix.php?id=40656&r=dst[/url]
IIS Stability: [url]http://bugs.php.net/fix.php?id=40656&r=isapi[/url]
Install GNU Sed: [url]http://bugs.php.net/fix.php?id=40656&r=gnused[/url]
Floating point limitations: [url]http://bugs.php.net/fix.php?id=40656&r=float[/url]
No Zend Extensions: [url]http://bugs.php.net/fix.php?id=40656&r=nozend[/url]
MySQL Configuration Error: [url]http://bugs.php.net/fix.php?id=40656&r=mysqlcfg[/url]

[rrichards@php.net]

ID: 40656
Updated by: [email]rrichards@php.net[/email]
Reported By: bugs at php dot frankkleine dot de
-Status: Open
+Status: Bogus
Bug Type: Feature/Change Request
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
[url]http://www.php.net/manual/[/url] and the instructions on how to report
a bug at [url]http://bugs.php.net/how-to-report.php[/url]

url_stat needs to return an array


Previous Comments:
------------------------------------------------------------------------

[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 [url]http://bugs.php.net/?id=40656&edit=1[/url]

[bugs at php dot frankkleine dot de]

ID: 40656
User updated by: bugs at php dot frankkleine dot de
-Summary: DOMDocument::load() does not support stream wrappers
Reported By: bugs at php dot frankkleine dot de
-Status: Bogus
+Status: Open
-Bug Type: Feature/Change Request
+Bug Type: DOM XML related
Operating System: irrelevant
PHP Version: 5.2.1
New Comment:

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.


Previous Comments:
------------------------------------------------------------------------

[2007-02-27 11:22:35] [email]rrichards@php.net[/email]

Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
[url]http://www.php.net/manual/[/url] and the instructions on how to report
a bug at [url]http://bugs.php.net/how-to-report.php[/url]

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 [url]http://bugs.php.net/?id=40656&edit=1[/url]

[rrichards@php.net]

ID: 40656
Updated by: [email]rrichards@php.net[/email]
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
[url]http://www.php.net/manual/[/url] and the instructions on how to report
a bug at [url]http://bugs.php.net/how-to-report.php[/url]

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] [email]rrichards@php.net[/email]

Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
[url]http://www.php.net/manual/[/url] and the instructions on how to report
a bug at [url]http://bugs.php.net/how-to-report.php[/url]

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 [url]http://bugs.php.net/?id=40656&edit=1[/url]