ID:               40477
 Updated by:       [EMAIL PROTECTED]
 Reported By:      piotrek dot pokora at gmail dot com
 Status:           Bogus
 Bug Type:         Variables related
 Operating System: debian/linux
 PHP Version:      5.2.1
 New Comment:

>As it works without get_properties.
Oh, yes it does work, but you can't get the list of properties without
implementing get_properties().

>It's enough to add own read and write property hook
No, it's obviously not enough. See below why.

>Look at mysqli implementation.
>It DOES NOT contain get_properties hook and works. Why?

var_dump($mysqli_object);
Do you see anything? No. 
That's why.


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

[2007-02-14 14:49:56] piotrek dot pokora at gmail dot com

>Of course get_properties() is required, how else do you >think it
should work?

As it works without get_properties.
It's enough to add own read and write property hook and some function
which "registers" properties of an object in constructor.  

It works _perfect_ for reading property, assigning value to it. And the
same for derived classes.

Look at mysqli implementation.
It DOES NOT contain get_properties hook and works. Why?

>You need to know the name of the property to be able to call
>read_property().

Hook function requires zval *member as second argument.
I do not know internals , please tell me :)

read_property hook works _perfect_ without get_properties hook.

>The fact that you can't find any docs for >get_properties()(which is
>implemented quite a number of times in PHP sources) is >hardly a PHP
bug.

Yes, but the logic bug here is the fact that always internal zend
functions "operate" in object's handlers with this little exception of
print_t or var_dump when custom hash table is taken into account.

So , to sum up. I can not find any not working part of code with only
read and write property hook , except this one for print_r and
var_dump.

>Please discuss custom extension writing in >[EMAIL PROTECTED]
mailing list.

Will try :)

------------------------------------------------------------------------

[2007-02-14 11:03:32] [EMAIL PROTECTED]

Of course get_properties() is required, how else do you think it should
work? You need to know the name of the property to be able to call
read_property().
The fact that you can't find any docs for get_properties()(which is
implemented quite a number of times in PHP sources) is hardly a PHP
bug.
Please discuss custom extension writing in [EMAIL PROTECTED]
mailing list.

------------------------------------------------------------------------

[2007-02-14 09:56:01] piotrek dot pokora at gmail dot com

Description:
------------
zend_print_zval_r_ex uses object's get_properties handler and prints
properties HashTable without using read_property handler.

For custom object handlers hooks it means that get_properties should be
also implemented as hook which leads to too much overloaded
implementation as get_properties seems to be invoked for read and write
hooks.

Taking performance into account , one would sacrifice performance
problem for print_r ( which is not used in real production environment
) which could use object's read_property handler hook. 

Reproduce code:
---------------
Implement read and write property hooks for object.
Assuming that property's data is set in C extension one is able to
call:

echo $object->prop; 

which prints property's value, while 

print_r($object);

prints object's properties without values. 

I know I can implement own get_properties hook , but unfortunatelly I
can not find suitable docs for such implementation.

I think one should be able to reproduce it with mysqli extension which
implement only get and read property hooks.



------------------------------------------------------------------------


-- 
Edit this bug report at http://bugs.php.net/?id=40477&edit=1

Reply via email to