eager loading not working with nested populate using find query

[dimitriSaplatkin]

Describe the bug

When I call a find query with a populate then the eager loading is not working for the nested populated entity.
Let's say we have the following query:

const authors = await orm.em.find(
      Author,
      { firstName: "Jon" },
      { populate: ["books", "books.genre"] },
    );

On the Genre entity there is another ManyToOne property with the option eager: true.

I would now expect this property is also populated. This case was working for us on mirko-orm v5.

You can see the full reproduciton in the provided github repo

Reproduction

https://github.com/dimitriSaplatkin/reproduction

What driver are you using?

@mikro-orm/postgresql

MikroORM version

6

Node.js version

20

Operating system

Ubuntu 23

Validations

• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Slack.
• The provided reproduction is a minimal reproducible example of the bug.

[B4nan]

This is actually caused by you passing the books hint twice, if you omit the first one it works fine:

const authors = await orm.em.find(
  Author,
  { firstName: 'Jon' },
  { populate: ['books.genre'] },
);

[dimitriSaplatkin]

Hmm okay. But it feels a bit like inconsistent behavior because it works for a findOne query in my example to populate

{ populate: ["books", "books.genre"] },

So I would expect that it also should not work for a fineOne query

[B4nan]

I closed the issue with a fix, I guess you missed that? Otherwise I'd rather expect "thank you" instead of "hmm okay" :D

[dimitriSaplatkin]

Sorry, my bad. I didn't saw the fix. Thanks for fixing and Happy New Year 🎉.