Skip to content

Conversation

@richard67
Copy link
Member

Pull Request for Issue # .

Summary of Changes

This pull request (PR) updates the composer dependency "joomla/test" from version 3.0.3 to version 3.0.4 for the 5.4-dev branch.

The only change pulled with this update is the one from PR joomla-framework/test#19 to fix a deprecation notice on PHP 8.5 for the setAccessible() methods of various Reflection objects.

Release note: https://github.com/joomla-framework/test/releases/tag/3.0.4 .

See also https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_reflectionsetaccessible .

Testing Instructions

  • Use php 8.5 (latest pre-version).
  • Run composer install
  • Run unit tests ./libraries/vendor/bin/phpunit --testsuite Unit
  • Check PHP error log

Actual result BEFORE applying this Pull Request

Deprecation messages ReflectionMethod::setAccessible() is deprecated.

Expected result AFTER applying this Pull Request

None of these messages.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@muhme
Copy link
Contributor

muhme commented Oct 17, 2025

✅ Tested with JBT php8.5-rc

  • Before the PR seen 23 ReflectionMethod::setAccessible() is deprecated
  • Applied PR by gh pr checkout 46309 and running composer i
    • ✅ no more ReflectionMethod::setAccessible() is deprecated
    • ℹ️ For Info: Still there are other deprecations

@muhme muhme merged commit 446b596 into joomla:5.4-dev Oct 17, 2025
72 of 78 checks passed
@muhme muhme added this to the Joomla! 5.4.1 milestone Oct 17, 2025
@muhme
Copy link
Contributor

muhme commented Oct 17, 2025

Thank you @richard67 for your contribution.

@richard67 richard67 deleted the 5.4-dev-composer-update-joomla-test-3.0.4 branch October 17, 2025 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants