[hygiene] Rename logSql to onSqlCommand

Author: jamesgpearce

src/persisters/persister-cr-sqlite-wasm.ts

@@ -9,7 +9,7 @@ export const createCrSqliteWasmPersister = ((
   store: Store,
   db: DB,
   configOrStoreTableName?: DatabasePersisterConfig | string,
-  logSql?: (sql: string, args?: any[]) => void,
+  onSqlCommand?: (sql: string, args?: any[]) => void,
   onIgnoredError?: (error: any) => void,
 ): Persister =>
   createSqlitePersister(
@@ -20,7 +20,7 @@ export const createCrSqliteWasmPersister = ((
     (listener: UpdateListener): (() => void) =>
       db.onUpdate((_, _2, tableName) => listener(tableName)),
     (removeListener: () => void): void => removeListener(),
-    logSql,
+    onSqlCommand,
     onIgnoredError,
     db,
   )) as typeof createCrSqliteWasmPersisterDecl;

src/persisters/persister-expo-sqlite.ts

@@ -11,7 +11,7 @@ export const createExpoSqlitePersister = ((
   store: Store,
   db: SQLiteDatabase,
   configOrStoreTableName?: DatabasePersisterConfig | string,
-  logSql?: (sql: string, args?: any[]) => void,
+  onSqlCommand?: (sql: string, args?: any[]) => void,
   onIgnoredError?: (error: any) => void,
 ): Persister =>
   createSqlitePersister(
@@ -22,7 +22,7 @@ export const createExpoSqlitePersister = ((
     (listener: UpdateListener): Subscription =>
       db.onDatabaseChange(({tableName}) => listener(tableName)),
     (subscription: Subscription) => subscription.remove(),
-    logSql,
+    onSqlCommand,
     onIgnoredError,
     db,
   )) as typeof createExpoSqlitePersisterDecl;

src/persisters/persister-sqlite-wasm.ts

@@ -9,7 +9,7 @@ export const createSqliteWasmPersister = ((
   sqlite3: any,
   db: any,
   configOrStoreTableName?: DatabasePersisterConfig | string,
-  logSql?: (sql: string, args?: any[]) => void,
+  onSqlCommand?: (sql: string, args?: any[]) => void,
   onIgnoredError?: (error: any) => void,
 ): Persister =>
   createSqlitePersister(
@@ -26,7 +26,7 @@ export const createSqliteWasmPersister = ((
         0,
       ),
     (): void => sqlite3.capi.sqlite3_update_hook(db, () => 0, 0),
-    logSql,
+    onSqlCommand,
     onIgnoredError,
     db,
   )) as typeof createSqliteWasmPersisterDecl;

src/persisters/persister-sqlite3.ts

@@ -14,7 +14,7 @@ export const createSqlite3Persister = ((
   store: Store,
   db: Database,
   configOrStoreTableName?: DatabasePersisterConfig | string,
-  logSql?: (sql: string, args?: any[]) => void,
+  onSqlCommand?: (sql: string, args?: any[]) => void,
   onIgnoredError?: (error: any) => void,
 ): Persister =>
   createSqlitePersister(
@@ -35,7 +35,7 @@ export const createSqlite3Persister = ((
       return observer;
     },
     (observer: Observer): any => db.off(CHANGE, observer),
-    logSql,
+    onSqlCommand,
     onIgnoredError,
     db,
   )) as typeof createSqlite3PersisterDecl;

src/persisters/sqlite/create.ts

@@ -24,7 +24,7 @@ export const createSqlitePersister = <UpdateListeningHandle>(
   cmd: Cmd,
   addUpdateListener: (listener: UpdateListener) => UpdateListeningHandle,
   delUpdateListener: (listeningHandle: UpdateListeningHandle) => void,
-  logSql: ((sql: string, args?: any[]) => void) | undefined,
+  onSqlCommand: ((sql: string, args?: any[]) => void) | undefined,
   onIgnoredError: ((error: any) => void) | undefined,
   scheduleId: any,
 ): Persister => {
@@ -75,9 +75,9 @@ export const createSqlitePersister = <UpdateListeningHandle>(
 
   return (isJson ? createJsonSqlitePersister : createTabularSqlitePersister)(
     store,
-    logSql
+    onSqlCommand
       ? async (sql, args) => {
-          logSql(sql, args);
+          onSqlCommand(sql, args);
           return await cmd(sql, args);
         }
       : cmd,

test/unit/persisters/sqlite.ts

@@ -17,7 +17,7 @@ type SqliteVariant<Database> = [
     store: Store,
     db: Database,
     storeTableOrConfig?: string | DatabasePersisterConfig,
-    logSql?: (sql: string, args?: any[]) => void,
+    onSqlCommand?: (sql: string, args?: any[]) => void,
     onIgnoredError?: (error: any) => void,
   ) => Persister,
   cmd: (
@@ -37,14 +37,14 @@ export const VARIANTS: {[name: string]: SqliteVariant<any>} = {
       store: Store,
       db: Database,
       storeTableOrConfig?: string | DatabasePersisterConfig,
-      logSql?: (sql: string, args?: any[]) => void,
+      onSqlCommand?: (sql: string, args?: any[]) => void,
       onIgnoredError?: (error: any) => void,
     ) =>
       (createSqlite3Persister as any)(
         store,
         db,
         storeTableOrConfig,
-        logSql,
+        onSqlCommand,
         onIgnoredError,
       ),
     (
@@ -72,15 +72,15 @@ export const VARIANTS: {[name: string]: SqliteVariant<any>} = {
       store: Store,
       [sqlite3, db]: SqliteWasmDb,
       storeTableOrConfig?: string | DatabasePersisterConfig,
-      logSql?: (sql: string, args?: any[]) => void,
+      onSqlCommand?: (sql: string, args?: any[]) => void,
       onIgnoredError?: (error: any) => void,
     ) =>
       (createSqliteWasmPersister as any)(
         store,
         sqlite3,
         db,
         storeTableOrConfig,
-        logSql,
+        onSqlCommand,
         onIgnoredError,
       ),
     async ([_, db]: SqliteWasmDb, sql: string, args: any[] = []) =>
@@ -96,14 +96,14 @@ export const VARIANTS: {[name: string]: SqliteVariant<any>} = {
       store: Store,
       db: DB,
       storeTableOrConfig?: string | DatabasePersisterConfig,
-      logSql?: (sql: string, args?: any[]) => void,
+      onSqlCommand?: (sql: string, args?: any[]) => void,
       onIgnoredError?: (error: any) => void,
     ) =>
       (createCrSqliteWasmPersister as any)(
         store,
         db,
         storeTableOrConfig,
-        logSql,
+        onSqlCommand,
         onIgnoredError,
       ),
     async (db: DB, sql: string, args: any[] = []) => await db.execO(sql, args),