Opened 4 years ago
Closed 4 years ago
#54162 closed task (blessed) (fixed)
Update GetID3 to version 1.9.21
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| 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
@
4 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
#3
@
4 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.
4 years ago
#7
@
4 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: