i'll set a starting point...

CommentFileSizeAuthor
#8 mail_logger-1.patch696 bytescreaoy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geek-merlin’s picture

basicmagic.net’s picture

subscribe

Rosamunda’s picture

Any news on this? :)

deekayen’s picture

codeelegance is working on it right now

codeelegance’s picture

Initial branch of 7.x has been committed. Feedback would be greatly appreciated.

Rick G’s picture

Feedback:

Initial d7 module works on my d7 install, with one minor patch:

Line 187 in mail_logger.module is:
$output = drupal_render(drupal_get_form('mail_logger_form_overview'));

it should be done like this:

  $form = drupal_get_form('mail_logger_form_overview');
  $output = drupal_render($form);

Otherwise, it throws a fatal "Only variables can be passed by reference" error in PHP 5.3

deekayen’s picture

Status: Active » Fixed

Committed #6.

creaoy’s picture

FileSize
696 bytes

Here is patch for notice fixes for 7.x version:

"Notice: Undefined property: stdClass::$to i mail_logger_overview() (linje 227 af /var/www/html/dev/sites/all/modules/contrib/mail_logger/mail_logger.module).
Notice: Undefined property: stdClass::$from i mail_logger_overview() (linje 228 af /var/www/html/dev/sites/all/modules/contrib/mail_logger/mail_logger.module)."

deekayen’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev

Applied #8.

Status: Fixed » Closed (fixed)

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