[RESOLVED] Check Input Against List of Acceptable Answers

rsmith

Hi all,
Here's my question. I have a form that's used to authorize the addition of a new user to the network. The form must be filled out by 1 of 8 branch managers. I'm running into a wall trying to figure out how I would take the name of the person requesting the addition and compare it to a list of names to make sure they have the authority to submit. If their name is not on the list, they get redirected to an error stating so (I have that part done). Would I set the names of the managers into an array and iterate through it by comparing the submitted name to the list? If so, how would I go about doing that? Here's what I have so far...

<?php
$mgrArray = ("Richard","Mary","Mark");

	if(isset($approvedBy)){
		if(!in_array($approvedBy, $mgrArray)){
			echo "You are not authorized to make this request.";
		}
	}
?>

TIA for any help!

NogDog

You might want to merge the isset() and in_array() into one conditional:

if(!isset($approvedBy) or !in_array($approvedBy, $mgrArray))
{
   // error message
}

rsmith

Thanks I'll try that. It makes more sense that way but now I'm running into an unexpected ',' in the line that specifies the array? I don't see any syntax errors.

NogDog

Make sure the quotes are balanced, that there is a comma between each element, and that there is not a comma after the last element. (Those are the most common mistakes I make. 🙂 )

rsmith

Same here. I've quadruple checked it though. The only thing is I have the person's last name as well such as "John Doe", "Jane Doe" etc.

bpat1434

You need the keyword "array" prior to the opening parenthesis "(" on your second line of code:

$mgrArray = array("Richard","Mary","Mark");

NogDog

bpat1434 wrote:
You need the keyword "array" prior to the opening parenthesis "(" on your second line of code:
$mgrArray = array("Richard","Mary","Mark");

LOL - I didn't even look at the original post. For some reason I assumed it was new code written since then. :rolleyes:

rsmith

Wow I'm a genius. Hahaha. Chalk this up to one of those times you've been working so long you can't see the simple answer. Thanks!