Skip to content
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

Convert package to be compatible to Joomla 4+ #33

Merged
merged 116 commits into from
Jan 2, 2024

Conversation

parapente
Copy link
Collaborator

@parapente parapente commented Aug 29, 2023

This is a work in progress. It is not ready for production but should be ok to install in an isolated environment for testing. What works:

  • Showing attachments in the editor and in the frontend
  • Adding attachments to articles through the add attachment admin menu option
  • Adding/Editing/Removing attachments using the editor

What doesn't work as expected:

  • Add attachment link in the front end doesn't open in a modal
  • Edit/Remove in the front end when not using the Cassiopeia theme

Needs more testing:

  • Admin utils
  • Adding attachments with an editor other than TinyMCE

What doesn't work:

  • Unit tests

There are a couple of minor problems with translated strings.

Sorry for the large number of commits but there were a lot of changes needed in the codebase and too little documentation from Joomla about how you can easily convert old components/plugins to Joomla 4.

I tried to change all functions marked as deprecated in Joomla 4 to prevent breaking the code when Joomla 5 is released.

NOTE: I cannot find anything about AttachmentsRemapper class. Could it be old unnecessary code that needs deletion?

@jmcameron
Copy link
Owner

@parapente It looks like you are serious about this. Would you like me to give you permissions to change 'master'?

@parapente
Copy link
Collaborator Author

@jmcameron If you are ok with the changes I have made, certainly. I have completely dropped compatibility with older versions of Joomla and it should only work with Joomla 4+ versions. I have a couple of questions I'd like to ask:

  • Do you remember what AttachmentsRemapper class was about? Is it in a still usable plugin that is not included in the package?
  • Should I change the version number of the plugins/component to 4.0 to signify the Joomla compatibility?

@jmcameron
Copy link
Owner

jmcameron commented Sep 3, 2023 via email

@parapente parapente changed the title Convert package to be compatible to Joomla 4+ (WIP) Convert package to be compatible to Joomla 4+ Dec 24, 2023
@parapente parapente marked this pull request as ready for review December 24, 2023 18:01
@parapente
Copy link
Collaborator Author

I updated the title and the status of the merge request to signify that the packages are ready for use in case someone reads this. Merry Christmas!

@JLTRY JLTRY merged commit afb4e68 into jmcameron:master Jan 2, 2024
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.

None yet

3 participants