From 7592d966e684410f603942e34413375c8d98ac9e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?louiz=E2=80=99?= <louiz@louiz.org>
Date: Sun, 29 Apr 2018 01:40:46 +0200
Subject: Missing fields in a data-form response are now interpreted as an
 empty value

---
 src/database/column.hpp             | 16 ++++++++++++++--
 src/database/database.hpp           | 10 +++++-----
 src/database/row.hpp                | 21 ++++++++++++++++++---
 src/xmpp/biboumi_adhoc_commands.cpp |  5 ++++-
 4 files changed, 41 insertions(+), 11 deletions(-)

(limited to 'src')

diff --git a/src/database/column.hpp b/src/database/column.hpp
index 50c9c14..837aa3f 100644
--- a/src/database/column.hpp
+++ b/src/database/column.hpp
@@ -9,18 +9,30 @@ struct Column
         value{default_value} {}
     Column():
         value{} {}
+    void clear()
+    {
+      this->value = {};
+    }
     using real_type = T;
     T value{};
 };
 
+template <typename T>
+struct UnclearableColumn: public Column<T>
+{
+    using Column<T>::Column;
+    void clear()
+    { }
+};
+
 struct ForeignKey: Column<std::size_t> {
     static constexpr auto name = "fk_";
 };
 
-struct Id: Column<std::size_t> {
+struct Id: UnclearableColumn<std::size_t> {
     static constexpr std::size_t unset_value = static_cast<std::size_t>(-1);
     static constexpr auto name = "id_";
     static constexpr auto options = "PRIMARY KEY";
 
-    Id(): Column<std::size_t>(unset_value) {}
+    Id(): UnclearableColumn<std::size_t>(unset_value) {}
 };
diff --git a/src/database/database.hpp b/src/database/database.hpp
index d986ecc..30ffcb4 100644
--- a/src/database/database.hpp
+++ b/src/database/database.hpp
@@ -27,15 +27,15 @@ class Database
 
   struct Uuid: Column<std::string> { static constexpr auto name = "uuid_"; };
 
-  struct Owner: Column<std::string> { static constexpr auto name = "owner_"; };
+  struct Owner: UnclearableColumn<std::string> { static constexpr auto name = "owner_"; };
 
-  struct IrcChanName: Column<std::string> { static constexpr auto name = "ircchanname_"; };
+  struct IrcChanName: UnclearableColumn<std::string> { static constexpr auto name = "ircchanname_"; };
 
-  struct Channel: Column<std::string> { static constexpr auto name = "channel_"; };
+  struct Channel: UnclearableColumn<std::string> { static constexpr auto name = "channel_"; };
 
-  struct IrcServerName: Column<std::string> { static constexpr auto name = "ircservername_"; };
+  struct IrcServerName: UnclearableColumn<std::string> { static constexpr auto name = "ircservername_"; };
 
-  struct Server: Column<std::string> { static constexpr auto name = "server_"; };
+  struct Server: UnclearableColumn<std::string> { static constexpr auto name = "server_"; };
 
   struct Date: Column<time_point::rep> { static constexpr auto name = "date_"; };
 
diff --git a/src/database/row.hpp b/src/database/row.hpp
index 27caf43..1253f93 100644
--- a/src/database/row.hpp
+++ b/src/database/row.hpp
@@ -1,7 +1,5 @@
 #pragma once
 
-#include <utils/is_one_of.hpp>
-
 #include <type_traits>
 
 template <typename... T>
@@ -25,7 +23,24 @@ struct Row
     return col.value;
   }
 
-public:
+  void clear()
+  {
+    this->clear_col<0>();
+  }
+
   std::tuple<T...> columns;
   std::string table_name;
+
+private:
+  template <std::size_t N>
+  typename std::enable_if<N < sizeof...(T), void>::type
+  clear_col()
+  {
+    std::get<N>(this->columns).clear();
+    this->clear_col<N+1>();
+  }
+  template <std::size_t N>
+  typename std::enable_if<N == sizeof...(T), void>::type
+  clear_col()
+  { }
 };
diff --git a/src/xmpp/biboumi_adhoc_commands.cpp b/src/xmpp/biboumi_adhoc_commands.cpp
index 53806d6..b62dcfc 100644
--- a/src/xmpp/biboumi_adhoc_commands.cpp
+++ b/src/xmpp/biboumi_adhoc_commands.cpp
@@ -177,6 +177,7 @@ void ConfigureGlobalStep2(XmppComponent& xmpp_component, AdhocSession& session,
     {
       const Jid owner(session.get_owner_jid());
       auto options = Database::get_global_options(owner.bare());
+      options.clear();
       for (const XmlNode* field: x->get_children("field", "jabber:x:data"))
         {
           const XmlNode* value = field->get_child("value", "jabber:x:data");
@@ -400,7 +401,8 @@ void ConfigureIrcServerStep2(XmppComponent&, AdhocSession& session, XmlNode& com
         server_domain = target.local;
       auto options = Database::get_irc_server_options(owner.local + "@" + owner.domain,
                                                       server_domain);
-      auto commands = Database::get_after_connection_commands(options);
+      options.clear();
+      Database::AfterConnectionCommands commands{};
 
       for (const XmlNode* field: x->get_children("field", "jabber:x:data"))
         {
@@ -608,6 +610,7 @@ bool handle_irc_channel_configuration_form(XmppComponent& xmpp_component, const
           const Iid iid(target.local, {});
           auto options = Database::get_irc_channel_options(requester.bare(),
                                                            iid.get_server(), iid.get_local());
+          options.clear();
           for (const XmlNode *field: x->get_children("field", "jabber:x:data"))
             {
               const XmlNode *value = field->get_child("value", "jabber:x:data");
-- 
cgit v1.2.3