diff options
author | DukePyrolator <DukePyrolator@anope.org> | 2012-04-08 12:30:48 +0200 |
---|---|---|
committer | DukePyrolator <DukePyrolator@anope.org> | 2012-04-08 12:30:48 +0200 |
commit | 9e1fda2a44dee120e26acc6e51fbe779eea97712 (patch) | |
tree | ba47ff0354fe3b4302dffc4c7108e5bf26d3048c /modules | |
parent | 9d249ef96f7fa1e6460abb9263417652d7e49c3b (diff) |
Modified the SQL API to allow unescaped parameters (useful for passing row names and NULL values)
Diffstat (limited to 'modules')
-rw-r--r-- | modules/extra/m_mysql.cpp | 4 | ||||
-rw-r--r-- | modules/extra/sql.h | 14 |
2 files changed, 13 insertions, 5 deletions
diff --git a/modules/extra/m_mysql.cpp b/modules/extra/m_mysql.cpp index 138c726d8..e83e78c7c 100644 --- a/modules/extra/m_mysql.cpp +++ b/modules/extra/m_mysql.cpp @@ -423,8 +423,8 @@ Anope::string MySQLService::BuildQuery(const SQLQuery &q) { Anope::string real_query = q.query; - for (std::map<Anope::string, Anope::string>::const_iterator it = q.parameters.begin(), it_end = q.parameters.end(); it != it_end; ++it) - real_query = real_query.replace_all_cs("@" + it->first + "@", "'" + this->Escape(it->second) + "'"); + for (std::map<Anope::string, QueryData>::const_iterator it = q.parameters.begin(), it_end = q.parameters.end(); it != it_end; ++it) + real_query = real_query.replace_all_cs("@" + it->first + "@", (it->second.escape ? ("'" + this->Escape(it->second.data) + "'") : it->second.data)); return real_query; } diff --git a/modules/extra/sql.h b/modules/extra/sql.h index 0b27fe744..b23466dee 100644 --- a/modules/extra/sql.h +++ b/modules/extra/sql.h @@ -11,10 +11,17 @@ class SQLException : public ModuleException /** A SQL query */ + +struct QueryData +{ + Anope::string data; + bool escape; +}; + struct SQLQuery { Anope::string query; - std::map<Anope::string, Anope::string> parameters; + std::map<Anope::string, QueryData> parameters; SQLQuery() { } SQLQuery(const Anope::string &q) : query(q) { } @@ -36,12 +43,13 @@ struct SQLQuery return !(*this == other); } - template<typename T> void setValue(const Anope::string &key, const T& value) + template<typename T> void setValue(const Anope::string &key, const T& value, bool escape = true) { try { Anope::string string_value = stringify(value); - this->parameters[key] = string_value; + this->parameters[key].data = string_value; + this->parameters[key].escape = escape; } catch (const ConvertException &ex) { } } |