Make WordPress Core

Opened 8 years ago

Closed 2 months ago

#41409 closed enhancement (wontfix)

No point in storing empty EXIF/IPTC data on post meta table

Reported by: geminorum's profile geminorum Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.9
Component: Media Keywords:
Focuses: Cc:

Description

  • Post meta tables are bloated with often not used EXIF/IPTC data for each image attachments.
  • wp_read_image_metadata() generates aperture, credit, camera, caption, created_timestamp, copyright, focal_length, iso, shutter_speed, title, orientation, keywords. and the data often stored as empty key/value pairs on each attachment metadata.
  • I see no problem on filling default/empty values upon retrieving the data.
  • and mentioning the core currently has no way of using stored image_meta.

Attachments (2)

41409.diff (1.6 KB) - added by geminorum 8 years ago.
41409.2.diff (1.6 KB) - added by geminorum 8 years ago.

Download all attachments as: .zip

Change History (5)

@geminorum
8 years ago

@geminorum
8 years ago

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


6 years ago

This ticket was mentioned in Slack in #core-performance by swissspidy. View the logs.


2 months ago

#3 @swissspidy
2 months ago

  • Focuses performance removed
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Consensus from when this was previously looked at was that this patch is not complete, and it might cause issues for plugins relying on the array keys to always be there. The performance impact is likely negligible.

Note: See TracTickets for help on using tickets.