Skip to content

Commit 7fa3789

Browse files
committed
MAGETWO-70279: Issue with the config merging introduced for the Analytics integration
1 parent 68a79d9 commit 7fa3789

File tree

15 files changed

+132
-476
lines changed

15 files changed

+132
-476
lines changed

app/code/Magento/Analytics/Model/Config/Data.php

Lines changed: 0 additions & 29 deletions
This file was deleted.

app/code/Magento/Analytics/Model/Config/Reader/Xml.php

Lines changed: 0 additions & 61 deletions
This file was deleted.

app/code/Magento/Analytics/Model/Config/SchemaLocator.php

Lines changed: 0 additions & 56 deletions
This file was deleted.

app/code/Magento/Analytics/ReportXml/Config.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
*/
66
namespace Magento\Analytics\ReportXml;
77

8-
use Magento\Analytics\ReportXml\Config\Data;
8+
use Magento\Framework\Config\DataInterface;
99

1010
/**
1111
* Class Config
@@ -15,17 +15,17 @@
1515
class Config implements ConfigInterface
1616
{
1717
/**
18-
* @var Data
18+
* @var DataInterface
1919
*/
2020
private $data;
2121

2222
/**
2323
* Config constructor.
2424
*
25-
* @param Data $data
25+
* @param DataInterface $data
2626
*/
2727
public function __construct(
28-
Data $data
28+
DataInterface $data
2929
) {
3030
$this->data = $data;
3131
}

app/code/Magento/Analytics/ReportXml/Config/Data.php

Lines changed: 0 additions & 29 deletions
This file was deleted.

app/code/Magento/Analytics/ReportXml/Config/Reader/Xml.php

Lines changed: 0 additions & 56 deletions
This file was deleted.

app/code/Magento/Analytics/ReportXml/Config/SchemaLocator.php

Lines changed: 0 additions & 51 deletions
This file was deleted.

app/code/Magento/Analytics/ReportXml/DB/ConditionResolver.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ private function getCondition(SelectBuilder $selectBuilder, $tableName, $conditi
116116
) {
117117
$expression = $columns[$condition['attribute']];
118118
} else {
119-
$expression = $tableName . '.' . $condition['attribute'];
119+
$expression = $this->getConnection()->quoteIdentifier($tableName . '.' . $condition['attribute']);
120120
}
121121
return sprintf(
122122
$this->conditionMap[$condition['operator']],
@@ -161,6 +161,6 @@ public function getFilter(SelectBuilder $selectBuilder, $filterConfig, $aliasNam
161161
}
162162
$filtersParts[] = '(' . implode(' ' . strtoupper($glue) . ' ', $parts) . ')';
163163
}
164-
return implode(' AND ', $filtersParts);
164+
return implode(' OR ', $filtersParts);
165165
}
166166
}

app/code/Magento/Analytics/ReportXml/ReportProvider.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,11 +64,9 @@ private function getIteratorName(Query $query)
6464
* Returns report data by name and criteria
6565
*
6666
* @param string $name
67-
* @param SearchCriteria|null $criteria
6867
* @return \IteratorIterator
69-
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
7068
*/
71-
public function getReport($name, SearchCriteria $criteria = null)
69+
public function getReport($name)
7270
{
7371
$query = $this->queryFactory->create($name);
7472
$connection = $this->connectionFactory->getConnection($query->getConnectionName());

0 commit comments

Comments
 (0)