feat: add support for getRawMany() and row count as it is

[okaprinarjaya]

Hi,

In this PR, I try to solve two following issues:

1. Need to include in query sum of amounts of relations #265 --> Primary issue that I need to solve.
2. GROUP BY removed in count query when using getManyAndCount() typeorm/typeorm#5127 , Bug - .groupBy is empty on count query, resulting in wrong count when .groupBy is used typeorm/typeorm#7454

I totally open for any feedback, and I do really need @ppetzold guidance to improve my changes in this PR.

Thank You

[ppetzold]

Looks solid at first sight. Could you add new config params description to README?

[okaprinarjaya]

Looks solid at first sight. Could you add new config params description to README?

Hi,
Thank you for your reply,
Ok I will help to update the documentation (README).

And I have some concerns

1. about usage of queryBuilderCloned . I need your opinion, about that. Is it better to use queryBuilderCloned or move lines: https://github.com/ppetzold/nestjs-paginate/blob/master/src/paginate.ts#L222-L230 into below / under line: 389, then we focus on process the counting first, save the counting result, and then execute data rows fetching. - Fixet at: b6d4478
2. about custom getCount() method that mimick typeorm's getCount() method. I need your opinion about this. Do we really need to create a custom getCount() method that "override" typeorm's getCount() method? because at line: 393 using typeorm's getManyAndCount() method. If you're Ok / align with my custom getCount() idea, then we can adjusts to remove the usage of typeorm's getManyAndCount() method. - Yes! we need it to easily switch to typeorm's original getCount() or to the as it is count.
3. I'm not confident with naming of two new config attribute: 1. getDataAsRaw , 2. rowCountAsItIs . Do you have better naming idea?

[Helveg]

Hi! I'd like to weigh in here on point 3:

• 3.1: returnRaw, fetchRaw
• 3.2: preserveRowCount, exactRowCount

[96Jerry]

Are there any updates here?

[msulieman10]

Hello @ppetzold
Kindly, can you merge this pull request?

Thank you in advance,

[msulieman10]

Are there any updates here?

[fpnuseis]

Hi @ppetzold 👋

It looks like the original author hasn’t responded for a while, and the remaining work on this PR seems to be mainly the README/documentation updates.

I’m interested in continuing this work.
I will add the missing README updates and also apply the config name change suggested by @Helveg, then submit a follow-up PR based on the current changes.

Would it be okay if I take this over?
Just want to confirm before proceeding. Thanks!

[ppetzold]

@fpnuseis yep sure, you are welcome to do so