Flexmailer - Switch defintion of "auto" to "flexmailer"

diff --git a/ext/flexmailer/settings/flexmailer.setting.php b/ext/flexmailer/settings/flexmailer.setting.php
index 6b51460e6bbc4af632b6866f7884d2738822a49d..ea227d9ef0f64e7837a97bccb92e728aad81393c 100644 (file)
--- a/ext/flexmailer/settings/flexmailer.setting.php
+++ b/ext/flexmailer/settings/flexmailer.setting.php
@@ -11,7 +11,7 @@ return [
     'html_type' => 'select',
     'html_attributes' => ['class' => 'crm-select2'],
     'pseudoconstant' => ['callback' => '_flexmailer_traditional_options'],
-    'default' => 'flexmailer',
+    'default' => 'auto',
     'add' => '5.13',
     'title' => E::ts('Traditional Mailing Handler'),
     'is_domain' => 1,

diff --git a/ext/flexmailer/src/Listener/Abdicator.php b/ext/flexmailer/src/Listener/Abdicator.php
index 5ec0077209bb605e5a1af33a2bf4a9daec67abf3..b237d5899e6ec2adc92038e7db2d1dd188f322d7 100644 (file)
--- a/ext/flexmailer/src/Listener/Abdicator.php
+++ b/ext/flexmailer/src/Listener/Abdicator.php
@@ -40,15 +40,10 @@ class Abdicator {
     }
 
     switch (\Civi::settings()->get('flexmailer_traditional')) {
-      case 'auto':
-        // Transitional support for old hidden setting "experimentalFlexMailerEngine" (bool)
-        // TODO: Remove this. Maybe after Q4 2019.
-        // TODO: Change this to default to flexmailer
-        return (bool) \Civi::settings()->get('experimentalFlexMailerEngine');
-
       case 'bao':
         return FALSE;
 
+      case 'auto':
       case 'flexmailer':
         return TRUE;
 

diff --git a/tests/phpunit/CRM/Mailing/MailingSystemTest.php b/tests/phpunit/CRM/Mailing/MailingSystemTest.php
index b1846cee7eea0c572106079a4e193aa39d5852ab..80e7938e69f2dbef3959c979bedf84d4150252a1 100644 (file)
--- a/tests/phpunit/CRM/Mailing/MailingSystemTest.php
+++ b/tests/phpunit/CRM/Mailing/MailingSystemTest.php
@@ -40,7 +40,9 @@ class CRM_Mailing_MailingSystemTest extends CRM_Mailing_BaseMailingSystemTest {
 
   public function setUp() {
     parent::setUp();
-    Civi::settings()->add(['experimentalFlexMailerEngine' => FALSE]);
+    // If we happen to execute with flexmailer active, use BAO mode.
+    // There is a parallel FlexMailerSystemTest which runs in flexmailer mode.
+    Civi::settings()->add(['flexmailer_traditional' => 'bao']);
 
     $hooks = \CRM_Utils_Hook::singleton();
     $hooks->setHook('civicrm_alterMailParams',