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.
If we create more groups with different conditions, the condition label apper next to field is incorrect. Please check attached screenshot.
Attached the patch file.
Steps to reproduce:
1. Install view conditional module
2. Create a new view for contents.
3. Add multiple filters using 'And' and 'OR' options.
Comment | File | Size | Author |
---|---|---|---|
#18 | Screenshot 2019-10-11 at 3.40.22 PM.png | 199.4 KB | naresh_bavaskar |
#13 | 2897644_#13_view_conditional_js_error_fix.patch | 1.43 KB | nikunj.shah |
Comments
Comment #2
sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commentedPlease review.
Comment #3
anand.toshniwal93 CreditAttribution: anand.toshniwal93 as a volunteer and at QED42 commented@sathish.redcrackle,
please verify the patch you have uploaded, it's meant for Drupal core.
Comment #4
sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commentedComment #5
sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commentedComment #6
LendudeMoving to the right queue.
Comment #7
sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commentedAttached the patch for drupal 8.5 version.
Comment #8
sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commentedAdded patch for version drupal 8.6.
Comment #9
Lendude@sathish.redcrackle thanks for looking at this. When making changes to javascript files we now need to modify the es6 file and then compile the js. See https://www.drupal.org/node/2815083 for steps to do this.
We will also need a test for this.
Comment #11
sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commentedAttached the patch file with .es6 file changes.
Comment #13
nikunj.shah CreditAttribution: nikunj.shah at QED42 commentedHi Sathish, I have created a patch for Drupal core 8.8.x. This patch is solving the error.
Thank you.
Comment #14
nikunj.shah CreditAttribution: nikunj.shah at QED42 commentedComment #15
ambuj_gupta CreditAttribution: ambuj_gupta at QED42 commentedComment #17
naresh_bavaskarComment #18
naresh_bavaskar@nikunj.shah
The Patch attached in #13 is worked for me. Please find the attached screenshot. Tested on Drupal version 8.9.x.
Thanks
Comment #19
naresh_bavaskarComment #20
lauriiiMoving back to needs work to add some test coverage for this bug fix.