Fejl i php script

prøv
$newpics1=mysql_query("SELECT billede FROM sider WHERE gruppe='".$gruppe."' ORDER BY id DESC limit ".$limit)or die(mysql_error());

$gruppe indeholder vel ikke noget i funktionen?

Prøv at ændre så følgende linier ligner:
function randompic($limit, $variabel){

$newpics1=mysql_query("SELECT billede FROM sider WHERE gruppe='$variabel' ORDER BY id DESC limit $limit")or die(mysql_error());

echo randompic("1", $gruppe).randompic("1", $gruppe).randompic("1", $gruppe);

Kan du se hvad jeg mener?

Når nu $gruppe indgår som argument i din SQL, bør den også indgår som argument til din funktion. Imidlertid bruges limit ikke til noget idet du faktisk alligevel kun bruger den første række - og denne bør alts fjernes! Der er endvider et par andre småting som bør rettes i samme omgang:

function randompic($gruppe)
{
    $sql = "SELECT billede FROM sider WHERE gruppe='" . $gruppe . "' ORDER BY id DESC LIMIT 0, 1;
    $query = mysql_query($sql) or die(mysql_error());
    $row = mysql_fetch_row($query);

    $dir = $row[0] . "/thumbs/";

    $filesArr = array();

    $handle = opendir($dir);
    while ($file = readdir($handle))
    {
        if ($file != "." && $file != "..")
        {
            $filesArr[] = $file;
        }
    }
    closedir($handle);

    $filesArrObj = array_rand($filesArr);
    $randomFile = $filesArr[$filesArrObj];
    $path = $dir . $randomFile;

    $sti = "<img  onmouseover=\"urlFullSizeImage = '" . $row[0] . "/pics/" . $randomFile . "'; StartTimer ( 10,window.event.clientY, viewDelay );\" onmouseout=\"StopTimer (); HideImage ();\" src=\"" . $path . "\"  border=\"0\" />";

    retuen $sti;
}

Tak for hjælpen!
Læg venligst et svar!

Hvem vil du have svar fra?

Jeg må indrømme jeg kunne ikke få nielle's svar til at virke, men jeg fik vel en idé af hvad jeg skulle gøre og det samme med jeres svar. Så jeg synes i skal dele pointene.

Jamn så lægger jeg da gerne et svar :)