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

drewish’s picture

humm... that is probably a bug. i'll get it patched tomorrow.

drewish’s picture

Status: Active » Fixed

i've commited the !, thanks for the bug report.

Anonymous’s picture

Status: Fixed » Closed (fixed)