Opened 3 years ago
Closed 3 years ago
#54162 closed task (blessed) (fixed)
Update GetID3 to version 1.9.21
Reported by: | jrf | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | External Libraries | Keywords: | needs-patch php81 |
Focuses: | Cc: |
Description
GetID3 1.9.21 has just been released and includes preliminary PHP 8.1 support, as well as a variety of bugfixes.
The version included with WP should be updated.
Full details:
- https://github.com/JamesHeinrich/getID3/releases/tag/v1.9.21
- https://github.com/JamesHeinrich/getID3/compare/v1.9.20...v1.9.21
I've NOT reviewed the diff yet, but would strongly recommend an update at the earliest convenience due to the PHP 8.1 fixes being included.
Attachments (1)
Change History (9)
#2
@
3 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
#3
@
3 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Looks like [51900] caused unit test failures:
There were 3 errors: 1) Tests_Media::test_wp_read_audio_metadata_adds_creation_date_with_mp4 Undefined index: created_timestamp /var/www/tests/phpunit/tests/media.php:2557 2) Tests_Media::test_wp_read_video_metadata_adds_creation_date_with_quicktime Undefined index: created_timestamp /var/www/tests/phpunit/tests/media.php:2567 3) Tests_Media::test_wp_read_video_metadata_adds_creation_date_with_mp4 Undefined index: created_timestamp /var/www/tests/phpunit/tests/media.php:2577
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
3 years ago
#7
@
3 years ago
Tracked the test failures to this commit setting the $options_audiovideo_quicktime_ReturnAtomData
property to false
instead of true
by default.
For WordPress, this leads to the data returned by $id3::analyze()
no longer containing the moov
element, which, among other things, includes the creation_time_unix
field that wp_get_media_creation_timestamp()
uses to retrieve the creation time.
Simply setting the property to true
makes the tests pass again, see 54162.diff.
In 51900: