Piszę instalator do swojego skryptu i mam problem z utrzymaniem połączenia z bazą danych MySQL. W instalatorze są trzy etapy instalacji, krok 1 to formularz z polami: host, username, password, database, do nawiązania połączenie z MySQL.

Po wpisaniu poprawnych danych skrypt łączy się z bazą danych i wyświetla komunikat o powodzeniu lub niepowodzeniu. Tak wygląda skrypt ten skrypt:

[PHP] pobierz, plaintext 
<?php } elseif ($_GET['step'] == 2) { ?>
	html
<?php
			if (@mysql_connect($_POST['mysql_host'], $_POST['mysql_username'], $_POST['mysql_password'])) {
				echo '<font color="green">.:: Poprawnie nawiązano połączenie z bazą danych MySQL ::.</font><br />';
				if (@mysql_select_db($_POST['mysql_database'])) {
					echo '<font color="green">.:: Poprawnie wybrano bazę MySQL ::.</font><br />';
				}
 
				else {
					echo '<font color="red">Nie mogę wybrać bazy!</font><br />';
					die;
				}
 
				create_tables();
 
			}
			else {
				echo '<font color="red">Brak połączenia z bazą danych!</font><br />';
				die;
			}
 
			echo '<br />';
		?>
[PHP] pobierz, plaintext 

I tu jest wszystko dobrze, po poprawnym wprowadzeniu danych, skrypt łączy się z bazą danych MySQL i tworzy tabele.

Krok 2 to formularz z głównymi ustawieniami skryptu (pola: login administratora, hasło, e-mail, itd). Kod wygląda tak:

[PHP] pobierz, plaintext 
<?php } elseif($_GET['step'] == 3) { ?>
 
		<?php
			if (mysql_query("INSERT INTO `settings` (`setting_name`, `value`) VALUES
					('default_language', 'pl'),
					('admin_email', '".$_POST['contact_email']."')")) {
					echo '<font color="green">.:: Poprawnie dodano rekordy do tabeli `settings` ::.</font><br />';
				}
				else {
					echo '<font color="red">.:: Błąd podczas dodawaniu rekordów do tabeli `settings` ::.</font><br />';
					die;
				}
		?>
[PHP] pobierz, plaintext 

I tu jest problem, ponieważ nie dodaje mi tych danych do tabeli `settings`. Wyrzuca takie błędy:

Warning: mysql_query() [function.mysql-query]: Access denied for user 'ODBC'@'localhost'
Warning: mysql_query() [function.mysql-query]: A link to the server could not be established

Wiem co te błędy oznaczają, ale nie rozumiem dlaczego występują, skoro połączenie powinno być nawiązane...

Z góry dzięki za pomoc.

albo zapisujesz gdzieś dane do logowania i w każdym formularzu się na nowo łączysz, albo w jakiś sposób przekazujesz "uchwyt" połączenia ($uchwyt = mysql_connect...), chociaż głowy nie dam, że drugi sposób zadziała

Edit: no tak, nie zauważyłem, że to ten sam plik... tak jak piszą niżej, przenieś połączenie z bazą przed ifa i będzie działać.

z twojego kodu wynika, że z bazą łączysz się tylko gdy $_GET['step']==2 więc jeżeli jest równe 3 to już nie wykonujesz mysql_connect, wywal mysql_connect przed blok warunkowy.

Poza tym nie tłum błędów za pomocą @ a najlepiej przenieś to z użyciem PDO które teraz powinno być już standardem w skryptach , poczytaj też o mysql injection.

Połączenie nawiązujesz tylko gdy $_GET['step'] == 2. Dla 3 już go nie ma, więc wywala Ci błąd. Albo połącz się w 3 jeszcze raz, albo przenieś połączenie z bazą przed całego if'a.

Dzięki za pomoc.



To jest tylko instalator więc nie ma sensu używać tutaj PDO żeby wykonać 2 proste zapytania. Co do sql injection to wiem, ale po co mam tutaj robić zabezpieczenia skoro to tylko instalator? przecież to jest tylko i wyłącznie dla moich potrzeb, więc bez sensu jest to robić, bo sam sobie raczej nie będę zagrażał

Czemu nie ma sensu? PDO jest tylko do większych projektów? Jak będziesz chciał wywiercić kilka dziur w desce to będziesz wolał ręczną wiertarkę, bo nie ma sensu podłączać elektrycznej?
O SQL injection nie ma co się martwić, ale wypada pisać kod zgodny ze standardami, a takim niepisanym standardem jest używanie PDO.