Closed (fixed)
Project:
Audio
Version:
4.7.x-1.x-dev
Component:
contrib
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Oct 2006 at 22:48 UTC
Updated:
25 Oct 2006 at 02:31 UTC
Apologies if this is just a misunderstanding on my behalf.
In audio_attach.module, in the function audio_attach_form_alter, there is a little bit of code:
elseif (audio_getid3_isfound(NULL, TRUE)) {
// the getid3 module will display the error for us
}
else {
$form['workflow']["audio_attach_$type"] = array(
'#type' => 'radios', '#title' => t('Attach audio files'),
'#default_value' => variable_get("audio_attach_$type", 0),
'#options' => array(0 => t('Disabled'), 1 => t('Enabled')),
'#description' => t('Should this node allows users to upload an audio file?'),
);
Should there be a ! before the audio_getid3_isfound ?
Since at the moment if audio_getid3_isfound() returns true (i.e. it is found) then you don't get the audio attach fieldset displayed...
Comments
Comment #1
drewish commentedhumm... that is probably a bug. i'll get it patched tomorrow.
Comment #2
drewish commentedi've commited the !, thanks for the bug report.
Comment #3
(not verified) commented