Index: casetracker_actions/casetracker_actions.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/casetracker/casetracker_actions/Attic/casetracker_actions.module,v
retrieving revision 1.1.2.4
diff -u -p -r1.1.2.4 casetracker_actions.module
--- casetracker_actions/casetracker_actions.module	12 Feb 2010 16:19:34 -0000	1.1.2.4
+++ casetracker_actions/casetracker_actions.module	27 May 2010 18:46:11 -0000
@@ -17,7 +17,7 @@ function casetracker_actions_action_info
       'description' => t('Set case priority'),
       'type' => 'node',  
       'configurable' => TRUE,
-      'hooks' => array('any' => TRUE),	// this is a sloppy cheat. 
+      'hooks' => array('any' => TRUE),  // this is a sloppy cheat. 
     ),
     'casetracker_actions_set_status_action' => array(
       'description' => t('Set case status'),
@@ -115,7 +115,7 @@ function casetracker_actions_take_case_a
 function casetracker_actions_set_priority_action_form($context) {
   $case_priority_options = casetracker_realm_load('priority');
   $default_priority = variable_get('casetracker_default_case_priority', 
-    key($case_priority_options));	
+    key($case_priority_options)); 
 
   $form['casetracker_actions_priority'] = array(
     '#title' => t('Priority'),
@@ -137,7 +137,7 @@ function casetracker_actions_set_priorit
 function casetracker_actions_set_status_action_form($context) {
   $case_status_options = casetracker_realm_load('status');
   $default_status = variable_get('casetracker_default_case_status', 
-    key($case_status_options));	
+    key($case_status_options)); 
 
   $form['casetracker_actions_status'] = array(
     '#title' => t('Status'),
@@ -246,15 +246,18 @@ function casetracker_actions_set_project
  * All casetracker actions have the same fundamental behavior.
  */
 function _casetracker_actions_act($nid, $field, $value) {
-  if(!$nid) return;
+  if (empty($nid)) {
+    return;
+  }
   // make sure we have a valid case node
-  $node = node_load(array('nid' => $nid, 'type' => 'casetracker_basic_case'));
- 
-  $success = _casetracker_actions_update_by_comment(
-    $field, $value, $node);
-  
-  if($success) {
-    _casetracker_actions_log($field, $value, $nid);
+  $node = node_load($nid);
+
+  if ($node->nid && casetracker_is_case($node)) {
+    $success = _casetracker_actions_update_by_comment($field, $value, $node);
+
+    if($success) {
+      _casetracker_actions_log($field, $value, $nid);
+    }
   }
 }
 
@@ -283,7 +286,7 @@ function _casetracker_actions_get_nid($o
 function _casetracker_actions_update_by_comment($field, $value, $node) {
   global $user;
   if (user_access('post comments') 
-    && (user_access('administer comments') || node_comment_mode($nid) 
+    && (user_access('administer comments') || node_comment_mode($node->nid) 
     == COMMENT_NODE_READ_WRITE)) {
 
       $case_info = get_object_vars($node->casetracker);
