query-core 0.1.6 → 0.1.10

package/src/services.ts

@@ -2,13 +2,13 @@ import {attributes, buildToDelete, buildToInsert, buildToUpdate, exist, metadata
 import {Attribute, Attributes, Statement, StringMap} from './metadata';
 import {SearchResult} from './search';
 
-export interface SearchModel {
+export interface Filter {
   fields?: string[];
   sort?: string;
   q?: string;
 }
 export class SqlLoader<T, ID> {
-  keys: Attribute[];
+  primaryKeys: Attribute[];
   map?: StringMap;
   attributes: Attributes;
   bools?: Attribute[];
@@ -18,35 +18,42 @@ export class SqlLoader<T, ID> {
     protected param: (i: number) => string,
     protected fromDB?: (v: T) => T) {
     if (Array.isArray(attrs)) {
-      this.keys = attributes(attrs);
+      this.primaryKeys = attributes(attrs);
+      this.attributes = {} as any;
     } else {
       const m = metadata(attrs);
       this.attributes = attrs;
-      this.keys = m.keys;
+      this.primaryKeys = m.keys;
       this.map = m.map;
       this.bools = m.bools;
     }
-    this.metadata = this.metadata.bind(this);
+    if (this.metadata) {
+      this.metadata = this.metadata.bind(this);
+    }
     this.all = this.all.bind(this);
     this.load = this.load.bind(this);
     this.exist = this.exist.bind(this);
   }
-  metadata(): Attributes {
+  metadata?(): Attributes|undefined {
     return this.attributes;
   }
   all(): Promise<T[]> {
     const sql = `select * from ${this.table}`;
     return this.query(sql, [], this.map);
   }
-  load(id: ID, ctx?: any): Promise<T> {
-    const stmt = select<ID>(id, this.table, this.keys, this.param);
-    if (this.fromDB) {
+  load(id: ID, ctx?: any): Promise<T|null> {
+    const stmt = select<ID>(id, this.table, this.primaryKeys, this.param);
+    if (!stmt) {
+      throw new Error('cannot build query by id');
+    }
+    const fn = this.fromDB;
+    if (fn) {
       return this.query(stmt.query, stmt.params, this.map, ctx).then(res => {
         if (!res || res.length === 0) {
           return null;
         } else {
           const obj = res[0];
-          return this.fromDB(obj);
+          return fn(obj);
         }
       });
     } else {
@@ -54,12 +61,15 @@ export class SqlLoader<T, ID> {
     }
   }
   exist(id: ID, ctx?: any): Promise<boolean> {
-    const field = (this.keys[0].field ? this.keys[0].field : this.keys[0].name);
-    const stmt = exist<ID>(id, this.table, this.keys, this.param, field);
+    const field = (this.primaryKeys[0].field ? this.primaryKeys[0].field : this.primaryKeys[0].name);
+    const stmt = exist<ID>(id, this.table, this.primaryKeys, this.param, field);
+    if (!stmt) {
+      throw new Error('cannot build query by id');
+    }
     return this.query(stmt.query, stmt.params, this.map, undefined, ctx).then(res => (!res || res.length === 0) ? false : true);
   }
 }
-export class SqlSearchLoader<T, ID, S extends SearchModel> extends SqlLoader<T, ID> {
+export class SqlSearchLoader<T, ID, S extends Filter> extends SqlLoader<T, ID> {
   constructor(
       protected find: (s: S, limit?: number, offset?: number|string, fields?: string[]) => Promise<SearchResult<T>>,
       table: string,
@@ -68,6 +78,7 @@ export class SqlSearchLoader<T, ID, S extends SearchModel> extends SqlLoader<T,
       param: (i: number) => string,
       fromDB?: (v: T) => T) {
     super(table, query, attrs, param, fromDB);
+    this.search = this.search.bind(this);
   }
   search(s: S, limit?: number, offset?: number|string, fields?: string[]): Promise<SearchResult<T>> {
     return this.find(s, limit, offset, fields);
@@ -76,7 +87,7 @@ export class SqlSearchLoader<T, ID, S extends SearchModel> extends SqlLoader<T,
 export interface Manager {
   exec(sql: string, args?: any[], ctx?: any): Promise<number>;
   execBatch(statements: Statement[], firstSuccess?: boolean, ctx?: any): Promise<number>;
-  query<T>(sql: string, args?: any[], m?: StringMap, fields?: string[], ctx?: any): Promise<T[]>;
+  query<T>(sql: string, args?: any[], m?: StringMap, bools?: Attribute[], ctx?: any): Promise<T[]>;
 }
 export function createSqlWriter<T, ID>(table: string,
     manager: Manager,
@@ -84,7 +95,7 @@ export function createSqlWriter<T, ID>(table: string,
     buildParam: (i: number) => string,
     toDB?: (v: T) => T,
     fromDB?: (v: T) => T) {
-  const writer = new SqlWriter<T, ID>(table, manager.query, manager.exec, attrs, buildParam, toDB, fromDB, manager.execBatch);
+  const writer = new SqlWriter<T, ID>(table, manager.query, manager.exec, attrs, buildParam, toDB, fromDB);
   return writer;
 }
 
@@ -96,8 +107,7 @@ export class SqlWriter<T, ID> extends SqlLoader<T, ID> {
     attrs: Attributes,
     buildParam: (i: number) => string,
     protected toDB?: (v: T) => T,
-    fromDB?: (v: T) => T,
-    public execBatch?: (statements: Statement[], firstSuccess?: boolean, ctx?: any) => Promise<number>) {
+    fromDB?: (v: T) => T) {
     super(table, query, attrs, buildParam, fromDB);
     const x = version(attrs);
     if (x) {
@@ -142,7 +152,7 @@ export class SqlWriter<T, ID> extends SqlLoader<T, ID> {
     return this.update(obj, ctx);
   }
   delete(id: ID, ctx?: any): Promise<number> {
-    const stmt = buildToDelete<ID>(id, this.table, this.keys, this.param);
+    const stmt = buildToDelete<ID>(id, this.table, this.primaryKeys, this.param);
     if (stmt) {
       return this.exec(stmt.query, stmt.params, ctx);
     } else {
@@ -150,7 +160,7 @@ export class SqlWriter<T, ID> extends SqlLoader<T, ID> {
     }
   }
 }
-export class SqlSearchWriter<T, ID, S extends SearchModel> extends SqlWriter<T, ID> {
+export class SqlSearchWriter<T, ID, S extends Filter> extends SqlWriter<T, ID> {
   constructor(
       protected find: (s: S, limit?: number, offset?: number|string, fields?: string[]) => Promise<SearchResult<T>>,
       table: string,
@@ -159,15 +169,15 @@ export class SqlSearchWriter<T, ID, S extends SearchModel> extends SqlWriter<T,
       attrs: Attributes,
       buildParam: (i: number) => string,
       toDB?: (v: T) => T,
-      fromDB?: (v: T) => T,
-      execBatch?: (statements: Statement[], firstSuccess?: boolean, ctx?: any) => Promise<number>) {
-    super(table, query, exec, attrs, buildParam, toDB, fromDB, execBatch);
+      fromDB?: (v: T) => T) {
+    super(table, query, exec, attrs, buildParam, toDB, fromDB);
+    this.search = this.search.bind(this);
   }
   search(s: S, limit?: number, offset?: number|string, fields?: string[]): Promise<SearchResult<T>> {
     return this.find(s, limit, offset, fields);
   }
 }
-export function createSqlSearchWriter<T, ID, S extends SearchModel>(
+export function createSqlSearchWriter<T, ID, S extends Filter>(
   find: (s: S, limit?: number, offset?: number|string, fields?: string[]) => Promise<SearchResult<T>>,
   table: string,
   manager: Manager,
@@ -175,6 +185,6 @@ export function createSqlSearchWriter<T, ID, S extends SearchModel>(
   buildParam: (i: number) => string,
   toDB?: (v: T) => T,
   fromDB?: (v: T) => T) {
-const writer = new SqlSearchWriter<T, ID, S>(find, table, manager.query, manager.exec, attrs, buildParam, toDB, fromDB, manager.execBatch);
+const writer = new SqlSearchWriter<T, ID, S>(find, table, manager.query, manager.exec, attrs, buildParam, toDB, fromDB);
 return writer;
 }

package/tsconfig.json

@@ -6,6 +6,7 @@
     "outDir": "./lib",
     "module": "commonjs",
     "moduleResolution": "node",
+    "strict": true,
     "pretty": true,
     "sourceMap": false,
     "declaration": false,