Trouble running tests on PHP 8 with github actions using this lib

[francislavoie]

See Vectorface/GoogleAuthenticator#2

You can see the test failures here: https://github.com/Vectorface/GoogleAuthenticator/pull/2/checks?check_run_id=1484500680

FYI the dependency of this lib that is causing failures in your builds is now fixed to support PHP 8: khanamiryan/php-qrcode-detector-decoder#98 with this release: https://github.com/khanamiryan/php-qrcode-detector-decoder/releases/tag/1.0.4

I'm not sure why GD would be failing on PHP 8. This lib doesn't emit any useful information as to why it might be the case. It definitely gets past the check for the extension being loaded, but then it fails on imagecreatetruecolor()

qr-code/src/Writer/PngWriter.php

Lines 71 to 75 in ad1aa6c

	$image = imagecreatetruecolor($data['block_count'] * $baseSize, $data['block_count'] * $baseSize);
	if (!is_resource($image)) {
	throw new GenerateImageException('Unable to generate image: check your GD installation');
	}

[francislavoie]

Ohhh - I think it's actually because gd doesn't return resources anymore in PHP 8, but objects now.

php/php-tasks#6

php/php-src#4714

[endroid]

Hi @francislavoie this should be fixed.

[acelaya]

Hey!

In what version is this fixed? Because I'm using 3.9.6 (which seems to be the latest one), and I can still see this errors.

Also, If I check the source code inside my vendor folder, it's still doing this is_resource calls.

Am I missing something? 🤔

[endroid]

Hi @acelaya this should be fixed in the 3.x branch and awaiting update of an external library before it completely supports PHP 8 and can be tagged.

[acelaya]

Perfect. Thanks for the info.

[ThibaudDauce]

@endroid Can we help for something to tag a new release?

[endroid]

@ThibaudDauce unfortunately I think not :) I hope to have a 4.x release soon which allows usage of the library without requiring that package but for now if you need to run in PHP 8 you can use the temporary workaround mentioned in #292.