From bcf60737f91990c417f36300f385ccc1d3c2a8e1 Mon Sep 17 00:00:00 2001 From: HanzZ Date: Mon, 14 Nov 2011 16:14:30 +0100 Subject: [PATCH] Don't send empty units/values in stats payload --- .../Serializer/PayloadSerializers/StatsSerializer.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/include/Swiften/Serializer/PayloadSerializers/StatsSerializer.cpp b/include/Swiften/Serializer/PayloadSerializers/StatsSerializer.cpp index e670a6ee..3675550f 100644 --- a/include/Swiften/Serializer/PayloadSerializers/StatsSerializer.cpp +++ b/include/Swiften/Serializer/PayloadSerializers/StatsSerializer.cpp @@ -23,8 +23,12 @@ std::string StatsSerializer::serializePayload(boost::shared_ptr st foreach(const StatsPayload::Item& item, stats->getItems()) { boost::shared_ptr statElement(new XMLElement("stat")); statElement->setAttribute("name", item.getName()); - statElement->setAttribute("units", item.getUnits()); - statElement->setAttribute("value", item.getValue()); + if (!item.getUnits().empty()) { + statElement->setAttribute("units", item.getUnits()); + } + if (!item.getUnits().empty()) { + statElement->setAttribute("value", item.getUnits()); + } queryElement.addNode(statElement); }