Updated ORM - Githubissues

dart-backend / angel

A polished, production-ready backend framework in Dart for the VM, AOT, and Flutter.

https://github.com/dukefirehawk/angel

BSD 3-Clause "New" or "Revised" License

171 stars 22 forks source link

Updated ORM #44

Closed dukefirehawk closed 2 years ago

dukefirehawk commented 2 years ago

Fixed the following issues with PR #42

Changed Query to Query() for generated belongsTo relationship
_index++ mapped to the wrong fields for some test cases, which requires _index+2 or _index+3. Reverted to literalNum(i++) for the correct mapping.

dukefirehawk commented 2 years ago

@debuggerx01 Please take note on the fix for the failed test cases in angel_orm_test with PR #42

debuggerx01 commented 2 years ago

@debuggerx01 Please take note on the fix for the failed test cases in angel_orm_test with PR #42

Yes, maybe this is not a suitable modification, because it will bring inconsistent behavior, and will bring too many null value to the results. So just revert it, I will try to find other ways to achieve similar effects.