Afform - Code cleanup in LoadAdminData API action
authorColeman Watts <coleman@civicrm.org>
Wed, 11 Aug 2021 02:06:49 +0000 (22:06 -0400)
committerColeman Watts <coleman@civicrm.org>
Wed, 11 Aug 2021 02:06:49 +0000 (22:06 -0400)
ext/afform/admin/Civi/Api4/Action/Afform/LoadAdminData.php

index f8bd4cfafc3f90af599e7ee7a1698d0f9ab1a24f..6caa45095252fdbcfc471f6d5d562101e2b9a7ce 100644 (file)
@@ -4,7 +4,7 @@ namespace Civi\Api4\Action\Afform;
 
 use Civi\AfformAdmin\AfformAdminMeta;
 use Civi\Api4\Afform;
-use Civi\Api4\Entity;
+use Civi\Api4\Utils\CoreUtil;
 use Civi\Api4\Query\SqlExpression;
 
 /**
@@ -209,6 +209,10 @@ class LoadAdminData extends \Civi\Api4\Generic\AbstractAction {
     $result[] = $info;
   }
 
+  /**
+   * @param string $name
+   * @return array|null
+   */
   private function loadForm($name) {
     return Afform::get($this->checkPermissions)
       ->setFormatWhitespace(TRUE)
@@ -262,10 +266,7 @@ class LoadAdminData extends \Civi\Api4\Generic\AbstractAction {
       else {
         $joinCount[$entityName] = 1;
       }
-      $label = Entity::get(FALSE)
-        ->addWhere('name', '=', $entityName)
-        ->addSelect('title')
-        ->execute()->first()['title'];
+      $label = CoreUtil::getInfoItem($entityName, 'title');
       $joinMap[$alias] = $label . $num;
     }
 
@@ -288,6 +289,9 @@ class LoadAdminData extends \Civi\Api4\Generic\AbstractAction {
     return $calcFields;
   }
 
+  /**
+   * @return array[]
+   */
   public function fields() {
     return [
       [