Make WordPress Core

Opened 4 months ago

Closed 3 months ago

#60946 closed defect (bug) (wontfix)

Images Being Renamed in 6.5

Reported by: roaming1's profile roaming1 Owned by: joedolson's profile joedolson
Milestone: Priority: normal
Severity: normal Version: 6.5
Component: Upload Keywords: close 2nd-opinion
Focuses: Cc:

Description

Wordpress 6.5 renaming uploaded images according to the image comment data. Reverting to 6.4.3 fixes the problem. I am using default (no plugins and default theme) instillation on local host with both versions. 6.5 is displaying the same behavior on live site.

Attachments (2)

image upload name changes versions.jpg (42.8 KB) - added by roaming1 4 months ago.
testing image.jpg (31.5 KB) - added by roaming1 4 months ago.
Testing Image

Download all attachments as: .zip

Change History (10)

#1 @joedolson
4 months ago

  • Owner set to joedolson
  • Status changed from new to accepted

This could be related to changes in https://core.trac.wordpress.org/changeset/57267. That did effect the fallback condition to use UserComment, although only in the case where there's no title. Might need to review that logic to check.

Can you provide the image from this upload, @roaming1?

@roaming1
4 months ago

Testing Image

#2 @krupalpanchal
4 months ago

I've tried uploading the image. Found this metadata of the image online. Is there any issue with it:

FileSize: 31 KiB
FileModifyDate: 2024-04-08T04:26:04.000+00:00
FileAccessDate: 2024-04-08T04:26:04.000+00:00
FileInodeChangeDate: 2024-04-08T04:26:04.000+00:00
FileType: JPEG
FileTypeExtension: jpg
MIMEType: image/jpeg
JFIFVersion: 1.01
ExifByteOrder: Little-endian (Intel, II)
Orientation: Horizontal (normal)
XResolution: 96
YResolution: 96
ResolutionUnit: inches
YCbCrPositioning: Centered
ExifVersion: 0231
ComponentsConfiguration: Y, Cb, Cr, -
FlashpixVersion: 0100
ColorSpace: Uncalibrated
ExifImageWidth: 600
ExifImageHeight: 600
UserComment: xr:d:DAFNOlA-gRc:159,j:5850599947607112990,t:24040615
Title: rottweiler picture facts about rottweilers petrage - Kamchatka Sled Dog
: 2
Author: Jae Cee
CreatorTool: Canva
ImageWidth: 600
ImageHeight: 600
EncodingProcess: Baseline DCT, Huffman coding
BitsPerSample: 8
ColorComponents: 3
YCbCrSubSampling: YCbCr4:2:0 (2 2)
ImageSize: 600x600
Megapixels: 0.36

#3 @roaming1
4 months ago

Stock image downloaded from Canva-was not manipulated after download.

#4 @joedolson
4 months ago

  • Keywords close 2nd-opinion added

One of the main problems here is that Canva appears to be putting inappropriate information in the
UserComment field.

It does have IPTC data, but doesn't have an APP13 marker, so WordPress doesn't look at the IPTC data.

In the past, there was no consideration of the UserComment exif data, so the image was uploaded with the file name as the title. Now that the UserComment data is also taken into consideration, this is being loaded with that data instead.

In my opinion, this behavior is correct and as expected for this file - the problem is really with the file. If it either used the UserComment field correctly or had an APP13 marker, it would behave as expected.

Marking this as close and 2nd opinion, because this is a pretty complicated area and I want to make sure somebody else gets a look at it as well.

#5 @roaming1
4 months ago

Interesting, I have been using Canva for several years with WordPress. Upon examination of previous images uploaded-they all have comment data but were never renamed until the changes in WP 6.5. I will reach out to Canva to see if this data can be eliminated or I will have to strip images manually before upload.

This ticket was mentioned in Slack in #core-media by joedolson. View the logs.


4 months ago

#7 @joedolson
4 months ago

What Canva should do is add an APP13 marker, so that the title field is pulled from the IPTC data. Adding this data into the UserComment field is wrong, but the title is present, and could be used *if* there was a marker to indicate that IPTC data was present.

This is caused by the changes in 6.5; I'm not disputing that; but I think that it's a problem with the image.

#8 @antpb
3 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from accepted to closed

Given that this is metadata that is out of our control, and because WordPress is using these meta fields to their specification, I don't see any possible actions for this ticket. There have been a couple reports now with reports going to Canva where the bug originates. I think we'll need to wait for them to have a stance on it, but as it looks today they are using the metadata in a nonstandard way and media maitainers agree WordPress should not accommodate or encourage usage of the metadata in this way.

Going to close, but if Canva has any more info on why the data is valid, we can reconsider.

Note: See TracTickets for help on using tickets.