@mikro-orm/core 7.0.0-dev.139 → 7.0.0-dev.140

package/drivers/DatabaseDriver.js

@@ -168,22 +168,51 @@ export class DatabaseDriver {
         };
     }
     createCursorCondition(definition, offsets, inverse, meta) {
-        const createCondition = (prop, direction, offset, eq = false) => {
-            if (offset === null) {
-                throw CursorError.missingValue(meta.className, prop);
-            }
+        const createCondition = (prop, direction, offset, eq = false, path = prop) => {
             if (Utils.isPlainObject(direction)) {
+                if (offset === undefined) {
+                    throw CursorError.missingValue(meta.className, path);
+                }
                 const value = Utils.keys(direction).reduce((o, key) => {
-                    if (Utils.isEmpty(offset[key])) {
-                        throw CursorError.missingValue(meta.className, `${prop}.${key}`);
-                    }
-                    Object.assign(o, createCondition(key, direction[key], offset[key], eq));
+                    Object.assign(o, createCondition(key, direction[key], offset?.[key], eq, `${path}.${key}`));
                     return o;
                 }, {});
-                return ({ [prop]: value });
+                return { [prop]: value };
             }
-            const desc = direction === QueryOrderNumeric.DESC || direction.toString().toLowerCase() === 'desc';
-            const operator = Utils.xor(desc, inverse) ? '$lt' : '$gt';
+            const isDesc = direction === QueryOrderNumeric.DESC || direction.toString().toLowerCase() === 'desc';
+            const dirStr = direction.toString().toLowerCase();
+            let nullsFirst;
+            if (dirStr.includes('nulls first')) {
+                nullsFirst = true;
+            }
+            else if (dirStr.includes('nulls last')) {
+                nullsFirst = false;
+            }
+            else {
+                // Default: NULLS LAST for ASC, NULLS FIRST for DESC (matches most databases)
+                nullsFirst = isDesc;
+            }
+            const operator = Utils.xor(isDesc, inverse) ? '$lt' : '$gt';
+            // For leaf-level properties, undefined means missing value
+            if (offset === undefined) {
+                throw CursorError.missingValue(meta.className, path);
+            }
+            // Handle null offset (intentional null cursor value)
+            if (offset === null) {
+                if (eq) {
+                    // Equal to null
+                    return { [prop]: null };
+                }
+                // Strict comparison with null cursor value
+                // hasItemsAfterNull: forward + nullsFirst, or backward + nullsLast
+                const hasItemsAfterNull = Utils.xor(nullsFirst, inverse);
+                if (hasItemsAfterNull) {
+                    return { [prop]: { $ne: null } };
+                }
+                // No items after null in this direction, return impossible condition
+                return { [prop]: [] };
+            }
+            // Non-null offset
             return { [prop]: { [operator + (eq ? 'e' : '')]: offset } };
         };
         const [order, ...otherOrders] = definition;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@mikro-orm/core",
   "type": "module",
-  "version": "7.0.0-dev.139",
+  "version": "7.0.0-dev.140",
   "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.",
   "exports": {
     "./package.json": "./package.json",

package/utils/Cursor.js

@@ -95,15 +95,22 @@ export class Cursor {
     from(entity) {
         const processEntity = (entity, prop, direction, object = false) => {
             if (Utils.isPlainObject(direction)) {
+                const unwrapped = Reference.unwrapReference(entity[prop]);
+                // Check if the relation is loaded - for nested properties, undefined means not populated
+                if (Utils.isEntity(unwrapped) && !helper(unwrapped).isInitialized()) {
+                    throw CursorError.entityNotPopulated(entity, prop);
+                }
                 return Utils.keys(direction).reduce((o, key) => {
-                    Object.assign(o, processEntity(Reference.unwrapReference(entity[prop]), key, direction[key], true));
+                    Object.assign(o, processEntity(unwrapped, key, direction[key], true));
                     return o;
                 }, {});
             }
-            if (entity[prop] == null) {
-                throw CursorError.entityNotPopulated(entity, prop);
-            }
             let value = entity[prop];
+            // Allow null/undefined values in cursor - they will be handled in createCursorCondition
+            // undefined can occur with forceUndefined config option which converts null to undefined
+            if (value == null) {
+                return object ? { [prop]: null } : null;
+            }
             if (Utils.isEntity(value, true)) {
                 value = helper(value).getPrimaryKey();
             }
@@ -131,7 +138,13 @@ export class Cursor {
      */
     static for(meta, entity, orderBy) {
         const definition = this.getDefinition(meta, orderBy);
-        return Cursor.encode(definition.map(([key]) => entity[key]));
+        return Cursor.encode(definition.map(([key]) => {
+            const value = entity[key];
+            if (value === undefined) {
+                throw CursorError.missingValue(meta.className, key);
+            }
+            return value;
+        }));
     }
     static encode(value) {
         return Buffer.from(JSON.stringify(value)).toString('base64url');

package/utils/Utils.js

@@ -123,7 +123,7 @@ export function parseJsonSafe(value) {
 }
 export class Utils {
     static PK_SEPARATOR = '~~~';
-    static #ORM_VERSION = '7.0.0-dev.139';
+    static #ORM_VERSION = '7.0.0-dev.140';
     /**
      * Checks if the argument is instance of `Object`. Returns false for arrays.
      */