Just wondering if there is any chance this is getting ported to D6? If not, it looks like a pretty simple module; let me know if you want someone else to do it ;)

CommentFileSizeAuthor
#2 civicrm_error-6.x-307569_2.tgz3.57 KBxurizaemon
#2 307569-2civicrm_error-d6_port.patch4.21 KBxurizaemon

Comments

dalin’s picture

dalin
he/him
Primary language English
Location 🇨🇦, 🇲🇽, 🌍
commented

Patches welcome. It should be very simple. I think just hook_menu() needs to be adjusted. See here:
http://api.drupal.org/api/function/hook_menu/6

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
StatusFileSize
new 307569-2civicrm_error-d6_port.patch4.21 KB
new civicrm_error-6.x-307569_2.tgz3.57 KB

Hopefully this is a start! Tarball and patch against current D5 CVS attached.

Changes were to hook_menu() and addition of hook_mail(), which I'm a bit unfamiliar with, so if my method of getting the message body from civicrm_error_handler() to civicrm_error_mail() is funky, please correct it :)

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Status: Active » Needs review

also need CVS tag created and version on this issue changed (if you think the code is worthy, dalin)

cheers

dalin’s picture

dalin
he/him
Primary language English
Location 🇨🇦, 🇲🇽, 🌍
commented
Status: Needs review » Fixed

Looks good. Committed. Will now create a D6 project.

Status: Fixed » Closed (fixed)

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