Skip to content

Conversation

@rdeutz
Copy link
Contributor

@rdeutz rdeutz commented Jan 29, 2026

Summary of Changes

This is a base implementation for module versioning.

Testing Instructions

  1. Apply patch
  2. Make sure Database changes are executed, check if you have an entry for modules (id=14) in #__content_types
  3. Enable Versioning in Module component configuration
  4. Do changes on modules and switch versions, back and force
  5. Try different module type also 3rd part modules

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

@brianteeman
Copy link
Contributor

@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

@rdeutz rdeutz changed the title Feature/module versions [6.1] Feature/module versions Jan 29, 2026
@rdeutz rdeutz added the Feature label Jan 29, 2026
@rdeutz rdeutz changed the title [6.1] Feature/module versions [6.1] Versions for Modules Jan 29, 2026
@brianteeman
Copy link
Contributor

Please add the version note field

@brianteeman
Copy link
Contributor

Please add Maximum Versions setting

@brianteeman
Copy link
Contributor

all is looking good so far

@exlemor
Copy link

exlemor commented Jan 29, 2026

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:
The file marked for modification does not exist: phpstan-baseline.neon

Use the Download package instead :)

@tecpromotion tecpromotion added the PBF Pizza, Bugs and Fun label Jan 29, 2026
@rdeutz
Copy link
Contributor Author

rdeutz commented Jan 29, 2026

@brianteeman version note and history limit added

@brianteeman
Copy link
Contributor

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

@brianteeman
Copy link
Contributor

Tested with core modules in the admin ui and module versioning is working as expected - cool stuff

@rdeutz
Copy link
Contributor Author

rdeutz commented Jan 29, 2026

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

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.

@vishalchincholi1
Copy link

Getting this error on locally while applying the patch
The file marked for modification does not exist: phpstan-baseline.neon

@brianteeman
Copy link
Contributor

@vishalchincholi1

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:
The file marked for modification does not exist: phpstan-baseline.neon

Use the Download package instead :)

@manojLondhe
Copy link
Contributor

I have tested this item 🔴 unsuccessfully on 473ab99

Getting error while applying patch as The file marked for modification does not exist: phpstan-baseline.neon


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

@brianteeman
Copy link
Contributor

This reverts commit 5c4fe50.
@crimle
Copy link

crimle commented Jan 30, 2026

I have tested this item 🔴 unsuccessfully on 24ca80a

The button [Versions] appears correctly, but no versions are available after submitting several changes to the module. I tested modules of the type «Custom», «Tags popular» and «Articles».


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

@brianteeman
Copy link
Contributor

@crimle did you run the sql before testing

@crimle
Copy link

crimle commented Jan 30, 2026

@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?

@idefax
Copy link

idefax commented Jan 30, 2026

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.

@crimle
Copy link

crimle commented Jan 30, 2026

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.

@crimle
Copy link

crimle commented Jan 30, 2026

After retesting I can confirm: it works!


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

@dautrich
Copy link

I have tested this item ✅ successfully on 24ca80a

Tested with a small module "Custom HTML" and with a third party module ("DWD Weather Warning" from Thomas Hunziker).


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

@paternax
Copy link

I have tested this item ✅ successfully on 24ca80a

The ‘Versions’ button is also displayed when versioning is deactivated. This is somewhat confusing, as it does not occur to you that versioning still needs to be activated.


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

@brianteeman
Copy link
Contributor

@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

@dautrich
Copy link

dautrich commented Jan 30, 2026

@paternax @brianteeman
On my test site, the "Versions" button is only displayed, when versioning is enabled in the configuration. I just tested this again.

@brianteeman
Copy link
Contributor

@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

@exlemor
Copy link

exlemor commented Jan 30, 2026

I have tested this item ✅ successfully on 24ca80a

I have successfully tested this with Olivier watching during PBF 2026 Friday... @rdeutz - please validate if this passes based on below:
Tested this on 2 installations, same results!

  1. Applying the PR, did NOT update the SQL for me, had to add SQL code into database via phpMyAdmin
  2. With core module and 3rd party extension changing 5-6 fields in multiples tabs including playing with Associations on a multi-lingual site, the PR does what it claims and shows/restores previous versions of the Module

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.

@idefax
Copy link

idefax commented Jan 31, 2026

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.

@gioacchino54
Copy link

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.

@dautrich
Copy link

@gioacchino54
As mentioned in the testing instructions, you need to apply the SQL script from the patch manually, i.e. via phpMyAdmin.

@dusries
Copy link

dusries commented Jan 31, 2026

I was able to test this successfully.
However, I did not see any (id=14) in #__content_types in the database.
Not even after clearing the browser cache.


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

@dautrich
Copy link

@dusries
As mentioned in the testing instructions, you need to apply the SQL script from the patch manually, i.e. via phpMyAdmin.

@Razzo1987
Copy link
Contributor

Razzo1987 commented Jan 31, 2026

I have tested this item ✅ successfully on 24ca80a

Tested with multiple modules.

eg:
image


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

@gioacchino54
Copy link

I have tested this item ✅ successfully on 24ca80a

Successfully applied


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

@gioacchino54
Copy link

@dautrich Sorry, I didn't see that.


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

@gioacchino54
Copy link

I have tested this item ✅ successfully on 24ca80a

Successfully applied


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

@gioacchino54
Copy link

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.

@rdeutz
Copy link
Contributor Author

rdeutz commented Jan 31, 2026

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.

@rdeutz
Copy link
Contributor Author

rdeutz commented Jan 31, 2026

Thanks to all the testers, RTC.


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 31, 2026
@brianteeman
Copy link
Contributor

@rdeutz are you leaving front end versioning to a new issue?

@rdeutz
Copy link
Contributor Author

rdeutz commented Jan 31, 2026

@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.

@torstenhoppe
Copy link

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.

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

Labels

Feature PBF Pizza, Bugs and Fun PR-6.1-dev RTC This Pull Request is Ready To Commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.