From 55711c3dcfe37b27b4c88b56f7f4604815d71dc5 Mon Sep 17 00:00:00 2001
From: Alex Robenko <arobenko@gmail.com>
Date: Mon, 9 Dec 2024 09:43:51 +1000
Subject: [PATCH] Fixing compilation with field marked having FixedValue.

---
 .../cc_tools_qt/details/ToolsIntFieldImpl.h   | 21 ++++++++++++++++++-
 .../details/ToolsUnsignedLongFieldImpl.h      | 20 +++++++++++++++++-
 2 files changed, 39 insertions(+), 2 deletions(-)

diff --git a/lib/include/cc_tools_qt/details/ToolsIntFieldImpl.h b/lib/include/cc_tools_qt/details/ToolsIntFieldImpl.h
index 050d944..4c11d99 100644
--- a/lib/include/cc_tools_qt/details/ToolsIntFieldImpl.h
+++ b/lib/include/cc_tools_qt/details/ToolsIntFieldImpl.h
@@ -144,7 +144,14 @@ class ToolsIntFieldImpl : public ToolsNumericFieldImpl<cc_tools_qt::field::Tools
 
     virtual void setDisplayValueImpl(UnderlyingType value) override
     {
-        Base::field().setDisplayValue(value);
+        using Tag = 
+            std::conditional_t<
+                Field::hasFixedValue(),
+                NoFeatureTag,
+                HasFeatureTag
+            >;
+
+        setDisplayValueInternal(value, Tag());
     }    
 
 private:
@@ -208,7 +215,19 @@ class ToolsIntFieldImpl : public ToolsNumericFieldImpl<cc_tools_qt::field::Tools
         [[maybe_unused]] static constexpr bool Must_not_be_called = false;
         assert(Must_not_be_called);
         return 0.0;
+    }    
+
+    void setDisplayValueInternal(UnderlyingType value, HasFeatureTag)
+    {
+        Base::field().setDisplayValue(value);
+    }   
+
+    void setDisplayValueInternal([[maybe_unused]] UnderlyingType value, NoFeatureTag)
+    {
+        [[maybe_unused]] static constexpr bool Must_not_be_called = false;
+        assert(Must_not_be_called);
     }      
+
 };
 
 template <typename TField>
diff --git a/lib/include/cc_tools_qt/details/ToolsUnsignedLongFieldImpl.h b/lib/include/cc_tools_qt/details/ToolsUnsignedLongFieldImpl.h
index 7a1653b..2d9cd91 100644
--- a/lib/include/cc_tools_qt/details/ToolsUnsignedLongFieldImpl.h
+++ b/lib/include/cc_tools_qt/details/ToolsUnsignedLongFieldImpl.h
@@ -132,7 +132,14 @@ class ToolsUnsignedLongFieldImpl : public ToolsNumericFieldImpl<cc_tools_qt::fie
 
     virtual void setDisplayValueImpl(UnderlyingType value) override
     {
-        Base::field().setDisplayValue(value);
+        using Tag = 
+            std::conditional_t<
+                Field::hasFixedValue(),
+                NoFeatureTag,
+                HasFeatureTag
+            >;
+
+        setDisplayValueInternal(value, Tag());        
     }       
 
 private:
@@ -172,6 +179,17 @@ class ToolsUnsignedLongFieldImpl : public ToolsNumericFieldImpl<cc_tools_qt::fie
 
         return result;
     }
+
+    void setDisplayValueInternal(UnderlyingType value, HasFeatureTag)
+    {
+        Base::field().setDisplayValue(value);
+    }   
+
+    void setDisplayValueInternal([[maybe_unused]] UnderlyingType value, NoFeatureTag)
+    {
+        [[maybe_unused]] static constexpr bool Must_not_be_called = false;
+        assert(Must_not_be_called);
+    }     
 };
 
 template <typename TField>