[MySQL][PHP]Pomoc z naprawie skryptu Opcje

[kawon]

Witam.

Prosiłbym o pomoc w naprawie skryptu:

[PHP] pobierz, plaintext 
<?PHP
ini_set( 'display_errors', 'On' ); 
error_reporting(E_ALL ^ E_NOTICE);
 
 
	include("../config.php");
	include("komunikaty.php");
	$grupa = mysql_query('SELECT `group` FROM `users` WHERE `id` = '.$_SESSION['id'].'');
	$grupa = mysql_fetch_assoc($grupa);
	$tytul = $_POST['tytul'];
	$tresc = $_POST['tresc'];
	$autor = $_SESSION['nick'];
	$data = time();
	$sql = mysql_query('INSERT INTO `news` (`data_dodania` , `tytul`, `tresc`, `autor`) VALUES ('.$data.', '.$tytul.', '.$tresc.', '.$autor.')');
	$sql = mysql_fetch_assoc($sql);
// jeśli user jest zalogowany mysql_error()
if($_SESSION['logged'])
{
	if ($grupa['group'] == 2)
	{
		if(isset($_POST['dodaj']))
		{
			if(!$tytul || !$tresc)
			{
			echo 'Musisz podać tytuł i treść komunikatu.';
			}
			else{
				if($sql)
				{
				header( "refresh:2;url=index.php" );
				echo 'Komunikat został dodany.';
 
 
 
				}
				else{
				echo 'Problem z zapytaniem<br>mysql_error()';
				}
			}
		}
		else{
		echo 'Problem z dodaniem.';
		}
	}
	else{
	echo 'Musisz się zalogować jako administrator żeby dodać komunikat.';
	}
}
?>
[PHP] pobierz, plaintext 

Wywala dwa błędy w tych linijkach:

[PHP] pobierz, plaintext 
$grupa = mysql_fetch_assoc($grupa);
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$sql = mysql_fetch_assoc($sql);
[PHP] pobierz, plaintext 

[XML] pobierz, plaintext 
Warning: mysql_fetch_assoc() expects parameter 1 to be resource, boolean given in
[XML] pobierz, plaintext 

Robi się cała biała strona i skrypt nie wykonuje się, już mi brakuje się. Jakieś wskazówki co z tym zrobić?
Pozdrawiam

Ten post edytował kawon 16.06.2011, 09:25:53

[nospor]

rozwiązanie Twojego problemu znajduje się tu:
Temat: Jak poprawnie zada pytanie
Zapoznaj się, zastosuj, zapytaj ponownie

ps:
error_reporting(E_ALL ^ E_NOTICE);
ma byc:
error_reporting(E_ALL);

ps2: tekst w zapytaniu bierze się w apostrofy.

[aachi]

mysql_query wykonało Ci się z błędem...
Połączony jesteś z bazą?

Zerknij w przykłady jak powinno się używać tej funkcji: mysql_query ta strona akurat jest przetłumaczona na polski. Dodaj sprawdzanie, czy funkcja wykonała się poprawnie i sprawdź jaki błąd wyświetla.

Swoją drogą wrzucasz dane od użytkownika bezpośrednio do zapytania przez co narażony jesteś na atak sql injection. Użyj mysql_real_escape_string() na danych które odbierasz z $_POST i wstawiasz je do zapytania sql.

Ten post edytował aachi 16.06.2011, 09:51:30

[kawon]

Poprawiłem cały kod i niestety nadal nic...

Zapytanie wykonuje się poprawnie, wkleiłem je do pma i wykonałem zapytanie z przykładowymi danymi i wykonało bez problemu..

[PHP] pobierz, plaintext 
<?PHP
	error_reporting(E_ALL);
	  ini_set('display_errors','1');
 
 
	include("../config.php");
	include("komunikaty.php");
 
	// jeśli user jest zalogowany mysql_error()
if($_SESSION['logged'])
{
	$grupa = mysql_query('SELECT `group` FROM `users` WHERE `id` = '.$_SESSION['id'].'') or die('BŁĄD:'.mysql_error());
	$grupa = mysql_fetch_assoc($grupa);
	if ($grupa['group'] == 2)
	{
		if(isset($_POST['dodaj']))
		{
			$tytul = mysql_real_escape_string($_POST['tytul']);
			$tresc = mysql_real_escape_string($_POST['tresc']);
			$autor = mysql_real_escape_string($_SESSION['nick']);
			$data = mysql_real_escape_string(time());
			$sql = mysql_query('INSERT INTO `news` (`data_dodania` , `tytul`, `tresc`, `autor`) VALUES ('.$data.', '.$tytul.', '.$tresc.', '.$autor.')') or die("BŁĄD ZAPYTANIA");
 
			if(!$tytul || !$tresc)
			{
			echo 'Musisz podać tytuł i treść komunikatu.';
			}
			else{
				if($sql)
				{
				header( "refresh:2;url=index.php" );
				echo 'Komunikat został dodany.';
 
 
 
				}
				else{
				echo 'Problem z zapytaniem<br>mysql_error()';
				}
			}
		}
		else{
		echo 'Problem z dodaniem.';
		}
	}
	else{
	echo 'Musisz się zalogować jako administrator żeby dodać komunikat.';
	}
}
?>
[PHP] pobierz, plaintext 

[nospor]

i wykonałem zapytanie z przykładowymi danymi i wykonało bez problemu..

Ty nie masz wykonać z przykłądowymi danymi, ale masz wykonać dokładnie takie samo jakie wykonujesz tu.

ps: po wprowadzeniu popraprawek nie wyswietliły ci się zadnego dodatkowe błędy?

[kawon]

Zrobiłem tak:

[PHP] pobierz, plaintext 
<?PHP
	error_reporting(E_ALL);
	  ini_set('display_errors','1');
session_start();
 
	include("../config.php");
 
	// jeśli user jest zalogowany mysql_error()
if($_SESSION['logged'])
{
	$grupa = mysql_query('SELECT `group` FROM `users` WHERE `id` = '.$_SESSION['id'].'') or die('BŁĄD:'.mysql_error());
	$grupa = mysql_fetch_assoc($grupa);
	if ($grupa['group'] == 2)
	{
		if(isset($_POST['dodaj']))
		{
			$tytul = mysql_real_escape_string($_POST['tytul']);
			$tresc = mysql_real_escape_string($_POST['tresc']);
			$autor = mysql_real_escape_string($_SESSION['nick']);
			$data = mysql_real_escape_string(time());
			$sql = mysql_query('INSERT INTO `news` (`data_dodania` , `tytul`, `tresc`, `autor`) VALUES ('.$data.', '.$tytul.', '.$tresc.', '.$autor.')') or die('BŁĄD:'.mysql_error());
 
			if(!$tytul || !$tresc)
			{
			echo 'Musisz podać tytuł i treść komunikatu.';
			}
			else{
				if($sql)
				{
				header( "refresh:2;url=index.php" );
				echo 'Komunikat został dodany.';
 
 
 
				}
				else{
				echo 'Problem z zapytaniem<br>mysql_error()';
				}
			}
		}
		else{
		echo 'Problem z dodaniem.';
		}
	}
	else{
	echo 'Musisz się zalogować jako administrator żeby dodać komunikat.';
	}
}
?>
[PHP] pobierz, plaintext 

I teraz gdy w tytule komunikatu lub w treści wpiszę LITERY to mam błąd np:
Unknown column 'test' in 'field list'

Jak wpiszę cyfry w obu np: 1221 to zaś wywala błąd autora komunikatu:
Unknown column 'kawon' in 'field list'

Kolumny w bazie są zapisane na pobieranie danych typu string text więc nie wiem o co chodzi

[nospor]

Przecież napisałem w poprzednim poście:

tekst w zapytaniu bierze się w apostrofy. Ty nie bierzesz.

[kawon]

[PHP] pobierz, plaintext 
$sql = mysql_query('INSERT INTO `news` (`data_dodania` , `tytul`, `tresc`, `autor`) VALUES ('.$data.', '.$tytul.', '.$tresc.', '.$autor.')') or die('BŁĄD:'.mysql_error());
[PHP] pobierz, plaintext 

Wszystko jest w apostrofach, wklejam całe zapytanie do bazy i normalnie dodaje. Więc w którym miejscu poprawka musi być?

[nospor]

NIe, nic nie jest w apostrofach. W apostrofy to ty kolego wstawiłeś zapytanie.

Jakbyś zastosował się do raz z tematu do którego cię odesłałem to byś zrobił tak:
$sql = 'INSERT INTO `news` (`data_dodania` , `tytul`, `tresc`, `autor`) VALUES ('.$data.', '.$tytul.', '.$tresc.', '.$autor.')';
echo $sql;

Nadal tam widzisz apostrofy w tekscie?

[kawon]

Co za głupi błąd.. Oczywiście teraz mi wywala że komunikat dodany i normalnie przerzuca do index.php tak jak miało być, jednak do bazy nie zostanie dodany ten news..

[nospor]

no ale mysql_query($sql); zrobiłeś?
No chyba nie ograniczyłeś się tylko do zrobienia echo $sql?

[kawon]

No tak i zaś problem z apostrofami chociaż mam tak jak kazałeś

[PHP] pobierz, plaintext 
<?PHP
	error_reporting(E_ALL);
	  ini_set('display_errors','1');
session_start();
 
	include("../config.php");
 
	// jeśli user jest zalogowany mysql_error()
if($_SESSION['logged'])
{
	$grupa = mysql_query('SELECT `group` FROM `users` WHERE `id` = '.$_SESSION['id'].'') or die('BŁĄD:'.mysql_error());
	$grupa = mysql_fetch_assoc($grupa);
	if ($grupa['group'] == 2)
	{
		if(isset($_POST['dodaj']))
		{
			$tytul = mysql_real_escape_string($_POST['tytul']);
			$tresc = mysql_real_escape_string($_POST['tresc']);
			$autor = mysql_real_escape_string($_SESSION['nick']);
			$data = mysql_real_escape_string(time());
			$sql = 'INSERT INTO `news` (`data_dodania` , `tytul`, `tresc`, `autor`) VALUES ('.$data.', '.$tytul.', '.$tresc.', '.$autor.')';
			if(!$tytul || !$tresc)
			{
			echo 'Musisz podać tytuł i treść komunikatu.';
			}
			else{
				if(mysql_query($sql))
				{
				header( "refresh:2;url=../index.php" );
				echo 'Komunikat został dodany.';
 
 
 
 
				}
				else{
				echo 'Problem z zapytaniem<br>';
				echo mysql_error();
				}
			}
		}
		else{
		echo 'Problem z dodaniem.';
		}
	}
	else{
	echo 'Musisz się zalogować jako administrator żeby dodać komunikat.';
	}
}
?>
[PHP] pobierz, plaintext 

Błąd mysql:

[XML] pobierz, plaintext 
Problem z zapytaniem
Unknown column 'sedfs' in 'field list'
[XML] pobierz, plaintext 

[nospor]

O ja pierdziu.... nadal nie wstawiłeś tekstu w zapytaniu w aspotrofy!

No ograniczyłeś się tylko do
$sql = 'INSERT INTO `news` (`data_dodania` , `tytul`, `tresc`, `autor`) VALUES ('.$data.', '.$tytul.', '.$tresc.', '.$autor.')';
No przecież ja ci to podałem byś zobaczył jaki masz błąd.
No przecież jak zrobisz
echo $sql;
to dopiero widać jak wygląda to zapytanie. I to zapytanie nie zawiera apostrofów. Ty te apostrofy masz dodać. No pomyśl trochę.

[kawon]

Udało się.. Muszę nauczyć się czytać