revert unnecessary changes

simonLeary42 · simonLeary42 · commit 8f2b2c629ea2 · 2025-05-23T16:05:18.000-04:00
diff --git a/webroot/panel/new_account.php b/webroot/panel/new_account.php
@@ -14,7 +14,7 @@
 
 if ($_SERVER["REQUEST_METHOD"] == "POST") {
     if (isset($_POST["new_user_sel"])) {
-        if (($_POST["eula"] ?? "disagree") != "agree") {
+        if (!isset($_POST["eula"]) || $_POST["eula"] != "agree") {
             UnitySite::badRequest("user did not agree to EULA");
         }
         if ($_POST["new_user_sel"] == "not_pi") {
@@ -25,7 +25,7 @@
             $form_group->newUserRequest($USER);
         }
         if ($_POST["new_user_sel"] == "pi") {
-            if (($_POST["confirm_pi"] ?? "disagree") != "agree") {
+        if (!isset($_POST["confirm_pi"]) || $_POST["confirm_pi"] != "agree") {
                 UnitySite::badRequest("user did not agree to account policy");
             }
             $USER->getPIGroup()->requestGroup($SEND_PIMESG_TO_ADMINS);

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`
`15`	`15`	`if ($_SERVER["REQUEST_METHOD"] == "POST") {`
`16`	`16`	`if (isset($_POST["new_user_sel"])) {`
`17`		`- if (($_POST["eula"] ?? "disagree") != "agree") {`
	`17`	`+ if (!isset($_POST["eula"]) \|\| $_POST["eula"] != "agree") {`
`18`	`18`	`UnitySite::badRequest("user did not agree to EULA");`
`19`	`19`	`}`
`20`	`20`	`if ($_POST["new_user_sel"] == "not_pi") {`
`@@ -25,7 +25,7 @@`
`25`	`25`	`$form_group->newUserRequest($USER);`
`26`	`26`	`}`
`27`	`27`	`if ($_POST["new_user_sel"] == "pi") {`
`28`		`- if (($_POST["confirm_pi"] ?? "disagree") != "agree") {`
	`28`	`+ if (!isset($_POST["confirm_pi"]) \|\| $_POST["confirm_pi"] != "agree") {`
`29`	`29`	`UnitySite::badRequest("user did not agree to account policy");`
`30`	`30`	`}`
`31`	`31`	`$USER->getPIGroup()->requestGroup($SEND_PIMESG_TO_ADMINS);`