diff --git a/core/modules/views/src/Plugin/views/join/JoinPluginBase.php b/core/modules/views/src/Plugin/views/join/JoinPluginBase.php index 8c79be9..12d75b9 100644 --- a/core/modules/views/src/Plugin/views/join/JoinPluginBase.php +++ b/core/modules/views/src/Plugin/views/join/JoinPluginBase.php @@ -303,15 +303,13 @@ public function buildJoin($select_query, $table, $view_query) { if (is_array($info['value']) && count($info['value']) == 1) { $info['value'] = array_shift($info['value']); } - $use_parenthesis = FALSE; if (is_array($info['value'])) { // We use an SA-CORE-2014-005 conformant placeholder for our array // of values. Also, note that the 'IN' operator is implicit. // @see https://www.drupal.org/node/2401615. - $placeholder = ':views_join_condition_' . $select_query->nextPlaceholder() . '[]'; + $placeholder = '( :views_join_condition_' . $select_query->nextPlaceholder() . '[] )'; $operator = !empty($info['operator']) ? $info['operator'] : 'IN'; $arguments[$placeholder] = $info['value']; - $use_parenthesis = TRUE; } else { // With a single value, the '=' operator is implicit. @@ -337,12 +335,7 @@ public function buildJoin($select_query, $table, $view_query) { $arguments[$placeholder] = $info['value']; } // Render out the SQL fragment with parameters. - if ($use_parenthesis) { - $extras[] = "$join_table_field $operator ( $placeholder )"; - } - else { - $extras[] = "$join_table_field $operator $placeholder"; - } + $extras[] = "$join_table_field $operator $placeholder"; } if ($extras) {