Skip to content

[5.3] File extensions #45716

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

Open
wants to merge 4 commits into
base: 5.3-dev
Choose a base branch
from
Open

[5.3] File extensions #45716

wants to merge 4 commits into from

Conversation

brianteeman
Copy link
Contributor

The default values for file extensions were out of sync. No idea why we have it in so many places but at least now they should all match

code review

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

The default values for file extensions were out of sync. No idea why we have it in so many places but at least now they should all match

code review

Signed-off-by: BrianTeeman <brian@teeman.net>
Signed-off-by: BrianTeeman <brian@teeman.net>
@ceford
Copy link
Contributor

ceford commented Jul 28, 2025

Could you explain what you mean by out of sync? I notice that some of the revised lists are not in alphabet order. Is that significant? The sql statements are difficult to read in the diff.


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

@brianteeman
Copy link
Contributor Author

Could you explain what you mean by out of sync?

the lists do not match - they should

@RickR2H
Copy link
Member

RickR2H commented Jul 31, 2025

I have tested this item ✅ successfully on 3841bed


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

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

Successfully merging this pull request may close these issues.

5 participants