MDL-40612 libraries: Tidy up and improvement of the code for additional name fields.
Includes an update to get_all_user_name_fields() and the inclusion of set_user_name_object()
This commit is contained in:
+2
-18
@@ -559,15 +559,7 @@ class question_bank_creator_name_column extends question_bank_column_base {
|
||||
protected function display_content($question, $rowclasses) {
|
||||
if (!empty($question->creatorfirstname) && !empty($question->creatorlastname)) {
|
||||
$u = new stdClass();
|
||||
$allnames = get_all_user_name_fields();
|
||||
foreach ($allnames as $allname) {
|
||||
$tempname = 'creator' . $allname;
|
||||
if (isset($question->$tempname)) {
|
||||
$u->$allname = $question->$tempname;
|
||||
} else {
|
||||
$u->$allname = '';
|
||||
}
|
||||
}
|
||||
$u = username_load_fields_from_object($u, $question, 'creator');
|
||||
echo fullname($u);
|
||||
}
|
||||
}
|
||||
@@ -612,15 +604,7 @@ class question_bank_modifier_name_column extends question_bank_column_base {
|
||||
protected function display_content($question, $rowclasses) {
|
||||
if (!empty($question->modifierfirstname) && !empty($question->modifierlastname)) {
|
||||
$u = new stdClass();
|
||||
$allnames = get_all_user_name_fields();
|
||||
foreach ($allnames as $allname) {
|
||||
$tempname = 'modifier' . $allname;
|
||||
if (isset($question->$tempname)) {
|
||||
$u->$allname = $question->$tempname;
|
||||
} else {
|
||||
$u->$allname = '';
|
||||
}
|
||||
}
|
||||
$u = username_load_fields_from_object($u, $question, 'modifier');
|
||||
echo fullname($u);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user