Skip to content

Commit b39565c

Browse files
committed
MAGETWO-49469: Contribute Ogre Bugfixes and Stories for Sprint 44
- reverting BiC changes
1 parent 0b4cb8c commit b39565c

File tree

2 files changed

+121
-12
lines changed
  • app/code/Magento

2 files changed

+121
-12
lines changed

app/code/Magento/CatalogSearch/Model/ResourceModel/Fulltext/Collection.php

Lines changed: 109 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,43 @@
88
use Magento\Framework\DB\Select;
99
use Magento\Framework\Exception\StateException;
1010
use Magento\Framework\Search\Adapter\Mysql\TemporaryStorage;
11+
use Magento\Framework\Search\Response\QueryResponse;
12+
use Magento\Framework\App\ObjectManager;
1113

1214
/**
1315
* Fulltext Collection
1416
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
1517
*/
1618
class Collection extends \Magento\Catalog\Model\ResourceModel\Product\Collection
1719
{
18-
/**
19-
* @var string
20+
/**
21+
* @var QueryResponse
22+
* @deprecated
23+
*/
24+
protected $queryResponse;
25+
26+
/**
27+
* Catalog search data
28+
*
29+
* @var \Magento\Search\Model\QueryFactory
30+
* @deprecated
31+
*/
32+
protected $queryFactory = null;
33+
34+
/**
35+
* @var \Magento\Framework\Search\Request\Builder
36+
* @deprecated
37+
*/
38+
private $requestBuilder;
39+
40+
/**
41+
* @var \Magento\Search\Model\SearchEngine
42+
* @deprecated
43+
*/
44+
private $searchEngine;
45+
46+
/**
47+
* @var string
2048
*/
2149
private $queryText;
2250

@@ -76,11 +104,11 @@ class Collection extends \Magento\Catalog\Model\ResourceModel\Product\Collection
76104
* @param \Magento\Framework\Stdlib\DateTime $dateTime
77105
* @param \Magento\Customer\Api\GroupManagementInterface $groupManagement
78106
* @param \Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitation $productLimitation
107+
* @param \Magento\Search\Model\QueryFactory $catalogSearchData
108+
* @param \Magento\Framework\Search\Request\Builder $requestBuilder
109+
* @param \Magento\Search\Model\SearchEngine $searchEngine
79110
* @param \Magento\Framework\Search\Adapter\Mysql\TemporaryStorageFactory $temporaryStorageFactory
80-
* @param \Magento\Search\Api\SearchInterface $search
81-
* @param \Magento\Framework\Api\Search\SearchCriteriaBuilder $searchCriteriaBuilder
82-
* @param \Magento\Framework\Api\FilterBuilder $filterBuilder
83-
* @param \Magento\Framework\DB\Adapter\AdapterInterface|null $connection
111+
* @param \Magento\Framework\DB\Adapter\AdapterInterface $connection
84112
* @param string $searchRequestName
85113
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
86114
*/
@@ -105,13 +133,14 @@ public function __construct(
105133
\Magento\Framework\Stdlib\DateTime $dateTime,
106134
\Magento\Customer\Api\GroupManagementInterface $groupManagement,
107135
\Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitation $productLimitation,
136+
\Magento\Search\Model\QueryFactory $catalogSearchData,
137+
\Magento\Framework\Search\Request\Builder $requestBuilder,
138+
\Magento\Search\Model\SearchEngine $searchEngine,
108139
\Magento\Framework\Search\Adapter\Mysql\TemporaryStorageFactory $temporaryStorageFactory,
109-
\Magento\Search\Api\SearchInterface $search,
110-
\Magento\Framework\Api\Search\SearchCriteriaBuilder $searchCriteriaBuilder,
111-
\Magento\Framework\Api\FilterBuilder $filterBuilder,
112140
\Magento\Framework\DB\Adapter\AdapterInterface $connection = null,
113141
$searchRequestName = 'catalog_view_container'
114142
) {
143+
$this->queryFactory = $catalogSearchData;
115144
parent::__construct(
116145
$entityFactory,
117146
$logger,
@@ -135,11 +164,73 @@ public function __construct(
135164
$productLimitation,
136165
$connection
137166
);
167+
$this->requestBuilder = $requestBuilder;
168+
$this->searchEngine = $searchEngine;
138169
$this->temporaryStorageFactory = $temporaryStorageFactory;
139170
$this->searchRequestName = $searchRequestName;
140-
$this->search = $search;
141-
$this->searchCriteriaBuilder = $searchCriteriaBuilder;
142-
$this->filterBuilder = $filterBuilder;
171+
}
172+
173+
174+
/**
175+
* @deprecated
176+
*/
177+
private function getSearch()
178+
{
179+
if ($this->search !== null) {
180+
$this->search = ObjectManager::getInstance()->get('\Magento\Search\Api\SearchInterface');
181+
}
182+
return $this->search;
183+
}
184+
185+
/**
186+
* @deprecated
187+
* @param \Magento\Search\Api\SearchInterface $object
188+
*/
189+
public function setSearch($object)
190+
{
191+
$this->search = $object;
192+
}
193+
194+
/**
195+
* @deprecated
196+
*/
197+
private function getSearchCriteriaBuilder()
198+
{
199+
if ($this->searchCriteriaBuilder !== null) {
200+
$this->searchCriteriaBuilder = ObjectManager::getInstance()
201+
->get('\Magento\Framework\Api\Search\SearchCriteriaBuilder');
202+
}
203+
return $this->searchCriteriaBuilder;
204+
}
205+
206+
/**
207+
* @deprecated
208+
* @param \Magento\Framework\Api\Search\SearchCriteriaBuilder $object
209+
*/
210+
public function setSearchCriteriaBuilder($object)
211+
{
212+
$this->searchCriteriaBuilder = $object;
213+
}
214+
215+
216+
/**
217+
* @deprecated
218+
*/
219+
private function getFilterBuilder()
220+
{
221+
if ($this->filterBuilder !== null) {
222+
$this->filterBuilder = ObjectManager::getInstance()->get('\Magento\Framework\Api\FilterBuilder');
223+
}
224+
return $this->filterBuilder;
225+
}
226+
227+
/**
228+
* @deprecated
229+
* @param \Magento\Framework\Api\FilterBuilder $object
230+
*/
231+
public function setFilterBuilder($object)
232+
{
233+
$this->filterBuilder = $object;
143234
}
144235

145236
/**
@@ -155,6 +246,8 @@ public function addFieldToFilter($field, $condition = null)
155246
throw new \RuntimeException('Illegal state');
156247
}
157248

249+
$this->getSearchCriteriaBuilder();
250+
$this->getFilterBuilder();
158251
if (!is_array($condition) || !in_array(key($condition), ['from', 'to'])) {
159252
$this->filterBuilder->setField($field);
160253
$this->filterBuilder->setValue($condition);
@@ -191,6 +284,10 @@ public function addSearchFilter($query)
191284
*/
192285
protected function _renderFiltersBefore()
193286
{
287+
$this->getSearchCriteriaBuilder();
288+
$this->getFilterBuilder();
289+
$this->getSearch();
290+
194291
if ($this->queryText) {
195292
$this->filterBuilder->setField('search_term');
196293
$this->filterBuilder->setValue($this->queryText);

app/code/Magento/PageCache/Block/System/Config/Form/Field/Export.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,4 +64,16 @@ protected function _getUrl($params = [])
6464
{
6565
return $this->getUrl('*/PageCache/exportVarnishConfig', $params);
6666
}
67+
68+
/**
69+
* Return PageCache TTL value from config
70+
* to avoid saving empty field
71+
*
72+
* @return string
73+
* @deprecated
74+
*/
75+
public function getTtlValue()
76+
{
77+
return $this->_scopeConfig->getValue(\Magento\PageCache\Model\Config::XML_PAGECACHE_TTL);
78+
}
6779
}

0 commit comments

Comments
 (0)