[SQL & PHP] Basic query werkt niet

1
2
3
4
5

DELETE FROM `reserveringen` 
WHERE `reservering_id` = 'een_integer' 
AND `gereserveerd_door` = 'een_username' 
OR `gereserveerd_voor` = 'een_username' 
OR naam.admins = 'een_username'

1
2
3
4
5

DELETE FROM `reserveringen` 
WHERE `reservering_id` = 'een_integer' 
AND `gereserveerd_door` = 'een_username' 
OR `gereserveerd_voor` = 'een_username' 
OR admins.naam = 'een_username'

1
2
3
4

DELETE FROM `reserveringen` 
WHERE `reservering_id` = 'een_integer' 
AND `gereserveerd_door` = 'een_username' 
OR `gereserveerd_voor` = 'een_username'

[ Voor 5% gewijzigd door CH4OS op 01-06-2005 13:04 ]

whoami schreef op woensdag 01 juni 2005 @ 13:04:
Rond numerieke waarden moet je geen quotes zetten.

1
2

OR 
admins.naam = 'een_user'

[ Voor 12% gewijzigd door CH4OS op 01-06-2005 13:05 ]

Sybr_E-N schreef op woensdag 01 juni 2005 @ 13:05:
Het feit dat MySQL het slikt wil niet zeggen dat het good practice is.

whoami schreef op woensdag 01 juni 2005 @ 13:10:
Het probleem zal 'm zitten in die OR condities.
Gebruik eens haakjes.

1
2
3
4
5
6
7
8
9
10

DELETE FROM 
reserveringen
WHERE 
reserveringen.reservering_id = 68 
AND
(reserveringen.gereserveerd_door = 'gertjanp' )
OR 
(reserveringen.gereserveerd_voor = 'gertjanp' 
OR 
admins.naam = 'gertjanp' )

1
2
3
4
5
6
7
8
9
10

DELETE FROM 
reserveringen
WHERE 
reserveringen.reservering_id = 68 
AND
(reserveringen.gereserveerd_door = 'gertjanp' )
OR 
(reserveringen.gereserveerd_voor = 'gertjanp')
OR 
(admins.naam = 'gertjanp' )

[ Voor 48% gewijzigd door CH4OS op 01-06-2005 13:13 ]

Nick The Heazk schreef op woensdag 01 juni 2005 @ 13:14:
Ik ga voor een typfout;

OR naam.admins = 'een_username' != OR admins.naam = 'een_username'

bovendien vermoed ik dat je je OR constructie verkeerd bet gemaakt. Nu evalueert hij: (dit EN dat) OF (dit) OF (dat). Ik denk dat je dit bedoelt: dit EN (dit OF dat OF ditter)

1
2
3
4
5
6
7
8
9
10

DELETE FROM 
reserveringen
WHERE 
(reserveringen.reservering_id = 68)
AND
(reserveringen.gereserveerd_door = 'een_username' 
OR 
reserveringen.gereserveerd_voor = 'een_username' )
OR 
admins.naam = 'een_username'

1
2
3
4
5
6
7
8
9
10

DELETE FROM 
reserveringen
WHERE 
(reservering_id = 68)
AND
(gereserveerd_door = 'een_username' 
OR 
gereserveerd_voor = 'een_username' )
OR 
admins.naam = 'een_username'

[ Voor 53% gewijzigd door CH4OS op 01-06-2005 13:24 ]

Ramon de Jesus schreef op woensdag 01 juni 2005 @ 13:25:
maar even voor de goede orde, je haalt uit de tabel admins het veld naam. Maar hoe weet MySQL op welke rij hij moet zoeken naar die naam?

1
2

OR 
admins.naam = 'een_username'

[ Voor 44% gewijzigd door CH4OS op 01-06-2005 13:29 ]

LuCarD schreef op woensdag 01 juni 2005 @ 13:30:
Maar je probeert toch wat te verwijderen in de reserveringen tabel en niet in de admin tabel.

[ Voor 66% gewijzigd door CH4OS op 01-06-2005 13:36 ]

BCC schreef op woensdag 01 juni 2005 @ 13:35:
Dat begrijpt Ramon ook wel. Punt is dat je niet zomaar even naar een regel in een tabel kan wijzen bij zo'n delete. Dat moet óf in een subquery of je moet eerst een losse query doen om te kijken of die user admin is. Bij bijvoorbeeld een select kan dit wel, omdat je daar met meerdere tables bezig kan zijn. Bij een Delete kan dit volgens mij niet.

T-MOB schreef op woensdag 01 juni 2005 @ 13:36:
Wat jij bedoeld met je admins.naam = 'admin_naam' is dat de naam moet voorkomen in de admins tabel.

Voor mysql betekent het echter dat er een rij wordt verwijderd waar al dan niet dmv JOINs een kolom admins.naam is waar de adminnaam gelijk is aan 'admin_naam'.

1
2

WHERE `reservering_id` = 68 
AND (`gereserveerd_voor` = 'een_user' OR `geserveerd_door` = 'een_user' OR JOIN { join syntax even opzoeken })

De makkelijkste manier is waarschijnlijk om gewoon aan het begin van je script te bepalen of iemand admin is. Admins zullen wel meer bijzondere rechten hebben in je pagina. Uitgaande van een 1 als admin en 0 voor normale users zet je dan het volgende in je query:

code:

1 2 3 4 5 6 7 8 9 10

DELETE FROM reserveringen WHERE (reservering_id = 68) AND (gereserveerd_door = 'een_username' OR gereserveerd_voor = 'een_username' OR $isAdmin )

storeman schreef op woensdag 01 juni 2005 @ 13:40:
Ik denk dat je beide tabellen moet noemen bij een delete query.

code:

1 2 3 4 5

DELETE reserveringen.* FROM `reserveringen`, 'admins' WHERE `reserveringen.reservering_id` = 'een_integer' AND `reserveringen.gereserveerd_door` = 'een_username' OR `reserveringen.gereserveerd_voor` = 'een_username' OR 'admins.naam' = 'een_username'

Het lijkt mij bovendien logisch dat de integer altijd moet bestaan. Gebruik daarom haakjes:

code:

1 2 3 4 5

DELETE reserveringen.* FROM `reserveringen`, 'admins' WHERE `reserveringen.reservering_id` = 'een_integer' AND (`reserveringen.gereserveerd_door` = 'een_username' OR `reserveringen.gereserveerd_voor` = 'een_username' OR 'admins.naam' = 'een_username')

Anders weet je nooit welk er verwijderd moet worden en kijkt ie alleen naar username. Zou je niet willen denk ik!

Succes!

1
2
3

ERROR 1064: You have an error in your SQL syntax near ' `admins`
WHERE `reserveringen.reservering_id` = 68
AND (`reserveringen.gereserv' at line 1

[ Voor 57% gewijzigd door CH4OS op 01-06-2005 13:56 ]

T-MOB schreef op woensdag 01 juni 2005 @ 13:56:
[...]


Een JOIN gebruiken wordt lastig aangezien er vziw geen link is tussen de tabel met admins en de tabel met reserveringen. De opmerking was meer in het algemeen bedoeld.

[...]

Die $isAdmin moet idd een integer zijn die 0 of 1 is. Hoe een standaard PHP boolean wordt omgezet in een query zou ik namelijk niet uit het hoofd durven zeggen. In code wordt dat dan zoiets:

PHP:

1 2	$query = mysql_query('SELECT naam FROM admin WHERE naam = "user_name" LIMIT 1', $db); $isAdmin = mysql_num_rows($query);

Die $isAdmin kun je dus gebruiken voor alle acties waarbij adminschap vereist is. Door PHP's loosetyping kun je hem gewoon als boolean aanspreken.

T-MOB schreef op woensdag 01 juni 2005 @ 13:56:
Let wel even op, dit is een SQL-injection waiting to happen

* CH4OS heeft weer een positief uitzicht op zijn stage

1
2
3
4
5

DELETE FROM `reserveringen`
WHERE reservering_id = x
AND (gereserveerd_door = 'een_user' 
OR gereserveerd_voor = 'een_user' 
OR $isAdmin = 1)

[ Voor 84% gewijzigd door CH4OS op 01-06-2005 14:31 ]