From 14217044a5cc931428efe86ac698ff4f1b139bf2 Mon Sep 17 00:00:00 2001 From: moodler Date: Thu, 2 Jan 2003 15:25:51 +0000 Subject: [PATCH] Some little fixes --- admin/auth.php | 3 ++- admin/config.php | 4 ++-- course/editsection.php | 2 +- lib/moodlelib.php | 2 +- login/change_password.php | 7 ++++--- login/forgot_password.php | 4 ++-- login/signup.php | 2 +- 7 files changed, 13 insertions(+), 11 deletions(-) diff --git a/admin/auth.php b/admin/auth.php index ea3da976e1..cf72f160a7 100644 --- a/admin/auth.php +++ b/admin/auth.php @@ -18,6 +18,7 @@ if ($config = data_submitted()) { + $config = (array)$config; validate_form($config, $err); if (count($err) == 0) { @@ -39,7 +40,7 @@ /// Otherwise fill and print the form. - if (!isset($config)) { + if (empty($config)) { $config = $CFG; } diff --git a/admin/config.php b/admin/config.php index 48a9b9c1f4..cb3430e0eb 100644 --- a/admin/config.php +++ b/admin/config.php @@ -68,10 +68,10 @@ /// Otherwise fill and print the form. - if (!isset($config)) { + if (empty($config)) { $config = $CFG; } - if (!isset($focus)) { + if (empty($focus)) { $focus = ""; } diff --git a/course/editsection.php b/course/editsection.php index 2d2169cf96..eadcdac5f2 100644 --- a/course/editsection.php +++ b/course/editsection.php @@ -39,7 +39,7 @@ /// Otherwise fill and print the form. - if (! $form ) { + if (empty($form)) { $form = $section; } diff --git a/lib/moodlelib.php b/lib/moodlelib.php index 599fe5329f..5c9e962bef 100644 --- a/lib/moodlelib.php +++ b/lib/moodlelib.php @@ -508,7 +508,7 @@ function authenticate_user_login($username, $password) { $md5password = md5($password); - if (!isset($CFG->auth)) { + if (empty($CFG->auth)) { $CFG->auth = "email"; // Default authentication module } diff --git a/login/change_password.php b/login/change_password.php index e9a6071e9f..27ce2cf7ae 100644 --- a/login/change_password.php +++ b/login/change_password.php @@ -65,17 +65,18 @@ $frm->id = $id; } - if (!$frm->username) + if (empty($frm->username)) { $frm->username = get_moodle_cookie(); + } - if ($frm->username) { + if (!empty($frm->username)) { $focus = "form.password"; } else { $focus = "form.username"; } $strchangepassword = get_string("changepassword"); - if ($course->id) { + if (!empty($course->id)) { print_header($strchangepassword, $strchangepassword, "wwwroot/course/view.php?id=$course->id\">$course->shortname -> wwwroot/user/index.php?id=$course->id\">".get_string("participants")." -> diff --git a/login/forgot_password.php b/login/forgot_password.php index 94cf72793a..ec5ffc7320 100644 --- a/login/forgot_password.php +++ b/login/forgot_password.php @@ -34,8 +34,8 @@ } } - if ( empty($frm->email) ) { - if ( $username = get_moodle_cookie() ) { + if (empty($frm->email)) { + if ($username = get_moodle_cookie() ) { $frm->email = get_field("user", "email", "username", "$username"); } } diff --git a/login/signup.php b/login/signup.php index b3fa090b1f..f07d3c509b 100644 --- a/login/signup.php +++ b/login/signup.php @@ -37,7 +37,7 @@ $focus = 'form.' . array_shift(array_flip(get_object_vars($err))); } - if (!$user->country and $CFG->country) { + if (empty($user->country) and !empty($CFG->country)) { $user->country = $CFG->country; } -- 2.39.5