$mform->addElement('date_selector', 'sessionenddate', get_string('endofperiod', 'attendance'));
$mform->disabledIf('sessionenddate', 'includeallsessions', 'checked');
- $mform->addElement('select', 'format', get_string('format'),
- array('excel' => get_string('downloadexcel', 'attendance'),
- 'ooo' => get_string('downloadooo', 'attendance'),
- 'text' => get_string('downloadtext', 'attendance')));
+ $formatoptions = array('excel' => get_string('downloadexcel', 'attendance'),
+ 'ooo' => get_string('downloadooo', 'attendance'),
+ 'text' => get_string('downloadtext', 'attendance'));
+ $mform->addElement('select', 'format', get_string('format'), $formatoptions);
$submitstring = get_string('ok');
$this->add_action_buttons(false, $submitstring);
public function validation($data, $files) {
$errors = parent::validation($data, $files);
- // Validate the 'users' field.
+ // Validate the 'users' field.
if ($data['selectedusers'] && empty($data['users'])) {
$errors['users'] = get_string('mustselectusers', 'mod_attendance');
}
$dta = $this->construct_date_time_actions($sessdata, $sess);
$table->data[$sess->id][] = $i;
- $table->data[$sess->id][] = $sess->groupid ? get_string('group') . ': ' . $sessdata->groups[$sess->groupid]->name
- : get_string('commonsession', 'attendance');
+ if ($sess->groupid) {
+ $table->data[$sess->id][] = get_string('group') . ': ' . $sessdata->groups[$sess->groupid]->name;
+ } else {
+ $table->data[$sess->id][] = get_string('commonsession', 'attendance');
+ }
+
$table->data[$sess->id][] = $dta['date'];
$table->data[$sess->id][] = $dta['time'];
$table->data[$sess->id][] = $sess->description;
$row = new html_table_row();
$row->cells[] = $i;
- $sessiontypeshort = $sess->groupid ? get_string('group') . ': ' . $userdata->groups[$sess->groupid]->name
- : get_string('commonsession', 'attendance');
+ if ($sess->groupid) {
+ $sessiontypeshort = get_string('group') . ': ' . $userdata->groups[$sess->groupid]->name;
+ } else {
+ $sessiontypeshort = get_string('commonsession', 'attendance');
+ }
+
$row->cells[] = html_writer::tag('nobr', $sessiontypeshort);
$row->cells[] = userdate($sess->sessdate, get_string('strftimedmyw', 'attendance'));
$row->cells[] = $this->construct_time($sess->sessdate, $sess->duration);
$sesstext = html_writer::link($reportdata->url_take($sess->id, $sess->groupid), $sesstext);
}
$sesstext .= html_writer::empty_tag('br');
- $sesstext .= $sess->groupid ? get_string('group') . ': ' . $reportdata->groups[$sess->groupid]->name
- : get_string('commonsession', 'attendance');
+ if ($sess->groupid) {
+ $sesstext .= get_string('group') . ': ' . $reportdata->groups[$sess->groupid]->name;
+ } else {
+ $sesstext .= get_string('commonsession', 'attendance');
+ }
$table->head[] = $sesstext;
$table->align[] = 'center';
if ($formdata = $mform->get_data()) {
$sessions = construct_sessions_data_for_add($formdata);
$att->add_sessions($sessions);
- $message = count($sessions) == 1 ? get_string('sessiongenerated', 'attendance') :
- get_string('sessionsgenerated', 'attendance', count($sessions));
+ if (count($sessions) == 1) {
+ $message = get_string('sessiongenerated', 'attendance');
+ } else {
+ $message = get_string('sessionsgenerated', 'attendance', count($sessions));
+ }
+
mod_attendance_notifyqueue::notify_success($message);
// Redirect to the sessions tab always showing all sessions.
$SESSION->attcurrentattview[$cm->course] = ATT_VIEW_ALL;
And I follow "Add"
And I set the following fields to these values:
| id_addmultiply | 0 |
- | Use status set | Status set 1 (P L E A) |
+ | Status set | Status set 1 (P L E A) |
| id_sestime_starthour | 10 |
| id_sestime_startminute | 0 |
And I click on "submitbutton" "button"
And I follow "Add"
And I set the following fields to these values:
| id_addmultiply | 0 |
- | Use status set | Status set 2 (G O B) |
+ | Status set | Status set 2 (G O B) |
| id_sestime_starthour | 11 |
| id_sestime_startminute | 0 |
And I click on "submitbutton" "button"