Commit a71163c4 authored by catch's avatar catch

Issue #2943163 by seanB, Lendude: Wrong views exposed raw input values for multivalue filter field

parent 6f2569da
......@@ -183,7 +183,7 @@ public function submitForm(array &$form, FormStateInterface $form_state) {
// https://www.drupal.org/node/342316 is resolved.
$checked = Checkboxes::getCheckedCheckboxes($value);
foreach ($checked as $option_id) {
$view->exposed_raw_input[$option_id] = $value[$option_id];
$view->exposed_raw_input[$key][] = $value[$option_id];
}
}
else {
......
......@@ -3,11 +3,12 @@
namespace Drupal\Tests\views\Kernel\Plugin;
use Drupal\Component\Utility\Html;
use Drupal\node\Entity\NodeType;
use Drupal\Tests\views\Kernel\ViewsKernelTestBase;
use Drupal\views\Views;
/**
* Tests the exposed form markup.
* Tests the exposed form.
*
* @group views
* @see \Drupal\views_test_data\Plugin\views\display_extender\DisplayExtenderTest
......@@ -52,4 +53,92 @@ public function testExposedFormRender() {
$this->assertEqual(count($result), 1, 'Filter description was found.');
}
/**
* Tests the exposed form raw input.
*/
public function testExposedFormRawInput() {
$node_type = NodeType::create(['type' => 'article']);
$node_type->save();
$view = Views::getView('test_exposed_form_buttons');
$view->setDisplay();
$view->displayHandlers->get('default')->overrideOption('filters', [
'type' => [
'exposed' => TRUE,
'field' => 'type',
'id' => 'type',
'table' => 'node_field_data',
'plugin_id' => 'in_operator',
'entity_type' => 'node',
'entity_field' => 'type',
'expose' => [
'identifier' => 'type',
'label' => 'Content: Type',
'operator_id' => 'type_op',
'reduce' => FALSE,
'multiple' => FALSE,
],
],
'type_with_default_value' => [
'exposed' => TRUE,
'field' => 'type',
'id' => 'type_with_default_value',
'table' => 'node_field_data',
'plugin_id' => 'in_operator',
'entity_type' => 'node',
'entity_field' => 'type',
'value' => ['article', 'article'],
'expose' => [
'identifier' => 'type_with_default_value',
'label' => 'Content: Type with value',
'operator_id' => 'type_op',
'reduce' => FALSE,
'multiple' => FALSE,
],
],
'multiple_types' => [
'exposed' => TRUE,
'field' => 'type',
'id' => 'multiple_types',
'table' => 'node_field_data',
'plugin_id' => 'in_operator',
'entity_type' => 'node',
'entity_field' => 'type',
'expose' => [
'identifier' => 'multiple_types',
'label' => 'Content: Type (multiple)',
'operator_id' => 'type_op',
'reduce' => FALSE,
'multiple' => TRUE,
],
],
'multiple_types_with_default_value' => [
'exposed' => TRUE,
'field' => 'type',
'id' => 'multiple_types_with_default_value',
'table' => 'node_field_data',
'plugin_id' => 'in_operator',
'entity_type' => 'node',
'entity_field' => 'type',
'value' => ['article', 'article'],
'expose' => [
'identifier' => 'multiple_types_with_default_value',
'label' => 'Content: Type with default value (multiple)',
'operator_id' => 'type_op',
'reduce' => FALSE,
'multiple' => TRUE,
],
],
]);
$view->save();
$this->executeView($view);
$expected = [
'type' => 'All',
'type_with_default_value' => 'article',
'multiple_types_with_default_value' => ['article'],
];
$this->assertSame($view->exposed_raw_input, $expected);
}
}
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment