Everyone really needs to learn regex, though it seems to be eternally relegated to people's todo list. :(

Anyway, looking at the latest posted code, problem is that the while loop doesn't have a brace block - when it was a single statement (echo) it worked, but now it's multiple statements you need to group them.

while($row = mysql_fetch_array($result, MYSQL_ASSOC))

{

    $blah = split(",", $row[images]);

    foreach ($blah as $beep)

    {

        $stringading .= "<img src=\"" . $beep . "\" />";

    }



    echo "

    ...

         ";

}

?>

I agree on the regex front. It seems to have been the fix for loads of stuff I've asked on here.

 

Also (hug) the extra braces worked!

It is responsible for adding paragraph tags but obeying valid HTML nesting and tag ordering.

Hmm, well must be something else that is stopping PHP being all coloured and stuff then?

In my previous post, there's extra line-breaks there, but in Ant's there aren't any. Might also be related in some way?

I disabled them because they were being added extra within the code tags. If I hadn't it would've looked like yours.

Ah, thought you might have done. However, it shouldn't be necessary - the line breaks shouldn't get added at all - content between code isn't supposed to be changed (excluding font/colour formatting).

 

Dunno what the cause is though - I did go look at the code Matt referred to and it's far too much effort trying to decipher it all, especially as tired as I currently am.

Drew needs to change his nick. Lucy is a girls name.

What's wrong with girls names, Gemma?

Personally I'd dump the list of image filenames to a JS array and build the HTML client-side with Mustache, but that's me, I love client-side templating.

Yay I'm rubbish :C Some code

$sold1 = $row[sold];
if ($sold1 == yes){
$soldstring .= "<img alt=\"sold\" src=\"./photos/{$row[thumb]}\" />"; }
else {
$soldstring .= "<img alt=\"notsold\" src=\"./photos/{$row[thumb]}\" />"; }

echoing

" . $soldstring . "

This does as expected for the first result from the DB. But the next result contains the first image as well, and the next one the previous and so on.

So the top result shows correctly, but each result after has extra images, meaning on the 10th result it shows the correct image plus the 9 previous.

Why? :(

.= adds to a string, = sets a string. You want = in this case. i.e.:

$sold1 = $row[sold];

if ($sold1 == yes){

$soldstring = "<img alt=\"sold\" src=\"./photos/{$row[thumb]}\" />"; }

else {

$soldstring = "<img alt=\"notsold\" src=\"./photos/{$row[thumb]}\" />"; }

(hug)

Nothing at all except he's not a girl like you!

Then who is he a girl like?