Skip to content

Commit

Permalink
MDL-35078 enrol: allow times to be chose on bulk enrolment forms
Browse files Browse the repository at this point in the history
Previously only a date was available, which made it ambigious whether
the the enrolment was inclusive of the date. Now we use a time to make
it less ambigious.

This matches changes done in other forms in MDL-41173.
  • Loading branch information
wiktorwandachowicz authored and danpoltawski committed Dec 17, 2014
1 parent 8f06fff commit 900a559
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions enrol/bulkchange_forms.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ protected function definition() {
$statusoptions = $this->get_status_options();
$form->addElement('html', $this->get_users_table($users, $statusoptions));
$form->addElement('select', 'status', get_string('alterstatus', 'enrol_manual'), $statusoptions, array('optional' => true));
$form->addElement('date_selector', 'timestart', get_string('altertimestart', 'enrol_manual'), array('optional' => true));
$form->addElement('date_selector', 'timeend', get_string('altertimeend', 'enrol_manual'), array('optional' => true));
$form->addElement('date_time_selector', 'timestart', get_string('altertimestart', 'enrol_manual'), array('optional' => true));
$form->addElement('date_time_selector', 'timeend', get_string('altertimeend', 'enrol_manual'), array('optional' => true));

$this->add_action_buttons();
}
Expand Down

0 comments on commit 900a559

Please sign in to comment.