From e7c5a118cb2fe0ce46b0a00ce6dc654328640556 Mon Sep 17 00:00:00 2001
From: colemanw <coleman@civicrm.org>
Date: Fri, 26 Jan 2024 13:34:06 -0500
Subject: [PATCH] dev/core#4905 - Make internal function private again

This can be private now that the DedupeMerger copy of getTree has been refactored out
---
 CRM/Core/BAO/CustomGroup.php | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/CRM/Core/BAO/CustomGroup.php b/CRM/Core/BAO/CustomGroup.php
index cf3332951398..738d7825d3a2 100644
--- a/CRM/Core/BAO/CustomGroup.php
+++ b/CRM/Core/BAO/CustomGroup.php
@@ -536,7 +536,7 @@ public static function getTree(
     ];
     if ($subTypes) {
       foreach ($subTypes as $subType) {
-        $filters['extends_entity_column_value'][] = CRM_Core_BAO_CustomGroup::validateSubTypeByEntity($entityType, $subType);
+        $filters['extends_entity_column_value'][] = self::validateSubTypeByEntity($entityType, $subType);
       }
       if (!$onlySubType) {
         $filters['extends_entity_column_value'][] = NULL;
@@ -665,15 +665,13 @@ private static function formatLegacyDbValues(array &$values): void {
    *
    * For all other entities, it doesn't validate except to check the subtype is an integer.
    *
-   * @internal this will be private again soon.
-   *
    * @param string $entityType
    * @param string $subType
    *
    * @return string
    * @throws \CRM_Core_Exception
    */
-  public static function validateSubTypeByEntity($entityType, $subType) {
+  private static function validateSubTypeByEntity($entityType, $subType) {
     $subType = trim($subType, CRM_Core_DAO::VALUE_SEPARATOR);
     if (is_numeric($subType)) {
       return $subType;