Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 291026

Update mail interfaces

$
0
0

Follow up of #1821420-73: Convert mail variables to cmi to update mail interfaces

+++ b/core/includes/mail.incundefined
@@ -231,7 +231,7 @@ function drupal_mail($module, $key, $to, $langcode, $params = array(), $from = N
- *   'default-system' => 'Drupal\Core\Mail\PhpMail',
+ *   'default' => 'Drupal\Core\Mail\PhpMail',
  *   'user' => 'DevelMailLog',

@@ -241,7 +241,7 @@ function drupal_mail($module, $key, $to, $langcode, $params = array(), $from = N
- *   'default-system' => 'Drupal\Core\Mail\PhpMail',
+ *   'default' => 'Drupal\Core\Mail\PhpMail',
  *   'user' => 'DevelMailLog',
  *   'contact_page_autoreply' => 'DrupalDevNullMailSend',

Can we open a follow-up to fix those examples? (Those are not PSR-0 classes, should be updated with how it's really called now in devel/ technically correct examples)

+++ b/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.phpundefined
@@ -854,7 +854,7 @@ protected function setUp() {
-    variable_set('mail_system', array('default-system' => 'Drupal\Core\Mail\VariableLog'));
+    config('system.mail')->set('interface.default', 'Drupal\Core\Mail\VariableLog')->save();

That should have been renamed when it was converted to use state and not a variable.

Viewing all articles
Browse latest Browse all 291026

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>