|
|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2019-03-31 22:32 UTC] alexey dot khokhryakov at gmail dot com
Description:
------------
SplFileObject::seek() works wrong for custom stream wrappers. It should work exactly the same as for native files.
In my example I created simple stream wrapper which just proxies all calls to native fopen/fread/ftell/feof/fseek/stat functions which demonstrates the problem.
Test script:
---------------
<?php
file_put_contents($tmpFilename = tempnam(sys_get_temp_dir(), 'test'), "one line\r\n");
stream_wrapper_register('foo', FooStreamWrapper::class);
foreach ([$tmpFilename, "foo:/{$tmpFilename}"] as $filename) {
$file = new SplFileObject($filename, 'r');
$file->seek($file->getSize());
var_dump($file->key());
}
class FooStreamWrapper {
private $r;
public function stream_open($path, $mode) { $this->r = fopen(str_replace('foo:/', '', $path), $mode); return true; }
public function stream_read($count) { return fread($this->r, $count); }
public function stream_tell() { return ftell($this->r); }
public function stream_eof() { return feof($this->r); }
public function stream_seek($offset, $whence) { return fseek($this->r, $offset, $whence) === 0; }
public function url_stat($path) { return stat(str_replace('foo:/', '', $path)); }
}
Expected result:
----------------
int(1)
int(1)
Actual result:
--------------
int(1)
int(0)
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||||||||||||
Copyright © 2001-2023 The PHP GroupAll rights reserved. |
Last updated: Tue Jun 06 16:03:37 2023 UTC |
It's not SplFileObject but your stream wrapper. PHP's file wrapper is reading lines, yours is reading bytes. If you edit yours to read lines as well public function stream_read($count) { return fgets($this->r, $count); } then it works as expected. Not sure the best way to fix this.I tried and I see that my test script works but it breaks SplFileObject::fread() functionality as I said. file_put_contents($tmpFilename = tempnam(sys_get_temp_dir(), 'test'), "one\r\ntwo\r\n"); stream_wrapper_register('foo', FooStreamWrapper::class); $file = new SplFileObject("foo:/{$tmpFilename}", 'r'); var_dump($file->fread($file->getSize())); Expected: string(10) "one two " Actual: string(5) "one "