[Resolved] Replacing the src of all images in a file with MIME placeholders

tomgosling

Hi,

I'm assuming that this is going to need to use something like ereg_replace or similar but as I'm not too familiar with regular expressions....Help!

Basically, I need to replace the src of all images in a file with a unique identifier (different for each unique image).

So in english... I want to say - replace everything between "<img" and the following occurance of ">" with my own incremental identifier. So we have to replace one image at a time.

At the moment I'm blagging it a bit with the following code....

$n_images = substr_count($text,"<img");
$starting_point=0;

for($i=0;$i<$n_images;$i++){

$stringstart = strpos($text,"<img",$starting_point);
$stringend = strpos($text,">",$stringstart);
$string_length = ($stringend+1)-$stringstart;

$string = substr($text,$stringstart,$string_length);
$text = str_replace($string,"<img src='cid:000".$i."'>",$text);
$starting_point = ($stringend+1);

$image_array[] = "cid:000".$i;

}

The problem is that when the <img> tag gets over about 70 characters in length it all goes wrong!

If anyone can help I would be most grateful

xblue

Hi,

the pattern shouldn't be much of a problem since you define quite "regex-friendly" what you want to be matched.

Incrementing could be achieved by using preg_replace_callback() instead of preg_replace(), maybe something along these lines:

$input = '<img scr="img1.gif">text text text <img name="test" scr="img2.jpg"  alt="test" /> text';

$output = preg_replace_callback("/<img[^>]*>/i", create_function(
           '$matches',
           'static $counter = 0;
            $counter++; 
            return "<img scr=\"cid:000".$counter."\">";'
           ), $input);

echo $output;

(this creates the function on the fly, but you can define it somewhere else in the script if you prefer, see examples and explanations on http://www.php.net/preg_replace_callback)

tomgosling

Thank you very much xblue, you're a hero!!!

xblue

you are welcome (I actually tested this one to be sure it worked)! just 2 things you may net know about this forum since you seem to be fairly new:

use [ php ] and [ /php ] (without spaces) around your code, makes it look more beautiful and helps to help.

there's an option to mark the thread as resolved somewhere, I think near the bottom of the form.

P.S.: I never thought that you would answer within minutes after your post has been left unanswered for more than a week. Thank you for not bumping it 🙂

tomgosling

Cheers xblue, I'll keep that in mind for the future

Thanks again for your help