Skip to content

Conversation

@HLeithner
Copy link
Member

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:

  • We need to activate the b/c plugin before upgrading 5.4 to 6.0 (it's not possible in the upgrade process)
  • We would like to force the user to disable the b/c plugin in 5.4 before upgrading (if it site crashes in 5.4 it could be easily recovered by reactivating the plugin, but if it crashes after uninstall in the 6.0 upgrade process there is no way back)
  • Both checks can be done in the pre update checker but only if we have 2 separated plugins

Summary of Changes

  • Create the compat6 plugin
  • Removed the compat plugin
  • Moved already merged changes from old to compat6 plugin
  • Removed all b/c functionally for 5 with the old compat plugin as discussed in the maintainers meeting from February

Testing Instructions

  • New install and test with and without extensions
  • Update from the latest 5.4 nightly build to the patched package or custom update URL created by Drone for this PR, having an extension installed on the 5.4 which will require the compat6 plugin to be enabled on Joomla 6.

Actual result BEFORE applying this Pull Request

Everything works without the old b/c plugin

After updating from 5.4, the old b/c plugin is still there.

Expected result AFTER applying this Pull Request

Everything works with the new b/c plugin

After updating from 5.4, the old b/c plugin is removed.

Link to documentations

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

HLeithner and others added 21 commits April 16, 2025 20:13
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-6.0-dev labels May 15, 2025
@LadySolveig
Copy link
Contributor

I have tested this item ✅ successfully on d8cbde2


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45480.

@richard67
Copy link
Member

I have tested this item ✅ successfully on d8cbde2

Tested update from 5.4 as well as new installation.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45480.

@richard67
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45480.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 25, 2025
@softforge softforge added this to the Joomla! 6.0.0 milestone May 25, 2025
@softforge softforge merged commit fbd724d into joomla:6.0-dev May 25, 2025
4 of 5 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 25, 2025
@softforge
Copy link
Contributor

softforge commented May 25, 2025

Thank you @HLeithner and @richard67 and all who worked on it to make it happen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-6.0-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants