Not restricting freetagging vocabularies in node form

Yeah it seems like freetagging forms have a different id to the select forms, so the module doesn't catch them. Should be a simple fix if I knew how to find that!

Bumping to critical since this is essentially half the functionality of the module.

So the relevant bit of a select form for taxonomy looks like:

select name="taxonomy[21][]" multiple="multiple"  class="form-select" id="edit-taxonomy-21"

for freetagging:

input type="text" maxlength="255" name="taxonomy[tags][2]" id="edit-taxonomy-tags-2"

So in the module, the references to ($form['taxonomy']) need this as well: ($form['taxonomy']['tags'])

Should be a pretty simple patch for someone who understands it more than me! Since I'm only using this for freetagging, I'm going to try hacking it in and check it works for now.

OK so this is a modified vocabperms_node_form_remove function that ONLY works for freetagging vocabularies. I don't yet have an idea how to do both at the same time. If you're only using it to restrict freetagging, it's a three line change to get working, and won't conflict with it getting fixed properly later. The only change is : ($form['taxonomy']) becomes ($form['taxonomy']['tags'])

Not a real patch, but marking as needs work presumptously because, well, it needs work.

function vocabperms_node_form_remove(&$form) {
  global $user;
  if ($user->uid != 1 && isset($form['taxonomy']['tags'])) {
    foreach ($form['taxonomy']['tags'] as $vid => $elem) {
      if (is_int($vid)) {
        // get the saved permissions for this vocabulary
        if ($perms = unserialize(variable_get('vocab_perms-'. $vid, ''))) {
          $perm = VOCABPERMS_NONE;
          foreach ($user->roles as $rid => $rolename) {
            if (isset($perms[$rid]) && $perms[$rid] < $perm) {
              $perm = $perms[$rid];
            }
          }
          if ($perm > VOCABPERMS_EDIT) {
            /* Convert the element to a value so any preset or existing values
             * are retained.  Use a default value if available.  Otherwise, use
             * the first option (which would be selected by default).
             */
            $value = isset($elem['#default_value']) ? $elem['#default_value'] : key($elem['#options']);
            $form['taxonomy']['tags'][$vid] = array(
              '#type' => 'value',
              '#value' => $value,
            );
          }
        }
      }
    }
  }

Thanks Doug, that's service for ya.