Merge pull request #14037 from colemanw/dev/core#562

dev/core#562 - Remove dao->free from api
civicrm · Apr 12, 2019 · 0dcf3c8 · 0dcf3c8
2 parents 356ec83 + de866e8
commit 0dcf3c8
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/Civi/API/SelectQuery.php b/Civi/API/SelectQuery.php
@@ -105,7 +105,6 @@ public function __construct($entity, $checkPermissions) {
     $this->apiFieldSpec = $this->getFields();
 
     $this->query = \CRM_Utils_SQL_Select::from($bao->tableName() . ' ' . self::MAIN_TABLE_ALIAS);
-    $bao->free();
 
     // Add ACLs first to avoid redundant subclauses
     $this->checkPermissions = $checkPermissions;
@@ -145,7 +144,6 @@ public function run() {
 
     while ($result_dao->fetch()) {
       if (in_array('count_rows', $this->select)) {
-        $result_dao->free();
         return (int) $result_dao->c;
       }
       $result_entities[$result_dao->id] = [];
@@ -166,7 +164,6 @@ public function run() {
         }
       };
     }
-    $result_dao->free();
     return $result_entities;
   }