Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-20833

Change namespace for APIv4 entities

    Details

    • Type: Patch
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.7.21
    • Fix Version/s: 4.7.22
    • Component/s: CiviCRM API
    • Labels:
    • Versioning Impact:
      Patch (backwards-compatible bug fixes)
    • Documentation Required?:
      None
    • Funding Source:
      Core Team Contract
    • Verified?:
      No
    • How it works currently:
      If you want to use api4 after the restructuring PR is closed you have to manually change the namespace in core code
    • How it should work:
      Core is compatible with the latest api4 changes.

      Description

      In the \Civi\API\Request the namespace for the APIv4 classes is hardcoded to "Civi\Api4".

      After the restructuring in https://github.com/civicrm/api4/pull/67 the namespace has changed to "Civi\API\V4\Entity".{{}}

      It might also be nice to allow this to be configured using an environmental variable. We might consider changing the logic here completely in the future since it seems strange that core would depend on an extension (api4) like this.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              michael Michael Devery
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: