User tests: Successful: Unsuccessful:
In the today's maintainer meeting we came to the conclusion that making a new plugin with a new name makes it easier
for us to update and maintain in the future.
Considerations:
Everything works without the old b/c plugin
After updating from 5.4, the old b/c plugin is still there.
Everything works with the new b/c plugin
After updating from 5.4, the old b/c plugin is removed.
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
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_admin Language & Strings SQL Installation Postgresql Front End Plugins |
Labels |
Added:
Language Change
PR-6.0-dev
|
Category | Administration com_admin Language & Strings SQL Installation Postgresql Front End Plugins | ⇒ | Administration com_admin Language & Strings Repository NPM Change SQL Installation Postgresql Front End Plugins |
Labels |
Added:
NPM Resource Changed
|
Category | Administration com_admin Language & Strings SQL Installation Postgresql Front End Plugins Repository NPM Change | ⇒ | Administration com_admin Language & Strings Repository NPM Change SQL Installation Postgresql Libraries Front End Plugins |
Upgrade testing needs a separate PR which installs the extension in 5.4, so can't be tested yet ( @richard67 when done please update this section)
@HLeithner PR for 5.4 is #45371 . Question:
I did my PR in the same way as PR PR #41738 did it on 4.4 for the 5 plugin. That means the plugin does not have any configuration parameters set in the database.
That would not be a problem when the default values which the 6 plugin uses when reading its configuration are always so that all functionality is enabled.
Is this the case?
If not: Shall I add the right configuration parameters in the database to my PR?
But this would mean we have to maintain them there and in 6.0-dev here when we add some new ones.
by default, all parameters will be active in the code, so no parameters in the sql config is needed.
thanks for the PR
@HLeithner Is the file name of file "plugins/behaviour/compat6/src/Extension/Compat.php" right? Or shouldn't it be "Compat6.php"
Update: I've just tested with my PR, and yes, the file name has to fit to the class name, so it has to be "Compat6.php".
@HLeithner Is the file name of file "plugins/behaviour/compat6/src/Extension/Compat.php" right? Or shouldn't it be "Compat6.php"
Update: I've just tested with my PR, and yes, the file name has to fit to the class name, so it has to be "Compat6.php".
@HLeithner I've allowed myself to rename the file in your PR.
thanks
Labels |
Added:
Release Blocker
|
Please enable the new compat6 plugin during installation in system-tests (#45414).
@heelc29 Sure you want to enable it? The PR you refer to disables the J5 plugin. In this PR here that should be changed to disable the new J6 plugin.
Sorry I mean DISABLE ;)
@heelc29 We will handle that after the upmerge when we update this PR to the upmerged changes in the base branch and solve the conflicts.
@HLeithner Now as @laoneo 's PR #44925 has been merged, this PR here has conflicts which can't be solved on GitHub. I think you know how to solve them. If not, let me know.
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-05-14 22:16:57 |
Closed_By | ⇒ | HLeithner |
@HLeithner You also have to change the extensions helper from "compat" to "compat6" here: https://github.com/joomla/joomla-cms/blob/6.0-dev/libraries/src/Extension/ExtensionHelper.php#L176