WordPress.org
Get WordPress

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#41496 closed defect (bug) (fixed)

Media Upload getid3 fatal errors on PHP7

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

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 9 years ago.
41496.diff​ (269.2 KB ) - added by wonderboymusic 9 years ago.

Download all attachments as: .zip

Change History (5)

@MyThemeShop
9 years ago

#1 @wonderboymusic
9 years ago

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

@wonderboymusic
9 years ago

#2 @wonderboymusic
9 years ago

  • Resolution → fixed
  • Status assigned → 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
9 years ago

#41568 was marked as a duplicate.

Note: See TracTickets for help on using tickets.