doc: $conf['picture_ext'] case insensitive #795

Open
opened 2026-10-05 00:17:41 +02:00 by adam · 2 comments
Owner

Thought picture_ext and file_ext were changes to be case insensitive it does not appear anywhere in the config file.
Also they are infect have to be written lower cased in the config otherwise the extension wont work.
Probably should mention it in the config file that all extensions should be written lower cased only

Thought picture_ext and file_ext were changes to be case insensitive it does not appear anywhere in the config file. Also they are infect have to be written lower cased in the config otherwise the extension wont work. Probably should mention it in the config file that all extensions should be written lower cased only <!-- github-sync-id:331576825 -->
Author
Owner

Hi @yonjah

Do you feel like doing a pull-request for that? (the difficulty is to make the pull-request, not to modify the include/config_default.inc.php file)

Hi @yonjah Do you feel like doing a pull-request for that? (the difficulty is to make the pull-request, not to modify the include/config_default.inc.php file) <!-- github-comment-id:396601522 -->
Author
Owner

@plegall could you please explain why not to change config_default.inc.php ?
My initial though was that just changing a comment or two in the file should be enough

@plegall could you please explain why not to change `config_default.inc.php` ? My initial though was that just changing a comment or two in the file should be enough <!-- github-comment-id:396606409 -->
Sign in to join this conversation.
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: adam/Piwigo#795