Mærkelig problem med function og database

Din kode?

Ved godt at koden kan optimeres, men det var en vi hurtigt lavede fordi den anden gav problemer og vi ikke ville bruge alle resourcer her nu, hvilket gav os en masse besvær alligevel. Men det er ved die() at det afgør om databasen gider huske det.

public function toggle($table, $column, $find, $what = 'id') {
    if (!$table || !$column || !is_numeric($find) || !$what)
        return "error_missing_information";
   
    $sql = $this->query_read("
        SELECT `". $column ."`
        FROM `". PREFIX . $table ."`
        WHERE `". $what ."` = ". $find ."
        LIMIT 1
    ");
   
    if ($this->num_rows($sql) == 0)
        return "error_record_not_found";
   
    $data = $this->fetch($sql);
   
    if ($data[$column] == 0)
        $newToggle = 1;
    else
        $newToggle = 0;
   
    $this->query_write("
        UPDATE `". PREFIX . $table ."`
        SET `". $column ."` = ". $newToggle ."
        WHERE `". $what ."` = ". $find ."
        LIMIT 1
    ");
    #die(print_r($data)); // virker\#virker ikke
   
    return $newToggle;
}

Jeg har en gang brugt 2 timer på at debugge noget lignende. Det viste sig at være en fantastisk dygtig grafiker der havde sat onclick event på et link, så der blev dobbelt postet, og funktionen derved blev kørt x2. Det underlige var at et dump viste den korrekte data. Er du grafiker :D?

hehe nej, og det er vidst heller ikke problemet

Prøvede lige med den her i starten af funktionen:

if ($_SESSION['qwerty'] = 12345)
    return false;
$_SESSION['qwerty'] = 12345;

og den ændrer stadig ikke noget

Det har selvfølgelig intet - i sig selv - at gøre med om den "die" står der. Så det må være noget andet. Ideen fra fant0mas er faktisk ret sjov - hvis du toggler to gange, så er resulatet uændret. Ved at sætte en "die" ind, så sker det kun første gang.

Vil du lave det lille hack, så hedder det ikke
  if ($_SESSION['qwerty'] = 12345)
men
  if ($_SESSION['qwerty'] == 12345)

Ja selvfølgelig, doh, man kan godt blive lidt træt i hovedet. Men ser ud til at den bliver kørt 2 gange af en eller anden mystisk grund, vender lige tilbage, for det bliver en større opgave at finde fejlen, medmindre man kan noget smart i PHP mht det.

Kan man evt. se, hvor en funktion bliver kaldt?

Du kan vel søge efter funktionsnavnet i din tekst-editor?

Funktionen er kun kaldt 1 sted fra og jeg har også prøvet at omdøbe den til noget helt andet. Problemet må ligge i at det lykkes systemet at loade filen 2 gange, det er ihvertfald mit bud

Det er altid en mulighed. Tæl en sessions-variabel op passende steder, og kig på den.

Selv om denne funktion kun kaldes et sted, kan den funktion, den kaldes i, selv kaldes 2 gange ... osv.

Den konkrete fejl er fundet, hvordan det kan lade sig gøre er en stor gåde for mig. Men det var det her kode som gjorde at det loadede 2 gange:

<div>
    <img src="" alt="" />
    <?php
        if (!$_GET['name'])
            echo '<a href="?" style="font-weight: bold;">Home</a>';
        else
            echo '<a href="?">Home</a>';
    ?>
</div>

<img src="" alt="" /> med tom src henter samme side, som <img>-tagget står på.

Det tyder på du har lavet din "toggle" med en GET, fx via et link. Det plejer at være god swahili at lave ændringer i databasen med POST-er.

Har jeg nu ikke, det er en class, men det var det at src ikke havde nogen værdi der gav problemer. Ved du om det kan tyde på et problem et andet sted, eller det være okay nu?

"Har jeg nu ikke," ... ikke forstået?  Om der er andre problemer ... det ved jeg da virkelig ikke. :)

Har kaldt den fra en class som var fast defineret for at teste:
$db->toggle('table', 'column', 1);
Så tror ikke det er det med GET.


Anyway, har tilpasset funktionen nu, til dem der skal bruge noget lign. istedet for det lort ovenover:

public function toggle($table, $column, $find, $what = 'id') {
    if (!$table || !$column || !is_numeric($find) || !$what)
        return "error_missing_information";
   
    $this->query_write("
        UPDATE `". PREFIX . $table ."`
        SET `". $column ."` = (CASE WHEN `". $column ."` = 1 THEN 0 ELSE 1 END)
        WHERE `". $what ."` = ". $find ."
        LIMIT 1
    ");
   
    return true;
}


og så var det vel egentlig fant0mas der løste det?

Jeg snakker om din HTML-kode. Står der <a href="....php?id=... ">Skift</a>, så bruger du "GET". Det kan give problemer. Med en <form method="post"...> ...osv... , og en test på serveren om den kommer fra "POST", sker det ikke.

Jeg springer over, Erik har vist brugt mest energi på at hjælpe, så stik ham dem.

Ingen point til mig, tak.

Svar