Mailcomment seems incompatible with Comment CCK - similar to the casetracker issue. Whenever I reply to a notification on a content/comment, Comment CCK sets all that content's CCK fields to null.

I looked at the custom casetracker fix module, and can't seem to get it working for comment cck - then again, I'm not a coder. Any tips?

Comments

greggles’s picture

I have a similar issue with Project* module. I'm not sure whether the fix will be in MailComment or in the other modules, but I hope we can find a solution.

obrienmd’s picture

I'm taking a look at that Casetracker fix now, maybe I can hack it enough that it works for Comment CCK... I'll report back.

obrienmd’s picture

No such luck - nothing seemed to work.

Ian Ward’s picture

If you're using mailcomment and not mail2web and you are trying the casetracker fix mentioned here http://drupal.org/node/563474#comment-2086374 then you'll need to rename that module like the following:

<?php

function mailcomment_casetracker_mailcomment_alter(&$node, $op) {
  switch ($op) {
    case 'pre':
      $parent = node_load($node->nid);
      $name = db_result(db_query("SELECT name FROM {users} WHERE uid = %d", $parent->casetracker->assign_to));
      $node->casetracker->case_priority_id = $parent->casetracker->case_priority_id;
      $node->casetracker->case_type_id = $parent->casetracker->case_type_id;
      $node->casetracker->assign_to = $name;
      $node->casetracker->case_status_id = $parent->casetracker->case_status_id;
      $node->casetracker->pid = $parent->casetracker->pid;
      return $node;
  }
}

and call the module file and info file mailcomment_casetracker.module / mailcomment_casetracker.info

In the case of Comment CCK, I've not used that module, but the problem does sound the same. It's possible there's a sweeping fix to fix this _kind_ of issue. I'll take a look soon if no one else gets to it before. If there's no sweeping fix, then it's probably worth packaging a couple of plugin modules for these cases.

obrienmd’s picture

Right, I attempted the modifications you suggest here and no luck - I really didn't expect this to fix it, given the differences between casetracker and comment cck... However, it was worth a try. I'd be happy to help w/ testing on any sub-module or sweeping fix you might have ideas on.

obrienmd’s picture

Ian, had you a chance to look at this further?

obrienmd’s picture

Status: Active » Closed (fixed)

I sponsored a fix for this over at Comment CCK - apparently that module was the problem.

http://drupal.org/node/660970

hanoii’s picture

Status: Closed (fixed) » Active

I think this is a valid issue still, as #660970: Incompatible with mail2web / mailcomment is for comment cck and this module can operate without it.

Openatrium doesn't use it for instance.

hanoii’s picture

Status: Active » Closed (fixed)

Oops, I missed that this issue was actually an incompatibility with comment cck after all. Sorry.

shaundychko’s picture

#4 is awesome!