Change for Construct class for PHP 8.1 compatibility

[SilinMykola]

Hello! We tried to update our project for PHP 8.1 compatibility but had an error with your module. I made some changes to fix compatibility with PHP8.1.

[SilinMykola]

@fgrosse Hi! Could you help with this PR please?
Thank you!

[andrewbess]

Hello @fgrosse
The PHP 8.1 has been released.
Making a compatibility of our project with PHP 8.1 relates from your extension.
Could you please share your plans with releasing new version of your extension with us?
Thank you in advance.

[xmav]

@afk11 @fgrosse Could you help to proceed with this ?

[stefandanaita]

@fgrosse it would be nice to get this in whenever you have some time 🙏🏻 😄

[fgrosse]

Hey all, sorry it took a month to review but I finally took the time and the change looks fine.

[TimWolla]

@fgrosse Awesome, thank you!

[fgrosse]

It looks like a minor change to me so I suppose bumping to v2.3.1 is the right thing? However maybe someone can confirm since I haven't touched any PHP in a long time and might misunderstand the implications of this change.

[xmav]

Fantastic! Will you do release with those changes soon?

[TimWolla]

It looks like a minor change to me so I suppose bumping to v2.3.1 is the right thing?

@fgrosse Yes, the change is fully backwards compatible / a bugfix. It just suppresses the compatibility warning from PHP and has no other effect.

[fgrosse]

Alright then, I also just realized there is another PR #85 which looks related. Should we try to include this in that release as well?

[TimWolla]

@fgrosse #85 breaks compatibility with PHP 7.0. The void type was introduced in PHP 7.1. So this would be a breaking change and should not be included. However I suggest you merge my #86 to re-enable a working CI.

[fgrosse]

Alright thanks for the input 👍