|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #44392 getFilePointer() for Childs of SplFileObject
Submitted: 2008-03-10 14:45 UTC Modified: 2015-03-25 21:39 UTC
Avg. Score:4.4 ± 0.8
Reproduced:20 of 20 (100.0%)
Same Version:4 (20.0%)
Same OS:4 (20.0%)
From: php at benjaminschulz dot com Assigned: danack
Status: Assigned Package: SPL related
PHP Version: 5.3CVS-2008-03-10 (CVS) OS:
Private report: No CVE-ID:
Have you experienced this issue?
Rate the importance of this bug to you:

 [2008-03-10 14:45 UTC] php at benjaminschulz dot com
It would be nice if it would be possible to get the underlying resource handle of an SplFileObject to be able to add stream filters on the file. Sadly the URI parser in PHP seems to be broken and URIs with filters like php://filter/read=convert.iconv.ISO-8859-15/UTF-8/resource=... cannot be used (encoding the slash doesn't work either (%2F)) therefore it would be nice if one could access the underlying resource handle f.e. by providing a protected $fp in SplFileObject one could use in a child class then and do the stream_filter_append() there.


Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2008-11-14 23:49 UTC] jordan dot raub at dataxltd dot com
add a protected member function to SplFileObject so that extending classes can have more control of the file handle... patch included against php5.2.6..

--- php-5.2.6/ext/spl/spl_directory.c   2008-02-13 04:23:26.000000000 -0800
+++ php52GetResource/ext/spl/spl_directory.c    2008-11-14 13:22:17.000000000 -0800
@@ -2218,6 +2218,15 @@
 } /* }}} */

+/* {{{ proto void SplFileObject::getFileResource()
+   Seek to specified line */
+SPL_METHOD(SplFileObject, getFileResource)
+       spl_filesystem_object *intern = (spl_filesystem_object*)zend_object_store_get_object(getThis() TSRMLS_CC);
+       php_stream_to_zval(intern->, return_value);
+} /* }}} */
 /* {{{ Function/Class/Method definitions */
 ZEND_BEGIN_ARG_INFO_EX(arginfo_file_object___construct, 0, 0, 1)
@@ -2310,6 +2319,7 @@
        SPL_ME(SplFileObject, getMaxLineLen,  NULL, ZEND_ACC_PUBLIC)
        SPL_ME(SplFileObject, hasChildren,    NULL, ZEND_ACC_PUBLIC)
        SPL_ME(SplFileObject, getChildren,    NULL, ZEND_ACC_PUBLIC)
+       SPL_ME(SplFileObject, getFileResource,NULL, ZEND_ACC_PROTECTED)
        SPL_ME(SplFileObject, seek,           arginfo_file_object_seek,          ZEND_ACC_PUBLIC)
        /* mappings */
        SPL_MA(SplFileObject, getCurrentLine, SplFileObject, fgets,      NULL, ZEND_ACC_PUBLIC)
 [2011-04-08 21:25 UTC]
-Package: Feature/Change Request +Package: SPL related
 [2012-11-20 23:38 UTC] mattsch at gmail dot com
What's the status of this bug?  SplFileObject is supposed to be an OO version of fopen but it's quite useless to pass into other functions like curl when those functions expect a resource.


$splFileObject = new SplFileObject('/tmp/foo', 'r');
$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, 's');
curl_setopt($ch, CURLOPT_UPLOAD, 1);
curl_setopt($ch, CURLOPT_INFILE, $splFileObject); # <-- won't work, must be file resource
curl_setopt($ch, CURLOPT_INFILESIZE, filesize($localfile));
curl_exec ($ch);
 [2012-12-09 01:26 UTC]
I don't see why this method shouldn't be public. I vote for extending its 
visibility to public.
 [2012-12-12 16:12 UTC] mattsch at gmail dot com
I agree that this method should also be public.  Please implement this method ASAP.
 [2013-12-04 13:05 UTC] drgomesp at gmail dot com
Any updates on this? It would be really useful to change the visibility of this method to public.
 [2014-05-13 14:50 UTC] nyamsprod at gmail dot com
Even thought making the SplFileObject::getFilePointer() method public would be a great addition I think there is a better solution which would be to create a abstract interface "à la" Traversable but called "Streamable". 
This interface would not be implemented alone but classes that implement it like SplFileObject would be usable directly on function like stream_append_filter, streamp_get_metadata of even curl_setopt. 
That way the SplFileObject file pointer would be "usable" but its file pointer property would remain protected from a developer who would otherwise use the SplFileObject::getFilePointer() result direclty on a fclose function.
 [2015-01-14 20:28 UTC] mattsch at gmail dot com
The Streamable interface sounds like a great idea.  Now is this bug ever going to get prioritized?
 [2015-03-25 21:39 UTC]
-Status: Open +Status: Assigned -Assigned To: +Assigned To: danack
PHP Copyright © 2001-2016 The PHP Group
All rights reserved.
Last updated: Sat Dec 03 06:01:41 2016 UTC