-
Notifications
You must be signed in to change notification settings - Fork 58
Open
Description
Description: In the following function, $sessiondate is never defined and so it will always write the session status for the session date column
Lines 1766 to 1799 in af644c7
| function facetoface_write_activity_attendance_helper( | |
| &$worksheet, | |
| $i, | |
| $session, | |
| $customsessionfields, | |
| $status, | |
| $dateformat, | |
| $starttime, | |
| $finishtime | |
| ) { | |
| global $DB; | |
| $j = 0; | |
| // Custom session fields. | |
| $customdata = $DB->get_records('facetoface_session_data', ['sessionid' => $session->id], '', 'fieldid, data'); | |
| foreach ($customsessionfields as $field) { | |
| if (empty($field->showinsummary)) { | |
| continue; // Skip. | |
| } | |
| $data = '-'; | |
| if (!empty($customdata[$field->id])) { | |
| if (CUSTOMFIELD_TYPE_MULTISELECT == $field->type) { | |
| $data = str_replace(CUSTOMFIELD_DELIMITER, "\n", $customdata[$field->id]->data); | |
| } else { | |
| $data = $customdata[$field->id]->data; | |
| } | |
| } | |
| $worksheet->write_string($i, $j++, $data); | |
| } | |
| if (empty($sessiondate)) { | |
| $worksheet->write_string($i, $j++, $status); // Session date. |
Metadata
Metadata
Assignees
Labels
No labels