$ROW inserten

Door Delano Velthuis op 17-04-2021 17:12 gewijzigd op 17-04-2021 17:13

1.780 views

Hey, ik ben bezig met mijn php code en ik loop ergens tegen aan.

Ik haal data op uit een database en wil deze vervolgens inserten naar een andere table maar dit werkt niet.


<?php

require_once 'config.php';
$yacht = $_POST['yacht'];
$from_time = $_POST['from_time'];
$to_time = $_POST['to_time'];
$paymentPreference = $_POST['paymentPreference'];

$catering = $_POST['catering'];
$insurance = $_POST['insurance'];
$childLifejackets = $_POST['childLifejackets'];
$paddleboard = $_POST['paddleboard'];
$skipper = $_POST['skipper'];
$flot = $_POST['flot'];
$fishinggear = $_POST['fishinggear'];
$groceries = $_POST['groceries'];
$transfer = $_POST['transfer'];
$terms = $_POST['terms'];
$email = $_POST['email'];
$password = $_POST['password'];

$sql = "SELECT * FROM costumers WHERE email = '$email' LIMIT 1"; 
$result = $conn->query($sql);

if( $result->num_rows == 1) {
    $row = $result->fetch_assoc();
    
    if($password == $row['password']) {
        $sql = "INSERT INTO `bookings`(`costumers_costumerID`, `Yachts_yachtID`, `paymentPreference`, `date_start`, `date_end`, `catering`, `skipper`, `flot`, `groceries`, `transfer`, `insurance`, `childLifejackets`, `fishinggear`, `paddlebloard`, `aggreedToTerms`) 
        VALUES ('". $row['customerID'] . "', '". $yacht . "', '". $paymentPreference . "', '". $from_time . "', '". $to_time . "', '". $catering . "', '". $skipper . "', '" . $flot . "', '" . $groceries . "', '" . $transfer . "', '" . $insurance . "', '" . $fishinggear . "', '" . $paddleboard . "', '" . $terms . "')";

        if( $conn->query( $sql )) {
            echo "Doet het";
        } else {
            echo "Doet het niet";
        }
    }
} else {
    echo "Gebruiker niet gevonden";
}

?>

Error:

Warning: Undefined array key "customerID" in C:\xampp\htdocs\coral-yachts-delano-burhan\templates\RoyalCarsHtml\includes\booking_customers.php on line 30

Zouden jullie hier mij mee kunnen helpen?

- Ariën -

17-04-2021 17:50 gewijzigd op 17-04-2021 17:52

$row['customerID'] bestaat blijkbaar niet, en het veld customerID ook niet.

Ook was ik benieuwd of je tabelnaam klopt. Deze heet costumers. Lijkt mij een typfout.

En verder zie ik SQL-injectie bij je POST-variabelen.

En verder zie ik ook dat je het password niet beveiligd opslaat!!! Gevaarlijk!!!

Ralf d

17-04-2021 17:51

O en de U omdraaien?
Lijkt mij het meest waarschijnlijke.

Maar als je dat er zelf niet uit kan halen mag je wel serieus afvragen of je dit wilt gebruiken in een live omgeving.
Denk aan de volgende dingen.
Beveiliging, escape, paswoord hashing.

Delano Velthuis

17-04-2021 18:06 gewijzigd op 17-04-2021 18:09

- Ariën - op 17/04/2021 17:50:31

$row['customerID'] bestaat niet, een het veld customerID ook niet.

Ook was ik benieuwd of je tabel naam klopt. Deze heet costumers. Lijkt mij een typfout.

En verder zie ik SQL-injectie bij je POST-variabelen.

Hey Arien, ja klopt ja.. was me niet opgevallen die typfouten. Nu heb ik de typfouten opgelost en dan krijg ik dat hij

if( $conn->query( $sql )) {
echo "Doet het";
echo $sql;
} else {
echo "Doet het niet";
echo $sql;

}

de return geeft dat hij het niet doet. Wat raar is aangezien ik alles juist heb ingevuld vanuit de form

[size=xsmall]Toevoeging op 17/04/2021 18:07:01:[/size]

Ralf d op 17/04/2021 17:51:41

O en de U omdraaien?
Lijkt mij het meest waarschijnlijke.

Maar als je dat er zelf niet uit kan halen mag je wel serieus afvragen of je dit wilt gebruiken in een live omgeving.
Denk aan de volgende dingen.
Beveiliging, escape, paswoord hashing.

Hey Ralf,

Ga het gelukkig ook niet online gebruiken. Was mee een eigen project waar ik mee bezig ben. Wel bedankt voor je antwoord :D ik ga daar goed opletten!

- Ariën -

17-04-2021 18:10 gewijzigd op 17-04-2021 18:12

Of je het wel of niet online gebruikt: Escaping is noodzakelijk. Jacques 'd Ancona kan zich namelijk nooit registreren omdat zijn naam je query corrupt maakt.

Over je probleem: echo eens $sql.
Ook $conn->error moet meer duidelijkheid geven.

Delano Velthuis

17-04-2021 18:13

- Ariën - op 17/04/2021 18:10:55

Of je het wel of niet online gebruikt: Escaping is noodzakelijk. Jacques 'd Ancona kon zich namelijk nooit registreren omdat zijn naam je query corrupt maakt.

Over je probleem: echo eens $sql.
Ook $conn->error moet meer duidelijkheid geven.

INSERT INTO `bookings`(`customers_customerID`, `Yachts_yachtID`, `status`, `paymentStatus`, `paymentPreference`, `date_start`, `date_end`, `catering`, `skipper`, `flot`, `groceries`, `transfer`, `insurance`, `childLifejackets`, `fishinggear`, `paddlebloard`, `aggreedToTerms`) VALUES ('11', '1', 'Niet voltooid', 'Niet betaald', 'Paypal', '04/17/2021 4:40 PM', '04/19/2021 4:39 PM', 'N', 'Y', 'Y', 'Y', 'Y', 'N', 'Y', 'N', 'Y')

en

Column count doesn't match value count at row 1

is de response die ik krijg.

[size=xsmall]Toevoeging op 17/04/2021 18:15:39:[/size]

Heb dit nu opgelost en vervolgens krijg ik deze error:
You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ' 'N', 'Y', 'N', 'Y')' at line 2.

Excusses voor de soms wat domme vragen. ik ben hier redelijk nieuw in na een lange afwezigheid ermee

- Ariën -

17-04-2021 18:17

Vrij vertaald: Het aantal kolommen komt niet overeen met het aantal waarden in de rij

Is het nu duidelijker? ;-)

Delano Velthuis

17-04-2021 18:18

- Ariën - op 17/04/2021 18:17:25

Vrij vertaald: Het aantal kolommen komt niet overeen met het aantal waarden in de rij

Is het nu duidelijker? ;-)

Yes dankjewel, ik had dat opgelost alleen ik krijg nu deze error

You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ' 'N', 'Y', 'N', 'Y')' at line 2

- Ariën -

17-04-2021 18:20

Ik denk dat je ergens een quote mist.
Zet wat enters in je query, zodat je een mooi overzicht hebt.

Ralf d

17-04-2021 18:21

Op de een of andere manier was mijn vorige reactie leeg.

Delano probeer het jezelf meteen goed aan te leren maak een paar functies die het voor je doen en zet die in een bestand functies.php include dat bestand.

Dat bestand zou je in ieder project kunnen gebruiken als je een vaste manier hebt van
Programeren.

Delano Velthuis

17-04-2021 18:23

- Ariën - op 17/04/2021 18:20:00

Ik denk dat je ergens een quote mist.
Zet wat enters in je query, zodat je een mooi overzicht hebt.

Top dankjewel!
Heb dit gedaan en het werkt nu, heb de fouten eruit gevonden.
Hartelijk dank voor je tijd en excuses voor de soms nog al slechte vragen.

[size=xsmall]Toevoeging op 17/04/2021 18:24:05:[/size]

Ralf d op 17/04/2021 18:21:55

Op de een of andere manier was mijn vorige reactie leeg.

Delano probeer het jezelf meteen goed aan te leren maak een paar functies die het voor je doen en zet die in een bestand functies.php include dat bestand.

Dat bestand zou je in ieder project kunnen gebruiken als je een vaste manier hebt van
Programeren.

Goede tip!
Dankjewel zal er meteen naar kijken!

Reageren

Inloggen om te reageren