enable file attachement for specific node types

I got my hand on this.

To add radios on admin/content/type/x:
in function webfm_form_alter, add

  if ($form_id == 'node_type_form') {
    $form['workflow']['webfm_attach'] = array(
      '#type' => 'radios',
      '#title' => t('Attach files'),
      '#default_value' => variable_get('webfm_attach_'. $form['#node_type']->type, 0),
      '#options' => array(0 => t('Disabled'), 1 => t('Enabled')),
      '#description' => t('Should this node allows users to upload and attach files ?'),
      );
  }

To show the form on allowed node types only, replace the line
if (user_access('attach WebFM files') && $form['type']['#value'] .'_node_form' == $form_id && variable_get("wfm_attach_$node->type", TRUE)) {
by

 _get("wfm_attach_$node->type", TRUE)) {
    if (user_access('attach WebFM files') && $form['type']['#value'] .'_node_form' == $form_id && variable_get("webfm_attach_$node->type", 0)) { 

in the same function.

It seems the wfm_attach_$node->type veriable is some junk of previous version, or whetever. I didn't look at all the code.

I don't post a patch because my local version has some other changes (it only shows one collapsible form showing attached files & file browser).

Thanks,

Charly

hi,

sorry for delay,

this patch can be applied on developement version. it just keeps wfm_attach_ variable (cause used elsewhere).

  function webfm_form_alter($form_id, &$form) {
   global $base_url;
 
+  if ($form_id == 'node_type_form') {
+    $form['workflow']['wfm_attach'] = array(
+      '#type' => 'radios',
+      '#title' => t('WebFM Attachments'),
+      '#default_value' => variable_get('wfm_attach_'. $form['#node_type']->type, 1),
+      '#options' => array(0 => t('Disabled'), 1 => t('Enabled')),
+      '#description' => t('Should this node allow users to upload and attach files via WebFM?'),
+    );
+  }
   if (isset($form['type'])) {
-    if ($form['type']['#value'] .'_node_settings' == $form_id) {
-      $form['workflow']['wfm_attach_'. $form['type']['#value']] = array(
-        '#type' => 'radios', '#title' => t('Webfm Attachments'), '#default_value' => variable_get('wfm_attach_'. $form['type']['#value'], 1),
-        '#options' => array(t('Disabled'), t('Enabled')),
-      );
-    }
-
     $node = $form['#node'];
-    if (user_access('attach WebFM files') && $form['type']['#value'] .'_node_form' == $form_id && variable_get("wfm_attach_$node->type", TRUE)) {
+    if (user_access('attach WebFM files') && $form['type']['#value'] .'_node_form' == $form_id && variable_get("wfm_attach_$node->type", 1)) {
 
       $modulepath = drupal_get_path('module', 'webfm');
       drupal_add_js($modulepath.'/js/webfm.js');

I don't see difference, but it's right it's better to always use same syntax

Regards,