Incompatible Composer PHP requirement in packages since 0.10.24

[lernhart]

It seems that support for PHP 7.4 was intentionally dropped in the latest 0.10.24 release. The base composer.json of the repo has been updated to reflect that properly.
However, the composer.json files in the affected packages still list PHP 7.4 as a valid alternative to PHP 8.0, leading to compatibility issues.

In the following example, running the affected enqueue/enqueue package on PHP 7.4 produces errors like the following, as trailing constructor commas were introduced to the enqueu package:

#message: "syntax error, unexpected ')', expecting variable (T_VARIABLE)"
#code: 0 
#file: "./vendor/enqueue/enqueue/Symfony/Consumption/ConfigurableConsumeCommand.php" 
#line: 50

Not sure, how packages are relesed through this monorepo, but a valid fix should be to update the packages' composer.json's to only allow >=PHP8.1?

[Steveb-p]

If there are any composer.json's that are missing the PHP dependency change, then we should definitely update those. Feel free to make the change and submit the PR if you have spare time for it.

As for the previous versions, it's very hard to fix a mistake like this. We could in theory add conflict declarations in against the offending packages and release a new version of 0.10 too, which would reduce the impact. But we can't really remove the package versions containing invalid declarations.

[lernhart]

@Steveb-p created a PR. Simply bumped the composer versions in there. CI Actions need approval to run, I think?

[cceconi]

The problem was not on the composer file but in the code. I made a PR which resolve the issue : php-enqueue/enqueue#14

I also have a remark on this line of code which will cause problems at runtime:
throw new \LogicException(sprintf('Transport "%s" is not supported.', $transport), null, $e);

the null param must be an int and I've seen that one in some files, maybe another PR ?

[JimTools]

Hi @lernhart, this issue should have been sorted now.