From 5a488b3d77326a0d957c1233493061da1b6ec207 Mon Sep 17 00:00:00 2001
From: Andrew Chow <achow101-github@achow101.com>
Date: Tue, 26 May 2020 20:53:06 -0400
Subject: [PATCH] Constructors, destructors, and relevant private fields for
 SQLiteDatabase/Batch

---
 src/wallet/sqlite.cpp | 11 +++++++++++
 src/wallet/sqlite.h   |  5 +++++
 2 files changed, 16 insertions(+)

diff --git a/src/wallet/sqlite.cpp b/src/wallet/sqlite.cpp
index c406269cabc..9a9904e17d8 100644
--- a/src/wallet/sqlite.cpp
+++ b/src/wallet/sqlite.cpp
@@ -4,6 +4,7 @@
 
 #include <wallet/sqlite.h>
 
+#include <logging.h>
 #include <util/memory.h>
 #include <util/strencodings.h>
 #include <util/translation.h>
@@ -17,10 +18,15 @@ static const char* const DATABASE_FILENAME = "wallet.dat";
 SQLiteDatabase::SQLiteDatabase(const fs::path& dir_path, const fs::path& file_path, bool mock)
     : WalletDatabase(), m_mock(mock), m_dir_path(dir_path.string()), m_file_path(file_path.string())
 {
+    LogPrintf("Using SQLite Version %s\n", SQLiteDatabaseVersion());
+    LogPrintf("Using wallet %s\n", m_dir_path);
+
+    Open();
 }
 
 SQLiteDatabase::~SQLiteDatabase()
 {
+    Close();
 }
 
 void SQLiteDatabase::Open()
@@ -46,6 +52,11 @@ std::unique_ptr<DatabaseBatch> SQLiteDatabase::MakeBatch(bool flush_on_close)
     return nullptr;
 }
 
+SQLiteBatch::SQLiteBatch(SQLiteDatabase& database)
+    : m_database(database)
+{
+}
+
 void SQLiteBatch::Close()
 {
 }
diff --git a/src/wallet/sqlite.h b/src/wallet/sqlite.h
index b5293017713..e56533b7b6d 100644
--- a/src/wallet/sqlite.h
+++ b/src/wallet/sqlite.h
@@ -7,6 +7,8 @@
 
 #include <wallet/db.h>
 
+#include <sqlite3.h>
+
 struct bilingual_str;
 class SQLiteDatabase;
 
@@ -23,6 +25,7 @@ private:
 
 public:
     explicit SQLiteBatch(SQLiteDatabase& database);
+    ~SQLiteBatch() override { Close(); }
 
     /* No-op. See commeng on SQLiteDatabase::Flush */
     void Flush() override {}
@@ -91,6 +94,8 @@ public:
 
     /** Make a SQLiteBatch connected to this database */
     std::unique_ptr<DatabaseBatch> MakeBatch(bool flush_on_close = true) override;
+
+    sqlite3* m_db{nullptr};
 };
 
 bool ExistsSQLiteDatabase(const fs::path& path);