Problema &quot;id&quot;

#1
Danbog_1920

Posted 13 February 2010 - 21:05

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

Buna seara !

Am o comanda SQL care imi extrage din baza de date doar datele care au un intr-un camp o anumita valoare. Am folosit un while() si daca am 2, cu line break-ul pus mi se afiseaza pe amandoua. Acum... Pentru a modifica campul acela, am folosit urmatorul cod:

while(...){
//aici sunt echo-urile care extrag din bd acele date care au in camp valoarea respectiva
if(isset($_POST['optiune'])){
$eschiuel = mysql_query("UPDATE `users` SET acel_camp='Da' WHERE id=".$ha['id']." ");
if(!$eschiuel){
echo 'Eroare: mysql_error() ';
}else{
echo 'Anuntul a fost aprobat cu succes';
}
}else{
echo '<form method="POST" action="" >
<input type="submit" name="optiune" value="Aprobare" /> <br /> <br />
</form> ';
}

, unde $ha este mysql_fetch_array($comandaSQL)

Acum...acel buton de Submit apare la fiecare data in parte... dar apasat, schimba cu "Da" la amandoua. Cum as putea face ca atunci cand dau click pe submitul respectivelor date, sa ii modifice doar la acele date campul....

Sper ca s-a inteles... Daca nu dau mai multe explicatii

Edited by Bogdan2901, 13 February 2010 - 21:07.

Back to top

#2
neobluenet

Posted 14 February 2010 - 10:48

Member

Grup: Members
Posts: 489
Înscris: 13.08.2004

Bogdan2901, on 13th February 2010, 21:05, said:

while(...){
	//aici sunt echo-urile care extrag din bd acele date care au in camp valoarea respectiva
	if(isset($_POST['optiune'])){
		$eschiuel = mysql_query("UPDATE `users` SET acel_camp='Da' WHERE id=".$ha['id']." ");
		if(!$eschiuel){
			echo 'Eroare: mysql_error() ';
		}
		else{
			echo 'Anuntul a fost aprobat cu succes';
		}
   }
   else{
	   echo '<form method="POST" action="" >
	   <input type="submit" name="optiune" value="Aprobare" /> <br /> <br />
		</form> ';

   }//asta lipseste in codul tau

}

Mai iti trebuie un hidden input in care sa pui id-ul inregistrarii din baza de date, ca daca dai submit (vezi ca ii trebuie si action la formular) el o sa trimita doar optiune. Pe langa faptul ca trebuie sa modifice, trebuie sa ii spui is ce sa modifice.
Data viitoare scrie codul mai "frumos", ca sa il poata citi lumea mai usor.

Back to top

#3
DarkAngelBv

Posted 14 February 2010 - 12:17

=== Proud owner of === ++++ Audi & Opel ++++

Grup: Senior Members
Posts: 12,747
Înscris: 24.03.2006

neobluenet, on 14th February 2010, 10:48, said:

Mai iti trebuie un hidden input in care sa pui id-ul inregistrarii din baza de date, ca daca dai submit (vezi ca ii trebuie si action la formular) el o sa trimita doar optiune. Pe langa faptul ca trebuie sa modifice, trebuie sa ii spui is ce sa modifice.
Data viitoare scrie codul mai "frumos", ca sa il poata citi lumea mai usor.

Solutia ta nu este tocmai corecta.

Sunt 2 solutii mari si late.

1. adauga form pentru fiecare articol cu input hidden.
2. creeaza butoanele submit de tip imagine si la campul "value" pune valoarea id-ului.

Back to top

#4
Danbog_1920

Posted 14 February 2010 - 12:28

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

Salut ! Intradevar merge, aproba doar cate unul... dar dupa ce am apasat odata dispar toate butoanele de submit

. La hidden am folosit:

<input type="hidden" name="aprobare" value="'.$ha['id'].'" />

Iar la update:

$iid = $_POST['aprobare'];
$eschiuel = mysql_query("UPDATE `users` SET Aprobat='Da' WHERE id='$iid' ");

Back to top

#5
Danbog_1920

Posted 14 February 2010 - 13:16

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

DarkAngelBv, on 14th February 2010, 12:17, said:

Solutia ta nu este tocmai corecta.

Sunt 2 solutii mari si late.

1. adauga form pentru fiecare articol cu input hidden.
2. creeaza butoanele submit de tip imagine si la campul "value" pune valoarea id-ului.

1. Pai eu le iau cu un while din baza de date... Deci echo $ha['Camp1']; e scris odata, dar apare de cate ori se gaseste in baza de date, cu valorile respetive

2. De ce chiar imagine ?

Back to top

#6
DarkAngelBv

Posted 14 February 2010 - 14:48

=== Proud owner of === ++++ Audi & Opel ++++

Grup: Senior Members
Posts: 12,747
Înscris: 24.03.2006

Imagine deoarece la submit-ul de tip imagine proprietatea "value" este ascunsa ochiului.

in loc sa ai

<form1>
<hidden1><submit1>
</form1>
<form2>
<hidden2><submit2>
</form2>
<form3>
<hidden3><submit3>
</form3>

vei avea ceva de genul
<form>
<submit1>
<submit2>
<submit3>
<submit4>
....
</form>

deci mult mai ok din punct de vedere al codului.

Back to top

#7
Danbog_1920

Posted 14 February 2010 - 15:47

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

Bun...dar cum pot pune un buton la fiecare articol ? Si dupa ce accept unul, sa apara in continuare celelalte butoane, iar daca le apas etc

Edited by Bogdan2901, 14 February 2010 - 15:48.

Back to top

#8
DarkAngelBv

Posted 15 February 2010 - 03:14

=== Proud owner of === ++++ Audi & Opel ++++

Grup: Senior Members
Posts: 12,747
Înscris: 24.03.2006

Ca sa ai interactivitate trebuie sa folosesti ajax.
Sau poti folosi solutia mai "din topor", adica sa faci submit catre aceiasi pagina in care sa verifici $_POST.

Back to top

#9
Danbog_1920

Posted 15 February 2010 - 15:04

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

Hm... Pana la urma am rezolvat-o in urmatorul mod:

- am mai creat un fisier in care am pus codul PHP pentru UPDATE;
- ID-ul anuntului este pus intr-un hidden input;
- in al doilea fisier verific daca s-a apasat inaitne butonul, astfel incat daca s-a incercat sa se intre direct pe pagina, nu il va lasa;
- am pus un http-equiv="refresh" , care duce inapoi spre pagina de admin daca totul a decurs perfect. (Cei care intra direct pe pagina pot astepta dupa refresh si o luan de zile....)

Rezultatul: A mers !!!

Multumesc tuturor !

Back to top

#10
OriginalCopy

Posted 15 February 2010 - 15:42

I'm harmful, fear me please! :))

Grup: Senior Members
Posts: 27,268
Înscris: 10.08.2006

Sa vezi ce merge pentru oricine care stie ce face, sa-ti ocoloeasca "protectia".

Usor usor incepi sa ma convingi ca ai talent la a face lucrurile gresit.

De ce nu faci un checkbox pentru fiecare rand cu numele "accept[<id>]", un singur buton de submit in formular, si in procesare iterezi peste $_POST['accept'] ?

Edited by OriginalCopy, 15 February 2010 - 15:47.

Back to top

#11
Danbog_1920

Posted 15 February 2010 - 15:59

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

Hm... as putea incerca. Daca zici ca nu e bine cum am facut eu.

Back to top

#12
OriginalCopy

Posted 15 February 2010 - 17:40

I'm harmful, fear me please! :))

Grup: Senior Members
Posts: 27,268
Înscris: 10.08.2006

Si mai bine ar fi sa te intrebi "de ce nu e bine?", "unde e gaura de securitate/functionalitate?". Nu e "meseria" mea sa-ti bag pe gat informatii. E treaba ta sa te intrebi, eventual sa intrebi pe forum.

Back to top

#13
Danbog_1920

Posted 15 February 2010 - 17:41

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

Bun atunci... cat de curand o sa modific algoritmul codului. Unde e gaura de securitate la versiunea actuala ?

Back to top

#14
OriginalCopy

Posted 15 February 2010 - 17:52

I'm harmful, fear me please! :))

Grup: Senior Members
Posts: 27,268
Înscris: 10.08.2006

Nu e neaparat de securitate (depinde cum folosesti html redirect, poate escala intr-una de securitate), e de functionalitate. Nu poti garanta ca userul care intra pe site-ul tau face asta cu un browser. Poate sa intre cu un program mai mult sau mai putin propriu, sau sa dezactiveze "html refresh" in browserul sau, si atunci nu va fi redirectionat. Va vedea codul HTML, ceea ce e de ajuns. Explicatii gasesti in "resurse pt studiu individual", punctele II. 1. 1, II. 1. 2, II. 1. 3.

Edited by OriginalCopy, 15 February 2010 - 17:53.

Back to top

#15
DarkAngelBv

Posted 15 February 2010 - 19:09

=== Proud owner of === ++++ Audi & Opel ++++

Grup: Senior Members
Posts: 12,747
Înscris: 24.03.2006

Din cate am inteles eu, el doreste acest script pentru administrare. Presupunand ca de administrarea sitului se ocupa el sau cineva de incredere, nu o putem numi gaura de securitate. Cel mult neglijenta.

Back to top

#16
Danbog_1920

Posted 15 February 2010 - 20:01

Active Member

Grup: Banned
Posts: 1,910
Înscris: 24.10.2008

OriginalCopy, on 15th February 2010, 17:52, said:

Nu e neaparat de securitate (depinde cum folosesti html redirect, poate escala intr-una de securitate), e de functionalitate. Nu poti garanta ca userul care intra pe site-ul tau face asta cu un browser. Poate sa intre cu un program mai mult sau mai putin propriu, sau sa dezactiveze "html refresh" in browserul sau, si atunci nu va fi redirectionat. Va vedea codul HTML, ceea ce e de ajuns. Explicatii gasesti in "resurse pt studiu individual", punctele II. 1. 1, II. 1. 2, II. 1. 3.

1. O sa fie sistem de autentificare pe aceea pagina, cu un singur cont creat... fara posibilitatea de a se creea unul nou

2. NU are cum sa ajunga pe pagina aceea ca utilizator capabil sa aprobe un articol. Ii va da : 1. eroarea pe prima pagina ca nu e logat, 2. eroarea pe a doua pagina ca butonul submit nu a fost apasat (adica a intrat direct pe pagina). Redirectul e pus intre if, nu in afara lui sau intre else

Back to top

Anunturi

Chirurgia cranio-cerebrală minim invazivă

Tehnicile minim invazive impun utilizarea unei tehnologii ultramoderne.

Endoscoapele operatorii de diverse tipuri, microscopul operator dedicat, neuronavigația, neuroelectrofiziologia, tehnicile avansate de anestezie, chirurgia cu pacientul treaz reprezintă armamentarium fără de care neurochirurgia prin "gaura cheii" nu ar fi posibilă. Folosind tehnicile de mai sus, tratăm un spectru larg de patologii cranio-cerebrale.

www.neurohope.ro

Problema "id"

#1
Danbog_1920

Posted 13 February 2010 - 21:05

#2
neobluenet

Posted 14 February 2010 - 10:48

#3
DarkAngelBv

Posted 14 February 2010 - 12:17

#4
Danbog_1920

Posted 14 February 2010 - 12:28

#5
Danbog_1920

Posted 14 February 2010 - 13:16

#6
DarkAngelBv

Posted 14 February 2010 - 14:48

#7
Danbog_1920

Posted 14 February 2010 - 15:47

#8
DarkAngelBv

Posted 15 February 2010 - 03:14

#9
Danbog_1920

Posted 15 February 2010 - 15:04

#10
OriginalCopy

Posted 15 February 2010 - 15:42

#11
Danbog_1920

Posted 15 February 2010 - 15:59

#12
OriginalCopy

Posted 15 February 2010 - 17:40

#13
Danbog_1920

Posted 15 February 2010 - 17:41

#14
OriginalCopy

Posted 15 February 2010 - 17:52

#15
DarkAngelBv

Posted 15 February 2010 - 19:09

#16
Danbog_1920

Posted 15 February 2010 - 20:01

Anunturi

▶ 0 user(s) are reading this topic

Problema "id"

#1 Danbog_1920 Posted 13 February 2010 - 21:05

#2 neobluenet Posted 14 February 2010 - 10:48

#3 DarkAngelBv Posted 14 February 2010 - 12:17

#4 Danbog_1920 Posted 14 February 2010 - 12:28

#5 Danbog_1920 Posted 14 February 2010 - 13:16

#6 DarkAngelBv Posted 14 February 2010 - 14:48

#7 Danbog_1920 Posted 14 February 2010 - 15:47

#8 DarkAngelBv Posted 15 February 2010 - 03:14

#9 Danbog_1920 Posted 15 February 2010 - 15:04

#10 OriginalCopy Posted 15 February 2010 - 15:42

#11 Danbog_1920 Posted 15 February 2010 - 15:59

#12 OriginalCopy Posted 15 February 2010 - 17:40

#13 Danbog_1920 Posted 15 February 2010 - 17:41

#14 OriginalCopy Posted 15 February 2010 - 17:52

#15 DarkAngelBv Posted 15 February 2010 - 19:09

#16 Danbog_1920 Posted 15 February 2010 - 20:01

Anunturi

▶ 0 user(s) are reading this topic

Sign In

#1
Danbog_1920

Posted 13 February 2010 - 21:05

#2
neobluenet

Posted 14 February 2010 - 10:48

#3
DarkAngelBv

Posted 14 February 2010 - 12:17

#4
Danbog_1920

Posted 14 February 2010 - 12:28

#5
Danbog_1920

Posted 14 February 2010 - 13:16

#6
DarkAngelBv

Posted 14 February 2010 - 14:48

#7
Danbog_1920

Posted 14 February 2010 - 15:47

#8
DarkAngelBv

Posted 15 February 2010 - 03:14

#9
Danbog_1920

Posted 15 February 2010 - 15:04

#10
OriginalCopy

Posted 15 February 2010 - 15:42

#11
Danbog_1920

Posted 15 February 2010 - 15:59

#12
OriginalCopy

Posted 15 February 2010 - 17:40

#13
Danbog_1920

Posted 15 February 2010 - 17:41

#14
OriginalCopy

Posted 15 February 2010 - 17:52

#15
DarkAngelBv

Posted 15 February 2010 - 19:09

#16
Danbog_1920

Posted 15 February 2010 - 20:01