Commit 4324551b authored by Leigh B. Stoller's avatar Leigh B. Stoller
Browse files

Minor big fix for returning users and form argument checks.

parent 73fe5ec9
...@@ -391,8 +391,7 @@ if (! isset($_POST['submit'])) { ...@@ -391,8 +391,7 @@ if (! isset($_POST['submit'])) {
else { else {
# Form submitted. Make sure we have a formfields array and a target_uid. # Form submitted. Make sure we have a formfields array and a target_uid.
if (!isset($_POST['formfields']) || if (!isset($_POST['formfields']) ||
!is_array($_POST['formfields']) || !is_array($_POST['formfields'])) {
!isset($_POST['formfields']['joining_uid'])) {
PAGEARGERROR("Invalid form arguments."); PAGEARGERROR("Invalid form arguments.");
} }
$formfields = $_POST['formfields']; $formfields = $_POST['formfields'];
...@@ -733,7 +732,8 @@ $leader_name = $row[0]; ...@@ -733,7 +732,8 @@ $leader_name = $row[0];
$leader_email = $row[1]; $leader_email = $row[1];
$leader_uid = $row[2]; $leader_uid = $row[2];
$allleaders = TBLeaderMailList($pid,$gid); if (($allleaders = TBLeaderMailList($pid,$gid)) == 0)
$allleaders = "";
# #
# The mail message to the leader. We send this for returning users # The mail message to the leader. We send this for returning users
......
...@@ -546,8 +546,7 @@ if (! isset($_POST['submit'])) { ...@@ -546,8 +546,7 @@ if (! isset($_POST['submit'])) {
else { else {
# Form submitted. Make sure we have a formfields array and a target_uid. # Form submitted. Make sure we have a formfields array and a target_uid.
if (!isset($_POST['formfields']) || if (!isset($_POST['formfields']) ||
!is_array($_POST['formfields']) || !is_array($_POST['formfields'])) {
!isset($_POST['formfields']['proj_head_uid'])) {
PAGEARGERROR("Invalid form arguments."); PAGEARGERROR("Invalid form arguments.");
} }
$formfields = $_POST['formfields']; $formfields = $_POST['formfields'];
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment