CRM-14990 is_reserved field cannot be set on custom groups via the API

          Details

          • Type: Bug
          • Status: Done/Fixed
          • Priority: Minor
          • Resolution: Fixed/Completed
          • Affects Version/s: 4.5
          • Fix Version/s: 4.5
          • Component/s: CiviCRM API, Core CiviCRM
          • Labels:
            None

            Description

            This isn't really the API's fault. The API basically just passes the params to CRM_Core_BAO_CustomGroup::create().

              Attachments

                Activity

                [CRM-14990] is_reserved field cannot be set on custom groups via the API
                Frank J. Gómez added a comment -

                PR: https://github.com/civicrm/civicrm-core/pull/3669

                David Greenberg added a comment -

                Merged that PR. I "think" we should also add this property to _civicrm_api3_custom_group_create_spec with default of 0. ??? Maybe check w/ Eileen to confirm. Else go ahead and close.

                Frank J. Gómez added a comment -

                PR #2 (https://github.com/civicrm/civicrm-core/pull/3687) adds pre- hook and doesn't always overwrite is_reserved.

                I don't think we need to update _civicrm_api3_custom_group_create_spec since we aren't providing/forcing a default; if no value is provided the database default of 0 wins.

                  People

                  • Assignee:
                    Eileen McNaughton
                    Reporter:
                    Frank J. Gómez

                    Dates

                    • Created:
                      Updated:
                      Resolved: