@mikro-orm/mongodb 7.3.0-dev.2 → 7.3.0-dev.4

package/MongoDriver.d.ts

@@ -35,6 +35,8 @@ export declare class MongoDriver extends DatabaseDriver<MongoConnection> {
     private buildQueryOptions;
     /** @internal */
     renameFields<T extends object>(entityName: EntityName<T>, data: T, dotPaths?: boolean, object?: boolean, root?: boolean): T;
+    /** Relations are stored as references, so conditions can only compare them by PK, e.g. `{ author: { id } }` becomes `{ author: _id }`. */
+    private inlineRelationPrimaryKey;
     private convertObjectIds;
     private buildFilterById;
     protected buildFields<T extends object, P extends string = never>(entityName: EntityName<T>, populate: PopulateOptions<T>[], fields?: readonly EntityField<T, P>[], exclude?: string[]): string[] | undefined;

package/MongoDriver.js

@@ -1,5 +1,5 @@
 import { ObjectId } from 'mongodb';
-import { DatabaseDriver, EntityManagerType, PolymorphicRef, QueryHelper, ReferenceKind, Utils, } from '@mikro-orm/core';
+import { DatabaseDriver, EntityManagerType, GroupOperator, PolymorphicRef, QueryHelper, ReferenceKind, Utils, } from '@mikro-orm/core';
 import { MongoConnection } from './MongoConnection.js';
 import { MongoPlatform } from './MongoPlatform.js';
 import { MongoEntityManager } from './MongoEntityManager.js';
@@ -446,6 +446,9 @@ export class MongoDriver extends DatabaseDriver {
                     const meta2 = this.metadata.find(prop.targetMeta.class);
                     const pk = meta2.properties[meta2.primaryKeys[0]];
                     isObjectId = pk.type === 'ObjectId';
+                    if (dotPaths) {
+                        copiedData[k] = this.inlineRelationPrimaryKey(meta, prop, meta2, copiedData[k]);
+                    }
                 }
                 if (isObjectId) {
                     copiedData[k] = this.convertObjectIds(copiedData[k]);
@@ -460,6 +463,23 @@ export class MongoDriver extends DatabaseDriver {
         });
         return copiedData;
     }
+    /** Relations are stored as references, so conditions can only compare them by PK, e.g. `{ author: { id } }` becomes `{ author: _id }`. */
+    inlineRelationPrimaryKey(meta, prop, meta2, value) {
+        if (prop.kind === ReferenceKind.ONE_TO_MANY || (prop.kind === ReferenceKind.MANY_TO_MANY && !prop.owner)) {
+            throw new Error(`Unsupported condition on inverse side ${meta.className}.${prop.name}, query the owning side ${meta2.className}.${prop.mappedBy} instead.`);
+        }
+        if (!Utils.isPlainObject(value)) {
+            return value;
+        }
+        const keys = Object.keys(value);
+        if (keys.length === 1 && [meta2.primaryKeys[0], meta2.serializedPrimaryKey].includes(keys[0])) {
+            return value[keys[0]];
+        }
+        if (keys.every(k => Utils.isOperator(k) && !(k in GroupOperator) && k !== '$not')) {
+            return value;
+        }
+        throw new Error(`Unsupported condition on relation ${meta.className}.${prop.name}, MongoDB can query relations only by primary key or entity reference.`);
+    }
     convertObjectIds(data) {
         if (data instanceof ObjectId) {
             return data;
@@ -515,6 +535,12 @@ export class MongoDriver extends DatabaseDriver {
                 }
             }
             ret.unshift(...meta.primaryKeys.filter(pk => !fields.includes(pk)));
+            // `targetKey` values identify the entity like its PK, references resolve it by them
+            for (const key of meta.root.targetKeys ?? []) {
+                if (meta.properties[key] && !fields.includes(key)) {
+                    ret.push(meta.properties[key].fieldNames[0]);
+                }
+            }
         }
         else if (!Utils.isEmpty(exclude) || lazyProps.some(p => !p.formula)) {
             const props = meta.props.filter(prop => this.platform.shouldHaveColumn(prop, populate, exclude));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/mongodb",
-  "version": "7.3.0-dev.2",
+  "version": "7.3.0-dev.4",
   "description": "TypeScript ORM for Node.js based on Data Mapper, Unit of Work and Identity Map patterns. Supports MongoDB, MySQL, PostgreSQL and SQLite databases as well as usage with vanilla JavaScript.",
   "keywords": [
     "data-mapper",
@@ -53,7 +53,7 @@
     "@mikro-orm/core": "^7.2.1"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.3.0-dev.2"
+    "@mikro-orm/core": "7.3.0-dev.4"
   },
   "engines": {
     "node": ">= 22.17.0"