@mikro-orm/core 7.0.0-dev.113 → 7.0.0-dev.115

package/EntityManager.d.ts

@@ -7,7 +7,7 @@ import { EntityLoader, type EntityLoaderOptions } from './entity/EntityLoader.js
 import { Reference } from './entity/Reference.js';
 import { UnitOfWork } from './unit-of-work/UnitOfWork.js';
 import type { CountOptions, DeleteOptions, FilterOptions, FindAllOptions, FindByCursorOptions, FindOneOptions, FindOneOrFailOptions, FindOptions, GetReferenceOptions, IDatabaseDriver, LockOptions, NativeInsertUpdateOptions, StreamOptions, UpdateOptions, UpsertManyOptions, UpsertOptions } from './drivers/IDatabaseDriver.js';
-import type { AnyString, ArrayElement, AutoPath, ConnectionType, Dictionary, EntityData, EntityDictionary, EntityDTO, EntityMetadata, EntityName, FilterDef, FilterQuery, FromEntityType, GetRepository, IHydrator, IsSubset, Loaded, MergeLoaded, MergeSelected, NoInfer, ObjectQuery, Primary, Ref, RequiredEntityData, UnboxArray } from './typings.js';
+import type { AnyString, ArrayElement, AutoPath, ConnectionType, Dictionary, EntityClass, EntityData, EntityDictionary, EntityDTO, EntityMetadata, EntityName, FilterDef, FilterQuery, FromEntityType, GetRepository, IHydrator, IsSubset, Loaded, MergeLoaded, MergeSelected, NoInfer, ObjectQuery, Primary, Ref, RequiredEntityData, UnboxArray } from './typings.js';
 import { FlushMode, LockMode, PopulatePath, type TransactionOptions } from './enums.js';
 import type { MetadataStorage } from './metadata/MetadataStorage.js';
 import type { Transaction } from './connections/Connection.js';
@@ -99,7 +99,7 @@ export declare class EntityManager<Driver extends IDatabaseDriver = IDatabaseDri
     /**
      * Registers global filter to this entity manager. Global filters are enabled by default (unless disabled via last parameter).
      */
-    addFilter<T extends EntityName<any> | readonly EntityName<any>[]>(options: FilterDef<T>): void;
+    addFilter<T extends EntityName | readonly EntityName[]>(options: FilterDef<T>): void;
     /**
      * Sets filter parameter values globally inside context defined by this entity manager.
      * If you want to set shared value for all contexts, be sure to use the root entity manager.
@@ -120,21 +120,21 @@ export declare class EntityManager<Driver extends IDatabaseDriver = IDatabaseDri
         disableContextResolution?: boolean;
     }): T;
     setFlushMode(flushMode?: FlushMode | `${FlushMode}`): void;
-    protected processWhere<Entity extends object, Hint extends string = never, Fields extends string = '*', Excludes extends string = never>(entityName: string, where: FilterQuery<Entity>, options: FindOptions<Entity, Hint, Fields, Excludes> | FindOneOptions<Entity, Hint, Fields, Excludes>, type: 'read' | 'update' | 'delete'): Promise<FilterQuery<Entity>>;
-    protected applyDiscriminatorCondition<Entity extends object>(entityName: string, where: FilterQuery<Entity>): FilterQuery<Entity>;
+    protected processWhere<Entity extends object, Hint extends string = never, Fields extends string = '*', Excludes extends string = never>(entityName: EntityName<Entity>, where: FilterQuery<Entity>, options: FindOptions<Entity, Hint, Fields, Excludes> | FindOneOptions<Entity, Hint, Fields, Excludes>, type: 'read' | 'update' | 'delete'): Promise<FilterQuery<Entity>>;
+    protected applyDiscriminatorCondition<Entity extends object>(entityName: EntityName<Entity>, where: FilterQuery<Entity>): FilterQuery<Entity>;
     protected createPopulateWhere<Entity extends object>(cond: ObjectQuery<Entity>, options: FindOptions<Entity, any, any, any> | FindOneOptions<Entity, any, any, any> | CountOptions<Entity, any>): ObjectQuery<Entity>;
     protected getJoinedFilters<Entity extends object>(meta: EntityMetadata<Entity>, options: FindOptions<Entity, any, any, any> | FindOneOptions<Entity, any, any, any>): Promise<ObjectQuery<Entity> | undefined>;
     /**
      * When filters are active on M:1 or 1:1 relations, we need to ref join them eagerly as they might affect the FK value.
      */
     protected autoJoinRefsForFilters<T extends object>(meta: EntityMetadata<T>, options: FindOptions<T, any, any, any> | FindOneOptions<T, any, any, any>, parent?: {
-        className: string;
+        class: EntityClass;
         propName: string;
     }): Promise<void>;
     /**
      * @internal
      */
-    applyFilters<Entity extends object>(entityName: string, where: FilterQuery<Entity> | undefined, options: FilterOptions | undefined, type: 'read' | 'update' | 'delete', findOptions?: FindOptions<any, any, any, any> | FindOneOptions<any, any, any, any>): Promise<FilterQuery<Entity> | undefined>;
+    applyFilters<Entity extends object>(entityName: EntityName<Entity>, where: FilterQuery<Entity> | undefined, options: FilterOptions | undefined, type: 'read' | 'update' | 'delete', findOptions?: FindOptions<any, any, any, any> | FindOneOptions<any, any, any, any>): Promise<FilterQuery<Entity> | undefined>;
     /**
      * Calls `em.find()` and `em.count()` with the same arguments (where applicable) and returns the results as tuple
      * where the first element is the array of entities, and the second is the count.
@@ -508,11 +508,11 @@ export declare class EntityManager<Driver extends IDatabaseDriver = IDatabaseDri
     /**
      * @internal
      */
-    cacheKey<T extends object>(entityName: string, options: FindOptions<T, any, any, any> | FindOneOptions<T, any, any, any> | CountOptions<T, any>, method: string, where: FilterQuery<T>): unknown[];
+    cacheKey<T extends object>(entityName: EntityName<T>, options: FindOptions<T, any, any, any> | FindOneOptions<T, any, any, any> | CountOptions<T, any>, method: string, where: FilterQuery<T>): unknown[];
     /**
      * @internal
      */
-    tryCache<T extends object, R>(entityName: string, config: boolean | number | [string, number] | undefined, key: unknown, refresh?: boolean, merge?: boolean): Promise<{
+    tryCache<T extends object, R>(entityName: EntityName<T>, config: boolean | number | [string, number] | undefined, key: unknown, refresh?: boolean, merge?: boolean): Promise<{
         data?: R | null;
         key: string;
     } | undefined>;

package/EntityManager.js

@@ -4,7 +4,7 @@ import { Cursor } from './utils/Cursor.js';
 import { DataloaderUtils } from './utils/DataloaderUtils.js';
 import { QueryHelper } from './utils/QueryHelper.js';
 import { TransactionContext } from './utils/TransactionContext.js';
-import { isRaw, RawQueryFragment } from './utils/RawQueryFragment.js';
+import { isRaw, Raw } from './utils/RawQueryFragment.js';
 import { EntityFactory } from './entity/EntityFactory.js';
 import { EntityAssigner } from './entity/EntityAssigner.js';
 import { validateEmptyWhere, validateParams, validatePrimaryKey, validateProperty } from './entity/validators.js';
@@ -35,7 +35,7 @@ export class EntityManager {
     global = false;
     name;
     loaders = {};
-    repositoryMap = {};
+    repositoryMap = new Map();
     entityLoader;
     comparator;
     entityFactory;
@@ -88,13 +88,12 @@ export class EntityManager {
      * Gets repository for given entity. You can pass either string name or entity class reference.
      */
     getRepository(entityName) {
-        entityName = Utils.className(entityName);
-        if (!this.repositoryMap[entityName]) {
-            const meta = this.metadata.get(entityName);
+        const meta = this.metadata.get(entityName);
+        if (!this.repositoryMap.has(meta)) {
             const RepositoryClass = this.config.getRepositoryClass(meta.repository);
-            this.repositoryMap[entityName] = new RepositoryClass(this, entityName);
+            this.repositoryMap.set(meta, new RepositoryClass(this, entityName));
         }
-        return this.repositoryMap[entityName];
+        return this.repositoryMap.get(meta);
     }
     /**
      * Shortcut for `em.getRepository()`.
@@ -116,7 +115,6 @@ export class EntityManager {
         const em = this.getContext();
         em.prepareOptions(options);
         await em.tryFlush(entityName, options);
-        entityName = Utils.className(entityName);
         where = await em.processWhere(entityName, where, options, 'read');
         validateParams(where);
         options.orderBy = options.orderBy || {};
@@ -192,7 +190,6 @@ export class EntityManager {
         em.prepareOptions(options);
         options.strategy = 'joined';
         await em.tryFlush(entityName, options);
-        entityName = Utils.className(entityName);
         const where = await em.processWhere(entityName, options.where ?? {}, options, 'read');
         validateParams(where);
         options.orderBy = options.orderBy || {};
@@ -303,15 +300,15 @@ export class EntityManager {
         if (!meta?.discriminatorValue) {
             return where;
         }
-        const types = Object.values(meta.root.discriminatorMap).map(cls => this.metadata.find(cls));
+        const types = Object.values(meta.root.discriminatorMap).map(cls => this.metadata.get(cls));
         const children = [];
         const lookUpChildren = (ret, type) => {
             const children = types.filter(meta2 => meta2.extends === type);
-            children.forEach(m => lookUpChildren(ret, m.className));
+            children.forEach(m => lookUpChildren(ret, m.class));
             ret.push(...children.filter(c => c.discriminatorValue));
             return children;
         };
-        lookUpChildren(children, meta.className);
+        lookUpChildren(children, meta.class);
         /* v8 ignore next */
         where[meta.root.discriminatorColumn] = children.length > 0 ? { $in: [meta.discriminatorValue, ...children.map(c => c.discriminatorValue)] } : meta.discriminatorValue;
         return where;
@@ -341,7 +338,7 @@ export class EntityManager {
                 continue;
             }
             const filters = QueryHelper.mergePropertyFilters(prop.filters, options.filters);
-            const where = await this.applyFilters(prop.type, {}, filters, 'read', {
+            const where = await this.applyFilters(prop.targetMeta.class, {}, filters, 'read', {
                 ...options,
                 populate: hint.children,
             });
@@ -377,11 +374,11 @@ export class EntityManager {
             if (prop.object
                 || ![ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind)
                 || !((options.fields?.length ?? 0) === 0 || options.fields?.some(f => prop.name === f || prop.name.startsWith(`${String(f)}.`)))
-                || (parent?.className === prop.targetMeta.root.className && parent.propName === prop.inversedBy)) {
+                || (parent?.class === prop.targetMeta.root.class && parent.propName === prop.inversedBy)) {
                 continue;
             }
             options = { ...options, filters: QueryHelper.mergePropertyFilters(prop.filters, options.filters) };
-            const cond = await this.applyFilters(prop.type, {}, options.filters, 'read', options);
+            const cond = await this.applyFilters(prop.targetMeta.class, {}, options.filters, 'read', options);
             if (!Utils.isEmpty(cond)) {
                 const populated = options.populate.filter(({ field }) => field.split(':')[0] === prop.name);
                 let found = false;
@@ -404,7 +401,7 @@ export class EntityManager {
             const prop = meta?.properties[field];
             if (prop && !ref) {
                 hint.children ??= [];
-                await this.autoJoinRefsForFilters(prop.targetMeta, { ...options, populate: hint.children }, { className: meta.root.className, propName: prop.name });
+                await this.autoJoinRefsForFilters(prop.targetMeta, { ...options, populate: hint.children }, { class: meta.root.class, propName: prop.name });
             }
         }
     }
@@ -412,16 +409,13 @@ export class EntityManager {
      * @internal
      */
     async applyFilters(entityName, where, options, type, findOptions) {
-        const meta = this.metadata.find(entityName);
+        const meta = this.metadata.get(entityName);
         const filters = [];
         const ret = [];
-        if (!meta) {
-            return where;
-        }
         const active = new Set();
         const push = (source) => {
             const activeFilters = QueryHelper
-                .getActiveFilters(entityName, options, source)
+                .getActiveFilters(meta, options, source)
                 .filter(f => !active.has(f.name));
             filters.push(...activeFilters);
             activeFilters.forEach(f => active.add(f.name));
@@ -440,7 +434,7 @@ export class EntityManager {
                 if (!args && filter.cond.length > 0 && filter.args !== false) {
                     throw new Error(`No arguments provided for filter '${filter.name}'`);
                 }
-                cond = await filter.cond(args, type, this, findOptions, entityName);
+                cond = await filter.cond(args, type, this, findOptions, Utils.className(entityName));
             }
             else {
                 cond = filter.cond;
@@ -457,7 +451,7 @@ export class EntityManager {
             }
             ret.push(cond);
         }
-        const conds = [...ret, where].filter(c => Utils.hasObjectKeys(c));
+        const conds = [...ret, where].filter(c => Utils.hasObjectKeys(c) || Raw.hasObjectFragments(c));
         return conds.length > 1 ? { $and: conds } : conds[0];
     }
     /**
@@ -468,12 +462,10 @@ export class EntityManager {
         const em = this.getContext(false);
         await em.tryFlush(entityName, options);
         options.flushMode = 'commit'; // do not try to auto flush again
-        return RawQueryFragment.run(async () => {
-            return Promise.all([
-                em.find(entityName, where, options),
-                em.count(entityName, where, options),
-            ]);
-        });
+        return Promise.all([
+            em.find(entityName, where, options),
+            em.count(entityName, where, options),
+        ]);
     }
     /**
      * Calls `em.find()` and `em.count()` with the same arguments (where applicable) and returns the results as {@apilink Cursor} object.
@@ -533,10 +525,9 @@ export class EntityManager {
      */
     async findByCursor(entityName, options) {
         const em = this.getContext(false);
-        entityName = Utils.className(entityName);
         options.overfetch ??= true;
         options.where ??= {};
-        if (Utils.isEmpty(options.orderBy)) {
+        if (Utils.isEmpty(options.orderBy) && !Raw.hasObjectFragments(options.orderBy)) {
             throw new Error('Explicit `orderBy` option required');
         }
         const [entities, count] = options.includeCount !== false
@@ -553,9 +544,9 @@ export class EntityManager {
         const ret = await this.refresh(entity, options);
         if (!ret) {
             options.failHandler ??= this.config.get('findOneOrFailHandler');
-            const entityName = entity.constructor.name;
-            const where = helper(entity).getPrimaryKey();
-            throw options.failHandler(entityName, where);
+            const wrapped = helper(entity);
+            const where = wrapped.getPrimaryKey();
+            throw options.failHandler(wrapped.__meta.className, where);
         }
         return ret;
     }
@@ -566,9 +557,8 @@ export class EntityManager {
      */
     async refresh(entity, options = {}) {
         const fork = this.fork({ keepTransactionContext: true });
-        const entityName = entity.constructor.name;
         const wrapped = helper(entity);
-        const reloaded = await fork.findOne(entityName, entity, {
+        const reloaded = await fork.findOne(wrapped.__meta.class, entity, {
             schema: wrapped.__schema,
             ...options,
             flushMode: FlushMode.COMMIT,
@@ -580,7 +570,7 @@ export class EntityManager {
         }
         let found = false;
         for (const e of fork.unitOfWork.getIdentityMap()) {
-            const ref = em.getReference(e.constructor.name, helper(e).getPrimaryKey());
+            const ref = em.getReference(e.constructor, helper(e).getPrimaryKey());
             const data = helper(e).serialize({ ignoreSerializers: true, includeHidden: true });
             em.config.getHydrator(this.metadata).hydrate(ref, helper(ref).__meta, data, em.entityFactory, 'full', false, true);
             Utils.merge(helper(ref).__originalEntityData, this.comparator.prepareEntity(e));
@@ -605,7 +595,6 @@ export class EntityManager {
             return ret;
         }
         const em = this.getContext();
-        entityName = Utils.className(entityName);
         em.prepareOptions(options);
         let entity = em.unitOfWork.tryGetById(entityName, where, options.schema);
         // query for a not managed entity which is already in the identity map as it
@@ -683,10 +672,10 @@ export class EntityManager {
         if (!entity || isStrictViolation) {
             const key = options.strict ? 'findExactlyOneOrFailHandler' : 'findOneOrFailHandler';
             options.failHandler ??= this.config.get(key);
-            entityName = Utils.className(entityName);
+            const name = Utils.className(entityName);
             /* v8 ignore next */
             where = Utils.isEntity(where) ? helper(where).getPrimaryKey() : where;
-            throw options.failHandler(entityName, where);
+            throw options.failHandler(name, where);
         }
         return entity;
     }
@@ -726,11 +715,11 @@ export class EntityManager {
         let where;
         let entity = null;
         if (data === undefined) {
-            entityName = entityNameOrEntity.constructor.name;
+            entityName = entityNameOrEntity.constructor;
             data = entityNameOrEntity;
         }
         else {
-            entityName = Utils.className(entityNameOrEntity);
+            entityName = entityNameOrEntity;
         }
         const meta = this.metadata.get(entityName);
         const convertCustomTypes = !Utils.isEntity(data);
@@ -793,7 +782,7 @@ export class EntityManager {
                     where[meta.primaryKeys[0]] = ret.insertId;
                 }
             }
-            const data2 = await this.driver.findOne(meta.className, where, {
+            const data2 = await this.driver.findOne(meta.class, where, {
                 fields: returning,
                 ctx: em.transactionContext,
                 convertCustomTypes: true,
@@ -848,11 +837,11 @@ export class EntityManager {
         let entityName;
         let propIndex;
         if (data === undefined) {
-            entityName = entityNameOrEntity[0].constructor.name;
+            entityName = entityNameOrEntity[0].constructor;
             data = entityNameOrEntity;
         }
         else {
-            entityName = Utils.className(entityNameOrEntity);
+            entityName = entityNameOrEntity;
         }
         const batchSize = options.batchSize ?? this.config.get('batchSize');
         if (data.length > batchSize) {
@@ -960,7 +949,7 @@ export class EntityManager {
                     where.$or[idx][prop] = item[prop];
                 });
             });
-            const data2 = await this.driver.find(meta.className, where, {
+            const data2 = await this.driver.find(meta.class, where, {
                 fields: returning.concat(...add).concat(...(Array.isArray(uniqueFields) ? uniqueFields : [])),
                 ctx: em.transactionContext,
                 convertCustomTypes: true,
@@ -1113,11 +1102,11 @@ export class EntityManager {
         em.prepareOptions(options);
         let entityName;
         if (data === undefined) {
-            entityName = entityNameOrEntity.constructor.name;
+            entityName = entityNameOrEntity.constructor;
             data = entityNameOrEntity;
         }
         else {
-            entityName = Utils.className(entityNameOrEntity);
+            entityName = entityNameOrEntity;
         }
         if (Utils.isEntity(data)) {
             if (options.schema && helper(data).getSchema() == null) {
@@ -1148,11 +1137,11 @@ export class EntityManager {
         em.prepareOptions(options);
         let entityName;
         if (data === undefined) {
-            entityName = entityNameOrEntities[0].constructor.name;
+            entityName = entityNameOrEntities[0].constructor;
             data = entityNameOrEntities;
         }
         else {
-            entityName = Utils.className(entityNameOrEntities);
+            entityName = entityNameOrEntities;
         }
         if (data.length === 0) {
             return [];
@@ -1189,7 +1178,6 @@ export class EntityManager {
     async nativeUpdate(entityName, where, data, options = {}) {
         const em = this.getContext(false);
         em.prepareOptions(options);
-        entityName = Utils.className(entityName);
         data = QueryHelper.processObjectParams(data);
         where = await em.processWhere(entityName, where, { ...options, convertCustomTypes: false }, 'update');
         validateParams(data, 'update data');
@@ -1203,7 +1191,6 @@ export class EntityManager {
     async nativeDelete(entityName, where, options = {}) {
         const em = this.getContext(false);
         em.prepareOptions(options);
-        entityName = Utils.className(entityName);
         where = await em.processWhere(entityName, where, options, 'delete');
         validateParams(where, 'delete condition');
         const res = await em.driver.nativeDelete(entityName, where, { ctx: em.transactionContext, ...options });
@@ -1213,7 +1200,6 @@ export class EntityManager {
      * Maps raw database result to an entity and merges it to this EntityManager.
      */
     map(entityName, result, options = {}) {
-        entityName = Utils.className(entityName);
         const meta = this.metadata.get(entityName);
         const data = this.driver.mapResult(result, meta);
         for (const k of Object.keys(data)) {
@@ -1235,13 +1221,12 @@ export class EntityManager {
      */
     merge(entityName, data, options = {}) {
         if (Utils.isEntity(entityName)) {
-            return this.merge(entityName.constructor.name, entityName, data);
+            return this.merge(entityName.constructor, entityName, data);
         }
         const em = options.disableContextResolution ? this : this.getContext();
         options.schema ??= em._schema;
         options.validate ??= true;
         options.cascade ??= true;
-        entityName = Utils.className(entityName);
         validatePrimaryKey(data, em.metadata.get(entityName));
         let entity = em.unitOfWork.tryGetById(entityName, data, options.schema, false);
         if (entity && helper(entity).__managed && helper(entity).__initialized && !options.refresh) {
@@ -1295,7 +1280,7 @@ export class EntityManager {
     getReference(entityName, id, options = {}) {
         options.schema ??= this.schema;
         options.convertCustomTypes ??= false;
-        const meta = this.metadata.get(Utils.className(entityName));
+        const meta = this.metadata.get(entityName);
         if (Utils.isPrimaryKey(id)) {
             if (meta.compositePK) {
                 throw ValidationError.invalidCompositeIdentifier(meta);
@@ -1316,7 +1301,6 @@ export class EntityManager {
         // Shallow copy options since the object will be modified when deleting orderBy
         options = { ...options };
         em.prepareOptions(options);
-        entityName = Utils.className(entityName);
         await em.tryFlush(entityName, options);
         where = await em.processWhere(entityName, where, options, 'read');
         options.populate = await em.preparePopulate(entityName, options);
@@ -1352,7 +1336,7 @@ export class EntityManager {
         for (const ent of entities) {
             if (!Utils.isEntity(ent, true)) {
                 /* v8 ignore next */
-                const meta = typeof ent === 'object' ? em.metadata.find(ent.constructor.name) : undefined;
+                const meta = typeof ent === 'object' ? em.metadata.find(ent.constructor) : undefined;
                 throw ValidationError.notDiscoveredEntity(ent, meta);
             }
             // do not cascade just yet, cascading of entities in persist stack is done when flushing
@@ -1396,7 +1380,6 @@ export class EntityManager {
     async tryFlush(entityName, options) {
         const em = this.getContext();
         const flushMode = options.flushMode ?? em.flushMode ?? em.config.get('flushMode');
-        entityName = Utils.className(entityName);
         const meta = em.metadata.get(entityName);
         if (flushMode === FlushMode.COMMIT) {
             return;
@@ -1415,7 +1398,6 @@ export class EntityManager {
      * Checks whether given property can be populated on the entity.
      */
     canPopulate(entityName, property) {
-        entityName = Utils.className(entityName);
         // eslint-disable-next-line prefer-const
         let [p, ...parts] = property.split('.');
         const meta = this.metadata.find(entityName);
@@ -1426,11 +1408,8 @@ export class EntityManager {
             p = p.split(':', 2)[0];
         }
         const ret = p in meta.root.properties;
-        if (!ret) {
-            return !!this.metadata.find(property)?.pivotTable;
-        }
         if (parts.length > 0) {
-            return this.canPopulate((meta.root.properties)[p].type, parts.join('.'));
+            return this.canPopulate(meta.root.properties[p].targetMeta.class, parts.join('.'));
         }
         return ret;
     }
@@ -1444,7 +1423,7 @@ export class EntityManager {
         }
         const em = this.getContext();
         em.prepareOptions(options);
-        const entityName = arr[0].constructor.name;
+        const entityName = arr[0].constructor;
         const preparedPopulate = await em.preparePopulate(entityName, { populate: populate, filters: options.filters }, options.validate);
         await em.entityLoader.populate(entityName, arr, preparedPopulate, options);
         return entities;
@@ -1573,7 +1552,6 @@ export class EntityManager {
      */
     getMetadata(entityName) {
         if (entityName) {
-            entityName = Utils.className(entityName);
             return this.metadata.get(entityName);
         }
         return this.metadata;
@@ -1602,8 +1580,8 @@ export class EntityManager {
                 lockTableAliases: options.lockTableAliases,
             });
         }
-        const preparedPopulate = await this.preparePopulate(meta.className, options);
-        await this.entityLoader.populate(meta.className, [entity], preparedPopulate, {
+        const preparedPopulate = await this.preparePopulate(meta.class, options);
+        await this.entityLoader.populate(meta.class, [entity], preparedPopulate, {
             ...options,
             ...this.getPopulateWhere(where, options),
             orderBy: options.populateOrderBy ?? options.orderBy,
@@ -1740,7 +1718,7 @@ export class EntityManager {
         for (const k of ['ctx', 'strategy', 'flushMode', 'logging', 'loggerContext']) {
             delete opts[k];
         }
-        return [entityName, method, opts, where];
+        return [Utils.className(entityName), method, opts, where];
     }
     /**
      * @internal

package/MikroORM.d.ts

@@ -94,7 +94,7 @@ export declare class MikroORM<Driver extends IDatabaseDriver = IDatabaseDriver,
     /**
      * Allows dynamically discovering new entity by reference, handy for testing schema diffing.
      */
-    discoverEntity<T extends Constructor | EntitySchema>(entities: T | T[], reset?: string | string[]): void;
+    discoverEntity<T extends Constructor | EntitySchema>(entities: T | T[], reset?: EntityName | EntityName[]): void;
     /**
      * Gets the SchemaGenerator.
      */

package/MikroORM.js

@@ -158,7 +158,6 @@ export class MikroORM {
      */
     getMetadata(entityName) {
         if (entityName) {
-            entityName = Utils.className(entityName);
             return this.metadata.get(entityName);
         }
         return this.metadata;
@@ -181,8 +180,8 @@ export class MikroORM {
         const tmp = this.discovery.discoverReferences(Utils.asArray(entities));
         const metadata = this.discovery.processDiscoveredEntities(tmp);
         for (const meta of metadata) {
-            this.metadata.set(meta.className, meta);
-            meta.root = this.metadata.get(meta.root.className);
+            this.metadata.set(meta.class, meta);
+            meta.root = this.metadata.get(meta.root.class);
         }
         this.metadata.decorate(this.em);
     }

package/drivers/DatabaseDriver.d.ts

@@ -22,18 +22,18 @@ export declare abstract class DatabaseDriver<C extends Connection> implements ID
     protected comparator: EntityComparator;
     protected metadata: MetadataStorage;
     protected constructor(config: Configuration, dependencies: string[]);
-    abstract find<T extends object, P extends string = never, F extends string = '*', E extends string = never>(entityName: string, where: FilterQuery<T>, options?: FindOptions<T, P, F, E>): Promise<EntityData<T>[]>;
-    abstract findOne<T extends object, P extends string = never, F extends string = '*', E extends string = never>(entityName: string, where: FilterQuery<T>, options?: FindOneOptions<T, P, F, E>): Promise<EntityData<T> | null>;
-    abstract nativeInsert<T extends object>(entityName: string, data: EntityDictionary<T>, options?: NativeInsertUpdateOptions<T>): Promise<QueryResult<T>>;
-    abstract nativeInsertMany<T extends object>(entityName: string, data: EntityDictionary<T>[], options?: NativeInsertUpdateManyOptions<T>, transform?: (sql: string) => string): Promise<QueryResult<T>>;
-    abstract nativeUpdate<T extends object>(entityName: string, where: FilterQuery<T>, data: EntityDictionary<T>, options?: NativeInsertUpdateOptions<T>): Promise<QueryResult<T>>;
-    nativeUpdateMany<T extends object>(entityName: string, where: FilterQuery<T>[], data: EntityDictionary<T>[], options?: NativeInsertUpdateManyOptions<T>): Promise<QueryResult<T>>;
-    abstract nativeDelete<T extends object>(entityName: string, where: FilterQuery<T>, options?: DeleteOptions<T>): Promise<QueryResult<T>>;
-    abstract count<T extends object, P extends string = never>(entityName: string, where: FilterQuery<T>, options?: CountOptions<T, P>): Promise<number>;
+    abstract find<T extends object, P extends string = never, F extends string = '*', E extends string = never>(entityName: EntityName<T>, where: FilterQuery<T>, options?: FindOptions<T, P, F, E>): Promise<EntityData<T>[]>;
+    abstract findOne<T extends object, P extends string = never, F extends string = '*', E extends string = never>(entityName: EntityName<T>, where: FilterQuery<T>, options?: FindOneOptions<T, P, F, E>): Promise<EntityData<T> | null>;
+    abstract nativeInsert<T extends object>(entityName: EntityName<T>, data: EntityDictionary<T>, options?: NativeInsertUpdateOptions<T>): Promise<QueryResult<T>>;
+    abstract nativeInsertMany<T extends object>(entityName: EntityName<T>, data: EntityDictionary<T>[], options?: NativeInsertUpdateManyOptions<T>, transform?: (sql: string) => string): Promise<QueryResult<T>>;
+    abstract nativeUpdate<T extends object>(entityName: EntityName<T>, where: FilterQuery<T>, data: EntityDictionary<T>, options?: NativeInsertUpdateOptions<T>): Promise<QueryResult<T>>;
+    nativeUpdateMany<T extends object>(entityName: EntityName<T>, where: FilterQuery<T>[], data: EntityDictionary<T>[], options?: NativeInsertUpdateManyOptions<T>): Promise<QueryResult<T>>;
+    abstract nativeDelete<T extends object>(entityName: EntityName<T>, where: FilterQuery<T>, options?: DeleteOptions<T>): Promise<QueryResult<T>>;
+    abstract count<T extends object, P extends string = never>(entityName: EntityName<T>, where: FilterQuery<T>, options?: CountOptions<T, P>): Promise<number>;
     createEntityManager(useContext?: boolean): this[typeof EntityManagerType];
     findVirtual<T extends object>(entityName: EntityName<T>, where: FilterQuery<T>, options: FindOptions<T, any, any, any>): Promise<EntityData<T>[]>;
-    countVirtual<T extends object>(entityName: string, where: FilterQuery<T>, options: CountOptions<T, any>): Promise<number>;
-    aggregate(entityName: string, pipeline: any[]): Promise<any[]>;
+    countVirtual<T extends object>(entityName: EntityName<T>, where: FilterQuery<T>, options: CountOptions<T, any>): Promise<number>;
+    aggregate(entityName: EntityName, pipeline: any[]): Promise<any[]>;
     loadFromPivotTable<T extends object, O extends object>(prop: EntityProperty, owners: Primary<O>[][], where?: FilterQuery<any>, orderBy?: OrderDefinition<T>, ctx?: Transaction, options?: FindOptions<T, any, any, any>, pivotJoin?: boolean): Promise<Dictionary<T[]>>;
     syncCollections<T extends object, O extends object>(collections: Iterable<Collection<T, O>>, options?: DriverMethodOptions): Promise<void>;
     mapResult<T extends object>(result: EntityDictionary<T>, meta?: EntityMetadata<T>, populate?: PopulateOptions<T>[]): EntityData<T> | null;
@@ -58,7 +58,7 @@ export declare abstract class DatabaseDriver<C extends Connection> implements ID
     /** @internal */
     mapDataToFieldNames(data: Dictionary, stringifyJsonArrays: boolean, properties?: Record<string, EntityProperty>, convertCustomTypes?: boolean, object?: boolean): Dictionary;
     protected inlineEmbeddables<T extends object>(meta: EntityMetadata<T>, data: T, where?: boolean): void;
-    protected getPrimaryKeyFields(entityName: string): string[];
+    protected getPrimaryKeyFields<T>(meta: EntityMetadata<T>): string[];
     protected createReplicas(cb: (c: ConnectionOptions) => C): C[];
     lockPessimistic<T extends object>(entity: T, options: LockOptions): Promise<void>;
     abstract stream<T extends object>(entityName: EntityName<T>, where: FilterQuery<T>, options: StreamOptions<T>): AsyncIterableIterator<T>;

package/drivers/DatabaseDriver.js

@@ -59,7 +59,7 @@ export class DatabaseDriver {
             {
                 const pk = coll.property.targetMeta.primaryKeys[0];
                 const data = { [coll.property.name]: coll.getIdentifiers(pk) };
-                await this.nativeUpdate(coll.owner.constructor.name, helper(coll.owner).getPrimaryKey(), data, options);
+                await this.nativeUpdate(coll.owner.constructor, helper(coll.owner).getPrimaryKey(), data, options);
             }
         }
     }
@@ -67,7 +67,7 @@ export class DatabaseDriver {
         if (!result || !meta) {
             return result ?? null;
         }
-        return this.comparator.mapResult(meta.className, result);
+        return this.comparator.mapResult(meta, result);
     }
     async connect(options) {
         await this.connection.connect(options);
@@ -152,7 +152,7 @@ export class DatabaseDriver {
         }
         const createOrderBy = (prop, direction) => {
             if (Utils.isPlainObject(direction)) {
-                const value = Utils.keys(direction).reduce((o, key) => {
+                const value = Utils.getObjectQueryKeys(direction).reduce((o, key) => {
                     Object.assign(o, createOrderBy(key, direction[key]));
                     return o;
                 }, {});
@@ -277,14 +277,14 @@ export class DatabaseDriver {
                     // explicitly allow `$exists`, `$eq` and `$ne` operators here as they can't be misused this way
                     const operator = Object.keys(data[prop.name]).some(f => Utils.isOperator(f) && !['$exists', '$ne', '$eq'].includes(f));
                     if (operator) {
-                        throw ValidationError.cannotUseOperatorsInsideEmbeddables(meta.className, prop.name, data);
+                        throw ValidationError.cannotUseOperatorsInsideEmbeddables(meta.class, prop.name, data);
                     }
                     if (prop.object && where) {
                         const inline = (payload, sub, path) => {
                             if (sub.kind === ReferenceKind.EMBEDDED && Utils.isObject(payload[sub.embedded[1]])) {
                                 return Object.keys(payload[sub.embedded[1]]).forEach(kkk => {
                                     if (!sub.embeddedProps[kkk]) {
-                                        throw ValidationError.invalidEmbeddableQuery(meta.className, kkk, sub.type);
+                                        throw ValidationError.invalidEmbeddableQuery(meta.class, kkk, sub.type);
                                     }
                                     inline(payload[sub.embedded[1]], sub.embeddedProps[kkk], [...path, sub.embedded[1]]);
                                 });
@@ -308,7 +308,7 @@ export class DatabaseDriver {
                         data[props[kk].name] = data[prop.name][props[kk].embedded[1]];
                     }
                     else {
-                        throw ValidationError.invalidEmbeddableQuery(meta.className, kk, prop.type);
+                        throw ValidationError.invalidEmbeddableQuery(meta.class, kk, prop.type);
                     }
                 });
                 if (!unknownProp) {
@@ -317,9 +317,8 @@ export class DatabaseDriver {
             }
         });
     }
-    getPrimaryKeyFields(entityName) {
-        const meta = this.metadata.find(entityName);
-        return meta ? Utils.flatten(meta.getPrimaryProps().map(pk => pk.fieldNames)) : [this.config.getNamingStrategy().referenceColumnName()];
+    getPrimaryKeyFields(meta) {
+        return meta.getPrimaryProps().flatMap(pk => pk.fieldNames);
     }
     createReplicas(cb) {
         const replicas = this.config.get('replicas', []);