The getID3() 2.0.0 Thread - PHP5 Only

[Allan Hansen]

The next major version of getID3() is underway. It will require PHP5 and a slightly different syntax from the user.

The 1.7 branch will be supported for some time for PHP4.

Proposed changes (possible very incomplete).

1. Make full use of protected/private/public members and values.
2. Replace 'error' with thrown exceptions.
3. Reduce size of getid3_lib - a lot.
4. Remove redundant code.
5. Create abstract class that all modules extends.
6. Make all constants per class.
7. Remove EmbeddedLookup() if possible.
8. Replace header comment block with PEAR compatible one http://pear.php.net/manual/en/standards.header.php
9. Use extra line feed to seperate sections and increase readablity.
one linefeed between small logical groups
two linefeeds between larger group of small groups
three linefeeds between functions
four linefeeds between classes
A. Use CVS
<strike>[REMOVED]B. Change result array to return complex object instead of multidim array.</strike>
C. Change all modules - give them a pointer/handle to the main getid3 class instead of fd and ThisFileInfo.
D. Get rid of getid3_lib::IncludeDependency()
E. Reduce code complexity by using $cond and FunctionCall().
F. Make full use of multi dim static arrays. Saves lots of memory.
G. Reduce number of functions and lines of code.
H. Replace many !empty($var)/isset($var) with @$var - easier to read, poossible faster.
I. Remove cleanup code. Instead of removing unwanted keys we do not create them in the first place.
J. Make all variable names lower case.

A public alpha version will be released very soon. It will be very limited, only a few modules, but it will show some of the principles mentioned above.

Comments and suggestions are more than welcome

[Allan Hansen]

* Everything related to md5_data and sha1_data moved to seperate module. Saves memory if these features are not used.

* New iconv-replacement module - loads automatically if iconv is not present. Saves memory if iconv is present.

[Allan Hansen]

Proposed changes to default options:

$option_tags_html => false
$option_max_2gb_check => false

[tim]

is there a fix in the new version for the determination of average bitrate of VBR mpeg video that show a playtime very strangely?

[James Heinrich]

is there a fix in the new version for the determination of average bitrate of VBR mpeg video that show a playtime very strangely?

No.

[WaldoMonster]

PHP 5.x has a new MySQL engine that only supports MySQL 4.1 and newer.
MySQL 4.1.3 is still beta.
Why not wait with PHP 5 support until MySQL is out of beta?

[Allan Hansen]

No - there is no mysql requirement/usage in getID3() except for a demo and an optional extention.

[WaldoMonster]

Ok,

Be aware that most projects that use getID3() also use MySQL :wink:

[WaldoMonster]

Good news, MySQL 4.1x is out of beta:
http://lists.mysql.com/announce/226

[koguma]

Open Source Beta = Commercial Code Quality

[Allan Hansen]

Open Source Beta = Commercial Code Quality

I disagree somewhat. I still use PHP 4.3 and MySQL 3.23. Would still be using Apache 1.3 if i was available as standard package in my linux dist.

Anyway, getID3 2.0.0beta is BETA quality software and not suitable for production. The MP3 module still needs a lot of work.