Image replacement is case-sensitive when it should not be #4848

Closed
opened 2026-02-05 09:20:44 +03:00 by OVERLORD · 1 comment
Owner

Originally created by @DanielGordonIT on GitHub (Jul 3, 2024).

Describe the Bug

When replacing an image in the image editor, if the case of the extension differs the replacement fails.

Steps to Reproduce

  1. Upload an image named "file.PNG"
  2. Attempt to replace the image with "newfile.png"
  3. Observe the error

Expected Behaviour

As png and PNG are not indicators of different files, you should be able to replace a file ending in PNG with one ending in png.

Screenshots or Additional Context

No response

Browser Details

No response

Exact BookStack Version

v24.02.2

Originally created by @DanielGordonIT on GitHub (Jul 3, 2024). ### Describe the Bug When replacing an image in the image editor, if the case of the extension differs the replacement fails. ### Steps to Reproduce 1. Upload an image named "file.PNG" 2. Attempt to replace the image with "newfile.png" 3. Observe the error ### Expected Behaviour As `png` and `PNG` are not indicators of different files, you should be able to replace a file ending in PNG with one ending in png. ### Screenshots or Additional Context _No response_ ### Browser Details _No response_ ### Exact BookStack Version v24.02.2
OVERLORD added the 🐛 Bug label 2026-02-05 09:20:44 +03:00
Author
Owner

@DanielGordonIT commented on GitHub (Jul 3, 2024):

Pull request wrapping the relevant segments of code in strtolower() has been created

@DanielGordonIT commented on GitHub (Jul 3, 2024): Pull request wrapping the relevant segments of code in `strtolower()` has been created
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#4848