diff options
author | Jannis Voelker <jannis.voelker@basyskom.com> | 2018-02-23 09:36:56 +0100 |
---|---|---|
committer | Jannis Völker <jannis.voelker@basyskom.com> | 2018-02-28 07:19:34 +0000 |
commit | f6ecf08115f6615b058cc7800246d5dc6dd54704 (patch) | |
tree | 677b24bc302f5793ba616c45bbfae59559b561fb | |
parent | 4c0d82ca8086f149cb4d7c2c38ba48b81d8df735 (diff) |
Use status code names in open62541 warnings
Open62541 provides a function to get the status code name from an
UA_StatusCode. This patch makes use of this function to generate
human readable warning messages.
Change-Id: I788aa55465d9661c761478a7e0a4dede1e7f8bc2
Reviewed-by: Frank Meerkoetter <frank.meerkoetter@basyskom.com>
-rw-r--r-- | src/plugins/opcua/open62541/qopen62541backend.cpp | 2 | ||||
-rw-r--r-- | src/plugins/opcua/open62541/qopen62541subscription.cpp | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/plugins/opcua/open62541/qopen62541backend.cpp b/src/plugins/opcua/open62541/qopen62541backend.cpp index a29b847..341d5b4 100644 --- a/src/plugins/opcua/open62541/qopen62541backend.cpp +++ b/src/plugins/opcua/open62541/qopen62541backend.cpp @@ -318,7 +318,7 @@ void Open62541AsyncBackend::callMethod(uintptr_t handle, UA_NodeId objectId, UA_ UA_StatusCode res = UA_Client_call(m_uaclient, objectId, methodId, args.size(), inputArgs, &outputSize, &outputArguments); if (res != UA_STATUSCODE_GOOD) - qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not call method:" << res; + qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not call method:" << UA_StatusCode_name(res); QVariant result; diff --git a/src/plugins/opcua/open62541/qopen62541subscription.cpp b/src/plugins/opcua/open62541/qopen62541subscription.cpp index 7154684..cfe6d91 100644 --- a/src/plugins/opcua/open62541/qopen62541subscription.cpp +++ b/src/plugins/opcua/open62541/qopen62541subscription.cpp @@ -80,7 +80,7 @@ UA_UInt32 QOpen62541Subscription::createOnServer() UA_CreateSubscriptionResponse res = UA_Client_Subscriptions_create(m_backend->m_uaclient, req, this, NULL, NULL); if (res.responseHeader.serviceResult != UA_STATUSCODE_GOOD) { - qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not create subscription with interval" << m_interval << res.responseHeader.serviceResult; + qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not create subscription with interval" << m_interval << UA_StatusCode_name(res.responseHeader.serviceResult); return 0; } @@ -237,7 +237,7 @@ bool QOpen62541Subscription::addAttributeMonitoredItem(uintptr_t handle, QOpcUa: qFuzzyCompare(settings.samplingInterval(), 0.0) ? m_interval : settings.samplingInterval()); if (ret != UA_STATUSCODE_GOOD) { - qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not add monitored item to subscription" << m_subscriptionId << ":" << ret; + qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not add monitored item to subscription" << m_subscriptionId << ":" << UA_StatusCode_name(ret); QOpcUaMonitoringParameters s; s.setStatusCode(static_cast<QOpcUa::UaStatusCode>(ret)); emit m_backend->monitoringEnableDisable(handle, attr, true, s); @@ -273,7 +273,7 @@ bool QOpen62541Subscription::removeAttributeMonitoredItem(uintptr_t handle, QOpc UA_StatusCode res = UA_Client_MonitoredItems_deleteSingle(m_backend->m_uaclient, m_subscriptionId, item->monitoredItemId); if (res != UA_STATUSCODE_GOOD) - qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not remove monitored item" << item->monitoredItemId << "from subscription" << m_subscriptionId << ":" << res; + qCWarning(QT_OPCUA_PLUGINS_OPEN62541) << "Could not remove monitored item" << item->monitoredItemId << "from subscription" << m_subscriptionId << ":" << UA_StatusCode_name(res); m_itemIdToItemMapping.remove(item->monitoredItemId); auto it = m_handleToItemMapping.find(handle); |