Make getID3() library location user assignable
mikey_p - May 17, 2007 - 07:43
| Project: | Media Field |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
Hi, I'm doing some install profile work and trying to figure out a way to make my system easiest to upgrade, and it works much easier if the location of the getid3 lib is user assignable. For example, I try to keep all non-core files to either sites/all, sites/site.domain, or in profiles/profilename. (places where they don't get mixed with core files, such as in /misc.
I've tested the patch and unless the settings are changed at the admin page, it works exactly as it did previously.
-Mike
| Attachment | Size |
|---|---|
| mediafield_0.patch | 2.12 KB |
#1
I will review and commit your patch shortly. It looks good at the first site. No problems to have such option.
There were many requests from people who had troubles with locations... which is strange, I thought misc/lib is a good option because Drupal has no special directory for 3d party components so misc/lib looked a good one.
#2
I just contributed the getID3() module, which provides an API to facilitate an installation of getID3. MediaField should move to start using this instead of just assuming that getID3 is installed in drupal/misc/lib/getid3.
The benefits to using this getID3 module is that it abstracts the getID3 functionality away from MediaField, making it accessible by other modules. It also gives additional functionality, including a requirements warning in admin/logs/status when getID3 is not installed, an easy way to include getID3 in other modules via a single call to
getid3_load, and (eventually) the ability for the user to change the location where getID3 is installed.This API is still young, so some additional functionality will become available in the future. I'll send in a patch to MediaField in the next two days which will use this instead of its assuming getID3 is installed in 'misc/lib/getid3'.
#3
This patch will make MediaField use the getID3 Drupal API Module instead of assuming getID3 is installed in drupal/misc/lib/getid3.
#4
#5
Committed to HEAD (http://drupal.org/cvs?commit=123615) with changes due to bug in getId3 see http://drupal.org/node/275151 for details.
#6
#7
#8
Automatically closed -- issue fixed for two weeks with no activity.