Allow extensions to access internal MagickWand objects

[jorissteyn]

Past versions of imagick included php_imagick_defs.h in installations of imagick (config.m4: PHP_INSTALL_HEADERS). This allowed other extensions to access the internal data of Imagick objects.

For example, the accessor php_imagick_get_class_entry allows extensions to handle Imagick objects, but in order to access the internal MagickWand, the extension needs to know at what offset the magic_wand field exists. Currently, the only way to know that is to include the definition of php_imagick_object.

See the pending discussion here and how I work around it in zbarcode.

[Danack]

Past versions of imagick included php_imagick_defs.h in installations of imagick (config.m4: PHP_INSTALL_HEADERS).

I'm not aware of the change that you're referring to - can you do me a favour and link to the commit where you think something changed?

See the pending discussion here and how I work around it in zbarcode.

Is the link to the discussion meant to be a different link?

[Danack]

Oh, i see - mkoppanen/php-zbarcode#21

But that change was done in 3.2.0RC1 - which was a 'while' ago: 9785713

I'll have a think - probably the best thing is making a proper function to get the MagickWand object from a zval that is a php_imagick_object

[jorissteyn]

But that change was done in 3.2.0RC1

Ah I read that information wrong, thought it was more recent.

I think my actual request is not necessarily "restore the header" but a more general "help me fix the zbarcode use-case", I'll update the issue title accordingly.

making a proper function to get the MagickWand object from a zval that is a php_imagick_object

That sounds like a great solution!

[Caffe1neAdd1ct]

Any update on this one? Found a strange issue decoding barcodes with GD not always working compared with passing in an Imagick object, would be great to re-implement this integration with Imagick / zbarcode.