From 2ec3cc9078d25f3fcb9dd8d5269422315e18d573 Mon Sep 17 00:00:00 2001
From: Gal Katz <galk@Fox.co.il>
Date: Sun, 11 Jan 2026 10:44:07 +0200
Subject: [PATCH] Fix: prevent measures from defaulting to Auto format when
 type is specified2

---
 GINQO_MasterItemManager.js | 60 +++++++++++++++++++-------------------
 1 file changed, 30 insertions(+), 30 deletions(-)

diff --git a/GINQO_MasterItemManager.js b/GINQO_MasterItemManager.js
index a662956..843205d 100644
--- a/GINQO_MasterItemManager.js
+++ b/GINQO_MasterItemManager.js
@@ -477,36 +477,36 @@ function ($, qlik, mainModalWindow, helpModalWindow, dimModalWindow, dimModalCon
 				const formatType = getValueByKey(cells, "%MI%MeasureFormatType", "qText");
 			
 				// Check if number format properties are provided
-				if (formatType.length > 0) {
-			
-					const formatFmt = getValueByKey(cells, "%MI%MeasureFormatFmt", "qText");
-					const formatDec = getValueByKey(cells, "%MI%MeasureFormatDec", "qText");
-					const formatThou = getValueByKey(cells, "%MI%MeasureFormatThou", "qText");
-
-					var formatNDec;
-					if (formatType == 'U') {
-						formatNDec = 10;
-					} else {
-						formatNDec = 2;
-					}
-					const formatUseThou = 0;  // always 0, parseInt(getValueByKey(cells, "%MI%MeasureFormatUseThou", "qText")); 
-
-					const numberFormat = {
-						qnDec: formatNDec,
-						qUseThou: formatUseThou,
-						qFmt: formatFmt,
-						qDec: formatDec,
-						qType: formatType,
-						qThou: formatThou
-					}
-			
-					// Assign the number format to the measure properties
-					properties.qMeasure.qNumFormat = numberFormat;
-			
-				} else {
-					// If the number format is not provided, use existing values
-					properties.qMeasure.qNumFormat = existingNumber;
-				}
+				if (formatType && formatType.length > 0) {
+        const formatFmt = getValueByKey(cells, "%MI%MeasureFormatFmt", "qText");
+        const formatDec = getValueByKey(cells, "%MI%MeasureFormatDec", "qText");
+        const formatThou = getValueByKey(cells, "%MI%MeasureFormatThou", "qText");
+
+    // Logic: 'U' (Unknown) gets 10 decimals, others (Fixed, Money, etc.) get 2
+    let formatNDec = (formatType === 'U') ? 10 : 2;
+
+    properties.qMeasure.qNumFormat = {
+        qType: formatType,
+        qnDec: formatNDec,
+        qUseThou: 0,
+        qFmt: formatFmt || '',
+        qDec: formatDec || '',
+        qThou: formatThou || ''
+    };
+
+    // FIX: Explicitly disable Auto Format so Qlik uses the object above
+    properties.qMeasure.qIsAutoFormat = false;
+
+} else {
+    // If Excel cell is blank, keep the existing Master Item format
+    if (existingNumber && Object.keys(existingNumber).length > 0) {
+        properties.qMeasure.qNumFormat = existingNumber;
+        properties.qMeasure.qIsAutoFormat = false;
+    } else {
+        // Fallback to Auto only if no format exists at all
+        properties.qMeasure.qIsAutoFormat = true;
+    }
+}
 			
 				// Determine the action (create or update) and perform the appropriate action
 				if (actionType === 'UPDATE') {