@mikro-orm/core 7.0.0-dev.99 → 7.0.0-rc.1

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';
@@ -17,7 +17,7 @@ import { EventType, FlushMode, LoadStrategy, LockMode, PopulateHint, PopulatePat
 import { EventManager } from './events/EventManager.js';
 import { TransactionEventBroadcaster } from './events/TransactionEventBroadcaster.js';
 import { OptimisticLockError, ValidationError } from './errors.js';
-import { getLoadingStrategy } from './entity/utils.js';
+import { applyPopulateHints, getLoadingStrategy } from './entity/utils.js';
 import { TransactionManager } from './utils/TransactionManager.js';
 /**
  * The EntityManager is the central access point to ORM functionality. It is a facade to all different ORM subsystems
@@ -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,10 +115,15 @@ 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 || {};
+        const meta = this.metadata.get(entityName);
+        if (meta.orderBy) {
+            options.orderBy = QueryHelper.mergeOrderBy(options.orderBy, meta.orderBy);
+        }
+        else {
+            options.orderBy ??= {};
+        }
         options.populate = await em.preparePopulate(entityName, options);
         const populate = options.populate;
         const cacheKey = em.cacheKey(entityName, options, 'em.find', where);
@@ -133,7 +137,6 @@ export class EntityManager {
             });
             return cached.data;
         }
-        const meta = this.metadata.get(entityName);
         options = { ...options };
         // save the original hint value so we know it was infer/all
         options._populateWhere = options.populateWhere ?? this.config.get('populateWhere');
@@ -192,7 +195,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 || {};
@@ -300,18 +302,18 @@ export class EntityManager {
     // this method only handles the problem for mongo driver, SQL drivers have their implementation inside QueryBuilder
     applyDiscriminatorCondition(entityName, where) {
         const meta = this.metadata.find(entityName);
-        if (!meta?.discriminatorValue) {
+        if (meta?.root.inheritanceType !== 'sti' || !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;
@@ -328,8 +330,10 @@ export class EntityManager {
         return ret;
     }
     async getJoinedFilters(meta, options) {
+        // If user provided populateFilter, merge it with computed filters
+        const userFilter = options.populateFilter;
         if (!this.config.get('filtersOnRelations') || !options.populate) {
-            return undefined;
+            return userFilter;
         }
         const ret = {};
         for (const hint of options.populate) {
@@ -341,7 +345,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,
             });
@@ -363,7 +367,11 @@ export class EntityManager {
                 }
             }
         }
-        return ret;
+        // Merge user-provided populateFilter with computed filters
+        if (userFilter) {
+            Utils.merge(ret, userFilter);
+        }
+        return Utils.hasObjectKeys(ret) ? ret : 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.
@@ -377,11 +385,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 +412,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 +420,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 +445,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 +462,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 +473,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.
@@ -489,21 +492,21 @@ export class EntityManager {
      *     - POJO/entity instance
      *
      * ```ts
-     * const currentCursor = await em.findByCursor(User, {}, {
+     * const currentCursor = await em.findByCursor(User, {
      *   first: 10,
      *   after: previousCursor, // cursor instance
      *   orderBy: { id: 'desc' },
      * });
      *
      * // to fetch next page
-     * const nextCursor = await em.findByCursor(User, {}, {
+     * const nextCursor = await em.findByCursor(User, {
      *   first: 10,
      *   after: currentCursor.endCursor, // opaque string
      *   orderBy: { id: 'desc' },
      * });
      *
      * // to fetch next page
-     * const nextCursor2 = await em.findByCursor(User, {}, {
+     * const nextCursor2 = await em.findByCursor(User, {
      *   first: 10,
      *   after: { id: lastSeenId }, // entity-like POJO
      *   orderBy: { id: 'desc' },
@@ -531,16 +534,16 @@ export class EntityManager {
      * }
      * ```
      */
-    async findByCursor(entityName, where, options) {
+    async findByCursor(entityName, options) {
         const em = this.getContext(false);
-        entityName = Utils.className(entityName);
         options.overfetch ??= true;
-        if (Utils.isEmpty(options.orderBy)) {
+        options.where ??= {};
+        if (Utils.isEmpty(options.orderBy) && !Raw.hasObjectFragments(options.orderBy)) {
             throw new Error('Explicit `orderBy` option required');
         }
         const [entities, count] = options.includeCount !== false
-            ? await em.findAndCount(entityName, where, options)
-            : [await em.find(entityName, where, options)];
+            ? await em.findAndCount(entityName, options.where, options)
+            : [await em.find(entityName, options.where, options)];
         return new Cursor(entities, count, options, this.metadata.get(entityName));
     }
     /**
@@ -552,9 +555,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;
     }
@@ -565,9 +568,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,
@@ -579,14 +581,14 @@ export class EntityManager {
         }
         let found = false;
         for (const e of fork.unitOfWork.getIdentityMap()) {
-            const ref = em.getReference(e.constructor.name, 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);
+            const ref = em.getReference(e.constructor, helper(e).getPrimaryKey());
+            const data = helper(e).serialize({ ignoreSerializers: true, includeHidden: true, convertCustomTypes: false });
+            em.config.getHydrator(this.metadata).hydrate(ref, helper(ref).__meta, data, em.entityFactory, 'full', false, false);
             Utils.merge(helper(ref).__originalEntityData, this.comparator.prepareEntity(e));
             found ||= ref === entity;
         }
         if (!found) {
-            const data = helper(reloaded).serialize({ ignoreSerializers: true, includeHidden: true });
+            const data = helper(reloaded).serialize({ ignoreSerializers: true, includeHidden: true, convertCustomTypes: true });
             em.config.getHydrator(this.metadata).hydrate(entity, wrapped.__meta, data, em.entityFactory, 'full', false, true);
             Utils.merge(wrapped.__originalEntityData, this.comparator.prepareEntity(reloaded));
         }
@@ -604,7 +606,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
@@ -682,10 +683,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;
     }
@@ -725,11 +726,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);
@@ -792,7 +793,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,
@@ -847,11 +848,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) {
@@ -959,7 +960,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,
@@ -1112,11 +1113,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) {
@@ -1147,11 +1148,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 [];
@@ -1188,7 +1189,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');
@@ -1202,7 +1202,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 });
@@ -1212,7 +1211,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)) {
@@ -1234,13 +1232,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) {
@@ -1294,7 +1291,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);
@@ -1315,7 +1312,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);
@@ -1351,7 +1347,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
@@ -1395,7 +1391,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;
@@ -1414,7 +1409,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);
@@ -1424,12 +1418,11 @@ export class EntityManager {
         if (p.includes(':')) {
             p = p.split(':', 2)[0];
         }
-        const ret = p in meta.root.properties;
-        if (!ret) {
-            return !!this.metadata.find(property)?.pivotTable;
-        }
+        // For TPT inheritance, check the entity's own properties, not just the root's
+        // For STI, meta.properties includes all properties anyway
+        const ret = p in meta.properties;
         if (parts.length > 0) {
-            return this.canPopulate((meta.root.properties)[p].type, parts.join('.'));
+            return this.canPopulate(meta.properties[p].targetMeta.class, parts.join('.'));
         }
         return ret;
     }
@@ -1443,7 +1436,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;
@@ -1572,7 +1565,6 @@ export class EntityManager {
      */
     getMetadata(entityName) {
         if (entityName) {
-            entityName = Utils.className(entityName);
             return this.metadata.get(entityName);
         }
         return this.metadata;
@@ -1601,8 +1593,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,
@@ -1622,6 +1614,7 @@ export class EntityManager {
             return ret;
         }, []);
     }
+    /** @internal */
     async preparePopulate(entityName, options, validate = true) {
         if (options.populate === false) {
             return [];
@@ -1662,7 +1655,7 @@ export class EntityManager {
             options.populate = pruneToOneRelations(meta, this.buildFields(options.fields));
         }
         if (!options.populate) {
-            const populate = this.entityLoader.normalizePopulate(entityName, [], options.strategy);
+            const populate = this.entityLoader.normalizePopulate(entityName, [], options.strategy, true, options.exclude);
             await this.autoJoinRefsForFilters(meta, { ...options, populate });
             return populate;
         }
@@ -1684,18 +1677,21 @@ export class EntityManager {
                 return [field];
             }).flat();
         }
-        const populate = this.entityLoader.normalizePopulate(entityName, options.populate, options.strategy);
+        const populate = this.entityLoader.normalizePopulate(entityName, options.populate, options.strategy, true, options.exclude);
         const invalid = populate.find(({ field }) => !this.canPopulate(entityName, field));
         if (validate && invalid) {
             throw ValidationError.invalidPropertyName(entityName, invalid.field);
         }
         await this.autoJoinRefsForFilters(meta, { ...options, populate });
-        return populate.map(field => {
+        for (const field of populate) {
             // force select-in strategy when populating all relations as otherwise we could cause infinite loops when self-referencing
             const all = field.all ?? (Array.isArray(options.populate) && options.populate.includes('*'));
             field.strategy = all ? LoadStrategy.SELECT_IN : (options.strategy ?? field.strategy);
-            return field;
-        });
+        }
+        if (options.populateHints) {
+            applyPopulateHints(populate, options.populateHints);
+        }
+        return populate;
     }
     /**
      * when the entity is found in identity map, we check if it was partially loaded or we are trying to populate
@@ -1715,7 +1711,7 @@ export class EntityManager {
                 return !inlineEmbedded && !prop.lazy && !helper(entity).__loadedProperties.has(prop.name);
             });
         }
-        if (autoRefresh) {
+        if (autoRefresh || options.filters) {
             return true;
         }
         if (Array.isArray(options.populate)) {
@@ -1739,7 +1735,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
@@ -1756,21 +1752,17 @@ export class EntityManager {
             return { key: cacheKey, data: cached };
         }
         let data;
+        const createOptions = {
+            merge: true,
+            convertCustomTypes: false,
+            refresh,
+            recomputeSnapshot: true,
+        };
         if (Array.isArray(cached) && merge) {
-            data = cached.map(item => em.entityFactory.create(entityName, item, {
-                merge: true,
-                convertCustomTypes: true,
-                refresh,
-                recomputeSnapshot: true,
-            }));
+            data = cached.map(item => em.entityFactory.create(entityName, item, createOptions));
         }
         else if (Utils.isObject(cached) && merge) {
-            data = em.entityFactory.create(entityName, cached, {
-                merge: true,
-                convertCustomTypes: true,
-                refresh,
-                recomputeSnapshot: true,
-            });
+            data = em.entityFactory.create(entityName, cached, createOptions);
         }
         else {
             data = cached;

package/MikroORM.d.ts

@@ -5,7 +5,7 @@ import { Configuration, type Options } from './utils/Configuration.js';
 import type { EntityManager } from './EntityManager.js';
 import type { AnyEntity, Constructor, EntityClass, EntityMetadata, EntityName, IEntityGenerator, IMigrator, ISeedManager } from './typings.js';
 /** @internal */
-export declare function lookupExtensions(options: Options): Promise<void>;
+export declare function loadOptionalDependencies(options: Options): Promise<void>;
 /**
  * The main class used to configure and bootstrap the ORM.
  *
@@ -51,9 +51,9 @@ export declare class MikroORM<Driver extends IDatabaseDriver = IDatabaseDriver,
     static init<D extends IDatabaseDriver = IDatabaseDriver, EM extends D[typeof EntityManagerType] & EntityManager<D> = D[typeof EntityManagerType] & EntityManager<D>, Entities extends (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]>(options: Options<D, EM, Entities>): Promise<MikroORM<D, EM, Entities>>;
     /**
      * Synchronous variant of the `init` method with some limitations:
-     * - database connection will be established when you first interact with the database (or you can use `orm.connect()` explicitly)
-     * - no loading of the `config` file, `options` parameter is mandatory
-     * - no support for folder based discovery
+     * - folder-based discovery not supported
+     * - ORM extensions are not autoloaded
+     * - when metadata cache is enabled, `FileCacheAdapter` needs to be explicitly set in the config
      */
     constructor(options: Options<Driver, EM, Entities>);
     /**
@@ -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.
      */