A Complicated preg_replace

slevytam

Hi,

I am using a preg_replace to search for image tags and wrap the image tag with a hyperlink tag to open a shadowbox/lightbox.

Problem is I just realized that now I can't make any images actually work as a hyperlink. Is there anyway I can make the preg_replace not add the wrapper if the image tag is already surrounded by a link?

This is the preg_replace I am currently using:

$bloghtml = preg_replace('/<img src="(.*)"(.*)\/>/isUe', "'<a href=\"'.str_replace('-500','','$1').'\" rel=\"shadowbox[entry]\"><img src=\"$1\" '.stripslashes('$2').'/></a>'", $bloghtml);

Any help would be more than appreciated!

Thanks,

slevytam

jim_keller

Often, when doing replaces of this complexity, instead of using preg_replace, use preg_match_all(). To match the img tag and surrounding <a> tag if it exists, try something like:

preg_match_all( '/(\<\s*a.*href=["\'](.*)["\'].*\>\s*)?(\<img.*\>)/siU', $html, $matches );

for( $j = 0; $j < $matches[0]; $j++ ) {
    $entire_match = $matches[0][$j];
    $link_tag = $matches[1][$j];
    $img_tag = $matches[2][$j];

//$changed_img_tag = 'do your parsing here';

if ( !$link_tag ) { //skip if this image has a link
    $html = str_replace($entire_match, $changed_img_tag, $html);
   }
}

note: I didn't test the regexp above, i typed it out just for the post. Let me know if it doesn't work.

slevytam

Hi Jim,

Thanks for the reply!

The code was crashing at first but I think thats because you have to use sizeof($matches[0]) in the for loop ?

The preg_match_all seems to matching any link followed by an image tag. Is that correct? Problem is that the text could have alot of links. I'm trying to ignore the <img tags wrapped in a link. ie <a href><img src /></a> and only catch the images that are on their own ie. <a href></a> <img src /> <a href> </a>

Appreciate the help!

slevytam

Hi Jim,

Thanks for the reply!

The code was crashing at first but I think thats because you have to use sizeof($matches[0]) in the for loop ?

Appreciate the help!