When using a view that has a block display, the results are not shown. This is what the tests in facets are failing over: #2790133: Fix tests with the newest search api commits..

CommentFileSizeAuthor
#6 views_with_a_block-2792569-6.patch1.19 KBborisson_
#6 interdiff.txt540 bytesborisson_
#4 2792569-4--fix_views_block_test.patch678 bytesdrunken monkey
#2 views_with_a_block-2792569-2.patch1.72 KBborisson_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

borisson_ created an issue. See original summary.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Active » Needs review
FileSize
views_with_a_block-2792569-2.patch1.72 KB

Status: Needs review » Needs work

The last submitted patch, 2: views_with_a_block-2792569-2.patch, failed testing.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Project: Search API » Facets
Component: Views integration » Code
Status: Needs work » Needs review
FileSize
2792569-4--fix_views_block_test.patch678 bytes

This should fix it.
The problem is that block displays with disabled AJAX can't have exposed forms, so the filters' default values are taken and applied – filtering for created date of 0 and empty strings as ID and keywords.

Status: Needs review » Needs work

The last submitted patch, 4: 2792569-4--fix_views_block_test.patch, failed testing.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Needs work » Needs review
FileSize
interdiff.txt540 bytes
views_with_a_block-2792569-6.patch1.19 KB

re-enable the test to make sure that this works.

  • borisson_ committed 399747a on 8.x-1.x 
    Issue #2792569 by borisson_, drunken monkey: Views with a block display...
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Needs review » Fixed

Awesome, thanks so much for helping us figure that out @drunken monkey!

Status: Fixed » Closed (fixed)

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

Status: Closed (fixed) » Needs work

The last submitted patch, 6: views_with_a_block-2792569-6.patch, failed testing.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Needs work » Closed (works as designed)

Was already committed, bad testbot.