Skip to content

[FEATURE] Use PHP 8.4/8.5#11238

Open
mjansenDatabay wants to merge 4 commits intoILIAS-eLearning:trunkfrom
mjansenDatabay:improvement/12/php-versions
Open

[FEATURE] Use PHP 8.4/8.5#11238
mjansenDatabay wants to merge 4 commits intoILIAS-eLearning:trunkfrom
mjansenDatabay:improvement/12/php-versions

Conversation

@mjansenDatabay
Copy link
Contributor

This PR adds the support for PHP 8.5 (and drops the PHP 8.3 support) for the next ILIAS major release (ILIAS 12) by ...

  • ... modifying the installation documentation
  • ... adjusting versions in our composer.json file (I upgraded the dependencies by using PHP 8.4 as the lower version, but this PR does not include the dependency consolidation for ILIAS 12)
  • ... changing the configuration for our GitHub CI pipeline configuration files
  • ... change the version constants in the setup component

Required and Supported Software ILIAS 12: https://docu.ilias.de/go/wiki/wpage_8819_1357

@mjansenDatabay mjansenDatabay added improvement dependencies Pull requests that update a dependency file documentation Pull requests that only update or add documentation. labels Mar 10, 2026
@mjansenDatabay mjansenDatabay force-pushed the improvement/12/php-versions branch 2 times, most recently from ccc903e to 265bb09 Compare March 10, 2026 09:51
@mjansenDatabay mjansenDatabay force-pushed the improvement/12/php-versions branch from 386c9a7 to b1a4d81 Compare March 10, 2026 12:19
@mjansenDatabay
Copy link
Contributor Author

Dear T&A authorities,

to make our unit tests work with PHP 8.5, I had to adjust components/ILIAS/TestQuestionPool/classes/class.assErrorText.php. In some situations, $position may be null, which is not a valid argument for array_key_exists and also not a valid array offset.

Best regards,
Michael

@mjansenDatabay
Copy link
Contributor Author

Dear component authorities,

I removed all usages/occurrences of ReflectionMethod::setAccessible(...) from our code base, since this method has been deprecated as of PHP 8.5. In addition, calling it has had no effect since PHP 8.1.

I will not request a review here, as this is a trivial change.

Best regards,
Michael

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file documentation Pull requests that only update or add documentation. improvement technical board

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant