]> git.mjollnir.org Git - moodle.git/commitdiff
fixes for extend enrol to make it less restrictive MDL-10181
authortoyomoyo <toyomoyo>
Wed, 4 Jul 2007 02:38:24 +0000 (02:38 +0000)
committertoyomoyo <toyomoyo>
Wed, 4 Jul 2007 02:38:24 +0000 (02:38 +0000)
user/extendenrol.php
user/groupextendenrol.php

index f0ae686917c46894325a1514afc3015f2fc4a329..a13a226595ddaffafe7625a728a3f330226d651b 100644 (file)
@@ -13,7 +13,8 @@ require_login($course->id);
 
 // to extend enrolments current user needs to be able to do role assignments
 require_capability('moodle/role:assign', $context);
-
+$today = time();
+$today = make_timestamp(date('Y', $today), date('m', $today), date('d', $today), 0, 0, 0);
 if ((count($users) > 0) and ($form = data_submitted()) and confirm_sesskey()) {
     if (count($form->userid) != count($form->extendperiod) || count($form->userid) != count($form->extendbase)) {
         error('Parameters malformation', $CFG->wwwroot.'/user/index.php?id='.$id);
@@ -55,9 +56,7 @@ if ((count($users) > 0) and ($form = data_submitted()) and confirm_sesskey()) {
                                     }
                                     break;
                                 case 3: // current date
-                                    if(time() > $course->startdate) {
-                                        $student->timeend = time() + $form->extendperiod[$k];
-                                    }
+                                    $student->timeend = $today + $form->extendperiod[$k];
                                     break;
                                 case 4: // course enrolment start date
                                     if($course->enrolstartdate > 0) {
@@ -103,8 +102,8 @@ for ($i=1; $i<=365; $i++) {
 $basemenu[0] = get_string('startdate') . ' (' . userdate($course->startdate, $timeformat) . ')';
 $basemenu[1] = get_string('enrolmentstart');
 $basemenu[2] = get_string('enrolmentend');
-if(time() > $course->startdate) {
-    $basemenu[3] = get_string('today') . ' (' . userdate(time(), $timeformat) . ')' ;
+if($course->enrollable != 2 || ($course->enrolstartdate == 0 || $course->enrolstartdate <= $today) && ($course->enrolenddate == 0 || $course->enrolenddate > $today)) {
+    $basemenu[3] = get_string('today') . ' (' . userdate($today, $timeformat) . ')' ;
 }
 if($course->enrollable == 2) {
     if($course->enrolstartdate > 0) {
index 23d1b56cde6af4996d55ad0041c3ec8e37afa949..18b6c78087fddbe00ef376e52c0235776605fe18 100755 (executable)
-<?php  // $Id$\r
-require_once("../config.php");\r
-\r
-$id    = required_param('id', PARAM_INT);              // course id\r
-$users = optional_param('userid', array(), PARAM_INT); // array of user id\r
-\r
-if (! $course = get_record('course', 'id', $id)) {\r
-    error("Course ID is incorrect");\r
-}\r
-\r
-$context = get_context_instance(CONTEXT_COURSE, $id);\r
-require_login($course->id);\r
-\r
-// to extend enrolments current user needs to be able to do role assignments\r
-require_capability('moodle/role:assign', $context);\r
-\r
-if ((count($users) > 0) and ($form = data_submitted()) and confirm_sesskey()) {\r
-\r
-    foreach ($form->userid as $k => $v) {\r
-        // find all roles this student have in this course  \r
-        if ($students = get_records_sql("SELECT ra.id, ra.roleid, ra.timestart, ra.timeend\r
-                                       FROM {$CFG->prefix}role_assignments ra\r
-                                       WHERE userid = $v\r
-                                       AND contextid = $context->id")) {\r
-            // enrol these users again, with time extension\r
-            // not that this is not necessarily a student role\r
-            foreach ($students as $student) {\r
-                // only extend if the user can make role assignments on this role\r
-                if (user_can_assign($context, $student->roleid)) {\r
-                    switch($form->extendperiod) {\r
-                        case 0: // No change (currently this option is not available in dropdown list, but it might be ...)\r
-                            break;\r
-                        case -1: // unlimited\r
-                            $student->timeend = 0;\r
-                            break;\r
-                        default: // extend\r
-                            switch($form->extendbase) {\r
-                                case 0: // course start date\r
-                                    $student->timeend = $course->startdate + $form->extendperiod;\r
-                                    break;\r
-                                case 1: // student enrolment start date\r
-                                    // we check for student enrolment date because Moodle versions before 1.9 did not set this for\r
-                                    // unlimited enrolment courses, so it might be 0\r
-                                    if($student->timestart > 0) {\r
-                                        $student->timeend = $student->timestart + $form->extendperiod;\r
-                                    }\r
-                                    break;\r
-                                case 2: // student enrolment start date\r
-                                    // enrolment end equals 0 means Unlimited, so adding some time to that will still yield Unlimited\r
-                                    if($student->timeend > 0) {\r
-                                        $student->timeend = $student->timeend + $form->extendperiod;\r
-                                    }\r
-                                    break;\r
-                                case 3: // current date\r
-                                    if(time() > $course->startdate) {\r
-                                        $student->timeend = time() + $form->extendperiod;\r
-                                    }\r
-                                    break;\r
-                                case 4: // course enrolment start date\r
-                                    if($course->enrolstartdate > 0) {\r
-                                        $student->timeend = $course->enrolstartdate + $form->extendperiod;\r
-                                    }\r
-                                    break;\r
-                                case 5: // course enrolment end date\r
-                                    if($course->enrolenddate > 0) {\r
-                                        $student->timeend = $course->enrolenddate + $form->extendperiod;\r
-                                    }\r
-                                    break;\r
-                            }\r
-                    } \r
-                    role_assign($student->roleid, $v, 0, $context->id, $student->timestart, $student->timeend, 0);\r
-                }\r
-            }\r
-        }\r
-    }\r
-\r
-    redirect("$CFG->wwwroot/user/index.php?id=$id", get_string('changessaved'));\r
-}\r
-\r
-/// Print headers\r
-\r
-if ($course->id != SITEID) {\r
-    print_header("$course->shortname: ".get_string('extendenrol'), $course->fullname,\r
-    "<a href=\"../course/view.php?id=$course->id\">$course->shortname</a> -> ".\r
-    get_string('extendenrol'), "", "", true, "&nbsp;", navmenu($course));\r
-} else {\r
-    print_header("$course->shortname: ".get_string('extendenrol'), $course->fullname,\r
-    get_string('extendenrol'), "", "", true, "&nbsp;", navmenu($course));\r
-}\r
-\r
-$timeformat = get_string('strftimedate');\r
-$unlimited = get_string('unlimited');\r
-$periodmenu[-1] = $unlimited;\r
-for ($i=1; $i<=365; $i++) {\r
-    $seconds = $i * 86400;\r
-    $periodmenu[$seconds] = get_string('numdays', '', $i);\r
-}\r
-\r
-$basemenu[0] = get_string('startdate') . ' (' . userdate($course->startdate, $timeformat) . ')';\r
-$basemenu[1] = get_string('enrolmentstart');\r
-$basemenu[2] = get_string('enrolmentend');\r
-if(time() > $course->startdate) {\r
-    $basemenu[3] = get_string('today') . ' (' . userdate(time(), $timeformat) . ')' ;\r
-}\r
-if($course->enrollable == 2) {\r
-    if($course->enrolstartdate > 0) {\r
-        $basemenu[4] = get_string('courseenrolstartdate') . ' (' . userdate($course->enrolstartdate, $timeformat) . ')';\r
-    }\r
-    if($course->enrolenddate > 0) {\r
-        $basemenu[5] = get_string('courseenrolenddate') . ' (' . userdate($course->enrolenddate, $timeformat) . ')';\r
-    }\r
-}\r
-\r
-$title = get_string('groupextendenrol');\r
-print_heading($title . helpbutton('groupextendenrol', $title, 'moodle', true, false, '', true));\r
-echo '<form method="post" action="groupextendenrol.php">';\r
-echo '<input type="hidden" name="id" value="'.$course->id.'" />';\r
-echo '<input type="hidden" name="sesskey" value="'.$USER->sesskey.'" />';\r
-$table->head  = array (get_string('fullname'), get_string('enrolmentstart'), get_string('enrolmentend'));\r
-$table->align = array ('left', 'center', 'center', 'center');\r
-$table->width = "600";\r
-$nochange = get_string('nochange');\r
-$notavailable = get_string('notavailable');\r
-\r
-foreach ($_POST as $k => $v) {\r
-    if (preg_match('/^user(\d+)$/',$k,$m)) {\r
-        \r
-        if (!($user = get_record_sql("SELECT * FROM {$CFG->prefix}user u \r
-                                    INNER JOIN {$CFG->prefix}role_assignments ra ON u.id=ra.userid \r
-                                    WHERE u.id={$m[1]} AND ra.contextid = $context->id"))) {\r
-            continue;\r
-        }\r
-        if ($user->timestart) {\r
-            $timestart = userdate($user->timestart, $timeformat);\r
-        } else {\r
-            $timestart = $notavailable;\r
-        }\r
-        if ($user->timeend) {\r
-            $timeend = userdate($user->timeend, $timeformat);\r
-        } else {\r
-            $timeend = $unlimited;\r
-        }\r
-        $table->data[] = array(\r
-        fullname($user, true),\r
-        $timestart,\r
-        $timeend . '<input type="hidden" name="userid['.$m[1].']" value="'.$m[1].'" />'\r
-        );\r
-    }\r
-}\r
-print_table($table);\r
-echo '<div style="width:100%;text-align:center;"><strong>';\r
-echo get_string('extendperiod') . ' ';\r
-choose_from_menu($periodmenu, "extendperiod");\r
-echo ' ' . get_string('startingfrom') . ' ';\r
-choose_from_menu($basemenu, "extendbase", "2", "");\r
-echo '</strong><br />';\r
-echo '<input type="submit" value="'.get_string('savechanges').'" />';\r
-echo '</div></form>';\r
-\r
-print_footer($course);\r
+<?php  // $Id$
+require_once("../config.php");
+
+$id    = required_param('id', PARAM_INT);              // course id
+$users = optional_param('userid', array(), PARAM_INT); // array of user id
+
+if (! $course = get_record('course', 'id', $id)) {
+    error("Course ID is incorrect");
+}
+
+$context = get_context_instance(CONTEXT_COURSE, $id);
+require_login($course->id);
+
+// to extend enrolments current user needs to be able to do role assignments
+require_capability('moodle/role:assign', $context);
+$today = time();
+$today = make_timestamp(date('Y', $today), date('m', $today), date('d', $today), 0, 0, 0);
+
+if ((count($users) > 0) and ($form = data_submitted()) and confirm_sesskey()) {
+
+    foreach ($form->userid as $k => $v) {
+        // find all roles this student have in this course  
+        if ($students = get_records_sql("SELECT ra.id, ra.roleid, ra.timestart, ra.timeend
+                                       FROM {$CFG->prefix}role_assignments ra
+                                       WHERE userid = $v
+                                       AND contextid = $context->id")) {
+            // enrol these users again, with time extension
+            // not that this is not necessarily a student role
+            foreach ($students as $student) {
+                // only extend if the user can make role assignments on this role
+                if (user_can_assign($context, $student->roleid)) {
+                    switch($form->extendperiod) {
+                        case 0: // No change (currently this option is not available in dropdown list, but it might be ...)
+                            break;
+                        case -1: // unlimited
+                            $student->timeend = 0;
+                            break;
+                        default: // extend
+                            switch($form->extendbase) {
+                                case 0: // course start date
+                                    $student->timeend = $course->startdate + $form->extendperiod;
+                                    break;
+                                case 1: // student enrolment start date
+                                    // we check for student enrolment date because Moodle versions before 1.9 did not set this for
+                                    // unlimited enrolment courses, so it might be 0
+                                    if($student->timestart > 0) {
+                                        $student->timeend = $student->timestart + $form->extendperiod;
+                                    }
+                                    break;
+                                case 2: // student enrolment start date
+                                    // enrolment end equals 0 means Unlimited, so adding some time to that will still yield Unlimited
+                                    if($student->timeend > 0) {
+                                        $student->timeend = $student->timeend + $form->extendperiod;
+                                    }
+                                    break;
+                                case 3: // current date
+                                    $student->timeend = $today + $form->extendperiod;
+                                    break;
+                                case 4: // course enrolment start date
+                                    if($course->enrolstartdate > 0) {
+                                        $student->timeend = $course->enrolstartdate + $form->extendperiod;
+                                    }
+                                    break;
+                                case 5: // course enrolment end date
+                                    if($course->enrolenddate > 0) {
+                                        $student->timeend = $course->enrolenddate + $form->extendperiod;
+                                    }
+                                    break;
+                            }
+                    } 
+                    role_assign($student->roleid, $v, 0, $context->id, $student->timestart, $student->timeend, 0);
+                }
+            }
+        }
+    }
+
+    redirect("$CFG->wwwroot/user/index.php?id=$id", get_string('changessaved'));
+}
+
+/// Print headers
+
+if ($course->id != SITEID) {
+    print_header("$course->shortname: ".get_string('extendenrol'), $course->fullname,
+    "<a href=\"../course/view.php?id=$course->id\">$course->shortname</a> -> ".
+    get_string('extendenrol'), "", "", true, "&nbsp;", navmenu($course));
+} else {
+    print_header("$course->shortname: ".get_string('extendenrol'), $course->fullname,
+    get_string('extendenrol'), "", "", true, "&nbsp;", navmenu($course));
+}
+
+$timeformat = get_string('strftimedate');
+$unlimited = get_string('unlimited');
+$periodmenu[-1] = $unlimited;
+for ($i=1; $i<=365; $i++) {
+    $seconds = $i * 86400;
+    $periodmenu[$seconds] = get_string('numdays', '', $i);
+}
+
+$basemenu[0] = get_string('startdate') . ' (' . userdate($course->startdate, $timeformat) . ')';
+$basemenu[1] = get_string('enrolmentstart');
+$basemenu[2] = get_string('enrolmentend');
+if($course->enrollable != 2 || ($course->enrolstartdate == 0 || $course->enrolstartdate <= $today) && ($course->enrolenddate == 0 || $course->enrolenddate > $today)) {
+    $basemenu[3] = get_string('today') . ' (' . userdate($today, $timeformat) . ')' ;
+}
+if($course->enrollable == 2) {
+    if($course->enrolstartdate > 0) {
+        $basemenu[4] = get_string('courseenrolstartdate') . ' (' . userdate($course->enrolstartdate, $timeformat) . ')';
+    }
+    if($course->enrolenddate > 0) {
+        $basemenu[5] = get_string('courseenrolenddate') . ' (' . userdate($course->enrolenddate, $timeformat) . ')';
+    }
+}
+
+$title = get_string('groupextendenrol');
+print_heading($title . helpbutton('groupextendenrol', $title, 'moodle', true, false, '', true));
+echo '<form method="post" action="groupextendenrol.php">';
+echo '<input type="hidden" name="id" value="'.$course->id.'" />';
+echo '<input type="hidden" name="sesskey" value="'.$USER->sesskey.'" />';
+$table->head  = array (get_string('fullname'), get_string('enrolmentstart'), get_string('enrolmentend'));
+$table->align = array ('left', 'center', 'center', 'center');
+$table->width = "600";
+$nochange = get_string('nochange');
+$notavailable = get_string('notavailable');
+
+foreach ($_POST as $k => $v) {
+    if (preg_match('/^user(\d+)$/',$k,$m)) {
+        
+        if (!($user = get_record_sql("SELECT * FROM {$CFG->prefix}user u 
+                                    INNER JOIN {$CFG->prefix}role_assignments ra ON u.id=ra.userid 
+                                    WHERE u.id={$m[1]} AND ra.contextid = $context->id"))) {
+            continue;
+        }
+        if ($user->timestart) {
+            $timestart = userdate($user->timestart, $timeformat);
+        } else {
+            $timestart = $notavailable;
+        }
+        if ($user->timeend) {
+            $timeend = userdate($user->timeend, $timeformat);
+        } else {
+            $timeend = $unlimited;
+        }
+        $table->data[] = array(
+        fullname($user, true),
+        $timestart,
+        $timeend . '<input type="hidden" name="userid['.$m[1].']" value="'.$m[1].'" />'
+        );
+    }
+}
+print_table($table);
+echo '<div style="width:100%;text-align:center;"><strong>';
+echo get_string('extendperiod') . ' ';
+choose_from_menu($periodmenu, "extendperiod");
+echo ' ' . get_string('startingfrom') . ' ';
+choose_from_menu($basemenu, "extendbase", "2", "");
+echo '</strong><br />';
+echo '<input type="submit" value="'.get_string('savechanges').'" />';
+echo '</div></form>';
+
+print_footer($course);
 ?>
\ No newline at end of file