The query function in the views relationship is ignored, as the parent function is called first. It needs to be called last, to build the query with the desired extra join.
The old extra join array is producing error and needs to be fixed too.
This issue can be reproduced, when making two queues, one simple and one multi. Add an entity with the same id in both of them. Then add a views relation for the subqueue, and the item in from simple queue is also taken.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | 2837841-followup.patch | 544 bytes | amateescu |
| #11 | subqueue_relation_query-2837841-11.patch | 1.11 KB | ytsurk |
| #3 | subqueue_relation_query-2837841-3.patch | 1.1 KB | ytsurk |
Comments
Comment #2
ytsurkHere the patch reversing the order, building the query correctly.
Comment #3
ytsurkThis patch now also fixed the join_extra "array syntax".
Comment #4
ytsurkComment #5
ytsurkComment #6
ytsurkComment #8
amateescu commentedCommitted to 8.x-1.x. Thanks!
Comment #9
josebc commentedthis patch have an issue, parent::query(); should be inside query()
Comment #10
josebc commentedComment #11
ytsurkShit .. so true .... @josebc - you got a good eye :D
I'm sorry for this mistake!
Comment #13
amateescu commentedOops :) I just committed the patch attached to fix it quickly.
Comment #14
ytsurkwe're in the fast lane here :D
Comment #15
mfernea commentedI tested this by applying patches at #3 and #13 on alpha6 and it works fine.
Comment #16
ytsurkyes it's #3 and #13 (as commited to the dev branch) or #11 ;)
Thx for testing!
Comment #17
giuliovale commentedI also confirm #3 and #13 over alpha6 work fine