PHP / Wordpress preg_match

Question

Ive got the following function wrapping all iframe's in a div class="video-container"

I want to target only iframes that contain a src beginning with src="www.youtube

Is there a way to modify this function do be more specific?

Thanks in advance.

function div_wrapper($content) {
// match any iframes
$pattern = '~<iframe.*</iframe>~';
preg_match_all($pattern, $content, $matches);

foreach ($matches[0] as $match) {
    // wrap matched iframe with div
    $wrappedframe = '<div class="video-container">' . $match . '</div>';

    //replace original iframe with new in content
    $content = str_replace($match, $wrappedframe, $content);
}

return $content;    
}
add_filter('the_content', 'div_wrapper');

`if (!condition) continue;` inside the foreach as the first line. Make condition the target condition (src begins with whatever) — Patashu, Feb 06 '13 at 06:09

score 0 · Accepted Answer · answered Feb 06 '13 at 06:13

0

Perform a lazy match (*?), and check if the string that you're looking for is present in it:

// match any iframes
$pattern = '~<iframe.*?</iframe>~';
$content = preg_replace_callback($pattern, function($matches){

  if(strpos($matches[0], 'youtube') !== false) 
    return '<div class="video-container">' . $matches[0] . '</div>';

  return $matches[0];

}, $content);

Needless to say this will fail in many cases. If you want something reliable use an XML parser (see DomDocument::getElementsByTagName and DomElement::getAttribute.

You can also try to style your iframe with pseudo-selectors like :before or :after. That way you don't need a wrapper element (eg. iframe[src~=youtube]:after)

answered Feb 06 '13 at 06:13

nice ass

16,471
7
50
89

Ah yes, I didnt think of using pseudo's to wrap the iframe. Ill give them a go. – user1385827 Feb 06 '13 at 21:48
the :before pseudo element won't work according to my tests and this post http://stackoverflow.com/a/13056674/1385827 – user1385827 Feb 06 '13 at 22:13
The php youve supplied above works. Im not sure why you say it will fail in many cases? Thanks again for your help. – user1385827 Feb 06 '13 at 22:23
Parsing HTML with regular expressions is generally not a good idea, because there will always be some string combination that your expression may not match. Indeed :before doesn't seem to work with iframes, so that leaves you with the other two options :) – nice ass Feb 06 '13 at 22:58

Tomas Ramirez Sarduy · Answer 2 · 2013-02-06T07:10:32.193

0

I think you just need to change the $pattern variable. Try with:

$pattern = '~<iframe.*src*=\'.*(http\:?)*\/\/*(www\.?)*youtube\..*</iframe>~';

Note: As @alex pointed, you are missing the protocol, so it should be http://youtube., //youtube. or http://www.youtube.

edited Feb 06 '13 at 07:10

answered Feb 06 '13 at 06:20

Tomas Ramirez Sarduy

17,294
8
69
85

It shouldn't work as missing the protocol will mean the `iframe` won't work. – alex Feb 06 '13 at 06:21
@alex: thanks for point that, I forgotten because is not in the question... updated – Tomas Ramirez Sarduy Feb 06 '13 at 06:38
I don't think you want it in a character class. If it's optional, use parenthesis to wrap it. – alex Feb 06 '13 at 06:40
@alex: like this? `(http\:?)` – Tomas Ramirez Sarduy Feb 06 '13 at 07:11

PHP / Wordpress preg_match

2 Answers2