diff --git a/modules/Calendar/views/TaskManagement.php b/modules/Calendar/views/TaskManagement.php
index d9c154120d9376d09a25a5bc626e2de5ca280bfb..8081ef89312cfaea53675ed1bef99dd7cb2ae142 100644
--- a/modules/Calendar/views/TaskManagement.php
+++ b/modules/Calendar/views/TaskManagement.php
@@ -98,8 +98,8 @@ class Calendar_TaskManagement_View extends Vtiger_Index_View {
 
 		if (!$request->get('colors')) {
 			$colors = array();
-			foreach ($priorities as $key => $value) {
-				$colors[$key] = $this->getColors($key,$field->getName());
+			foreach ($priorities as $priority => $value) {
+				$colors[$priority] = $this->getColors($priority, $field->getName());
 			}
 		} else {
 			$colors = $request->get('colors');
@@ -107,14 +107,14 @@ class Calendar_TaskManagement_View extends Vtiger_Index_View {
 		return $colors;
 	}
 
-	public function getColors($priorities,$fieldname){
+	public function getColors($priority, $fieldname){
 		$db=PearDatabase::getInstance();
-		if(isset($priorities)){
+		if(isset($priority)){
 			$tableName = "vtiger_$fieldname";
-			$result=$db->pquery("SELECT color FROM $tableName WHERE $fieldname=?",array($priorities));
+			$result=$db->pquery("SELECT color FROM $tableName WHERE $fieldname=?",array($priority));
 			$no_of_row=$db->num_rows($result);
 			for($i=0;$i<$no_of_row;$i++){
-				$color = $db->query_result($result,$i,'color');
+				$color = $db->query_result($result, $i, 'color');
 			}
 		}
 		if($color=='#ffffff' || empty($color)) {