diff options
author | Andrew Dolgov <[email protected]> | 2021-02-06 00:12:15 +0300 |
---|---|---|
committer | Andrew Dolgov <[email protected]> | 2021-02-06 00:12:15 +0300 |
commit | 6e774a58fe6c509e2cd16d222ed414edc9f10b17 (patch) | |
tree | b6a6d054677596b3ef79e806a44f7f8931dd84d6 /include | |
parent | 403dca154c6b539de221f9e16174a0fdd0a1e896 (diff) |
more php8 fixes mostly related to login
Diffstat (limited to 'include')
-rw-r--r-- | include/functions.php | 2 | ||||
-rwxr-xr-x | include/login_form.php | 6 | ||||
-rw-r--r-- | include/sessions.php | 2 |
3 files changed, 5 insertions, 5 deletions
diff --git a/include/functions.php b/include/functions.php index 0ec0ba30b..2db559a92 100644 --- a/include/functions.php +++ b/include/functions.php @@ -148,7 +148,7 @@ $lang = _TRANSLATION_OVERRIDE_DEFAULT; } - if ($_SESSION["uid"] && get_schema_version() >= 120) { + if (!empty($_SESSION["uid"]) && get_schema_version() >= 120) { $pref_lang = get_pref("USER_LANGUAGE", $_SESSION["uid"]); if ($pref_lang && $pref_lang != 'auto') { diff --git a/include/login_form.php b/include/login_form.php index 586d6fe78..0e8f8389e 100755 --- a/include/login_form.php +++ b/include/login_form.php @@ -99,7 +99,7 @@ <?php print_hidden("op", "login"); ?> - <?php if ($_SESSION["login_error_msg"]) { ?> + <?php if (!empty($_SESSION["login_error_msg"])) { ?> <?php echo format_error($_SESSION["login_error_msg"]) ?> <?php $_SESSION["login_error_msg"] = ""; ?> <?php } ?> @@ -110,7 +110,7 @@ onchange="UtilityApp.fetchProfiles()" onfocus="UtilityApp.fetchProfiles()" onblur="UtilityApp.fetchProfiles()" - required="1" value="<?php echo $_SESSION["fake_login"] ?>" /> + required="1" value="<?php echo $_SESSION["fake_login"] ?? "" ?>" /> </fieldset> <fieldset> @@ -122,7 +122,7 @@ onchange="UtilityApp.fetchProfiles()" onfocus="UtilityApp.fetchProfiles()" onblur="UtilityApp.fetchProfiles()" - value="<?php echo $_SESSION["fake_password"] ?>"/> + value="<?php echo $_SESSION["fake_password"] ?? "" ?>"/> </fieldset> <?php if (strpos(PLUGINS, "auth_internal") !== false) { ?> <fieldset class="align-right"> diff --git a/include/sessions.php b/include/sessions.php index 75d4671e8..15725c1f9 100644 --- a/include/sessions.php +++ b/include/sessions.php @@ -46,7 +46,7 @@ } $pdo = Db::pdo(); - if ($_SESSION["uid"]) { + if (!empty($_SESSION["uid"])) { if (!defined('_SESSION_SKIP_UA_CHECKS') && $_SESSION["user_agent"] != sha1($_SERVER['HTTP_USER_AGENT'])) { $_SESSION["login_error_msg"] = __("Session failed to validate (UA changed)."); |