From 9443af56db58b91de407d014a57485ab27ff16a3 Mon Sep 17 00:00:00 2001 From: demeritcowboy Date: Mon, 8 Aug 2022 16:56:00 -0400 Subject: [PATCH] should null and '' be treated the same here? --- CRM/Utils/System/UnitTests.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/CRM/Utils/System/UnitTests.php b/CRM/Utils/System/UnitTests.php index 547b93f98d..868367da48 100644 --- a/CRM/Utils/System/UnitTests.php +++ b/CRM/Utils/System/UnitTests.php @@ -103,8 +103,8 @@ class CRM_Utils_System_UnitTests extends CRM_Utils_System_Base { $separator = ($htmlize && $frontend) ? '&' : '&'; if (!$config->cleanURL) { - if (isset($path)) { - if (isset($query)) { + if ($path !== NULL && $path !== '' && $path !== FALSE) { + if ($query !== NULL && $query !== '' && $query !== FALSE) { return $base . $script . '?q=' . $path . $separator . $query . $fragment; } else { @@ -112,7 +112,7 @@ class CRM_Utils_System_UnitTests extends CRM_Utils_System_Base { } } else { - if (isset($query)) { + if ($query !== NULL && $query !== '' && $query !== FALSE) { return $base . $script . '?' . $query . $fragment; } else { -- 2.25.1