Step to replicate:

1) Add a Filter to a View
2) Expose the filter
3) Create a group (Hybrid Filter)
4) Un-Expose the filter

The filter is not configurable anymore.

CommentFileSizeAuthor
#7 views-1719496-7.patch6.23 KBtim.plunkett
#2 1719496.tests_only.4.patch3.87 KBdagmar
#2 1719496.tests_and_fix.4.patch6.54 KBdagmar
#1 1719496.hybrid_follow_up.patch2.61 KBdagmar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
CreditAttribution: dagmar commented
Status: Active » Needs review
FileSize
1719496.hybrid_follow_up.patch2.61 KB
dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
CreditAttribution: dagmar commented
FileSize
1719496.tests_and_fix.4.patch6.54 KB
1719496.tests_only.4.patch3.87 KB

With Tests.

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Needs review » Reviewed & tested by the community

After spotting this yesterday, have just tested the patch and works great now. good job dagmar.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Version: 7.x-3.x-dev » 8.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Awesome, thanks for both the fix and the tests!

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Patch (to be ported) » Postponed

postponed, as #731662: Hybrid Exposed Filters is not in the 8.x branch yet.

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Postponed » Active

Good to go again on this one.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Active » Needs review
FileSize
views-1719496-7.patch6.23 KB

Rerolled!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Fixed

Cool!

Committed to 8-3.x

Automatically closed -- issue fixed for 2 weeks with no activity.