Posted by drewish on September 6, 2007 at 7:30pm
| Project: | getID3() |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | major |
| Assigned: | hswong3i |
| Status: | needs review |
Issue Summary
I think you'll find that digging thorough the audio_getid3.module in the audio module would be a good source for code for dealing with getid3. i've got a couple of years worth of time in dealing with it and i'd hate to see you duplicate it.
Comments
#1
Audio_getid3.module assisted greatly when working on this. Is there anything else that could be abstracted away from audio_getid3.module that would help other modules using GetID3?
#2
Moving the GetID3 module over to Libraries would allow us to easily change the path..... Otherwise I think we should close this issue.
#3
Libraries still needs work. They don't have a stable release so I'm not in any hurry to migrate to it.
#4
getID3() with Drupal 7.x Library API integration. This will also works within installation profile, and able to auto detect the default value of getID3 library path. Fully tested with Drupal 7.x installation profile development.
Moreover, this patch also simplify some dummy error message handling, plus coding style cleanup with coder.module.
Finally, it also synchronize coding style with that of patching colorbox.module with Drupal 7.x Library API support (http://drupal.org/node/989590).
#5
I committed some of the comments and the t() -> st() fix in the .install file. Thanks!
However, I don't understand why you haven't fully ported getID3() to Libraries API. The patch above only uses the libraries_get_path() function if the the Libraries API is installed. The patch doesn't define any of its hooks or declare it as a dependency.
#6
This is because during installation libraries API may not yet successfully load in and so if we call it directly, the call will be failed. Therefore we need to manually include libraries.module.
On the other hand, libraries API just give a hand for us to figure out the correct path of library, and it is only useful IF user are not placing the files under module's path; besides this, nothing at all. So I would like to suggest NOT with enforced dependency with libraries API since it is not a most ;-)
Patch reroll via GIT 7.x-1.x.