No Code Attached Yet
avatar thetexian
thetexian
19 Apr 2024

Steps to reproduce the issue

File named IMG_455.JPG
In media manager edit mode, resized and cropped. Upon save, nothing changed. No messages, no acknowledgement.

Expected result

Expected result is change to image

Actual result

No change. No message

System information (as much as possible)

J 5.0.3 Media Manager

Additional comments

When file renamed to IMG_455.jpg, worked as designed
Media manager doesn't like capitalized filetypes apparently.

avatar thetexian thetexian - open - 19 Apr 2024
avatar thetexian thetexian - change - 19 Apr 2024
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 19 Apr 2024
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Apr 2024
avatar brianteeman
brianteeman - comment - 19 Apr 2024

I just added this for you to the relevant open issue. #43220

avatar dgrammatiko
dgrammatiko - comment - 20 Apr 2024

Should be fixed with #43295

avatar richard67 richard67 - close - 20 Apr 2024
avatar richard67
richard67 - comment - 20 Apr 2024

Closing as fixed with #43295 . Please test.

avatar richard67 richard67 - change - 20 Apr 2024
Status New Closed
Closed_Date 0000-00-00 00:00:00 2024-04-20 11:36:18
Closed_By richard67
avatar brianteeman
brianteeman - comment - 20 Apr 2024

sorry to say but 43295 does not fix this

avatar richard67 richard67 - change - 20 Apr 2024
Status Closed New
Closed_Date 2024-04-20 11:36:18
Closed_By richard67
avatar richard67 richard67 - reopen - 20 Apr 2024
avatar dgrammatiko
dgrammatiko - comment - 21 Apr 2024

FWIW it's not the filename the problem but the capitalized extension. I thought the mentioned PR would fix this but I had to revert some code when I realized that the canvas element has some inconsistent functionality across browsers and also realizing that introducing more formats for the editing part is a major change that can't happen before 6.0 or it would break all existing plugins.

avatar dgrammatiko
dgrammatiko - comment - 21 Apr 2024

Please test: #43325

avatar richard67 richard67 - change - 21 Apr 2024
Status New Closed
Closed_Date 0000-00-00 00:00:00 2024-04-21 17:07:02
Closed_By richard67
avatar richard67 richard67 - close - 21 Apr 2024
avatar richard67
richard67 - comment - 21 Apr 2024

Closing as having a pull request. Please test #43325 . Thanks in advance.

Add a Comment

Login with GitHub to post a comment