Skip to content
Snippets Groups Projects
Verified Commit 88e0d463 authored by Jess's avatar Jess
Browse files

Issue #3180227 by danflanagan8, robertom, xjm, Lendude, quietone: Notice:...

Issue #3180227 by danflanagan8, robertom, xjm, Lendude, quietone: Notice: Trying to access array offset on value of type null in Drupal\views\Plugin\views\display\EntityReference->query()

(cherry picked from commit 0f6101a7)
parent 17889f47
Branches
Tags
21 merge requests!8394[warning] array_flip(): Can only flip STRING and INTEGER values, when saving a non-revisionable custom content entity,!7780issue 3443822: fix for 'No route found for the specified format html. Supported formats: json, xml.',!5013Issue #3071143: Table Render Array Example Is Incorrect,!4848Issue #1566662: Update module should send notifications on Thursdays,!4792Issue #2230689: Remove redundant "Italic" style,!4220Issue #3368223: Link field > Access to internal links is not checked on display.,!3884Issue #3356842,!3870Issue #3087868,!3812Draft: Issue #3339373 by alexpott, andypost, mondrake:...,!3686Issue #3219967 against 9.5.x,!3683Issue #2939397: Clearing AliasManager cache with root path raises warning,!3543Issue #3344259: Allow ajax dialog to have focus configurable,!3356Issue #3209129: Scrolling problems when adding a block via layout builder,!2205Quote all names in the regions section.,!2050Issue #3272969: Remove UnqiueField constraint.,!1956Issue #3268872: hook_views_invalidate_cache not called when a view is deleted,!1893Issue #3217260: Add a way to make media captions not editable in CKEditor,!1459Issue #3087632: menu_name max length is too long,!878Issue #3221534: throw an exception when IDs passed to loadMultiple() are badly formed,!866Issue #2845319: The highlighting of the 'Home' menu-link does not respect query strings and fragment identifiers,!204Issue #3040556: It is not possible to react to an entity being duplicated
......@@ -155,7 +155,18 @@ public function query() {
$id_table = $this->view->storage->get('base_table');
$this->id_field_alias = $this->view->query->addField($id_table, $id_field);
$options = $this->getOption('entity_reference_options');
$options = $this->getOption('entity_reference_options') ?? [];
// The entity_reference_options are typically set during a call to
// Drupal\views\Plugin\EntityReferenceSelection\ViewsSelection::initializeView().
// If any entity_reference_options are not yet set, we apply the same
// default values that would typically be added by that method.
$default_options = [
'match' => NULL,
'match_operator' => 'CONTAINS',
'limit' => 0,
'ids' => NULL,
];
$options += $default_options;
// Restrict the autocomplete options based on what's been typed already.
if (isset($options['match'])) {
......
......@@ -275,6 +275,23 @@ public function testEntityReferenceDisplay() {
$view->setDisplay('entity_reference_1');
$render = $view->display_handler->render();
$this->assertSame([], $render, 'Render returned empty array');
// Execute the View without setting the 'entity_reference_options'.
// This is equivalent to using the following as entity_reference_options.
// @code
// $options = [
// 'match' => NULL,
// 'match_operator' => 'CONTAINS',
// 'limit' => 0,
// 'ids' => NULL,
// ];
// @endcode
// Assert that this view returns a row for each test entity.
$view->destroy();
$view = Views::getView('test_display_entity_reference');
$view->setDisplay('entity_reference_1');
$this->executeView($view);
$this->assertCount(13, $view->result, 'Search returned thirteen rows');
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment