[Opserty]

Curly braces won't affect the code execution unless you have not matched them up correctly. They are there to make code more readable.

Why did you insert the code I gave in my first post? (I suggested different code in my second post...)

Try this:

PHP Code:

if ($vbulletin->options['eic_enable'])
{
    session_start();
    if (!isset($_SESSION['secure']))
    {
        if (!$_GET['clicked']) 
        {
            $amount_of_images_to_show = $vbulletin->options['eic_numimgs'];
              $numx = $vbulletin->options['eic_numx'];
            $answer_position = rand(1,$amount_of_images_to_show);
            $d = dir($vbulletin->options['eic_imgpath']);
            
            while (false !== ($entry = $d->read()))
            {
                if (substr($entry,-4) == '.jpg')
                {
                    $answers[] .= $entry; 
                }
            }
    
            $amount = count($answers);
            $the_answer = rand(1,$amount);
            
            for($i=1;$i<=$amount_of_images_to_show;$i++) 
            {
                if ($answer_position == $i) 
                {
                     $show_image[] = $the_answer;
                }
                
                $tmp = rand(1,$amount);
                
                while ($tmp == $the_answer || (is_array($show_image) && in_array($tmp,$show_image))) 
                {
                    $tmp = rand(1,$amount);
                }
                
                $show_image[] = $tmp;
                
                eval('$eic_imagebits .= "' . fetch_template('eic_imagebits') . '";');
                /** ---------------------
                
                // If counter is a multiple of 5 then execute this code...
                if(($i % 5) == 0))
                {
                   $eic_imagebits .= '<br />';
                }  
                -------------- */
                
            }
            if ($answer_position == $i) 
            {
                 $show_image[] = $the_answer;
            }
            $d->close();
    
            $_SESSION['answer_position'] = $answer_position;
            $_SESSION['show_images'] = $show_image;
            $question = substr($answers[$the_answer-1],0,-4);
        }
        else 
        {
            if ($_GET['clicked'] == $_SESSION['answer_position']) 
            {
                 $_SESSION['secure'] = 1;
            }
            else
            {
                 $badchoice = 1;
            }
        }
        if (!isset($_SESSION['secure'])) 
        {
            eval('$navbar = "' . fetch_template('navbar') . '";');
            eval('print_output("' . fetch_template('eic_main_template') . '");');
        }
    }
} 


I've commented out my code to highlight it to you, uncomment it and test it out.