mirror of
https://github.com/moodle/moodle.git
synced 2025-04-23 09:23:09 +02:00
rating MDL-21657 added aliasing and unaliasing of the fields returned by user_picture::fields
This commit is contained in:
parent
f3afba4e19
commit
c538a23d9d
@ -78,7 +78,13 @@ if (!$ratings) {
|
||||
echo "<th class=\"header\" scope=\"col\" style=\"width:100%\"><a href=\"report.php?id=$itemid&sort=rating\">$strrating</a></th>";
|
||||
echo "<th class=\"header\" scope=\"col\"><a href=\"report.php?id=$itemid&sort=time\">$strtime</a></th>";
|
||||
echo "</tr>";
|
||||
$user = null;
|
||||
foreach ($ratings as $rating) {
|
||||
//undo the aliasing necessary for user_picture::fields
|
||||
$user = clone($rating);//could get away with just overwriting rating->id and not cloning
|
||||
//the rating object as we don't use rating->id again. That just seems like a bad idea.
|
||||
$user->id = $user->uid;
|
||||
|
||||
echo '<tr class="ratingitemheader">';
|
||||
echo "<td>";
|
||||
if($course && $course->id) {
|
||||
|
@ -183,7 +183,7 @@ class rating implements renderable {
|
||||
public static function load_ratings_for_item($context, $itemid, $sort) {
|
||||
global $DB;
|
||||
|
||||
$userfields = user_picture::fields('u');
|
||||
$userfields = user_picture::fields('u','uid');
|
||||
$sql = "SELECT r.id, r.rating, r.itemid, r.userid, r.timemodified,
|
||||
$userfields
|
||||
FROM {rating} r
|
||||
|
Loading…
x
Reference in New Issue
Block a user