Nested foreach() loop php Codeigniter-Php error

Question

I'm getting Message: Invalid argument supplied for foreach()

on this line foreach($element->action as $URI){ // find action="" attr of <form> element

My code is a nested foreach() loop:

           $siteToSearch = file_get_html($prefix.$sss);
                foreach($siteToSearch->find('form') as $element){ // find <form> element
                    foreach($element->action as $URI){ // find action="" attr of <form> element
                        $submit_vars["name"] = "' OR ''='";                     
                        $submit_vars["passwd"] = "' OR ''='";                   
                        $submit_vars["submit"] = "Submit";  
                        $snoopy->submit($URI,$submit_vars);
                        echo "response code: ".$snoopy->response_code."<br>\n";
                        print $snoopy->results;
                    }
   }

I tried defining $URI on the first line as $URI = $element->action; but this doesnt seem to fix it

Why is this nested loops? The outer loop seems to be looping through all the <form> elements, but the inner loop only makes sense if you're assuming each <form> might have multiple actions. — John Flatness
– John Flatness, Commented Feb 14, 2013 at 5:02

John Flatness · Accepted Answer · 2013-02-14 19:00:38Z

1

You get the "Invalid argument supplied for foreach()" message when the argument (here it's, $element->action) isn't an array (or something array-like).

I think the problem here is that you really don't need the inner foreach loop at all. $element->action seems to be representing the "action" attribute of an HTML form element. Each form will only have one action, which will probably be represented in PHP as a string, not an array.

Taking out the inner loop and simply adding a line like you mentioned, $URI = $element->action, should fix this problem.

answered Feb 14, 2013 at 19:00

John Flatness

34.1k5 gold badges81 silver badges81 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

CodeTalk Over a year ago

Great idea, got confused in my own logic here.Thanks!!

keshu_vats · Accepted Answer · 2013-02-14 07:57:58Z

please check that $element->action have some value or not please use print_r( $element); after the first loop and please put the inner loop in the contion ==>if($element->action) { }

the code will be something like given below

$siteToSearch = file_get_html($prefix.$sss); foreach($siteToSearch->find('form') as $element){ // find <form> element if($element->action) { foreach($element->action as $URI){ // find action="" attr of <form> element $submit_vars["name"] = "' OR ''='"; $submit_vars["passwd"] = "' OR ''='"; $submit_vars["submit"] = "Submit"; $snoopy->submit($URI,$submit_vars); echo "response code: ".$snoopy->response_code."<br>\n"; print $snoopy->results; } } }

Collectives™ on Stack Overflow

Nested foreach() loop php Codeigniter-Php error

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related