-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
[6.1] Versions for Modules #46772
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: 6.1-dev
Are you sure you want to change the base?
[6.1] Versions for Modules #46772
Conversation
|
@rdeutz please follow the convention to out the joomla version in the PR title - it makes it easier to see the version the pr should be tested against |
|
Please add the version note field |
|
Please add Maximum Versions setting |
|
all is looking good so far |
administrator/components/com_admin/sql/updates/mysql/6.1.0-2026-01-29.sql
Show resolved
Hide resolved
|
For other testers, if you try to test this PR via Joomla Patch Tester even with the latest Nightly Build, you will get the error: Use the Download package instead :) |
|
@brianteeman version note and history limit added |
|
Content versioning is available in both the site and admin so I would expect the same for module versioning. Currently its only enabled for the admin |
|
Tested with core modules in the admin ui and module versioning is working as expected - cool stuff |
I think it is a fair request. I have looked into it and a polite description how module edit is implemented in the frontend is: interesting. I would like to discuss this with the maintainers and don't start implementing it at this stage. I will come back with a decision. |
|
Getting this error on locally while applying the patch |
|
|
I have tested this item 🔴 unsuccessfully on 473ab99 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
This reverts commit 5c4fe50.
|
I have tested this item 🔴 unsuccessfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
@crimle did you run the sql before testing |
No, I did not, sorry. There is no testing instruction that tells me to run SQL before testing. Apart from that, how could I possibly do this on my Test Server? |
|
I have tested this successfully. I used core custom module and a 3rd party slider module. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I have tested this item ✅ successfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
After retesting I can confirm: it works! This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I have tested this item ✅ successfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I have tested this item ✅ successfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
@paternax that is not the intended behaviour. the versions button should not be present if versions is not enabled. can you please retest. Otherwise you should change your success to a failed test |
|
@paternax @brianteeman |
|
@dautrich thats my experience as well which is why I asked for a retest - maybe there is some scenario we didnt consider causing the button to be permanently visible |
|
I have tested this item ✅ successfully on 24ca80a
HOWEVER, and this may be outside of scope or on purpose, the Permissions Tab values/data does NOT get restored (i.e. if you Allow, Deny or Inherit) the values stay the same even after a or multiple restores Personally impressed that Versioning worked even with complex setup + multilingual associations = 🥰 😍 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I have tested this item ✅ successfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I applied the patch, but id=14 is not reported in the database. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
@gioacchino54 |
|
I was able to test this successfully. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
@dusries |
|
I have tested this item ✅ successfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I have tested this item ✅ successfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
@dautrich Sorry, I didn't see that. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I have tested this item ✅ successfully on 24ca80a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
Tested correctly and I can see the module versions. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
I reapplied the PHP Stan fix to get the testing green, I would say we have enough manual tests and it is save to merge. |
|
Thanks to all the testers, RTC. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |
|
@rdeutz are you leaving front end versioning to a new issue? |
Kind of, I will wait till a decision is made and then look what we can do. |
|
I have tested this item ✅ successfully on 0bdb14b This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46772. |

Summary of Changes
This is a base implementation for module versioning.
Testing Instructions
Actual result BEFORE applying this Pull Request
No Versions for modules available
Expected result AFTER applying this Pull Request
Versions for modules available
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