[php] Probleem met functie om post te editten.

[Verwijderd]

Hallo medetweakers,

ik zit weer met een probleem en ik hoop dat jullie me kunnen helpen want ik ben hier nu al een uur naar de bug aant zoeken, nochtans vind ik hem niet!
Het probleem is simpel:

Stel dat ik een gewone user ben met rank 0, ik heet user a, user b heeft het gepost, maar ik wil zijn bericht editten! Dit mag niet!
Nu als ik zijn bericht wil editten en daarna op de submit button druk die vervolgens door de request method POST wordt verzonden, dan edit hij het toch.
User a is ook geen moderator van dat bepaalde forum..

ik heb al vanalles geprobeerd, aparte filetjes gemaakt en daar zei hij dat hij do_error(2) moest doen, wat perfect klopt!
Nochtans doet hij het niet in mijn posting.php

Wat kan er hier fout gaan?


De 2 functies die ik gebruik:

PHP:

<?php

//
// Mod Functions
//

function moderate_forum () 

{
 
global $rank;

 $sql = "SELECT rank, id FROM users WHERE username = '".$_COOKIE['username']."'";
 $query = mysql_query($sql);


 while($userdata = mysql_fetch_array($query)) 
 {
    $rank = $userdata['rank'];

   $sql = "SELECT mod_id, forum_id FROM mod WHERE forum_id = '".$_GET['forum_id']."'";
   $query = mysql_query($sql) or die(mysql_error());


  while($moddata = mysql_fetch_array($query)) 
  {

   if($userdata['id'] == $moddata['mod_id'] && $rank == 2) 
   {
    define('AUTHORISED', true);
   }
   else 
   {

    define('AUTHORISED', false);
    
   }
  
  }


 }


}

//
// Let's get the editpost id!
//

function get_editpost_username () {

 global $editpost_username;

 $sql = "SELECT username FROM reply WHERE id = '".$_GET['id']."' AND forum_id = '".$_GET['forum_id']."'";
 $query = mysql_query($sql);

 $fetch = mysql_fetch_array($query);

 $editpost_username = $fetch[0];

 return $editpost_username;

}

?>

En nu de PHP die kijkt of na het wijzigen de post mag geupdate worden (geedit)

PHP:

<?php

} elseif($_POST['mode'] == "editpost" && isset($_POST['forum_id'])) {

            moderate_forum();

            get_editpost_username();

             if ( AUTHORISED == true || $rank == 3 || $rank == 4 || get_editpost_username() == $_COOKIE['username'] )

             {

    
                        $text = addslashes(htmlspecialchars($_POST['text']));
        
                    update_edited_topic();


            
                    header("Location: viewforum.php");

             }
             else
             {

              do_error(2);

              exit;

             }

?>

Mvg groeten,

Kris.

(Ik zou het erg appreciëren moest er iemand mij kunnen helpen! )

[ Voor 8% gewijzigd door Verwijderd op 03-06-2003 20:58 ]

[thomaske]

Als je gaat debuggen (welke variable heeft welke waarde op welke moment) kan je toch precies zien wanneer welke conditie true danwel false is. Waardoor je dus precies kan zien waar het fout gaat..

[edit]
in de FAQ staat een mooi stukje over debuggen

[ Voor 26% gewijzigd door thomaske op 03-06-2003 21:38 ]

[DUX]

Kun je niet gewoon het icoontje verborgen houden bij posts die hij toch niet kan editen?

[thomaske]

DUX schreef op 03 June 2003 @ 21:41:
Kun je niet gewoon het icoontje verborgen houden bij posts die hij toch niet kan editen?

En stel, ik heb zin om stiekem toch jouw post te editten, en klik niet op het icoontje, maar vul gewoon jouw post-id in...

Lijkt me niet de veiligste oplossing, mjah zolang niemand het probeert/weet, kan er niets gebeuren

_{'Security by Obscurity'}

[ Voor 6% gewijzigd door thomaske op 03-06-2003 21:48 ]

[Dennis]

@DUX: Je maakt hier een enorme denkfout, ik hoop niet dat jij ook zo je programma's maakt. Clientside-validatie (daar is dit gedeeltelijk een vorm van) is enorm fout.

@bReEzAh: Schaf eerst die globals eens af en roep functies met relevante parameters aan:

PHP:

voorbeeld($rank);

PHP:

function voorbeeld($rank)
{
  //bla
}

Constanten gebruiken waar je eigenlijk variabelen moet gebruiken krijg ik ook nachtmerries van.

[DUX]

offtopic:
Good point.

En nee, meestal maak ik mijn creaties ook op alle fronten dicht, maar zoals mij op school is geleerd: "Programmeurs zijn lui"

[Erkens]

DUX schreef op 03 juni 2003 @ 21:53:

offtopic:
Good point.

En nee, meestal maak ik mijn creaties ook op alle fronten dicht, maar zoals mij op school is geleerd: "Programmeurs zijn lui"

offtopic:
een programmeur mag dan wel lui zijn, maar dit soort "foutjes" kan je je irl (dus buiten school) niet veroorloven

[Verwijderd]

code:

AUTHORISED == true || $rank == 3 || $rank == 4 || get_editpost_username() == $_COOKIE['username']

lekker is dat, hier staat dus:

code:

AUTHORISED == (true || $rank == (3 || $rank == (4 || get_editpost_username() == $_COOKIE['username'])))

vraag me af of dat je bedoeling was.

[ACM]

Sinds wanneer is de precedence van de || operator hoger dan die van de == ?

Zie ook: http://www.php.net/manual/en/language.operators.php
Daar staat dus gewoon:
(AUTHORISED == true) || ($rank == 3) || ($rank == 4) || (get_editpost_username() == $_COOKIE['username'])

[zeroxcool]

offtopic:
lama

[ Voor 90% gewijzigd door zeroxcool op 03-06-2003 23:23 ]

[Verwijderd]

oo - php eigenaardigheidje, bij delphi is dat dus duidelijk anders, excuse me.
desalniettetoch kan het imho nooit kwaad wat extra haakjes te gebruiken.

[ACM]

Verwijderd schreef op 03 juni 2003 @ 23:24:
oo - php eigenaardigheidje, bij delphi is dat dus duidelijk anders, excuse me.
desalniettetoch kan het imho nooit kwaad wat extra haakjes te gebruiken.

Dan is dat, imho, een eigenaardigheidje van delphi

Want in elke taal waar i kmee gewerkt hebt is de comparisonoperator hoger in precedence dan de or-operator (of lager, afhankelijk van hoe je de tabel wilt aflezen).

Misschien ben je in de war met de assignment? (= in php, := in delphi )

[ Voor 6% gewijzigd door ACM op 03-06-2003 23:25 ]

[Verwijderd]

uit de delphi manual:

Operators Precedence
@, not first (highest)
*, /, div, mod, and, shl, shr, as second
+, –, or, xor third
=, <>, <, >, <=, >=, in, is fourth (lowest)

dus 'or' of 'and' gaan boven '=' in delphi.
daarom, haakjes kunnen nooit kwaad....

[ Voor 25% gewijzigd door Verwijderd op 03-06-2003 23:30 ]

[ACM]

das dan toch net als met php?

en dus:
1 = 1 or 2 = 2
==
(1 = 1) or (2 = 2)

[Verwijderd]

nee,
1 = 1 or 2 = 2 wordt dus
( 1 = (1 or 2) ) = 2
in delphi...

verders heeft php nog wel eens last van 'bugjes' en andere inconsequenties
ik hou me er nooit zo mee bezig en forceer uit gewoonte altijd de volgorde.

[ Voor 73% gewijzigd door Verwijderd op 03-06-2003 23:45 ]

[Dennis]

In php is OR en || ook niet geheel gelijk qua volgorde geloof ik (weet niet zeker).

[stekkel]

echo is your friend :-)

[drm]

ddc:
In php is OR en || ook niet geheel gelijk qua volgorde geloof ik (weet niet zeker).

de or operator heeft inderdaad een lagere precedence dan een || operator.

Precedence van operators in PHP

[Olaf van der Spek]

Waarom is de ? : operator in PHP left associative? Die hoort toch right associative te zijn?

[Sn3akz]

Vinden jullie zelf ook niet dat we nu redelijk off-topic gaan??