Forum PHP.pl

mam taki problem: zrobiłem sobie formularz, który pobiera dane do pól, a następnie umożliwia ich zmiane(można powiedzieć zwyczajny formularz). Niestety nie wiem dlaczego nie chce działać podstawowa jego funkcja ->UPDATE<-
oto i on:

[PHP] pobierz, plaintext 
<?php
//update_reklamacja.php 
 
if($akcja=='zmiendane') { 
$sql_U = "UPDATE reklamacja 
		SET id_klienta='$id_klienta', 
		data_reklamacji='$data_reklamacji', data_zwrotu='$data_zwrotu', 
		stan='$stan', 
		opis_usterki='$opis_usterki' , dodatkowe_info='$dodatkowe_info', 
		os_przyj='$os_przyj', os_modyf='$os_modyf' 
		WHERE id_reklamacji='$id_reklamacji' "; 
 
$wynik_u=mysql_query($sql_U); 
 
if (!$wynik_u) { 
	echo 'Zmiany sie nie powiodły'; 
} 
else 
{ 
	echo 'Zmiany się powiodły'; 
	 
		} 
 
	} 
	 
	$sql="select id_reklamacji, id_klienta, data_reklamacji, data_zwrotu, stan, 
				 opis_usterki, dodatkowe_info, os_przyj, os_modyf from reklamacja"; 
	 
	$wynik=mysql_query($sql); 
	 
	$ile=mysql_num_rows($wynik); 
	 
	if (!$ile) { 
	print ("<p>Brak zanowień</p><br/>"); 
} 
else { 
 
	echo "<form action='update_reklamacja.php' method='POST'>"; 
	echo "<input type='hidden' name='akcja' value='zmiendane'>";	//tablea 
	echo '<table align=center border=2>'; 
	echo '<tr> 
	<td>ID REK </td> 
	<td>ID KLIENTA </td> 
	<td>DATA REKLAMACJI </td> 
	<td>DATA ZWROTU </td> 
	<td>STAN </td> 
	<td>OPIS USTERKI </TD> 
	<td>INFO DLA KLIENTA </td> 
	<td>OS PRZYJM. USTERKE </td> 
	<td>OS MODYF. </td> 
	</tr>'; 
	 
	 
	for ($i=0 ; $i<$ile ; $i++) 
	{ 
	$wiersz=mysql_fetch_row($wynik); 
	 
	echo "<tr><td>"; 
	echo "<input type='text' size='3' name='id_reklamacji' value='$wiersz[0]' DISABLED>"; 
	echo "</td>"; 
	echo "<td>"; 
	echo "<input type='text' size='2' name='id_klienta' value='$wiersz[1]'>"; 
	echo "</td>"; 
	echo "<td>"; 
	echo "<input type='text' size='10' name='data_reklamacji' value='$wiersz[2]'>"; 
	echo "</td>"; 
	echo "<td>"; 
	echo "<input type='text' size='10' name='data_zwrotu' value='$wiersz[3]'>"; 
	echo "</td>"; 
	echo "<td>  "; 
	echo "<SELECT NAME='stan'> 
	 <OPTION>Do odbioru 
	 <OPTION>W naprawie 
	 <OPTION  SELECTED DISABLED>$wiersz[4] 
		</SELECT>" ; 
	echo "</td> "; 
	echo "<td>"; 
	echo "<textarea name='opis_usterki'cols='17' rows='6'>$wiersz[5]</textarea>"; 
	echo "</td>"; 
	echo "<td>"; 
	echo "<textarea name='dodatkowe_info'cols='17' rows='6'>$wiersz[6]</textarea>"; 
	echo "</td>"; 
	echo "<td>"; 
	echo "<input type='text' name='os_przyj' value='$wiersz[7]' DISABLED>"; 
	echo "</td>"; 
	echo "<td>"; 
	echo "<input type='text' name='os_modyf' value='$wiersz[8]'>"; 
	echo "</td>"; 
	echo "<td>"; 
	echo "</td></tr>"; 
 
	} 
	echo '</table>'; 
	//koniec tablea 
	 
	echo "<input align='center' type='submit' name='akcja' value='zmień' onclick='confirm('JESTEŚ PEWNY !!! (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ? ')'>"; 
echo "</form>"; 
 
	}
 
?>
[PHP] pobierz, plaintext 

działanie jego pokazane jest na stronie http://radin.pl/sklep/sys/update_reklamacja.php

mógłby ktoś powiedzieć gdzie robie błąd ?

Zamiast $id_klienta napisz $_POST["id_klienta"], to samo z pozostałymi zmiennymi.

nie pomogło (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

jaki blad wywala?

o to chodzi że błędu nie wywala ale także nie chce zrobić update danych i nie wiem dlaczego

Warunek if($akcja=='zmiendane') { jest niespelniony.

a mógbyś porawić to tak aby działało bo ja nie mam pomysłu, nad taką głupotą siedze cały dzień

A próbowałeś:

[PHP] pobierz, plaintext 
<?php
if($_POST['akcja']=='zmiendane')
?>
[PHP] pobierz, plaintext 

?

tak próbowałem nie skutkuje (IMG:http://forum.php.pl/style_emoticons/default/sadsmiley02.gif)

Powiedz mi asterix, czy to tak ciężko wyswietlic sobie zapytanie i sprawdzicm,czy oby napewno jest ono dobrze generowane? Jaksbys dal po utworzeniu zapytania:

[PHP] pobierz, plaintext 
<?php
echo $sql_U;
?>
[PHP] pobierz, plaintext 

i przyjrzal sie jemu, to bys zobaczyl co jest nie tak.

Robisz update ale na warunku: WHERE id_reklamacji='$id_reklamacji'
juz to powinines sprawdzic, czy oby napewno zmienna $id_reklamacji ma jakąś wartość. Gdybys to sprawdzilm, o bys zobaczyl ze nie ma zadnej wartosci. A wiesz czemu? Poniewaz dla pola input, ktory przechowuje tą wartość, nadajesz atrybut disabled. Inputy z takim attrybutem nie są przesyłane przez formularz, są poprostu olewane. Zrob z tego pola pole ukryte, lub nadaj mu atrybut readonly a nie disabled.

No idodatkowo zmodyfikuj kod o kontrole ewentualnych bledow:

[PHP] pobierz, plaintext 
<?php
 
//.....
echo $sql_U;
$wynik_u=mysql_query($sql_U) or die(mysql_error()) ;
//....
 
?>
[PHP] pobierz, plaintext 

wielkie dzięki za zjechanie mnie czasami sie to przydaje, ale no właśnie ale:
wynik tego jest na stronce:http://radin.pl/sklep/sys/update_reklamacja.php
UPDATE DZIAŁA TYLKO DLA OSTATNIEGO ID_REKLAMACJI (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) !

[SQL] pobierz, plaintext 
UPDATE reklamacja SET id_klienta='2', data_reklamacji='2005-10-11', data_zwrotu='2005-10-14', stan='W naprawie', opis_usterki='tutaj tylko sie robi update' , dodatkowe_info='tylko dla ostatniego wpisu, nie wiem dlaczego?', os_przyj='sta?', os_modyf='' WHERE id_reklamacji='4' Zmiany się powiodły
[SQL] pobierz, plaintext 

A tak nie może być, więc moje kolejne pytanie jest takie:
W jaki sposób przerobić zapytanie aby obojętnie dla jakiej krotki(ek) dokonam zmiany, były one odpowiednio updetowane?
Czyli np wpisuje jakąś zmianę do każdej reklamacji i tam sie dokonują zmiany, lub też wpisuje tylko w jednym(bądź dwum) id

moje zapytanie sql wygląda teraz tak:

[SQL] pobierz, plaintext 
$sql_U = "UPDATE reklamacja 
        SET id_klienta='$_POST[id_klienta]', 
        data_reklamacji='$_POST[data_reklamacji]', data_zwrotu='$_POST[data_zwrotu]', 
        stan='$_POST[stan]', 
        opis_usterki='$_POST[opis_usterki]' , dodatkowe_info='$_POST[dodatkowe_info]', 
        os_przyj='$_POST[os_przyj]', os_modyf='$_POST[os_modyf]' 
        WHERE id_reklamacji='$_POST[id_reklamacji]' ";
[SQL] pobierz, plaintext 

1) nie krzycz
2) nic dziwnego, gdyz nadpisujesz dane poprzez te same nazwy inputow

daj nazwy jako tablice, uzywajac []:
name='id_reklamacji[]'
i tak dal kazego inputa. Dobierac sie bedziesz
$_POST['id_reklamacji'][0],$_POST['id_reklamacji'][1], ....
i podobnie pozostale pola. bądz tez buduj recznie nazwy z $i:

[PHP] pobierz, plaintext 
<?php
 
//...petla
echo "<input type='text' size='3' name='id_reklamacji_$i' ....";
 
?>
[PHP] pobierz, plaintext 

a dobierasz sie tez w petli:

[PHP] pobierz, plaintext 
<?php
 
//....petla do updatea
echo $_POST['id_reklamacji_'.$i]; 
 
?>
[PHP] pobierz, plaintext 

obie metody byly juz na forum.

Jak chcesz to moge częściej (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Możesz zrobić jeszcze inaczej, a mianowicie:

[PHP] pobierz, plaintext 
<?php
 
//...petla
echo "<input type='text' size='3' name='id_reklamacji[$i]' ....";
 
?>
[PHP] pobierz, plaintext 

a następnie odbierasz zmienną która w tym momencie przyjęła wartość ARRAY

[PHP] pobierz, plaintext 
<?php
 
..... //odbieranie innych zmiennych formularza
$reklamacja_id=$_POST['id_reklamacji']; 
//gotowa tablica z wartościami z formularza
 
print_r ($reklamacja_id); 
//wyświetla ci całą zawartośc tablicy wraz z ich kluczami
 
?>
[PHP] pobierz, plaintext 

(IMG:http://forum.php.pl/style_emoticons/default/dry.gif) (IMG:http://forum.php.pl/style_emoticons/default/cool.gif)

@morhen to co zaproponowales jest jednoznaczne z moim [] Tyle tylko ze ty jawnie nadajesz indexy dla tablicy. zastosowanie samego [] powoduje ze on sam nadaje indexy od 0

zrobiłem tak jak powiedziałeś:

[PHP] pobierz, plaintext 
<?php
 
if($_POST['akcja']=='zmiendane') {
 
$id="select count(id_reklamacji) from reklamacja";
for ($i=0;$i<=$id;$i++)
{
	$sql_U = "UPDATE reklamacja 
		SET id_klienta='".$_POST['id_klienta_'.$i]."', 
		data_reklamacji='".$_POST['data_reklamacji_'.$i]."', data_zwrotu='".$_POST['data_zwrotu_'.$i]."', 
		stan='".$_POST['stan_'.$i]."', 
		opis_usterki='".$_POST['opis_usterki_'.$i]."' , dodatkowe_info='".$_POST['dodatkowe_info_'.$i]."', 
		os_przyj='".$_POST['os_przyj_'.$i]."', os_modyf='".$_POST['os_modyf_'.$i]."' 
		WHERE id_reklamacji='".$_POST['id_reklamacji_'.$i]."' "; 
 
 
 
echo $sql_U;
$wynik_u=mysql_query($sql_U) or die(mysql_error($sql_U)); 
 
if (!$wynik_u) { 
	echo 'Zmiany sie nie powiodły'; 
} 
else 
{ 
	echo 'Zmiany się powiodły'; 
		} 
 
	}
} 
?>
[PHP] pobierz, plaintext 

Dalej także pozmieniałem <inputy> na tego typu

[PHP] pobierz, plaintext 
<?php
 echo "<input type='hidden' size='3' name='id_reklamacji_$i' value='$wiersz[0]' >"; 
?>
[PHP] pobierz, plaintext 

no i wynik jest taki że zmienia mi tylko pierwszy rekord (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)
why (IMG:http://forum.php.pl/style_emoticons/default/withstupidsmiley.gif)
tutaj pokazane działanie

ta linijka:

[PHP] pobierz, plaintext 
<?php
$id="select count(id_reklamacji) from reklamacja";
?>
[PHP] pobierz, plaintext 

Tu niby pobierasz liczbę rekordów, ale ty do id przypisujesz zapytanie, a nie jego wynik. ma byc:

[PHP] pobierz, plaintext 
<?php
$sql="select count(id_reklamacji) from reklamacja";
$result = mysql_query($sql) or die(mysql_error());
$row = mysql_fetch_array($result);
$count = $row[0];
for ($i=0;$i<$count;$i++)
{
//......
}
 
?>
[PHP] pobierz, plaintext 

i nie <= a < (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) , gdyz jedziesz od 0

pozdro

poradziłem sobie troche inaczej z tym problemem po przez dodanie jeszcze jednego formulaża.

Niechcąc otwierać nowego topicu przedstawie swój kolejny problem(bardzo powiązany)
To samo(jak wyżej opisałem że zrobiłem) już nie może przejść mi w przypadku kiedy mam takie zapytanie w sql:

[PHP] pobierz, plaintext 
<?php
 
$sql="
UPDATE klient k, reklamacja r, zamowienia z set k.imie='".$_post['imie']."' , .... ,
r.opis_usterki='".$_post['opis_usterki']."' , ... , 
z.zlecenie='".$_post['zlecenie']."' , ... 
Where k.id_klienta='".$_POST['id_klienta']."' ";
 
?>
[PHP] pobierz, plaintext 

Zapytanie powodujące wyświetlenie danych w formularzu jest następujące:

[PHP] pobierz, plaintext 
<?php
 
$sql="select k.id_klienta, k.imie, k.nazwisko, k.nr_tel_dom, k.nr_tel_kom,
    z.id_zamowienia, z.data_zamowienia, z.data_finalizacji, z.stan_z, z.zlecenie
, z.dodatkowe_info, z.os_odbierajaca_zlecenie, z.os_modyfik_zlecenie,
        r.id_reklamacji, r.data_reklamacji, r.data_zwrotu, r.stan, r.opis_usterk
i, r.dodatkowe_info, r.os_przyj, r.os_modyf
          FROM klient k 
            left join zamowienia z on z.id_klienta=k.id_klienta
        left join reklamacja r on r.id_klienta=k.id_klienta 
        WHERE k.id_klienta='".$_GET['id']."' ";
 
?>
[PHP] pobierz, plaintext 

gdzie zmienna $_GET{'id'] przekazywana jest po przez formularz. Wszystkie dane są wyświetlane w odpowiednich inputach.

Mój problem jest taki że nie wiem jak zbudować odpowiednie zapytania/update aby po wyświetleniu się paru wyników można było zmienić każdy z wyników i żeby to zapamiętał. Sposoby które zostały podane wcześniej jakoś mi nie chcą działać (IMG:http://forum.php.pl/style_emoticons/default/sadsmiley02.gif)

Dodam że wyświetlanie wyników mam zrobione na takiej samej zasadzie jak wyżej

Prosze o pomoc

Pozdrawiam

Ten post edytował asterix1983 2.11.2005, 12:26:49

Morzesz cos w ten sposob zrobic

[SQL] pobierz, plaintext 
UPDATE tabela SET pole = pole + 1 WHERE ..........
[SQL] pobierz, plaintext 

Przed wyswietleniem wywolujesz UPDATE.