--- casetracker_actions/casetracker_actions.module	2009-07-26 19:37:30.000000000 -0700
+++ casetracker_actions/casetracker_actions-new.module	2009-08-31 21:20:23.831009925 -0700
@@ -61,7 +61,7 @@ function casetracker_actions_action_info
  */
 function casetracker_actions_set_priority_action(&$object, $context = array()) {
   $nid = _casetracker_actions_get_nid( $object->nid, $context['nid'] );
-  _casetracker_actions_act( $nid, 'case_priority_id', $context['priority'], $context['invisible']);
+  _casetracker_actions_act( $nid, 'case_priority_id', $context['priority']);
 }
 
 /** 
@@ -69,7 +69,7 @@ function casetracker_actions_set_priorit
  */
 function casetracker_actions_set_status_action(&$object, $context = array()) {
   $nid = _casetracker_actions_get_nid( $object->nid, $context['nid'] );
-  _casetracker_actions_act($nid, 'case_status_id', $context['status'], $context['invisible']);
+  _casetracker_actions_act($nid, 'case_status_id', $context['status']);
 }
 
 /** 
@@ -77,7 +77,7 @@ function casetracker_actions_set_status_
  */
 function casetracker_actions_set_type_action(&$object, $context = array()) {
   $nid = _casetracker_actions_get_nid( $object->nid, $context['nid'] );
-  _casetracker_actions_act($nid, 'case_type_id', $context['type'], $context['invisible']);
+  _casetracker_actions_act($nid, 'case_type_id', $context['type']);
 }
 
 /** 
@@ -85,7 +85,7 @@ function casetracker_actions_set_type_ac
  */
 function casetracker_actions_set_assign_to_action(&$object, $context = array()) {
   $nid = _casetracker_actions_get_nid( $object->nid, $context['nid'] );
-  _casetracker_actions_act($nid, 'assign_to', $context['assign_to'], $context['invisible']);
+  _casetracker_actions_act($nid, 'assign_to', $context['assign_to']);
 }
 
 /** 
@@ -93,7 +93,7 @@ function casetracker_actions_set_assign_
  */
 function casetracker_actions_set_project_action(&$object, $context = array()) {
   $nid = _casetracker_actions_get_nid($object->nid, $context['nid']);
-  _casetracker_actions_act($nid, 'pid', $context['project'], $context['invisible']);
+  _casetracker_actions_act($nid, 'pid', $context['project']);
 }
 
 /** 
@@ -102,7 +102,7 @@ function casetracker_actions_set_project
 function casetracker_actions_take_case_action(&$object, $context = array()) {
   global $user;
   $nid = _casetracker_actions_get_nid( $object->nid, $context['nid'] );  
-  _casetracker_actions_act($nid, 'assign_to', $user->name, $context['invisible']);
+  _casetracker_actions_act($nid, 'assign_to', $user->name);
 }
 
 /**
@@ -124,14 +124,11 @@ function casetracker_actions_set_priorit
     '#default_value' => $default_priority,
     '#options' => $case_priority_options,
   );
-  $form['casetracker_actions_invisible'] = _casetracker_actions_invisible_mode();
   return $form;
 }
 
 function casetracker_actions_set_priority_action_submit($form, $form_state) {
-  return array('priority' => $form_state['values']['casetracker_actions_priority'],
-    'invisible' => $form_state['values']['casetracker_actions_invisible']
-  );
+  return array('priority' => $form_state['values']['casetracker_actions_priority']);
 }
 
 /**
@@ -149,14 +146,11 @@ function casetracker_actions_set_status_
     '#default_value' => $default_status,
     '#options' => $case_status_options,
   );
-  $form['casetracker_actions_invisible'] = _casetracker_actions_invisible_mode();
   return $form;
 }
 
 function casetracker_actions_set_status_action_submit($form, $form_state) {
-  return array('status' => $form_state['values']['casetracker_actions_status'],
-    'invisible' => $form_state['values']['casetracker_actions_invisible']
-  );
+  return array('status' => $form_state['values']['casetracker_actions_status']);
 }
 
 /**
@@ -173,14 +167,11 @@ function casetracker_actions_set_type_ac
     '#default_value' => $default_type,
     '#options' => $case_type_options,
   );
-  $form['casetracker_actions_invisible'] = _casetracker_actions_invisible_mode();
   return $form;
 }
 
 function casetracker_actions_set_type_action_submit($form, $form_state) {
-  return array('type' => $form_state['values']['casetracker_actions_type'],
-    'invisible' => $form_state['values']['casetracker_actions_invisible']
-  );
+  return array('type' => $form_state['values']['casetracker_actions_type']);
 }
 
 /**
@@ -210,14 +201,11 @@ function casetracker_actions_set_assign_
 
   $form['casetracker_actions_assign_to']['#default_value'] = variable_get(
     'casetracker_default_assign_to', variable_get('anonymous', t('Anonymous')));
-  $form['casetracker_actions_invisible'] = _casetracker_actions_invisible_mode();
   return $form;
 }
 
 function casetracker_actions_set_assign_to_action_submit($form, $form_state) {
-  return array('assign_to' => $form_state['values']['casetracker_actions_assign_to'],
-    'invisible' => $form_state['values']['casetracker_actions_invisible']
-  );
+  return array('assign_to' => $form_state['values']['casetracker_actions_assign_to']);
 }
 
 /**
@@ -242,14 +230,11 @@ function casetracker_actions_set_project
       '#value' => !empty($default_project) ? $default_project : key($project_options),
     );
   }
-  $form['casetracker_actions_invisible'] = _casetracker_actions_invisible_mode();
   return $form;
 }
 
 function casetracker_actions_set_project_action_submit($form, $form_state) {
-  return array('project' => $form_state['values']['casetracker_actions_pid'],
-    'invisible' => $form_state['values']['casetracker_actions_invisible']
-  );
+  return array('project' => $form_state['values']['casetracker_actions_pid']);
 }
 
 /**
@@ -257,35 +242,17 @@ function casetracker_actions_set_project
  */
 
 /**
- * Creates checkbox toggle for action configuration form.
- * Only available to users with the Administer Cases permission.
- */
-function _casetracker_actions_invisible_mode() {
-  return array(
-    '#type' => 'checkbox',
-    '#title' => t('Update cases in invisible mode.'),
-    '#access' => user_access('administer case tracker'),
-  );
-}
-
-/**
  * generalized casetracker update function
  * All casetracker actions have the same fundamental behavior.
  */
-function _casetracker_actions_act($nid, $field, $value, $invisible) {
+function _casetracker_actions_act($nid, $field, $value) {
   if(!$nid) return;
   // make sure we have a valid case node
   $node = node_load(array('nid' => $nid, 'type' => 'casetracker_basic_case'));
  
-  $success = 0;
-  if($invisible) {
-    $success = _casetracker_actions_update_field(
-      $field, $value, $nid);	
-  }
-  else {
-    $success = _casetracker_actions_update_by_comment(
-      $field, $value, $node);
-  }
+  $success = _casetracker_actions_update_by_comment(
+    $field, $value, $node);
+  
   if($success) {
     _casetracker_actions_log($field, $value, $nid);
   }
@@ -311,15 +278,6 @@ function _casetracker_actions_get_nid($o
 }
 
 /**
- * Run "invisible mode" case update.
- */
-function _casetracker_actions_update_field($field, $value, $nid) {
-  return 
-    db_query('UPDATE {casetracker_case} SET %s = "%d" WHERE nid = "%d"', 
-      $field, $value, $nid);
-}
-
-/**
  * Update case by programmatically inserting a comment posted by current user.
  */
 function _casetracker_actions_update_by_comment($field, $value, $node) {
@@ -359,3 +317,4 @@ function _casetracker_actions_log($field
 
   drupal_set_message(t("Case #%nid: %field set to %value", array('%nid' => $nid, '%field' => $field, '%value' => $value)));
 }
+