Skip to content

Commit 2f4e160

Browse files
JackTheSpadesrauchbfh-pixeljstedfastCopilot
authored
Make SqlCertificateDatabase a proper base class for other database providers. (#1218)
Allow SqlCertificateDatabase subclasses to override parameter name syntax. --------- Co-authored-by: Benjamin Rauch <rauch.bfh@gmail.com> Co-authored-by: Jeffrey Stedfast <jestedfa@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent 2ffe5c3 commit 2f4e160

File tree

5 files changed

+199
-102
lines changed

5 files changed

+199
-102
lines changed

MimeKit/Cryptography/NpgsqlCertificateDatabase.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ protected override IList<DataColumn> GetTableColumns (DbConnection connection, s
128128
{
129129
using (var command = CreateCommand ()) {
130130
command.CommandText = $"PRAGMA table_info({tableName})";
131-
using (var reader = command.ExecuteReader ()) {
131+
using (var reader = ExecuteReader (command)) {
132132
var columns = new List<DataColumn> ();
133133

134134
while (reader.Read ()) {
@@ -232,7 +232,7 @@ protected override void CreateTable (DbConnection connection, DataTable table)
232232
using (var command = CreateCommand ()) {
233233
command.CommandText = statement.ToString ();
234234
command.CommandType = CommandType.Text;
235-
command.ExecuteNonQuery ();
235+
ExecuteNonQuery (command);
236236
}
237237
}
238238

@@ -257,7 +257,7 @@ protected override void AddTableColumn (DbConnection connection, DataTable table
257257
using (var command = CreateCommand ()) {
258258
command.CommandText = statement.ToString ();
259259
command.CommandType = CommandType.Text;
260-
command.ExecuteNonQuery ();
260+
ExecuteNonQuery (command);
261261
}
262262
}
263263
}

MimeKit/Cryptography/SQLServerCertificateDatabase.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ protected override void AddTableColumn (DbConnection connection, DataTable table
105105
using (var command = CreateCommand ()) {
106106
command.CommandText = statement.ToString ();
107107
command.CommandType = CommandType.Text;
108-
command.ExecuteNonQuery ();
108+
ExecuteNonQuery (command);
109109
}
110110
}
111111

@@ -137,7 +137,7 @@ protected override void CreateTable (DbConnection connection, DataTable table)
137137
using (var command = CreateCommand ()) {
138138
command.CommandText = statement.ToString ();
139139
command.CommandType = CommandType.Text;
140-
command.ExecuteNonQuery ();
140+
ExecuteNonQuery (command);
141141
}
142142
}
143143

@@ -190,7 +190,7 @@ protected override IList<DataColumn> GetTableColumns (DbConnection connection, s
190190
{
191191
using (var command = CreateCommand ()) {
192192
command.CommandText = $"select top 1 * from {tableName}";
193-
using (var reader = command.ExecuteReader ()) {
193+
using (var reader = ExecuteReader (command)) {
194194
var columns = new List<DataColumn> ();
195195
var table = reader.GetSchemaTable ();
196196

@@ -218,7 +218,7 @@ protected override void CreateIndex (DbConnection connection, string tableName,
218218

219219
using (var command = CreateCommand ()) {
220220
command.CommandText = query;
221-
command.ExecuteNonQuery ();
221+
ExecuteNonQuery (command);
222222
}
223223
}
224224

@@ -238,7 +238,7 @@ protected override void RemoveIndex (DbConnection connection, string tableName,
238238

239239
using (var command = CreateCommand ()) {
240240
command.CommandText = query;
241-
command.ExecuteNonQuery ();
241+
ExecuteNonQuery (command);
242242
}
243243
}
244244

0 commit comments

Comments
 (0)