CRM-18037: default value for preferred_mail_format should be 'Both'.
authorOlivier Tétard <olivier.tetard@miskin.fr>
Thu, 3 Mar 2016 10:47:05 +0000 (11:47 +0100)
committerOlivier Tétard <olivier.tetard@miskin.fr>
Thu, 3 Mar 2016 10:53:48 +0000 (11:53 +0100)
Default value should be “Both”, there is no need to find the
translated text for it. This prevents from having an empty default in
translated installations of CiviCRM.

CRM/Contact/Form/Edit/CommunicationPreferences.php

index ed3e4dbe8c272f19e24b6b5eec9b1d7238a76e55..03ba749b804688e66bc1e8a1eca8315db986f0a0 100644 (file)
@@ -166,7 +166,7 @@ class CRM_Contact_Form_Edit_CommunicationPreferences {
 
     // CRM-17778 -- set preferred_mail_format to default if unset
     if (empty($defaults['preferred_mail_format'])) {
-      $defaults['preferred_mail_format'] = array_search('Both', CRM_Core_SelectValues::pmf());
+      $defaults['preferred_mail_format'] = 'Both';
     }
 
     //set default from greeting types CRM-4575, CRM-9739