svn commit: r189 - trunk/mapbender/http/classes/class_administration.php

[uli at osgeo.org]

Author: uli
Date: 2006-05-05 09:13:04+0000
New Revision: 189

Modified:
   trunk/mapbender/http/classes/class_administration.php

Log:
bug fixed in function getGuisByPermission()


Modified: trunk/mapbender/http/classes/class_administration.php
Url: https://mapbender.osgeo.org/source/browse/mapbender/trunk/mapbender/http/classes/class_administration.php?view=diff&rev=189&p1=trunk/mapbender/http/classes/class_administration.php&p2=trunk/mapbender/http/classes/class_administration.php&r1=188&r2=189
==============================================================================
--- trunk/mapbender/http/classes/class_administration.php	(original)
+++ trunk/mapbender/http/classes/class_administration.php	2006-05-05 09:13:04+0000
@@ -493,7 +493,6 @@
 			$mb_user_groups[$cnt_groups] = $row["fkey_mb_group_id"];
 			$cnt_groups++;
 		}
-		$count_g = 0;
 		if($cnt_groups > 0){
 			$v = array();
 			$t = array();
@@ -508,24 +507,22 @@
 			$sql_g .= ") GROUP BY gui.gui_id";
 			$res_g = db_prep_query($sql_g,$v,$t);
 			while($row = db_fetch_array($res_g)){
-				$arrayGuis[$count_g] = $row["gui_id"];
-				$count_g++;
+				array_push($arrayGuis,$row["gui_id"]);
 			}				
-		}		
-
+		}
 		$sql_guis = "SELECT gui.gui_id FROM gui JOIN gui_mb_user ON gui.gui_id = gui_mb_user.fkey_gui_id";  
 		$sql_guis .= " WHERE (gui_mb_user.fkey_mb_user_id = $1) ";
 		$sql_guis .= " AND gui.gui_public = 1  GROUP BY gui.gui_id";  
 		$v = array($mb_user_id);
 		$t = array("i");
 		$res_guis = db_prep_query($sql_guis,$v,$t);
-		$count_guis = 0;
 		$guis = array();
 		while($row = db_fetch_array($res_guis)){
-			$guis[$count_guis] = $row['gui_id'];			
-			$count_guis++;
+			if(!in_array($row['gui_id'],$arrayGuis)){
+				array_push($arrayGuis,$row["gui_id"]);
+			}			
 		}
-		return $guis;
+		return $arrayGuis;
 	}
 		
 	function getWmsByOwnGuis($array_gui_ids){ 
@@ -596,7 +593,6 @@
  	}
    	
    	function checkModulePermission($arrayGuis, $modulePath, $column){
-   		// no prepared statements necessary - no client requests
    		$check = true;
    		if($check == true){
 	   		global $DBSERVER,$DB,$OWNER,$PW;