[wp-trac] [WordPress Trac] #58240: wp_read_image_metadata() doesn't handle Exif array values

WordPress Trac noreply at wordpress.org
Fri May 5 05:29:43 UTC 2023


#58240: wp_read_image_metadata() doesn't handle Exif array values
--------------------------+------------------------
 Reporter:  dd32          |       Owner:  joedolson
     Type:  defect (bug)  |      Status:  accepted
 Priority:  normal        |   Milestone:  6.3
Component:  Media         |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------------

Comment (by dd32):

 Replying to [comment:3 joedolson]:
 > I'm inclined to implode the array and store it as comma separated values

 That approach seems sane to me, although I question if it could just be
 stored as a literal array too.. but for compatibility, probably best to
 store it as a single stringy value.

 > although for these numerics, we should assume that there could be
 plugins using this data numerically

 That appears to be the case for the Photo Directory at least, although it
 looks like it's anticipating an array key anyway and imploding those:
 https://meta.trac.wordpress.org/browser/sites/trunk/wordpress.org/public_html
 /wp-content/plugins/photo-directory/inc/photo.php?marks=903#L841

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/58240#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list