Pending

User tests: Successful: Unsuccessful:

avatar n3t
n3t
4 Sep 2026

Function getOldManifest searched invalid path. It relied on extension_administrator path, which is available only for components. Changed to extension_root, which is available for all extensions, and in components installer fallback to extension_administrator.

Moreover path manifest contains complete path to manifest not only filename, therefore the result path was always invalid.

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

  • changed the InstallerScriptTrait::getOldManifest function to search in correct path
  • added parameters to error message (current and new version)

Testing Instructions

Use any extension using InstallerScriptTrait with disallowed downgrades (that is by default) and try to install an old version over existing newer version.

Actual result BEFORE applying this Pull Request

The extension is installed anyway,

Expected result AFTER applying this Pull Request

The extension is not installed, error message is displayed,

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar n3t n3t - open - 4 Sep 2026
avatar n3t n3t - change - 4 Sep 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Sep 2026
Category Libraries
avatar richard67
richard67 - comment - 4 Sep 2026

@n3t Is it a bug which happens also in 5.4? Or does it only happen in 6.x? If only in 6.x, the 6.1-dev branch is ok, but if the issue also happens in 5.4, the PR should be made for the 5.4-dev branch. See also https://github.com/joomla/joomla-cms#which-branch-should-my-pull-request-target .

avatar n3t
n3t - comment - 4 Sep 2026

@richard67 This is just Joomla 6.x related, as the InstallerScriptTrait was introduced in Joomla 6.0, should be replacement for InstallerScript class.

Add a Comment

Login with GitHub to post a comment