search

jmcameron / attachments

Attachments Extension for Joomla 3, 4, and 5
GNU General Public License v3.0
12 stars 19 forks source link

Reload iframe content before closing the modal #48

Closed parapente closed 2 months ago

parapente commented 2 months ago

Possible fix for #41. Can you test @JLTRY ?

JLTRY commented 2 months ago

@parapente it works , you can merge it I will add my other modifications (iframe size, compatibility for jomla_5.0, versions , history of modifications in an other pull request after your merge

parapente commented 2 months ago

ok @JLTRY thanks! Before you add your modifications I wanted to ask you about the check you added for restrict_uploads_extensions. I cannot find any resource for changing restrict_uploads_extensions to upload_extensions. Can you please link where you found this change? Even running grep on the source of Joomla 5 I cannot find any mentions of upload_extensions. It seems to be still restrict_uploads_extensions. It used to be upload_extensions in Joomla 2 & 3 I think.

parapente commented 2 months ago

You can push the rest of the changes to master.