Fix #1293: Exception: BelongsToMany Is Not supported for hybrid query constraints! #1324
Conversation
This temporarily skirts around the issue described in the following issue: mongodb#1293
Without this statement, the reaffirming of belongsToMany relations would fail. The original commit was to prove a point in an issue, but for it to work in the real world this extra bit is added.
|
Not sure I can do anything to make the Travis CI build pass. 🤔 |
|
@benargo just wondering how come you closed your PR? did you find an alternative fix to the problem or was it resolved through another PR? |
|
I need to do some more testing on it first, as I haven't attempted to replicate the issue on >3.2.3. I'm aiming to do this in the coming week. |
|
What ever happened to this fix? I'm having the same problem and the suggested fix works. It would be nice to get this into the mainline code. |
|
I'm really sorry about the delay. I had the day off today and took the opportunity to revisit this PR. I've made sure this change is compatible with the latest version (as I developed the original fix on a previous build). Your comments were very helpful with this as well. I'm going to resubmit the PR. |
|
Awesome, thanks Ben! |
Fix mongodb#1293: Exception: BelongsToMany Is Not supported for hybrid query constraints! mongodb#1324
Fix mongodb#1293: Exception: BelongsToMany Is Not supported for hybrid query constraints! mongodb#1324
This fixes issue #1293 where non-hybrid BelongsToMany relations were throwing the described exception.