This error comes up in php 5.3.x invironments:

Deprecated function: Function eregi() is deprecated in getID3->getID3() (line 63 of /var/www/d7test/sites/all/libraries/getid3/getid3/getid3.php).

I know this has to do with getID3 and that people should turn there for a solution, but I am pretty sure drupal users will come here first. So, this is to serve as a pointer to the right direction and a way to monitor any news on the matter. It is not a bug of this module, so I am filing it as a support request and since it has to do with php version used it applies to both 7.x and 6.x versions (but I had to choose one).

Here are two links to their support forums dealing with the issue: Use of ereg function & With PHP 5.3 all ereg functions are deprecated

CommentFileSizeAuthor
#8 getid3_886226.patch3.75 KBdrewish
#7 getid3_886226.patch2.43 KBdrewish
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented
Issue tags: +PHP 5.3 compatibility

Both current stable 1.7.9 version of the library as well as the 2.0.0b5 alpha have this issue. A note in the module's front page (pointing to this issue) warning users of this php 5.3.x issue would be nice.

+ added appropriate tag

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented
Status: Active » Postponed

Looks like we should fix it in GetID3 like you mentioned...
http://www.getid3.org/phpBB3/viewtopic.php?f=7&t=983
http://www.getid3.org/phpBB3/viewtopic.php?f=4&t=935

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented

1.8.1 and 2.0.0-b6 are out and are meant to be used with php 5.x. Lets see if that does it...

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Component: Miscellaneous » Documentation

...and it does!

We finally have some new releases of the getID3 library (more than 1 and a half year after the latest 1.7.x was released). It's a bit of a shame, cause php5 has been around since like 2004 and its considered stable(r) 5.2.0 version since 2006.

...actually it must have been 1.8.0 that solved the issue, but I somehow missed its release (2 days before 1.8.1). From the changelog:

1.8.0: [2010-11-23] James Heinrich
    » Changes required for PHP v5.3+ compatability, including:
      - change ereg* functions to preg_* equivalents
      - declare functions static as needed
      note: users of PHP v4.x may need to stay with getID3 v1.7.x
...

I only tested 1.8.1 and I think I am going to stick with it for a while. Someone test 2.0.0-b6 if you have some spare time in your lives and remember to report back here. I believe it should work fine anyways.

So, thanks to James' work we are good to go again ;)

PS: This now is nothing but a documentation issue. Lets just update the info in INSTALL.txt and the module's page that still point to 1.7.9.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented

...btw, I am really surprised that nobody else had this issue (given the fact that this module counts more than 20k setups).

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented
Category: support » task
Status: Postponed » Active

...setting right category and status too.

drewish’s picture

drewish CreditAttribution: drewish commented
Title: Deprecated function eregi() in getID3->getID3() (line 63 /sites/all/libraries/getid3/getid3.php) in php 5.3 » Bump suggested getID3 version to avoid Deprecated function eregi() warning
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Needs review
FileSize
getid3_886226.patch2.43 KB

Going to do this on 7.x first then we can backport.

drewish’s picture

drewish CreditAttribution: drewish commented
FileSize
getid3_886226.patch3.75 KB

Updating the INSTALL.txt as well.

drewish’s picture

drewish CreditAttribution: drewish commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)
drewish’s picture

drewish CreditAttribution: drewish commented
Status: Patch (to be ported) » Fixed

Committed this to 6.x-1.x. Since this was last touched getID3() has put out a 1.9.1 release. We should probably use this as a template and create a new issue.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

davidneedham’s picture

davidneedham
he/him
CreditAttribution: davidneedham commented

Was this ever ported? I still see this in D7 but I don't see any D7 issue related to this.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented
Status: Closed (fixed) » Active

...opening this one merely to bring under the maintainers' radar and get #12 answered.

Also, #10 suggested to open a new issue. Did that ever happen?

  • drewish committed 2a4317c on 7.x-2.x 
    #886226 by drewish: Bump suggested getID3 version to avoid Deprecated...

  • drewish committed 2a4317c on 8.x-1.x 
    #886226 by drewish: Bump suggested getID3 version to avoid Deprecated...