[MySQL][PHP]Zabawa z chechboxami, Działania na wybranych rekordach Opcje

[andrzejlechniak]

Hej wszystkim.

Mam skrypt, w którym za pomocą zaznaczenia opcji 'checkbox' chcę wykonywać parę działań. Ale coś mi to wszystko nie chce działać.

Oto skrypt 'index.php':

[PHP] pobierz, plaintext 
<?php
require_once ('baza/db.php');
error_reporting(E_ALL);
 
echo '<form action="action.php" method="POST">';
echo '<p>Tabela testu <span style="font-weight:bold;">checkboxa</span>.</p>';
 
$sql = 'SELECT * FROM tab_test';
$result = mysql_query($sql) or die(mysql_error());
while ($row = mysql_fetch_array($result))
{
	extract($row);
	echo '<div style="border: 1px solid #bbbbbb; margin-bottom: 3px;"><p>
			&nbsp '.$id.'.  
			Status - <input type="checkbox" title="zmien status" name="update" /> |
			List - <input type="checkbox" title="wyslij list" name="e_mail" value="'.$e_mail.'" /> |
			'.$pole_1.' - '.$pole_2.' - ';
			if($stat==1)
			{ 
				echo 'status Opublikowany';
			}
			else 
			{ 
				echo 'status Niepublikowany';
			}
	echo '</p></div>';
}
echo '<input type="checkbox" /> Zaznacz wszystkie osoby <br />';
echo '<input type="submit" name="zmien" value="Zmien status">';
echo '<input type="submit" name="napisz" value="Napisz list">';
echo '</form>';
[PHP] pobierz, plaintext 

a to skrypt, gdzie przesyłane są dane:

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL);
require_once ('baza/db.php');
 
if(isset($_POST['wyslij']))
{
	$tytul = mysql_real_escape_string(htmlspecialchars($_POST['tytul']));
	$tresc = mysql_real_escape_string(htmlspecialchars($_POST['tresc']));
 
	if(empty($tytul) || empty($tresc))
	{
		echo 'Jedno z pól jest puste.';
	}
	else
	{
		///Wysłanie listu. Pod ten adres zostanie wysłana wiadomosc
		$adresat = 'adresat@wp.pl'; //wybrane adresy osób z bazy
		$tytul = 'List z mojej strony';
		@$content = $_POST['tresc'];
		@$email = $_POST['e_mail'];
 
		if (mail($adresat, $tytul, $content, $email)) 
		{
			$adresat = $_POST['osoby'];
			$sql_subs = "INSERT INTO tab_testa (tresc,osoby,data) VALUES ('$tresc','$osoby',CURDATE());";
			$result = mysql_query($sql_subs) or die(mysql_error());
			if($result)
			{
				echo 'Zapisano w bazie.';
			}
			else
			{
				echo 'Nie udało się zapisać do bazy.';
			}
 
			echo '<p>Udało się</p><hr />';
			exit;
		}
	}
}
else 
{
	echo '<p>Nie udało się wysłać listu.</p><hr />';
}
 
////////////////////////////////////////////////////////////////////
if(isset($_POST['zmien']))
{
	if(empty($_POST['update']))
	{
		echo 'Nie wybrano akcji.<br />';
		echo 'Wróć do <a href="index.php">Strony głównej</a>.';
	}
	else
	{	
		$sql_update = "UPDATE tab_test SET stat='$stat'";
		$result = mysql_query($sql_update) or die(mysql_error());
 
		if($result)
		{
			echo 'Udało się.<br />';
			echo 'Wróć do <a href="index.php">Strony głównej</a>.';
		}
		else
		{
			echo 'Nie udało się.<br />';
			echo 'Wróć do <a href="index.php">Strony głównej</a>.';
		}
	}
}
else if(isset($_POST['napisz']))
{	
	if(empty($_POST['e_mail']))
	{
		echo 'Nie wybrano adresu.<br />';
		echo 'Wróć do <a href="index.php">Strony głównej</a>.';
	}
	else
	{
		$_POST['e_mail'] = implode(', ', $_POST['e_mail']);
		echo '<form action="" method="POST">';
		echo '<div style="font-weight:bold;">Napisz list.</div>';
		echo 'Wybrane osoby - '.$_POST['e_mail'].' <br />';
		echo 'Wpisz tytul: <input type="text" name="tytul" /><br />';
		echo 'Napisz tresc: <br /><textarea rows="10" cols="20" name="tresc"></textarea><br />';
		echo '<input type="submit" name="napisz_list" value="Wyslij" /><br/>';
		echo '</form>';
	}
}
 
?>
[PHP] pobierz, plaintext 

Taki mam problem. Chcę aktualizować status użytkownika i choć instrukcja if - else działa [tzn. pokazuje info, że udało się], to w bazie nie zmieniło się nic, czyli polecenie UPDATE nie zadziałało. Chodzi mi też o wysyłanie listu, gdy klikam na wybrane osoby [dla przykładu mam dwie osoby], to chyba funkcja explode() działa tak, że powinna rozpisać te stringi przesyłane przez formularz, np. jasio@o2.pl, malgosia@o2.pl. A mi to nie działa, gdzie tu popełniłem błąd?

[strife]

Najlepiej wyświetl sobie to zapytanie (sprintuj je) a potem wklej do konsoli mysql i zobacz jakie masz błędy.
Jak używasz mysql_real_escape_string to już nie potrzebujesz htmlspecialchars.

No i ostatnie, mówisz coś o funkcji explode(), ale nigdzie jej nie widzę w kodzie : >

[andrzejlechniak]

Co do updatu to mam tak:
Status - zrobiłem jak radziłeś i kod działa tylko nic nie wykazuje...

[PHP] pobierz, plaintext 
<input type="checkbox" title="zmien status" name="update" />
[PHP] pobierz, plaintext 

ale w pliku action.php dałem $stat = 1; co też nie wiele zmienia, bo zawsze jak zmieniam, to i tak zmienia się na wartość 1 [i to jest zrozumiałe], ew. można to zrobić za pomocą if -else, ale czy nie da się tak to napisać, aby automatycznie kod zmieniał na wartość odwrotną, np. gdy ma wartość 0 to zmienia na 1 i odwrotnie?

Co do adresów to zmieniłem trochę kod na:

[PHP] pobierz, plaintext 
else
	{
		$email = $_POST['e_mail'];
		$adresy = (explode(', ',$email));
 
		echo '<form action="" method="POST">';
		echo '<div style="font-weight:bold;">Napisz list.</div>';
		echo 'Wybrane osoby - '.$adresy.' <br />';
		echo 'Wpisz tytul: <input type="text" name="tytul" /><br />';
		echo 'Napisz tresc: <br /><textarea rows="10" cols="20" name="tresc"></textarea><br />';
		echo '<input type="submit" name="napisz_list" value="Wyslij" /><br/>';
		echo '</form>';
	}
[PHP] pobierz, plaintext 

co nie wiele zmienia, bo wyświetla się jedynie wiadomość, że jest to tablica.

Proszę, pomóżcie

Ten post edytował andrzejlechniak 20.02.2012, 20:36:12

[Daimos]

[PHP] pobierz, plaintext 
$sql_update = "UPDATE tab_test SET stat='$stat'";
[PHP] pobierz, plaintext 

nic dziwnego, bo w kodzie który podałeś, nie ma nigdzie zdefiniowanej zmiennej stat, a chyba nie korzystasz z register globals. Oprócz tego, pole checkbox o nazwie stat nie ma żadnej wartości (value=)

Poczytaj trochę o debugowaniu kodu, zaczynając od vardumpów co linijka (IMG:style_emoticons/default/wink.gif) przechwytywanie błędów, po różnego rodzaju firephp itp. Bez tego będziesz wiecznie błądził w kodzie i tracił czas na szukanie błędów

Ten post edytował Daimos 20.02.2012, 20:55:15

[andrzejlechniak]

Poczytam, poczytam. Zainstalowanie też jest pomysłem niezłym [już to zrobiłem]. Ale wracając do mojego problemu, cały czas kombinuję. Rozumiem, że każdy checkbox powinien przyjmować wartość, a więc zmieniłem checkboxa na:

[PHP] pobierz, plaintext 
Status - <input type="checkbox" title="zmien status" name="update[]" value="'.$id.'" /> |
[PHP] pobierz, plaintext 

Podobnie sprawa ma się jeśli chodzi o adresy e-mail:

[PHP] pobierz, plaintext 
List - <input type="checkbox" title="wyslij list" name="e_mail[]" /> |
[PHP] pobierz, plaintext 

Tylko czy też ten checkbox ma przyjmować wartość, chyba tak?

---

Może też chodzi o to, aby jeżeli wiele jest elementów takich samych, to w tablicy można je wyświetlać jako wartość przyjmując 'id' osoby, tylko teraz wyświetlić te elementy za pomocą explode(), bo to co napisałem wcześniej;

[PHP] pobierz, plaintext 
else if(isset($_POST['napisz']))
{	
	if(empty($_POST['e_mail']))
	{
		echo 'Nie wybrano adresu.<br />';
		echo 'Wróć do <a href="index.php">Strony głównej</a>.';
	}
	else
	{
		$email = $_POST['e_mail'];
		$email_1 = (explode(', ',$email));
 
		echo '<form action="" method="POST">';
		echo '<div style="font-weight:bold;">Napisz list.</div>';
		echo 'Wybrane osoby - '.$email_1.' <br />';
		echo 'Wpisz tytul: <input type="text" name="tytul" /><br />';
		echo 'Napisz tresc: <br /><textarea rows="10" cols="20" name="tresc"></textarea><br />';
		echo '<input type="submit" name="napisz_list" value="Wyslij" /><br/>';
		echo '</form>';
	}
}
[PHP] pobierz, plaintext 

to chyba nie do końca jest dobre i w ogóle nie działa.

[Daimos]

[HTML] pobierz, plaintext 
name="e_mail[]"
[HTML] pobierz, plaintext 

[] w nazwie, oznacza tablicę. Skoro wysyłasz tablicę, to po co dajesz później explode? Explode się robi na stringu
Druga sprawa, że explode zwraca tablicę, a Ty próbujesz to później wyświetlić w tekście
Więc lekcja debugowania nr. 1 - podstawy podstaw (IMG:style_emoticons/default/biggrin.gif)

[PHP] pobierz, plaintext 
$email = $_POST['e_mail'];
// dostawiasz:
var_dump($email);die();
// sprawdzasz co było, podejmujesz działania w związku z tym lub nie i dalej:
$email_1 = (explode(', ',$email));
var_dump($email_1); die();
// analogicznie :)
[PHP] pobierz, plaintext 

Ten post edytował Daimos 20.02.2012, 21:17:33

[andrzejlechniak]

Dobra, dzięki za cenną lekcję. Dodałem wartość:

[PHP] pobierz, plaintext 
List - <input type="checkbox" title="wyslij list" name="e_mail[]" value="'.$e_mail.'" /> |
[PHP] pobierz, plaintext 

Zmieniłem znów:

[PHP] pobierz, plaintext 
else
	{
		$email = $_POST['e_mail'];
		$email_1 = Array($email);
		echo $email_1;
 
		echo '<form action="" method="POST">';
		echo '<div style="font-weight:bold;">Napisz list.</div>';
		echo 'Wybrane osoby - '.$email_1.' <br />';
...
[PHP] pobierz, plaintext 

i... wyświetla mi się info, że jest to tablica - po prostu Array, a nie wyciąga adresów, a o to mi chodziło.

[Daimos]

bo to tablica, wyświetl w pętli, albo użyj implode

[PHP] pobierz, plaintext 
echo implode(', ',$email_1);
[PHP] pobierz, plaintext 

Ten post edytował Daimos 22.02.2012, 01:05:30

[andrzejlechniak]

Zrobiłem jak poradziłeś, ale efekt jest taki sam, a właściwie nie widzę zmian.

Już sobie poradziłem, ale dzięki wszystkim za dobre chęci. Pozdrawiam (IMG:style_emoticons/default/snitch.gif)

Ten post edytował andrzejlechniak 22.02.2012, 13:42:02