[Vtigercrm-commits] [vtiger-commits] r9888 - /vtigercrm/branches/5.0.3/modules/Reports/ReportRun.php

[vtigercrm-commits at vtiger.fosslabs.com]

Author: richie
Date: Tue Nov 21 23:59:34 2006
New Revision: 9888

Log:
Assigned_To field missing while viewing reports as non admin user- fixed. Fixes #2342 #2391

Modified:
    vtigercrm/branches/5.0.3/modules/Reports/ReportRun.php

Modified: vtigercrm/branches/5.0.3/modules/Reports/ReportRun.php
==============================================================================
--- vtigercrm/branches/5.0.3/modules/Reports/ReportRun.php (original)
+++ vtigercrm/branches/5.0.3/modules/Reports/ReportRun.php Tue Nov 21 23:59:34 2006
@@ -86,7 +86,7 @@
 
 			$querycolumns = $this->getEscapedColumns($selectedfields);
 					
-			if(sizeof($permitted_fields) != 0 && !in_array($fieldname,$permitted_fields))
+			if(sizeof($permitted_fields) != 0 && !in_array($colname,$permitted_fields))
 			{
 				continue;
 			}
@@ -124,6 +124,7 @@
 		while($collistrow = $adb->fetch_array($result))
 		{
 			$access_fields[] = $collistrow["fieldname"];
+
 		}
 		return $access_fields;
 	}
@@ -176,7 +177,7 @@
 		{
 			$sSQL .= $this->orderbylistsql.", ";	
 		}
-
+		
 		for($i=0; $i<$noofrows; $i++)
 		{
 			$fieldcolname = $adb->query_result($result,$i,"columnname");
@@ -206,7 +207,6 @@
 		$log->info("ReportRun :: Successfully returned getSelectedColumnsList".$reportid);
 		return $sSQL;
 	}
-
 	/** Function to get advanced comparator in query form for the given Comparator and value   
 	 *  @ param $comparator : Type String  
 	 *  @ param $value : Type String