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.
Same issue as https://www.drupal.org/node/1177882 but in relation to views 7.x-3.8.
I've re-rolled the patch given in #64 of https://www.drupal.org/node/1177882 for 7.x-3.8.
Comments
Comment #1
hughworm CreditAttribution: hughworm commentedI've re-rolled the patch from #64 of https://www.drupal.org/node/1177882 for 7.x-3.8, attached.
Comment #3
hughworm CreditAttribution: hughworm commentedHaving another crack at that re-rolled patch.
Comment #4
filipes CreditAttribution: filipes commentedThis patch solved my problems.
Thanks!
How can I appeal for this patch to be included into Views module, so it will also be available after the Views update?
Comment #5
-Mania- CreditAttribution: -Mania- commentedMy error did not disappear with the patch.
Comment #6
smallcoder CreditAttribution: smallcoder commentedThank you Hugh - my life and sanity have now been saved by your patch. I was starting to think Views was *gasp* not as amazing as everyone believes.
Still - it is daft this hasn't been committed as of yet?
Comment #7
Alan D. CreditAttribution: Alan D. commentedJust running the tests.
Some trailing whitespaces in the patch. Set your IDE to trim these on save :)
Comment #9
rbrownellAfter patching I still had this issue.
Comment #10
rbrownellComment #11
hughworm CreditAttribution: hughworm commentedAlan, have any idea why the patch is failing testing? It seems to work for myself and others.
rbrownell, can you elaborate at all?
Comment #13
Alan D. CreditAttribution: Alan D. commented@hughworm
That looks like an issue with the testing system. Re-queued.
[edit]
Sorry, no idea why this is doing this!
BTW, a code search finds about 40 all and 40 All in the code base ('all' and 'All'). I found trying to fix singular errors started to peculate additional bugs into other areas. I ended up hacking it by setting the correct $_GET parameter using hook_init() for the view that was having issues.
Comment #15
Rob230 CreditAttribution: Rob230 commentedI tried the patch in #3. It didn't fix it for me.
I got it working by using Alan D's hook_init() code which sets it to 'All' if it hasn't been set.
Comment #16
hughworm CreditAttribution: hughworm commentedFor what it's worth I think reason the patches fail testing is an issue in run-tests.sh. See https://www.drupal.org/node/2347827
Comment #17
demian1111 CreditAttribution: demian1111 commented...
Comment #18
demian1111 CreditAttribution: demian1111 commentedDon't know if this will help anyone...
but I was using group filters and had an extra BLANK field under my defined filters. Once I deleted the empty field the error went away.
Comment #19
luizsgpetri CreditAttribution: luizsgpetri at CI&T for Pfizer, Inc. commentedI had the same issue and created the following workaround. (Assume favorite as your filter name inside $parameters)
Comment #20
mpgeek CreditAttribution: mpgeek at Phase2 commentedThis is nearly necessary for #2578785: Large-scale location mapping in Drupal 8 (Views) if you want exposed filters on Leaflet GeoJSON views; adding as related. I don't yet know if this is even an issue in D8.
Comment #21
MustangGB CreditAttribution: MustangGB commentedComment #22
MustangGB CreditAttribution: MustangGB commentedI think this issue is pretty redundant now, further discussions over in: #1177882: Exposed filters fire "An illegal choice has been detected. Please contact the site administrator." error.