User tests: Successful: Unsuccessful:
Pull Request for Issue #16616
Make sure you delete the old autoload_psr4.php in libraries before.
Check that libraries/autoload_psr4.php is generated and paths are loaded.
Installation works
Installation is broken on default SQL credentials
Probably
Status | New | ⇒ | Pending |
Category | ⇒ | Libraries |
Labels |
Added:
?
|
Installation works fine now, autoload_psr4.php is generated properly. I tried to exclude Component from JLoader loadByExtension method https://github.com/joomla/joomla-cms/blob/4.0-dev/libraries/loader.php#L556 and everything still works well. Some notes if we decide to go with this approach:
There are still some components like com_categories, com_cpanel, com_fields... not registered in autoload_psr4.php
Do we want to support autoload for cli applications? Asking this question because we still register model class manually here https://github.com/joomla/joomla-cms/blob/4.0-dev/cli/finder_indexer.php#L238-L239
There are still some components like com_categories, com_cpanel, com_fields... not registered in autoload_psr4.php
They are missing from the install sql script and XML files - easy to fix.
Do we want to support autoload for cli applications? Asking this question because we still register model class manually here https://github.com/joomla/joomla-cms/blob/4.0-dev/cli/finder_indexer.php#L238-L239
Yes - but unsure where the best place to do that is. People may have to implement that manually?
They are missing from the install sql script and XML files - easy to fix.
I know. Just wanted to point out so that we won't forget
Yes - but unsure where the best place to do that is. People may have to implement that manually?
I am unsure, too as I haven't written any Cli application. I think it is inconsistent when we have autoload support in Site/Administrator but not in Cli.
The problem is all of our existing CLI scripts are all separate applications without any common structure and the last proposal to make something equivalent to a JApplicationCms
for CLI basically went nowhere because everything that is not serving HTML responses (JSON views, CLI) is still treated as a second class citizen in Joomla.
There are still some components like com_categories, com_cpanel, com_fields... not registered in autoload_psr4.php
They already autoload fine without this autoload_psr4.php. If we want to drop the existing autoload code for the "Joomla Namespace" in favor of this autoload_psr4 file, then we have to add the namespace to the database of those extensions and this script will pick it up. Otherwise it's not needed.
@joomdonation yep, but one step after another :-) Than we can continue updating the entries in the #__extensions table
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-06-15 11:39:01 |
Closed_By | ⇒ | wilsonge |
Merged so that we have a working installer for people
@yvesh PR for #16616?