fix issue #151, chaining method calls was not working as it should because the per-object refcount was not updated correctly, which caused an object to be destructed even when it already was assigned to a different variable

1 files changed, 5 insertions, 1 deletions

diff --git a/zend/objectimpl.h b/zend/objectimpl.h
index d72ddbd..4d20f3c 100644
--- a/zend/objectimpl.h
+++ b/zend/objectimpl.h
@@ -62,9 +62,10 @@ public:
      *
      *  @param  entry       Zend class entry
      *  @param  base        C++ object that already exists
+     *  @param  refcount    The initial refcount for the object
      *  @param  tsrm_ls     Optional threading data
      */
-    ObjectImpl(zend_class_entry *entry, Base *base TSRMLS_DC)
+    ObjectImpl(zend_class_entry *entry, Base *base, int refcount TSRMLS_DC)
     {
         // allocate a mixed object (for some reason this does not have to be deallocated)
         _mixed = (MixedObject *)emalloc(sizeof(MixedObject));
@@ -117,6 +118,9 @@ public:
         // seem to be necessary...
         _handle = zend_objects_store_put(php(), (zend_objects_store_dtor_t)destructMethod, (zend_objects_free_object_storage_t)freeMethod, NULL TSRMLS_CC);
         
+        // set the initial refcount (if it is different than one, because one is the default)
+        if (refcount != 1) EG(objects_store).object_buckets[_handle].bucket.obj.refcount = refcount;
+        
         // the object may remember that we are its implementation object
         base->_impl = this;
     }