Skip to content
Snippets Groups Projects
Unverified Commit 3663cc5b authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3221933 by marcoscano, alexpott, owenbush: PHP Notice when using...

Issue #3221933 by marcoscano, alexpott, owenbush: PHP Notice when using "left_formula" in views join

(cherry picked from commit 0926b0cc)
parent da276a38
Branches
Tags
9 merge requests!10011Issue #3200534 by quietone, longwave, Kristen Pol: Use dataprovider for...,!2571Issue #3000717: Missing mapping for "nodereference_url" widget,!2521Issue #3185775: Place Views preview on the side on large monitors,!1603Issue #3231707: mxr576's core patch playground,!1479Issue #3250298: Return empty string "" with JSON Serializer instead of FALSE,!1478Issue #3250298: Return empty string "" with JSON Serializer instead of FALSE,!1203Issue #3236191 Wrong group exposed form widgets and multiple selection error.,!1076Issue #2903336 Added node context for tokens.,!1015Issue #3226944: REST's Request handler doesn't resolve $data argument for put method
......@@ -266,7 +266,11 @@ public function __construct(array $configuration, $plugin_id, $plugin_definition
}
$this->leftTable = $configuration['left_table'];
$this->leftField = $configuration['left_field'];
if (!empty($configuration['left_field'])) {
$this->leftField = $configuration['left_field'];
}
$this->field = $configuration['field'];
if (!empty($configuration['left_formula'])) {
......
......@@ -205,6 +205,8 @@ public function testBasePlugin() {
// Test that joins using 'left_formula' are properly built.
$configuration['left_formula'] = 'MAX(views_test_data.uid)';
// When 'left_formula' is present, 'left_field' is no longer required.
unset($configuration['left_field']);
$join = $this->manager->createInstance('standard', $configuration);
$table = ['alias' => 'users6'];
$join->buildJoin($query, $table, $view->query);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment