From a8d779b4dec33788e3d72437e06a4c46e18b895d Mon Sep 17 00:00:00 2001 From: Sean Harvey Date: Thu, 15 Nov 2012 11:51:56 +1300 Subject: [PATCH] Removing deprecated DataObject::buildDataObjectSet() Use DataList API instead --- model/DataObject.php | 36 ------------------------------------ 1 file changed, 36 deletions(-) diff --git a/model/DataObject.php b/model/DataObject.php index e5bda66fb..11f68a66f 100644 --- a/model/DataObject.php +++ b/model/DataObject.php @@ -2738,42 +2738,6 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity return self::get($this->class, $filter, $sort, $join, $limit, $containerClass); } - /** - * Take a database {@link SS_Query} and instanciate an object for each record. - * - * @deprecated 3.0 Replaced by DataList - * - * @param SS_Query|array $records The database records, a {@link SS_Query} object or an array of maps. - * @param string $containerClass The class to place all of the objects into. - * - * @return mixed The new objects in an object of type $containerClass - */ - public function buildDataObjectSet($records, $containerClass = "DataObjectSet", $query = null, $baseClass = null) { - Deprecation::notice('3.0', 'Use DataList instead.'); - - foreach($records as $record) { - if(empty($record['RecordClassName'])) { - $record['RecordClassName'] = $record['ClassName']; - } - if(class_exists($record['RecordClassName'])) { - $results[] = new $record['RecordClassName']($record); - } else { - if(!$baseClass) { - user_error("Bad RecordClassName '{$record['RecordClassName']}' and " - . "\$baseClass not set", E_USER_ERROR); - } else if(!is_string($baseClass) || !class_exists($baseClass)) { - user_error("Bad RecordClassName '{$record['RecordClassName']}' and bad " - . "\$baseClass '$baseClass not set", E_USER_ERROR); - } - $results[] = new $baseClass($record); - } - } - - if(isset($results)) { - return new $containerClass($results); - } - } - /** * Return the first item matching the given query. * All calls to get_one() are cached.