svn commit: r248 - trunk/mapbender/http/php/mod_gui_filteredUser.php

uli at osgeo.org uli at osgeo.org
Thu May 11 10:47:33 EDT 2006


Author: uli
Date: 2006-05-11 14:47:33+0000
New Revision: 248

Modified:
   trunk/mapbender/http/php/mod_gui_filteredUser.php

Log:
db_prep_query included
verification of user permissions

Modified: trunk/mapbender/http/php/mod_gui_filteredUser.php
Url: https://mapbender.osgeo.org/source/browse/mapbender/trunk/mapbender/http/php/mod_gui_filteredUser.php?view=diff&rev=248&p1=trunk/mapbender/http/php/mod_gui_filteredUser.php&p2=trunk/mapbender/http/php/mod_gui_filteredUser.php&r1=247&r2=248
==============================================================================
--- trunk/mapbender/http/php/mod_gui_filteredUser.php	(original)
+++ trunk/mapbender/http/php/mod_gui_filteredUser.php	2006-05-11 14:47:33+0000
@@ -18,13 +18,11 @@
 # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
 
 import_request_variables("PG");
-session_start();
-
-require_once("../php/mb_validateSession.php");
 require_once("../../conf/mapbender.conf");
-$con = db_connect($DBSERVER,$OWNER,$PW);
+$con = db_connect(DBSERVER,OWNER,PW);
 db_select_db(DB,$con);
-$gui_id = $_SESSION["mb_user_gui"];
+require_once("../php/mb_validatePermission.php");
+$self = $PHP_SELF . "?".SID."&guiID=".$_REQUEST["guiID"]."&elementID=".$_REQUEST["elementID"];
 ?>
 <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
 <html>
@@ -63,10 +61,6 @@
 </head>
 <body>
 <?php
-require_once("../../conf/mapbender.conf");
-$con = db_connect($DBSERVER,$OWNER,$PW);
-db_select_db(DB,$con);
-
 require_once("../php/mb_getGUIs.php");
 
 $fieldHeight = 20;
@@ -81,19 +75,22 @@
 
 $logged_user_name=$_SESSION["mb_user_name"];
 $logged_user_id=$_SESSION["mb_user_id"];
-#echo $_SESSION["mb_user_name"];
 
-/*handle remove, update and insert**************************************************************************************/
+/*handle remove, update and insert*****************************************************************/
 if($insert){
 	if(count($selected_user)>0){
 		for($i=0; $i<count($selected_user); $i++){
             $exists = false;
-            $sql_insert = "SELECT * from gui_mb_user where fkey_gui_id = '".$selected_gui."' and fkey_mb_user_id = '".$selected_user[$i]."'";
-            $res_insert = db_query($sql_insert);
+            $sql_insert = "SELECT * from gui_mb_user where fkey_gui_id = $1 and fkey_mb_user_id = $2";
+            $v = array($selected_gui,$selected_user[$i]);
+            $t = array('s','i');
+            $res_insert = db_prep_query($sql_insert,$v,$t);
             while(db_fetch_row($res_insert)){$exists = true;}
             if($exists == false){
-                $sql_insert = "INSERT INTO gui_mb_user(fkey_gui_id, fkey_mb_user_id) VALUES('$selected_gui', '".$selected_user[$i]."');";
-                $res_insert = db_query($sql_insert);
+                $sql_insert = "INSERT INTO gui_mb_user(fkey_gui_id, fkey_mb_user_id) VALUES($1, $2);";
+                $v = array($selected_gui,$selected_user[$i]);
+                $t = array('s','i');
+                $res_insert = db_prep_query($sql_insert,$v,$t);
             }
 		}
 	}
@@ -101,17 +98,18 @@
 if($remove){
     if(count($remove_user)>0){
         for($i=0; $i<count($remove_user); $i++){
-            $sql_remove = "DELETE FROM gui_mb_user WHERE fkey_mb_user_id = '".$remove_user[$i]."' and fkey_gui_id = '$selected_gui'";
-            db_query($sql_remove);
+            $sql_remove = "DELETE FROM gui_mb_user WHERE fkey_mb_user_id = $1 and fkey_gui_id = $2";
+            $v = array($remove_user[$i],$selected_gui);
+            $t = array('i','s');
+            db_prep_query($sql_remove,$v,$t);
         }
     }
 }
 
 
-/*get all gui  ********************************************************************************************/
+/*get all gui  ************************************************************************************/
 $sql_gui = "SELECT * FROM gui ORDER BY gui_name";
 
-
 $res_gui = db_query($sql_gui);
 while($row = db_fetch_array($res_gui)){
     $gui_id_array[$cnt_gui] = $row["gui_id"];
@@ -119,26 +117,29 @@
     $cnt_gui++;
 }
 
-/*get only owner user **********************************************************************************************/
-$sql_user = "SELECT * FROM mb_user WHERE mb_user_owner=".$logged_user_id." ORDER BY mb_user_name";
-$res_user = db_query($sql_user);
+/*get only owner user *****************************************************************************/
+$sql_user = "SELECT * FROM mb_user WHERE mb_user_owner = $1 ORDER BY mb_user_name";
+$v = array($logged_user_id);
+$t = array('i');
+$res_user = db_prep_query($sql_user,$v,$t);
 while($row = db_fetch_array($res_user)){
 	$user_id[$cnt_user] = $row["mb_user_id"];
 	$user_name[$cnt_user] =  $row["mb_user_name"];
 	$cnt_user++;
 }
 
-/*get only owner user from selected gui******************************************************************************/
+/*get only owner user from selected gui************************************************************/
 $sql_gui_mb_user = "SELECT mb_user.mb_user_id, mb_user.mb_user_name, gui_mb_user.fkey_gui_id FROM gui_mb_user ";
 $sql_gui_mb_user .= "INNER JOIN mb_user ON gui_mb_user.fkey_mb_user_id = mb_user.mb_user_id ";
-$sql_gui_mb_user .= "WHERE gui_mb_user.fkey_gui_id= '";
-
-if(!$selected_gui){$sql_gui_mb_user .= $gui_id_array[0];}
-if($selected_gui){$sql_gui_mb_user .= $selected_gui;}
-#$sql_gui_mb_user .= "' AND  gui_mb_user.fkey_mb_user_id <> '".$logged_user_id."'";
-$sql_gui_mb_user .= "' AND  mb_user.mb_user_owner = '".$logged_user_id."'";
+$sql_gui_mb_user .= "WHERE gui_mb_user.fkey_gui_id = $1";
+if(!$selected_gui){$v = array($gui_id_array[0]);}
+if($selected_gui){$v = array($selected_gui);}
+$t = array('s');
+$sql_gui_mb_user .= " AND  mb_user.mb_user_owner = $2";
+array_push($v,$logged_user_id);
+array_push($t,'i');
 $sql_gui_mb_user .= " ORDER BY mb_user.mb_user_name";
-$res_gui_mb_user = db_query($sql_gui_mb_user);
+$res_gui_mb_user = db_prep_query($sql_gui_mb_user,$v,$t);
 while($row = db_fetch_array($res_gui_mb_user)){
 	$user_id_gui[$cnt_gui_user] = $row["mb_user_id"];
 	$user_name_gui[$cnt_gui_user] =  $row["mb_user_name"];
@@ -147,9 +148,9 @@
 
 
 /*INSERT HTML*/
-echo "<form name='form1' action='" . $PHP_SELF . "?".SID."&e_id_css=".$_REQUEST["e_id_css"]."' method='post'>";
+echo "<form name='form1' action='" . $self ."' method='post'>";
 
-/*insert guis in selectbox*************************************************************************************/
+/*insert guis in selectbox*************************************************************************/
 echo "<div class='text1'>GUI: </div>";
 echo "<select style='background:#ffffff' class='select1' name='selected_gui' onChange='submit()' size='10'>";
 for($i=0; $i<$cnt_gui; $i++){
@@ -161,7 +162,7 @@
                  }
 echo "</select>";
 
-/*insert all user in selectbox**************************************************************************/
+/*insert all user in selectbox*********************************************************************/
 echo "<div class='text2'>USER:</div>";
 echo "<select style='background:#ffffff' class='select2' multiple='multiple' name='selected_user[]' size='$fieldHeight' >";
 for($i=0; $i<$cnt_user; $i++){
@@ -169,7 +170,7 @@
 }
 echo "</select>";
 
-/*Button****************************************************************************************************/
+/*Button******************************************************************************************/
 
 echo "<div class='button1'><input type='button'  value='==>' onClick='validate(\"insert\")'></div>";
 echo "<input type='hidden' name='insert'>";




More information about the Mapbender_commits mailing list