'extra' join conditions leaking arguments

When specifying a views relationship, you can add extra conditions to the resulting join as such:

$data['my_table']['my_field']['relationship'] = array(
  'id' => 'standard',
  'base' => 'other_table',
  'base field' => 'other_field',
  'extra' => array(
    array(
      'field' => 'this',
      'value' => 1,
    ),
    array(
      'field' => 'that',
      'value' => array(3,4,5),
    ),
  ),
);

The above extra array should produce a join something like:

LEFT JOIN `other_table` ON `my_table`.`my_field` = `other_table`.`other_field` AND (`other_table`.`this` = 1 AND `other_table`.`that` IN (3, 4, 5))

However, currently it produces this:

LEFT JOIN `other_table` ON `my_table`.`my_field` = `other_table`.`other_field` AND (`other_table`.`this` = 1 AND `other_table`.`that` IN (1, 3, 4, 5))

Note that the argument from the first additional join condition is finding its way into the second additional join's list of arguments.

This is caused by core/modules/views/src/Plugin/views/join/JoinPluginBase.php on about line 257:

$placeholder = '( ' . implode(', ', array_keys($arguments)) . ' )';

The $arguments array is cumulative for all extra join conditions, meaning that any previous join conditions are imploded here for the current join condition. Instead, what is needed here is a local arguments list for the current join condition.