Closed (fixed)
Project:
Audio
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Aug 2006 at 23:04 UTC
Updated:
29 Aug 2006 at 07:45 UTC
I can upload mp3 with the Audio Module but I don't think the getid3 module is working with it because the Title and Artist tag is always blank and I have to fill that in. It also doesn't read the length, sample rate, bit rate... Everything use to work until I got the last version of the Audio Module that was dated 8/10/2006. I have attached a screen print. The mp3 file I'm uploading has proper tags in it.
| Comment | File | Size | Author |
|---|---|---|---|
| screenprint.gif | 13.53 KB | clashfan |
Comments
Comment #1
drewish commenteddid you make sure to enable the audio_getid3 module?
Comment #2
clashfan commentedYes the getid13 module is enabled. Like I said everything was working fine until I uploaded the newest cvs version.
Comment #3
sunAlso having the same problem with activated audio_getid3.module, downloaded getid3 1.7.7 library and configured path to getid3 library in settings of audio.module.
Uploaded file has proper id3v1 tags.
In difference to clashfan I'm running my sandbox on Windows.
Even with activated getid3 warnings there are no warnings displayed.
Comment #4
sunAlthough ID3 information is not read from my test files, the ID3 tags are written to disk.
btw: The written ID3v2 tags are corrupted - artist and title are shortened to a length of 7 chars and album title is shortened to 4 chars. Track number is also missing. ID3v1 tags are looking fine.
Comment #5
drewish commentedhumm, i can't make any promises about how well the module will work with HEAD. there's been a lot of big commits to Drupal HEAD that i haven't been able to account for. patches would be welcome.
Comment #6
sunbtw: Problems described by me occur in
audio-4.7.0.tar.gz, not HEAD.Comment #7
drewish commentedsun, just to be clear, you're running Drupal 4.7 correct?
Comment #8
sunCorrect, I'm running Drupal 4.7.2. I've also already tried getid3 v1.7.6 for troubleshooting but results are identical.
Comment #9
drewish commentedHumm, I haven't been able to recreate the problem. What version of PHP are you running?
Comment #10
sunPHP Version 4.3.11 on Windows Apache.
Comment #11
fractal commentedI am having the same issue here. I've tested it on debian sarge and ubuntu 6.06 with drupal 4.7.3 and have the same problem. I haven't got that problem before 10 aug. I think after 10 aug the audio-4.7.0.tar.gz changed. Is there a way to download that version of audio-4.7.0 (before 10 aug, I deleted it accidentally) which worked fine with plain install of drupal 4.7.3
Comment #12
drewish commentedfractal, same question, what version of PHP are you using? you can always pull out old versions of a module from CVS.
Comment #13
fractal commentedMy version is PHP 4.3.10(-16), Debian Sarge 3.1, getid3 1.7.7 .I change the permissions of drupal installation to default apache user (www-data). And for cvs I will check the command's syntax.
Comment #14
drewish commentedokay, it looks like this is PHP4 specific. after i broke down and installed an old version i was able to recreate the problem. i'll try to get a patch posted shortly.
Comment #15
drewish commentedokay, it turns out that module_invoke_all() will pass parameters by reference under PHP5 but not PHP4. I've added audio_invoke_audioapi() to work around this.
Comment #16
clashfan commentedI'm still having the same problems(no title, artist, bitrate...). Maybe I downloaded the wrong file? Can you provide a direct link to the download?
Comment #17
clashfan commentedNevermind I found the download. Works great now. Thanks for the fix.
Comment #18
drewish commentedwell, it kind of depends on when you downloaded it. looking at the project's release page it looks like a new version was built a few minutes before your bug was posted, but due to timezones that could be way off. i just double checked the contents of the 4.7 package and it's got my most recent changes. maybe you could try it again?
Comment #19
drewish commentedgreat! glad to hear it's working. if you run into any other bugs please open a new issue.
Comment #20
fractal commentedGreat! It's working here too. Thank You.
Comment #21
(not verified) commented