-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.4] Backward Compatibility Module #46424
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 5.4-dev
Are you sure you want to change the base?
Conversation
|
Its missing sql to add it to the extensions and assets tables on update |
@brianteeman Assets table we never insert on updates and leave that to the asset being created when the settings are saved. But with the extensions table you are right. |
administrator/modules/mod_backward/src/Dispatcher/Dispatcher.php
Outdated
Show resolved
Hide resolved
administrator/modules/mod_backward/src/Dispatcher/Dispatcher.php
Outdated
Show resolved
Hide resolved
Doesnt that lead to missing assets |
@brianteeman Yes, but that's not a problem as modules inherit from the component (part before the first dot in the asset name), so when you have no asset and now edit the module settings and save, the asset will be created exactly in that way. So when we added modules in update SQL scripts in past, we always added the extension record with asset id zero to the extensions table. |
phpstan Co-authored-by: Richard Fath <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
|
@alikon In the update SQL script we should insert things only if they don't exist yet. For this you can find some examples if you search for "WHERE NOT EXISTS" in the 5.4-dev branch. This will make sure that the extension and module will not be inserted multiple times when the script runs multiple times, e.g. after a failed update. Another thing is that if this PR counts as a new feature, it would have to go into 6.1-dev and be modified for checking the J6 and the J7 plugins. But a J7 b/c plugin will come very likely not with 6.4, if ever. We have to discuss that in the maintainers team. |
administrator/components/com_admin/sql/updates/mysql/5.4.0.2025-11.08.sql
Outdated
Show resolved
Hide resolved
…5-11.08.sql Co-authored-by: Richard Fath <[email protected]>
Pull Request for Issue #46400 .
Summary of Changes
This module displays the which Joomla! Backward Compatibility plugins are enabled and is intended to be displayed in the 'status' position.
Testing Instructions
use the Prebuilt package of this pr
Actual result BEFORE applying this Pull Request
N/A
Expected result AFTER applying this Pull Request
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