Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#41496 closed defect (bug) (fixed)

Media Upload getid3 fatal errors on PHP7

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

Description

Hello,

When uploading mp3 file ( https://ia802605.us.archive.org/7/items/HayseedRag/HayseedRag--theDizzyTrio_vbr.mp3 ) on PHP 7.1.1, getid3 is throwing fatal error - [] operator not supported for strings.

Not sure if whole getid3 library should be updated but we have added this commit - https://github.com/JamesHeinrich/getID3/pull/93/commits/7f069ac25b3d83e849aeefe4f7da660f358b6b25 in attached patch fixing just those errors.

Attachments (2)

getid3_erors.diff (103.2 KB) - added by MyThemeShop 8 years ago.
41496.diff (269.2 KB) - added by wonderboymusic 8 years ago.

Download all attachments as: .zip

Change History (5)

#1 @wonderboymusic
8 years ago

  • Milestone changed from Awaiting Review to 4.9
  • Owner set to wonderboymusic
  • Status changed from new to assigned

@wonderboymusic
8 years ago

#2 @wonderboymusic
8 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 41196:

Media: update the getID3 library to version 1.9.14 to avoid fatal errors in PHP7.

Props MyThemeShop for the initial patch.
Fixes #41496.

#3 @SergeyBiryukov
8 years ago

#41568 was marked as a duplicate.

Note: See TracTickets for help on using tickets.