Use proper $select for results

feathersjs-ecosystem · Nov 15, 2022 · ae974f2 · ae974f2
1 parent ec25764
commit ae974f2
Showing 1 changed file with 14 additions and 1 deletion.
diff --git a/src/adapter.ts b/src/adapter.ts
@@ -390,7 +390,20 @@ export class SequelizeAdapter<
   async $remove (id: NullableId, params: P = {} as P): Promise<T | T[]> {
     const Model = this.ModelWithScope(params);
 
-    const itemOrItems = await this._getOrFind(id, params);
+    const findParams = { ...params };
+    if (params.$returning === false) {
+      findParams.query = {
+        ...findParams.query,
+        $select: [this.id]
+      }
+    } else if (params.query?.$select) {
+      findParams.query = {
+        ...findParams.query,
+        $select: [...params.query.$select, this.id]
+      }
+    }
+
+    const itemOrItems = await this._getOrFind(id, findParams);
     const items = Array.isArray(itemOrItems) ? itemOrItems : [itemOrItems];
     const ids: Id[] = items.map(item => item[this.id]);
     const seqOptions = Object.assign(