PHP matching all

Question

I'm using a wordpress plugin called Movie Poster which grabs a movie's info from imdb and displays it in a post. How do I grab just the first Production Co before the first comma? I'm confused on how this code works. Here's the code which grabs all the production co:

$arr['productions'] = array();
foreach($this->match('/<a.*?>(.*?)<\/a>/ms', 
           $this->match('/Production Co.?:(.*?)(<\/div>|See more)/ms', $html, 1)
        , 1) as $m)
    array_push($arr['productions'], $m);`

Thanks for your time

you shouldn't be parsing html with regexes. that's why we have php.net/dom — Marc B
– Marc B, Commented Apr 30, 2013 at 17:06
@user2336811: Contact the developer of that plugin, tell him about your issue and negotiate a fix. — M8R-1jmw5r
– M8R-1jmw5r, Commented Apr 30, 2013 at 17:15

EnigmaRM · Accepted Answer · 2013-04-30 17:41:03Z

1

So you just want one Production Co's info?

Why not just get rid of the foreach? You currently have a loop going through and looking for all Production Co. and returning.

Or you could change it to a While statement and add a counter so that it only passes through once.

$arr['productions'] = array();
$i = 0;
while($i < 1) {
   $m = $this->match('/<a.*?>(.*?)<\/a>/ms', 
       $this->match('/Production Co.?:(.*?)(<\/div>|See more)/ms', $html, 1)
    , 1);
   array_push($arr['productions'], $m);
   $i++;
}

Not sure if the syntax is exactly correct, but should get you in the right direction. Keep in mind, I'm not familiar with the plugin or if I've understood exactly what you're trying to accomplish.

edited Apr 30, 2013 at 17:41

answered Apr 30, 2013 at 17:18

EnigmaRM

7,61211 gold badges48 silver badges74 bronze badges

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

7 Comments

Roku Over a year ago

Thanks for your answer but I'm getting this error "Parse error: syntax error, unexpected T_AS"

Roku Over a year ago

I'm getting "Fatal error: Maximum execution time of 30 seconds exceeded" in this line of code:

function match($regex, $str, $i = 0){         if(preg_match($regex, $str, $match) == 1)             return $match[$i];         else             return false;     }

Roku Over a year ago

Cool, I didnt get any errors but the post only displays a comma.

EnigmaRM Over a year ago

In your original code snippet, you have a ` at the end of the ;. Is that needed? Wasn't sure if that was just a typo, etc.

EnigmaRM Over a year ago

The problem may be that we're not assigning $m anywhere. Try changing it to $m = $this->match...

|

Collectives™ on Stack Overflow

PHP matching all

1 Answer 1

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related