From bc61aa40853a7f0a810d97355e0118d764075d43 Mon Sep 17 00:00:00 2001 From: Petr Skoda Date: Wed, 16 Mar 2011 19:59:33 +0100 Subject: [PATCH 1/2] MDL-26803 add missing forcing of password reset when new password created in cron --- lib/cronlib.php | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/cronlib.php b/lib/cronlib.php index 11d478acb7382..ea7c11507d5a3 100644 --- a/lib/cronlib.php +++ b/lib/cronlib.php @@ -287,22 +287,23 @@ function cron_run() { mtrace('checking for create_password'); if ($DB->count_records('user_preferences', array('name'=>'create_password', 'value'=>'1'))) { mtrace('creating passwords for new users'); - $newusers = $DB->get_records_sql("SELECT u.id as id, u.email, u.firstname, + $newusers = $DB->get_recordset_sql("SELECT u.id as id, u.email, u.firstname, u.lastname, u.username, p.id as prefid FROM {user} u JOIN {user_preferences} p ON u.id=p.userid WHERE p.name='create_password' AND p.value='1' AND u.email !='' "); - foreach ($newusers as $newuserid => $newuser) { + foreach ($newusers as $newuser) { // email user if (setnew_password_and_mail($newuser)) { - // remove user pref - $DB->delete_records('user_preferences', array('id'=>$newuser->prefid)); + unset_user_preference('create_password', $newuser); + set_user_preference('auth_forcepasswordchange', 1, $newuser); } else { trigger_error("Could not create and mail new user password!"); } } + $newusers->close(); } if (!empty($CFG->usetags)) { From 22a77963439e00441949440f0517135b3a5418da Mon Sep 17 00:00:00 2001 From: Petr Skoda Date: Wed, 16 Mar 2011 20:00:06 +0100 Subject: [PATCH 2/2] MDL-26803 fix switched values of autosubscribe defaults in user upload --- admin/uploaduser_form.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/admin/uploaduser_form.php b/admin/uploaduser_form.php index fe5311698abda..af0dd74cbf556 100644 --- a/admin/uploaduser_form.php +++ b/admin/uploaduser_form.php @@ -231,7 +231,7 @@ function definition () { $mform->setDefault('maildigest', 0); $mform->setAdvanced('maildigest'); - $choices = array(0 => get_string('autosubscribeyes'), 1 => get_string('autosubscribeno')); + $choices = array(1 => get_string('autosubscribeyes'), 0 => get_string('autosubscribeno')); $mform->addElement('select', 'autosubscribe', get_string('autosubscribe'), $choices); $mform->setDefault('autosubscribe', 1);