summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorlouiz’ <louiz@louiz.org>2018-04-29 01:40:46 +0200
committerlouiz’ <louiz@louiz.org>2018-04-29 01:40:46 +0200
commit7592d966e684410f603942e34413375c8d98ac9e (patch)
tree7b3af3008c6a6532edbf4f4fe3534778886e669e /src
parentefa63ee5cb4be3aebc1489dc9db936eea26660d4 (diff)
downloadbiboumi-7592d966e684410f603942e34413375c8d98ac9e.tar.gz
biboumi-7592d966e684410f603942e34413375c8d98ac9e.tar.bz2
biboumi-7592d966e684410f603942e34413375c8d98ac9e.tar.xz
biboumi-7592d966e684410f603942e34413375c8d98ac9e.zip
Missing fields in a data-form response are now interpreted as an empty value
Diffstat (limited to 'src')
-rw-r--r--src/database/column.hpp16
-rw-r--r--src/database/database.hpp10
-rw-r--r--src/database/row.hpp21
-rw-r--r--src/xmpp/biboumi_adhoc_commands.cpp5
4 files changed, 41 insertions, 11 deletions
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");