Skip to content

Preparing 3.10 for 4.0 content versioning#23479

Merged
zero-24 merged 9 commits intojoomla:3.10-devfrom
Hackwar:j3versioning
Dec 19, 2019
Merged

Preparing 3.10 for 4.0 content versioning#23479
zero-24 merged 9 commits intojoomla:3.10-devfrom
Hackwar:j3versioning

Conversation

@Hackwar
Copy link
Copy Markdown
Member

@Hackwar Hackwar commented Jan 8, 2019

This PR backports a few classes that would be necessary to implement versioning of content items in the same code for extensions in 3.x and 4.0. To support both 3.x-style and 4.0-style versioning, you'd have to implement the 3.x-style first. Good luck with that. To get 4.0-style working, simply let your table class implement the interface VersionableTableInterface (and the method that that defines) and let your item controller and model use the respective Versionable trait. In 3.x, these traits are empty, in 4.0 they contain all the logic necessary for versioning.

@Hackwar
Copy link
Copy Markdown
Member Author

Hackwar commented Jan 8, 2019

The original PR to refactor the versioning feature is #23432.

@ghost ghost added the J4 Issue label Apr 5, 2019
@ghost ghost removed the J4 Issue label Apr 13, 2019
@zero-24 zero-24 added this to the Joomla 3.10.0 milestone Dec 19, 2019
@zero-24 zero-24 merged commit ecf49a6 into joomla:3.10-dev Dec 19, 2019
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Dec 19, 2019

Merging for 3.10 thanks @Hackwar

@Hackwar
Copy link
Copy Markdown
Member Author

Hackwar commented Dec 19, 2019

This has not been merged into 4.0, so it seems not a good idea to merge this into 3.10

zero-24 added a commit that referenced this pull request Dec 19, 2019
zero-24 added a commit that referenced this pull request Dec 19, 2019
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Dec 19, 2019

Done thanks.

@zero-24 zero-24 removed this from the Joomla 3.10.0 milestone Dec 19, 2019
@Hackwar
Copy link
Copy Markdown
Member Author

Hackwar commented Jun 3, 2020

Hey, yeah, so... We actually did merge in this feature shortly before the beta release, so we need to add this to 3.10 again...

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Jun 4, 2020

Can you do a PR and tag me? I'm happy to review and merge as this PR here has been reverted.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants