diff --git a/core/modules/contact/lib/Drupal/contact/MessageFormController.php b/core/modules/contact/lib/Drupal/contact/MessageFormController.php index 20588871e9c232b04761cd0f75ce99fb6358acfe..da178478483cca0fdf1bf9f7bf2b03f41de67223 100644 --- a/core/modules/contact/lib/Drupal/contact/MessageFormController.php +++ b/core/modules/contact/lib/Drupal/contact/MessageFormController.php @@ -245,16 +245,16 @@ public function save(array $form, array &$form_state) { $this->flood->register('contact', $config->get('flood.interval')); if (!$message->isPersonal()) { watchdog('contact', '%sender-name (@sender-from) sent an e-mail regarding %category.', array( - '%sender-name' => $sender->name, + '%sender-name' => $sender->getUsername(), '@sender-from' => $sender->getEmail(), '%category' => $category->label(), )); } else { watchdog('contact', '%sender-name (@sender-from) sent %recipient-name an e-mail.', array( - '%sender-name' => $sender->name, + '%sender-name' => $sender->getUsername(), '@sender-from' => $sender->getEmail(), - '%recipient-name' => $message->recipient->name, + '%recipient-name' => $message->getPersonalRecipient()->getUsername(), )); } diff --git a/core/modules/contact/lib/Drupal/contact/Tests/ContactPersonalTest.php b/core/modules/contact/lib/Drupal/contact/Tests/ContactPersonalTest.php index c7056a7bca04110abb168088b232ccfe85f9ab59..f1df0789d8872a246e8a7954aecf84429245b26b 100644 --- a/core/modules/contact/lib/Drupal/contact/Tests/ContactPersonalTest.php +++ b/core/modules/contact/lib/Drupal/contact/Tests/ContactPersonalTest.php @@ -7,6 +7,7 @@ namespace Drupal\contact\Tests; +use Drupal\Component\Utility\String; use Drupal\simpletest\WebTestBase; /** @@ -19,7 +20,7 @@ class ContactPersonalTest extends WebTestBase { * * @var array */ - public static $modules = array('contact'); + public static $modules = array('contact', 'dblog'); /** * A user with some administrative permissions. @@ -54,7 +55,7 @@ function setUp() { parent::setUp(); // Create an admin user. - $this->admin_user = $this->drupalCreateUser(array('administer contact forms', 'administer users', 'administer account settings')); + $this->admin_user = $this->drupalCreateUser(array('administer contact forms', 'administer users', 'administer account settings', 'access site reports')); // Create some normal users with their contact forms enabled by default. \Drupal::config('contact.settings')->set('user_default_enabled', 1)->save(); @@ -85,6 +86,18 @@ function testSendPersonalContactMessage() { $this->assertTrue(strpos($mail['body'], t('Hello !recipient-name,', $variables)) !== FALSE, 'Recipient name is in sent message.'); $this->assertTrue(strpos($mail['body'], $this->web_user->getUsername()) !== FALSE, 'Sender name is in sent message.'); $this->assertTrue(strpos($mail['body'], $message['message']) !== FALSE, 'Message body is in sent message.'); + + // Check there was no problems raised during sending. + $this->drupalLogout(); + $this->drupalLogin($this->admin_user); + // Verify that the correct watchdog message has been logged. + $this->drupalGet('/admin/reports/dblog'); + $placeholders = array( + '@sender_name' => $this->web_user->username, + '@sender_email' => $this->web_user->getEmail(), + '@recipient_name' => $this->contact_user->getUsername() + ); + $this->assertText(String::format('@sender_name (@sender_email) sent @recipient_name an e-mail.', $placeholders)); } /**