I'm running Drupal 5.1 and Audio 5.x-0.3.
I go to Administer › Site configuration > Audio settings and change the players settings for both WAV and MP3.
When I go back to an existing audio node, or create a new one, I still have a 'click to play' link, which when clicked goes to a black page to play the audio clip.
Comments
Comment #1
joachim commentedWhat seems to be happening is that at line 1362,
$player = audio_get_players('name', $playername);
nothing is getting returned. The value of $player is empty, though the value of $playername is correctly set.
I don't understand what's going on in the function audio_get_players to try debugging any further.
Comment #2
drewish commentedwhat are you getting back from
_audio_player_build_list()?Comment #3
joachim commentedI just wrote a really long reply and Drupal ate it because my login cookie had expired :(
Basically, that function was returning false.
I chased things around, and eventually found that the fileformat for the node wasn't set.
Using phpmyadmin I found that this simply wasn't set in the 'audio' table -- the field for that node was empty.
I tried creating a second audio node, and this one worked. What had changed was that in the meantime, I'd installed the getid3 library.
I removed getid3, created a 3rd node, and this node had the original problem.
So the cause appears to be getid3 not installed. But the audio module doesn't state this as a requirement.
Comment #4
joachim commentedCorrection -- the Audio getID3 *module* needs to be enabled to fix the problem.
In other words, without that module, audio.module doesn't seem able to detect the format of a file.
Comment #5
drewish commentedright, it needs getid3 to detect it or you can specify it manually.
Comment #6
joachim commentedI don't see where to specify it manually on the add audio content page.
Is there something about this in the documentation?
Comment #7
drewish commentedDisable audio_getid3 and you'll get a bunch of edit boxes under "Audio File Info". One of those is "Format:" and you'd have to type in mp3. Not the most obvious I'll admit so we'll turn this into a feature request to make it clearer how to configure the node to work with a player.
Comment #8
drewish commentedComment #9
joachim commentedThere's no format property when I create a new node. Only when I edit it.
This means that unless you know (ie, unless the documentation covers it), you can't find this -- I didn't.
I would recommend:
- warning on the players selection page that without audiogetid3 enabled, you may only get the default players unless you manually specify format. I tore my hair out changing the settings here and getting no change of audio player.
- *strongly* suggesting audiogetid3 in both the module admin page description for audio.module and the documentation
Comment #10
(not verified) commentedComment #11
joachim commentedCould this be reopened please?
There's no bug in the code, but there are omissions in the documentation. That has the same effect as a bug: the user can't do something they want to do.
I lost about a week trying to figure this one out, so I really think it needs addressing.
Comment #12
varr commented- Omitted -
Unrelated - created as a separate distinct issue.
Comment #13
drewish commented