[#47637] - [5.4] Compile the manifest path from extension root
- Fixed in Code Base
- 5 Jun 2026
- Medium
- Build: 5.4-dev
- # 47637
- Diff
- Digital-Peak:installer
User tests: Successful: Unsuccessful:
Pull Request resolves #47635.
- 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
Load the extension mainfest file to create the class from the correct extension_root parameter as source is only compiled once.
Testing Instructions
- Extract the file plg_system_test1.zip into the folder /plugins/system/test1
- Extract the file plg_system_test2.zip into the folder /plugins/system/test2
- Discover install the extensions on the command line with the commands:
/cli/joomla.php extension:discover
/cli/joomla.php extension:discover:install
Actual result BEFORE applying this Pull Request
The following info text is displayed:
Postflight executed for test 1
Expected result AFTER applying this Pull Request
The following info text is displayed:
Postflight executed for test 1
Postflight executed for test 2
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Labels |
Added:
PR-5.4-dev
|
||
I discovered the issue when I install on cli. Will see if it only happens there.
| Labels |
Added:
bug
|
||
I can confirm that the problem only exists when you use the cli
php .\cli\joomla.php extension:discover:install
I can confirm that with this PR discover install using the web still works as before AND that it now works correctly from the CLI
PS D:\repos\j51> php .\cli\joomla.php extension:discover:install
Install Discovered Extensions
=============================
! [NOTE] 2 discovered extensions have been installed.
! [NOTE] Postflight executed for test 1
!
! Postflight executed for test 1
PS D:\repos\j51> php .\cli\joomla.php extension:discover
Discover Extensions
===================
! [NOTE] 2 extensions have been discovered.
PS D:\repos\j51> php .\cli\joomla.php extension:discover:install
Install Discovered Extensions
=============================
! [NOTE] 2 discovered extensions have been installed.
! [NOTE] Postflight executed for test 1
!
! Postflight executed for test 2
I have tested this item ✅ successfully on f5f446f
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47637.
I have tested this item ✅ successfully on f5f446f
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47637.
Thanks @brianteeman !! @exlemor do you have a chance to test it also on CLI?
before and after pr
I have tested this item ✅ successfully on f5f446f
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47637.
I have tested this item ✅ successfully on f5f446f
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47637.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47637.
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47637.
| Labels |
Added:
RTC
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-06-05 10:11:04 |
| Closed_By | ⇒ | richard67 |
Thanks @laoneo for this PR, and @brianteeman and @alikon for testing.
@laoneo Hi Allon, during the April 24th, 2026 PR Testing Group session with a Joomla Nightly Build from today, I was NOT able to replicate the BEFORE conditions/issue.
When selecting both plugins (which btw came up automatically on the Discover screen - didn't need to hit the Discover button), when I installed both I got:
info
Postflight executed for test 1
Postflight executed for test 2
×
success
Discover install successful.
Test 2
so based on the testing instructions, we did not match the BEFORE condition.
(if you meant that we should also have had Test 1 after Discover install successful so that it read:
info
Postflight executed for test 1
Postflight executed for test 2
×
success
Discover install successful.
Test 1
Test 2
then OK, but the testing instructions weren't clear for me - sorry ;(
Thanks for the work either way :)