Solved

Help with trivial validation function

Posted on 2009-07-06
4
228 Views
Last Modified: 2013-12-13
Hi everyone,
can anyone please help me do this right? I am trying to validate a form with category id, of which list I pull out of the DB to make sure the id is valid. But seem to get errors of invalid argument for "foreach". I am attaching the function below. I am a sort of beginner, so I am sure that everyone will have much better solution, but I am trying to do it myself (with this exception) and just need to find an error in my script.. not an entire new solution. that will be next :-)
$category_id_errors = check_cagegory_id(array('category')); 
 
 

function check_cagegory_id($check_category_id) {  //this is the funtion referred to in the code above

	$form_errors = array();

	

	$subject_set = get_all_subjects();

	if(mysql_num_rows($subject_set)>=1) {

		$subjects = array();

		while($subject = mysql_fetch_array($subject_set)) {

			$subjects[] = $subject['id'];

		}

		foreach($check_cagegory_id as $fieldname) {

			if((!isset($_POST[$fieldname]))||(!in_array($_POST[$fieldname], $subjects))) {

				$form_errors[] = $fieldname ."doesn't exist";

			}

		}

	} else {

		$form_errors[] = $fieldname." - no category to associate a new page with";

	}

	return $form_errors;

Open in new window

0
Comment
Question by:czechmate1976
  • 2
  • 2
4 Comments
 
LVL 12

Expert Comment

by:kevin_u
ID: 24786760
In the foreach,

$check_cagegory_id

is spelled wrong

It should be:

$check_category_id

0
 

Author Comment

by:czechmate1976
ID: 24787260
Thanks, I corrected it but still get "Warning: Invalid argument supplied for foreach() in C:\xampplite\htdocs\FGEnew\siteadmin\includes\functions.inc.php on line 218"

which is the following:
foreach($check_category_id as $fieldname) {
                  if((!isset($_POST[$fieldname]))||(!in_array($_POST[$fieldname], $subjects))) {
                        $form_errors[] = $fieldname ."doesn't exist";
                  }
0
 
LVL 12

Accepted Solution

by:
kevin_u earned 250 total points
ID: 24787336
The underlying issue is that $check_category_id is not an array.  Thats the cause of the error message.   In your short example, it is surely an array.  But is it called the same way in the full script?
0
 

Author Comment

by:czechmate1976
ID: 24787404
resolved. Sorry, after days staring at the code I can hardly see.. Thanks for your help
0

Featured Post

How to run any project with ease

Manage projects of all sizes how you want. Great for personal to-do lists, project milestones, team priorities and launch plans.
- Combine task lists, docs, spreadsheets, and chat in one
- View and edit from mobile/offline
- Cut down on emails

Join & Write a Comment

Foreword (July, 2015) Since I first wrote this article, years ago, a great many more people have begun using the internet.  They are coming online from every part of the globe, learning, reading, shopping and spending money at an ever-increasing ra…
Password hashing is better than message digests or encryption, and you should be using it instead of message digests or encryption.  Find out why and how in this article, which supplements the original article on PHP Client Registration, Login, Logo…
Viewers will get an overview of the benefits and risks of using Bitcoin to accept payments. What Bitcoin is: Legality: Risks: Benefits: Which businesses are best suited?: Other things you should know: How to get started:
The viewer will get a basic understanding of what section 508 compliance can entail, learn about skip navigation links, alt text, transcripts, and font size controls.

758 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

20 Experts available now in Live!

Get 1:1 Help Now