@sapui5/sap.gantt 1.108.9 → 1.108.10

package/src/sap/gantt/library.js

@@ -163,7 +163,7 @@ sap.ui.define([
 			}
 		},
 		noLibraryCSS: false,
-		version: "1.108.9"
+		version: "1.108.10"
 	});
 
 	this._oRb = sap.ui.getCore().getLibraryResourceBundle("sap.gantt");
@@ -291,6 +291,11 @@ sap.ui.define([
 	 */
 	oGanttLib.ValueSVGPaintServer = DataType.createType('sap.gantt.ValueSVGPaintServer', {
 		isValid : function(vValue) {
+			var sNewValue = vValue.substr(0, 1) === "@" ? vValue.substring(1) : vValue;
+			if (oGanttLib._colorCache[sNewValue]) {
+				// Check whether value is already retrieved and in the cache then simply return valid
+				return true;
+			}
 			var bValid = sap.m.ValueCSSColor.isValid(vValue);
 			if (!bValid){
 				bValid = /(?:url\(['|"]?)(.*?)(?:['|"]?\))|^[@|#](.*?)$|initial|transparent|none|inherit/.test(vValue);
@@ -298,6 +303,12 @@ sap.ui.define([
 			return bValid;
 		}
 	}, DataType.getType('string'));
+
+	/**
+	 * Stores color to access faster then fetching from theme parameters.
+	 */
+	oGanttLib._colorCache = {};
+
 	oGanttLib.ValueSVGPaintServer.setNormalizer(
 		/**
 		 * Check if the input value is an valid value for property fill and stroke. It allows hex/name color, URL pattern,
@@ -307,29 +318,39 @@ sap.ui.define([
 		 * @return {string} valid paint value
 		 */
 		function (vValue) {
-			var oValueColor = sap.m.ValueColor;
 			if (!vValue) {
-				// Return if input is undefined
+				// Return in case the argument is not passed
 				return vValue;
 			}
 
 			if (vValue.substr(0, 1) === "@") {
+				// In case there are annotation relevant special characters remove them before processing
 				vValue = vValue.substring(1);
 			}
 
-			var sRetValue = Parameters.get(vValue) || vValue;
+			if (oGanttLib._colorCache[vValue]) {
+				// Check whether value is already retrieved and in the cache then simply return from cache
+				return oGanttLib._colorCache[vValue];
+			}
 
+			var oValueColor = sap.m.ValueColor;
+			var sRetValue = Parameters.get(vValue) || vValue;
 			switch (sRetValue) {
 				case oValueColor.Critical:
-					return Parameters.get("sapUiChartCritical");
+					oGanttLib._colorCache[vValue] = Parameters.get("sapUiChartCritical");
+					return oGanttLib._colorCache[vValue];
 				case oValueColor.Error:
-					return Parameters.get("sapUiChartBad");
+					oGanttLib._colorCache[vValue] = Parameters.get("sapUiChartBad");
+					return oGanttLib._colorCache[vValue];
 				case oValueColor.Good:
-					return Parameters.get("sapUiChartGood");
+					oGanttLib._colorCache[vValue] = Parameters.get("sapUiChartGood");
+					return oGanttLib._colorCache[vValue];
 				case oValueColor.Neutral:
-					return Parameters.get("sapUiChartNeutral");
+					oGanttLib._colorCache[vValue] = Parameters.get("sapUiChartNeutral");
+					return oGanttLib._colorCache[vValue];
 				default:
-					return sRetValue;
+					oGanttLib._colorCache[vValue] = sRetValue;
+					return oGanttLib._colorCache[vValue];
 			}
 		}
 	);
@@ -713,7 +734,7 @@ sap.ui.define([
 
 	/**
 	 * Define the type of zoom control in global tool bar
-	 * @version 1.108.9
+	 * @version 1.108.10
 	 * @enum {string}
 	 * @public
 	 */