Hi all,

We're trying to port the casetracker_mail to Drupal 6. We hope to roll out a patch the next week or so.

Regards.

CommentFileSizeAuthor
#5 casetracker_mail.patch18.26 KBeclipsegc
#1 ct_mail-d6.patch11.56 KBimobachgs

Comments

imobachgs’s picture

imobachgs commented
StatusFileSize
new ct_mail-d6.patch11.56 KB

I attach a patch that allows to use casetracker_mail with Drupal 6. The major change is that the module use now the hook_mail() facility. Some minor fixes and some refactoring (to acomodate hook_mail()) have been also performed.

I guess that mailhandler integration is broken. I hope I can devote some time in a week or two. However, if anybody wants to fix it... go ahead :)

As usual, feedback is welcome!

PS: the code is not as clean as I'd like, so feel free to improve it :)

piotrsmykaj’s picture

piotrsmykaj commented

dont work with latest dev, on submit case print some email and its all. Case are added but mail not sended.

imobachgs’s picture

imobachgs commented

Ok, thanks for the feedback. I'll fix it this week.

jmiccolis’s picture

jmiccolis commented

FYI I've just made small change to the case state load functions, which I'm certain will effect mail functionality. Basically the two arguments in casetracker_case_state_load() have been reversed - sounds trivial, I know, but it allowed me to use that function as a loader in hook menu.

eclipsegc’s picture

eclipsegc commented
StatusFileSize
new casetracker_mail.patch18.26 KB

I had a working version of this for my own casetracker port. I've rerolled that against dev.

This patch is dependent upon #398458: Inconsistent $node->casetracker handling.

Eclipse

eclipsegc’s picture

eclipsegc commented
Status: Active » Needs review

status change

eclipsegc’s picture

eclipsegc commented
Version: master » 6.x-1.x-dev
jmiccolis’s picture

jmiccolis commented
Status: Needs review » Active

Hi All

I've committed (basically) the patch from #5 as it seemed to be more complete. Thanks to everybody for their help!

Before I close this ticket I'd really love for somebody to run casetracker_mail through it's paces... @osoh or @teszek could one or both of you verify that things are working as expected?

luco’s picture

luco commented

I didn't get it. so is this already commited to DEV?

I tried to set up email integration but was a total fail.

cheers,
Luciano

jmiccolis’s picture

jmiccolis commented
Status: Active » Closed (fixed)

Sorry for the confusion. Yes this has been commited to head. I'm setting this ticket to closed. However I'd still love for people to help me test, so if you find issues please open new tickets for them.