mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-17 21:49:15 +01:00
Code Issues Projects Releases Wiki Activity

MDL-66536 analytics: Bulk-fetch samples data

Browse Source 
For one_sample_per_analysable models.
This commit is contained in:
David Monllaó 2019-08-23 13:51:47 +08:00
parent 93e71c712d
commit b024720499
1 changed files with 20 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
analytics/classes/model.php
View File

@ -950,11 +950,9 @@ class model {
// the database, and we need to do it using one single database query (for performance reasons as well).
$predictionrecords = $this->add_prediction_ids($predictionrecords);
// Get \core_analytics\prediction objects also fetching the samplesdata. This costs us
// 1 db read, but we have to pay it if we want that our insights include links to the
// suggested actions.
$predictions = array_map(function($predictionobj) {
$prediction = new \core_analytics\prediction($predictionobj, $this->prediction_sample_data($predictionobj));
$samplesdata = $this->predictions_sample_data($predictionrecords);
$predictions = array_map(function($predictionobj) use ($samplesdata) {
$prediction = new \core_analytics\prediction($predictionobj, $samplesdata[$predictionobj->sampleid]);
return $prediction;
}, $predictionrecords);
} else {
@ -1411,6 +1409,23 @@ class model {
return $samplesdata[$predictionobj->sampleid];
}
/**
* Returns the samples data of the provided predictions.
*
* @param \stdClass[] $predictionrecords
* @return array
*/
public function predictions_sample_data(array $predictionrecords): array {
$sampleids = [];
foreach ($predictionrecords as $predictionobj) {
$sampleids[] = $predictionobj->sampleid;
}
list($sampleids, $samplesdata) = $this->get_analyser()->get_samples($sampleids);
return $samplesdata;
}
/**
* Returns the description of a sample
*