Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Queue is always required when using "Limit to a specific entity queue" even when specifying that this relation is not required from views, this results in that you cannot add multiple sorts and having a "not in the queue" relationship.
Comment | File | Size | Author |
---|---|---|---|
#8 | interdiff.txt | 1.89 KB | amateescu |
#8 | 2821631-8.patch | 1.69 KB | amateescu |
#5 | entityqueue-moved_queue_condition_to_join-2821631-5.patch | 1.64 KB | josebc |
Comments
Comment #2
amateescu CreditAttribution: amateescu commentedInteresting :) Do you think you can investigate a bit and post a patch for this? It would be nice to have it fixed in alpha5.
Comment #3
josebc CreditAttribution: josebc at Vardot commentedon it :)
Comment #4
josebc CreditAttribution: josebc at Vardot commentedComment #5
josebc CreditAttribution: josebc at Vardot commentedmoved condition to join instead of where
patch attached
Comment #6
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedRewarded work Jose!
Testing :)
Comment #7
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #8
amateescu CreditAttribution: amateescu as a volunteer commentedNice work! I tested the patch from #5 and it fixes only the first part of the problem: the queue is no longer required unless the relationship itself is required.
However, the "not in the queue" filter still doesn't behave as expected.
I was able to fix it easily by removing the extra WHERE from
\Drupal\entityqueue\Plugin\views\filter\EntityQueueInQueue
.Can you please test this patch and let me know if everything works correctly now?
Comment #9
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedTesting ......
Comment #10
amateescu CreditAttribution: amateescu as a volunteer commented@RajabNatshah, still testing? :)
Comment #11
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedTested;
I'm using your last patch now
https://github.com/Vardot/varbase/blob/8.x-4.x/composer.json
https://travis-ci.org/Vardot/varbase/jobs/182250366
Comment #12
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedThank you for your work on this.
I'm having a week for Automated Testing to add more testing features. at
https://github.com/Vardot/varbase/blob/8.x-4.x/tests/features/varbase/st...
https://github.com/Vardot/varbase/blob/8.x-4.x/tests/features/varbase/st...
https://github.com/Vardot/varbase/blob/8.x-4.x/tests/features/varbase/st...
https://github.com/Vardot/varbase/blob/8.x-4.x/tests/features/varbase/st...
#Automation: Have more automated functional testing gherkin features
I will do this issue after this post.
Comment #14
amateescu CreditAttribution: amateescu as a volunteer commentedCommitted the patch from #8 to 8.x-1.x. Thanks everyone :)
Comment #15
josebc CreditAttribution: josebc at Vardot commentedLatest patch will not limit the results to queue, moving parent::query(); above makes it ignore the extra
Comment #16
amateescu CreditAttribution: amateescu as a volunteer commentedThat's being fixed in #2837841: Subqueue views relation: Doubled/redundant entries.