This post is a summary of the latest weekly Media component meeting, which took place in #core-media on Slack, on February 17, 2017 2000 UTC. The purpose of these meetings are to move priority tasks forward, provide feedback on issues of interest, and review media focused tickets on Trac An open source project by Edgewall Software that serves as a bug tracker and project management tool for WordPress..
Attendees: @joemcgill, @sergeybiryukov, @dnavarrojr, @adamsilverstein, @pputzer, @mikeschroder, @desrosj, @gitlost.
Meeting Time Change
The group decided to move the weekly meeting to Thursdays at 1900 UTC, which is more convenient for participants.
We spent this meeting doing a bug A bug is an error or unexpected result. Performance improvements, code optimization, and are considered enhancements, not defects. After feature freeze, only bugs are dealt with, with regressions (adverse changes from the previous version) being the highest priority. scrub for tickets on the 4.7.3 milestone.
image_downsize assumption changes: Added to 4.7.3 milestone for prioritized discussion.
- #39774: PHP The web scripting language in which WordPress is primarily architected. WordPress requires PHP 5.6.20 or higher 7.1 warning during upload of file with the same name:
Approved for merge from 4.8 to 4.7.3. @sergeybiryukov to handle it.
- #37140: Reset of rotation orientation when image is rotated:
@mikeschroder tested, found fix for issue in tests. Reached out to @sanchothefat re: whether the current number of images being tested can be reduced to not slow down the tests. Otherwise, this looks ready.
- #39516: Media grid upload errors display below the grid:
Still needs testing. @joemcgill would appreciate additional eyes here.
- #39550/#39552: Some Non-image files fail to upload after 4.7.1:
@joemcgill to write up two possible solutions for the problem to address image files not properly validated with GD so that we can profile appropriately. There are performance concerns around adding
finfo_file() to all files uploaded.
- #39875: PDF previews overwrite existing images with the same name:
Worked through this in chat. @gitlost uploaded one approach to the ticket, with @desroj writing up another one based on our conversation in the meeting. Decision being that the filename saves should be handled in the same way that saves happen in the Media Gallery’s image editor. The problem is in https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/image.php#L254, with pre-existing method in https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/image-edit.php#L779.
Our next meeting is scheduled at the new day/time of February 23, 2017 at 1900 UTC.