projects / civicrm-core.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: df72de7)
Schema - Fix boolean fields in 'SMS'
authorMonish Deb <monish.deb@jmaconsulting.biz>
Fri, 8 Apr 2022 01:12:17 +0000 (18:12 -0700)
committerTim Otten <totten@civicrm.org>
Fri, 8 Apr 2022 01:12:17 +0000 (18:12 -0700)
CRM/SMS/DAO/Provider.php patch | blob | blame | history
CRM/Upgrade/Incremental/php/FiveFortyNine/SMS.bool.php [new file with mode: 0644] patch | blob
xml/schema/SMS/Provider.xml patch | blob | blame | history

diff --git a/CRM/SMS/DAO/Provider.php b/CRM/SMS/DAO/Provider.php
index 290bcf09559197ba6df8d481384d79b0a163f8f7..4605f19e5cc0573848108ae95c1a983d3e7878a7 100644 (file)
--- a/CRM/SMS/DAO/Provider.php
+++ b/CRM/SMS/DAO/Provider.php
@@ -6,7 +6,7 @@
  *
  * Generated from xml/schema/CRM/SMS/Provider.xml
  * DO NOT EDIT.  Generated by CRM_Core_CodeGen
- * (GenCodeChecksum:0fc3229e1bb7ff0e9203e6c5c7ba3d6a)
+ * (GenCodeChecksum:77bb8b25b596507446974d9dd3bd4c8e)
  */
 
 /**
@@ -97,14 +97,14 @@ class CRM_SMS_DAO_Provider extends CRM_Core_DAO {
   public $api_params;
 
   /**
-   * @var bool|string|null
+   * @var bool|string
    *   (SQL type: tinyint)
    *   Note that values will be retrieved from the database as a string.
    */
   public $is_default;
 
   /**
-   * @var bool|string|null
+   * @var bool|string
    *   (SQL type: tinyint)
    *   Note that values will be retrieved from the database as a string.
    */
@@ -295,6 +295,7 @@ class CRM_SMS_DAO_Provider extends CRM_Core_DAO {
           'name' => 'is_default',
           'type' => CRM_Utils_Type::T_BOOLEAN,
           'title' => ts('SMS Provider is Default?'),
+          'required' => TRUE,
           'where' => 'civicrm_sms_provider.is_default',
           'default' => '0',
           'table_name' => 'civicrm_sms_provider',
@@ -310,8 +311,9 @@ class CRM_SMS_DAO_Provider extends CRM_Core_DAO {
           'name' => 'is_active',
           'type' => CRM_Utils_Type::T_BOOLEAN,
           'title' => ts('SMS Provider is Active?'),
+          'required' => TRUE,
           'where' => 'civicrm_sms_provider.is_active',
-          'default' => '0',
+          'default' => '1',
           'table_name' => 'civicrm_sms_provider',
           'entity' => 'Provider',
           'bao' => 'CRM_SMS_BAO_Provider',
diff --git a/CRM/Upgrade/Incremental/php/FiveFortyNine/SMS.bool.php b/CRM/Upgrade/Incremental/php/FiveFortyNine/SMS.bool.php
new file mode 100644 (file)
index 0000000..a030d5c
--- /dev/null
+++ b/CRM/Upgrade/Incremental/php/FiveFortyNine/SMS.bool.php
@@ -0,0 +1,7 @@
+<?php
+return [
+  'civicrm_sms_provider' => [
+    'is_default' => "DEFAULT 0",
+    'is_active' => "DEFAULT 1",
+  ],
+];
diff --git a/xml/schema/SMS/Provider.xml b/xml/schema/SMS/Provider.xml
index 2c00293817c1e997faed47dac32c366172daf5b4..97f8aa6d7ee44636790527e1b3f477ed00c25fd9 100644 (file)
--- a/xml/schema/SMS/Provider.xml
+++ b/xml/schema/SMS/Provider.xml
@@ -99,6 +99,7 @@
     <title>SMS Provider is Default?</title>
     <type>boolean</type>
     <default>0</default>
+    <required>true</required>
     <add>4.2</add>
     <html>
       <type>CheckBox</type>
@@ -108,7 +109,8 @@
     <name>is_active</name>
     <title>SMS Provider is Active?</title>
     <type>boolean</type>
-    <default>0</default>
+    <default>1</default>
+    <required>true</required>
     <add>4.2</add>
     <html>
       <type>CheckBox</type>