Hi,

I'm having some problems with the mail module component in Case Tracker, the mail module does not appear to be functioning at all.

My understanding is that if a new case is created, the person it is assigned to will be notified via email, but this is not happeneing. If someone subscribes to a case and a comment is left then an email is generated and sent, but not when a case is first created (I believe the subscription email is part of a different module though).

I am using CSS and a custom page for the cases, so understand not all the case details can be addedd to the mail, however simple notification will suffice.

Any ideas what could be causing this?

Comments

szazo’s picture

Hi,

I have found that it is caused by a missing '$'. The short patch:

--- casetracker_mail.module.old	Mon Nov 19 08:36:58 2007
+++ casetracker_mail.module	Sun Dec 09 15:51:28 2007
@@ -58,7 +58,7 @@ function casetracker_mail_send($case, $c
   $project = db_fetch_object(db_query("SELECT cp.project_number, n.title FROM {casetracker_project} cp LEFT JOIN {node} n ON (n.vid = cp.vid) WHERE cp.nid = %d", $pid));
   
   // get the assigned to name
-  $assignToId = (isComment)
+  $assignToId = ($isComment)
   					? $comment['assign_to']
   					: $case->assign_to;
   $assignToName =  (is_numeric($assignToId))

Kind regards,
Szazo

dinis’s picture

Hi Szazo,

I have applied the changes in the patch, however I'm still not recieving emails when a case is raised or assigned to me. There is no error in the log file either, it's as if the mod is not triggering, or is triggering and not getting any errors.

Any other ideas, or any ideas how I can track the problem?

Cheers

Anonymous’s picture

Patch in #1 works for me.

Thanks,
txcrew

oprior’s picture

The patch didn't work for me

dinis’s picture

Still no email functionality for me here either.

I'm about set up a test server with a clean install of PHP/MySQL/Drupal 5.5 etc. and I'll report back.

bchoc’s picture

I actually am having this problem in 1.2. As I recall, this did work correctly in earlier versions.

(Also, the suggested code change does not appear to have resolved it.)

Anyway, I'm just sharing this comment for additional context.

mblazke’s picture

#1 patch works for me too.
Thanks!

AgnesCB’s picture

Status: Active » Reviewed & tested by the community

Patch works for me.

zero2one’s picture

Status: Reviewed & tested by the community » Fixed
zero2one’s picture

Assigned: Unassigned » zero2one
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.