search

thinkshout / mandrill

In order to better organize our issues we've moved all development to Drupal.org. Please visit the issue queue on Drupal.org to create new issues. http://drupal.org/project/mandrill
http://drupal.org/project/mandrill
1 stars 1 forks source link

Fixing all watchdog references in D8 code #41

Closed mshaver closed 8 years ago

mshaver commented 8 years ago

Issue reference #28

mshaver commented 8 years ago

@jamidwyer I found a few more places that need updating for logging in D8. I corrected a few of the references to the term watchdog, but there might need some additional work to actually log them correctly. Also, I believe the mandrill.module will need to use the new logging method where it is now using watchdog_exception:

catch (Exception $e) {
  \Drupal::logger('mailchimp')->error('An error occurred performing batch subscribe/update. "{message}"', array(
  'message' => $e->getMessage()));
}
mshaver commented 8 years ago

@jamidwyer moving this to "in progress" and "needs review" before we merge.