Skip to content
Snippets Groups Projects
Commit 696abd18 authored by Stuart Mark James's avatar Stuart Mark James
Browse files

Minor fix to DbConnection::storeParameterEvent()

parent f1d131c1
Branches
Tags
No related merge requests found
...@@ -261,15 +261,6 @@ namespace pqxx_conn ...@@ -261,15 +261,6 @@ namespace pqxx_conn
const string &description) const string &description)
{ {
assert(!full_attr_name.empty()); assert(!full_attr_name.empty());
assert(!label.empty());
assert(!unit.empty());
assert(!standard_unit.empty());
assert(!display_unit.empty());
assert(!format.empty());
assert(!archive_rel_change.empty());
assert(!archive_abs_change.empty());
assert(!archive_period.empty());
assert(!description.empty());
assert(_conn != nullptr); assert(_conn != nullptr);
assert(_conf_id_cache != nullptr); assert(_conf_id_cache != nullptr);
assert(_error_desc_id_cache != nullptr); assert(_error_desc_id_cache != nullptr);
...@@ -277,6 +268,20 @@ namespace pqxx_conn ...@@ -277,6 +268,20 @@ namespace pqxx_conn
spdlog::trace("Storing parameter event for attribute {}", full_attr_name); spdlog::trace("Storing parameter event for attribute {}", full_attr_name);
auto check_parameter = [](auto &name, auto &value) {
if (value.empty())
spdlog::warn("Parameter {} is empty. Please set in the device server", name);
};
check_parameter("label", label);
check_parameter("unit", unit);
check_parameter("standard_unit", standard_unit);
check_parameter("display_unit", display_unit);
check_parameter("archive_rel_change", archive_rel_change);
check_parameter("archive_abs_change", archive_abs_change);
check_parameter("archive_period", archive_period);
check_parameter("description", description);
spdlog::trace("Parmater event data: event_time {}, label {}, unit {}, standard_unit {}, display_unit {}, " spdlog::trace("Parmater event data: event_time {}, label {}, unit {}, standard_unit {}, display_unit {}, "
"format {}, archive_rel_change {}, archive_abs_change {}, archive_period {}, description {}", "format {}, archive_rel_change {}, archive_abs_change {}, archive_period {}, description {}",
event_time, event_time,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment