[4.0] Add library support to namespace mapper, other fixes#27479
[4.0] Add library support to namespace mapper, other fixes#27479wilsonge merged 9 commits intojoomla:4.0-devfrom SharkyKZ:j4/namespacemap-library
Conversation
|
@wilsonge How is this PR related to the Prepared Statements project? |
|
@SharkyKZ I can test later but only with Linux. Other testers please report back if Linux or Windows was used so we later can see if both was tested sufficiently. |
|
I have tested this item ✅ successfully on 282c48d This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27479. |
|
@810 Have you tested with webserver on Linux? Or Windows? Or both? |
|
windows only |
|
@810 Thanks for reporting back. Is good to have an overview who tested on what. |
|
I have tested this item ✅ successfully on 282c48d Have also compared newly created autoloader file with backup, using BeyondCompare (my favourite diff tool). A few frontent namespaces have been removed, guess these are the unused ones mentioned in the description. Frontend and backend still work fine. Now we have each one good test on Linux and Windows. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27479. |
|
@SharkyKZ We have 2 good tests, one for each server OS type. Shall I give RTC or do you want more tests? |
|
Have 2 tests so setting RTC. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27479. |
|
Thanks! |
Pull Request for Issue #27459.
Summary of Changes
Refactors
JNamespacePsr4Map::getNamespaces()code to be based on extension type rather than paths.Adds library extension support to namespace mapper.
Removes empty component frontend namespaces.
Removes trailing slash from paths.
Testing Instructions
This should be tested on both Linux and Windows.
Delete
libraries/autoload_psr4.phpfile.Test that Joomla! still works.
Test that
com_mediaworks.Test that
System - Debugplugin works when debug is enabled.Expected result
Works like before.
Documentation Changes Required
IDK.