querybuilder icon indicating copy to clipboard operation
querybuilder copied to clipboard

Fixes broken "INCLUDE/INCLUDEMANY"

Open chrisgate opened this issue 3 years ago • 6 comments

I managed to resolved Include and includeMany in SqlKata.Execution project at QueryFactory.cs file .

Please check if there no conflict with other feature of the library?

Cheers

chrisgate avatar Jun 18 '21 00:06 chrisgate

@chrisgate can you elaborate more on this?

ahmad-moussawi avatar Jun 23 '21 14:06 ahmad-moussawi

Thanks for your time, I opened this issue https://github.com/sqlkata/querybuilder/issues/433 long time ago and I found out many talk about it on different issues too. So, I clone and resolve it on my end and it is working again. But I am using PostgreSQL and I don't know if it will work with other compiler too but I think it should because all I edited was in this file QueryFactory.cs on SqlKata.Execution section.

Thanks

chrisgate avatar Jun 23 '21 15:06 chrisgate

Plus this my first pull request ever................at least I broke the York....lol

chrisgate avatar Jun 23 '21 15:06 chrisgate

@ahmad-moussawi

chrisgate avatar Aug 02 '21 11:08 chrisgate

it fixes also https://github.com/sqlkata/querybuilder/issues/311 @ahmad-moussawi

bsekin-jb avatar Dec 06 '22 10:12 bsekin-jb

@ahmad-moussawi

andrewweitzer avatar Mar 07 '24 05:03 andrewweitzer