"Nie działające" INSERT INTO w kodzie php

Question

Heja ponownie.

Znów z podobnym problemem otóż do pliku PHP, który odpowiada za wysyłanie maila z danymi z formularza na maila administratora strony postanowiłem dodać panel administratora, gdzie będzie znajdować się karta 'oczekujące i zaakceptowane wizyty". Problem pojawił się dosyć szybko, gdy postanowiłem go wdrożyć do tego właśnie pliku.
Kod wygląda w ten oto sposób;

<?php

	//Mail wizyt
	$od  = "From: wizyta@*********.pl \r\n"; 
	$od .= 'MIME-Version: 1.0'."\r\n";
	$od .= 'Content-type: text/html; charset=iso-8859-2'."\r\n";

	$adresat = '***********@gmail.com'; 
	$temat = 'Rezerwacja wizyty w Salonie Fryzjerskim';
	$name = $_POST['your-name'];
	$etel = $_POST['your-etel'];
	$date = $_POST['your-date'];
	$content = 'Osoba o imieniu i nazwisku: ' . $name . '<br />Kontaktujaca sie pod numerem lub mailem: ' . $etel . '<br />Chcialaby zarezerwowac wizyte dnia: ' . $date . '<br /> -- <br /> Wyslano z formularza kontaktowego na stronie ********.pl/wizyta;	
	
	//Wysylanie danych z formularza do tabeli `wizyty`
	$sql_conn = mysql_connect('localhost', '**********', '***********');
	mysql_select_db('***********');
	$zapytanie = "INSERT INTO `wizyty`(`id`, `your-name`, `your-etel`, `your-date`, `status`) VALUES ('NULL', '$name', '$etel', '$date', 'waiting')";
	mysql_query($zapytanie);
	mysql_close($sql_conn);

    if (mail($adresat, $temat, $content, $od)) 
       header("Location: finally.php"); 
    else 
       header("Location: lost.php"); 

?>

W czym problem? Ni cholery skrypt nie chce wysłać rekordu do bazy danych mimo, że wszystkie dane połączeniowe są poprawne. O przekierowywaniu na lost.php lub finally.php nie wspomnę. Mail działał normalnie póki nie dodałem połączenia z bazą danych.

@Edit: Kod do połączenia mysql, wysyłania rekordów brałem z jakiegoś tam poradnika, który podesłał mi kolega.

Pozdrawiam
Szczur
 

Comments

Wyskakują jakieś błędy? Masz w ogóle włączone ich wyświetlanie? Jaką masz wersję PHP?

---

Wersja PHP to 5.6, co do wyświetlania błędów to było lecz nie pokazywało de facto żadnego błędu. Pustka.

---

Na 5.6 powinno jeszcze działać. Pytam w kontekście funkcji zaczynających się od mysql_, które są już dawno zdeprecjonowane, a od PHP 7 usunięte - gdybyś miał 7, kod nie miałby szans zadziałać. Nawet jeśli nie masz, bardzo polecam zmianę na mysqli albo najlepiej PDO.

Na końcu linii 13 masz u siebie apostrof zamykający string? W powyżej podanym kodzie go nie ma.

---

Tak mam ten apostrof, pewno edytując komentarze (//) usunąłem go. Okej, poczytam na temat PDO i mysqli.

---

Świetnie, działa.  Użyłem mysqli, oczywiście były błędy, bo uczymy się na własnych błędach ;]

Pozdrawiam.

---

Cieszę się, powodzenia ;)

Jako dodatek mogę dopowiedzieć, że funkcja mail() też jest słaba, bywają z nią problemy i jest mało wygodna (co widać wyżej, ręczne ustawianie nagłówków itd.). Lepiej skorzystać z biblioteki PHPMailer lub SwiftMailer i wysyłać przez SMTP.

---

Jasne, wezmę sobie Twoją radę do serca ;)

Jeszcze jedno pytanie, bo nie chce otwierać nowego tematu, żeby otrzymać małą odpowiedź. Robie tabelę użytkowników i potrzebuję dodać coś do kodu, żeby mi brało każdego użytkownika do <table> z statusem 'waiting' a nie, że ja mam ID = 1 i w każdej rubryce wyświetla tylko mnie. Mam nadzieję, że rozumiesz o co mi chodzi. Jeśli nie - to chce mieć spis użytkowników o ID od 1 do nieskończoności ale TYLKO z statusem 'waiting'.
Obecnie kod wygląda w ten sposób; https://pastebin.com/EjqAb4Vd w tabeli mam wpisane <?php echo $row[0]; ?> i tak 5 razy z inną liczbą w nawiasie.

---

Hm, nie za bardzo rozumiem. Czyli potrzebujesz wybrać z bazy użytkowników tylko którzy mają status 'waiting'? Można to zrobić prostym zapytaniem dodając warunek, np. SELECT ... FROM ... WHERE status = 'waiting'. Następnie użyć fetch_all() i dostaniesz tablicę z listą rekordów, którą następnie możesz przeiterować np. pętlą foreach i zrobić co trzeba.

---

null nie powinno być w apostrofie ;)