From 82bb1af97a73931d7eb0f6ef520aacc7ebaa4c46 Mon Sep 17 00:00:00 2001
From: Magne Sjaastad <magne.sjaastad@ceetronsolutions.com>
Date: Thu, 20 Oct 2016 11:25:22 +0200
Subject: [PATCH] #922 Replace use of setValueFromUi

---
 .../ProjectDataModel/RimCommandObject.cpp            | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/ApplicationCode/ProjectDataModel/RimCommandObject.cpp b/ApplicationCode/ProjectDataModel/RimCommandObject.cpp
index 553c79d4ea..81276fd83e 100644
--- a/ApplicationCode/ProjectDataModel/RimCommandObject.cpp
+++ b/ApplicationCode/ProjectDataModel/RimCommandObject.cpp
@@ -28,6 +28,7 @@
 #include "cafPdmObjectGroup.h"
 #include "cafPdmUiPushButtonEditor.h"
 #include "cafPdmUiTextEditor.h"
+#include "cafPdmValueField.h"
 
 #include <QFile>
 
@@ -227,9 +228,16 @@ void RimCommandIssueFieldChanged::redo()
 
         if (fieldHandle && fieldHandle->uiCapability())
         {
+            caf::PdmValueField* valueField = dynamic_cast<caf::PdmValueField*>(fieldHandle);
+            CVF_ASSERT(valueField);
+
+            QVariant oldValue = valueField->toQVariant();
+            QVariant newValue(this->fieldValueToApply);
+
+            valueField->setFromQVariant(newValue);
+
             caf::PdmUiFieldHandle* uiFieldHandle = fieldHandle->uiCapability();
-            QVariant variantValue(this->fieldValueToApply);
-            uiFieldHandle->setValueFromUi(variantValue);
+            uiFieldHandle->notifyFieldChanged(oldValue, newValue);
         }
     }
 }