Skip to content

Commit 63523e5

Browse files
author
vpaladiychuk
committed
Merge remote-tracking branch 'origin/MAGETWO-26762' into MAGETWO-34988
2 parents 2d1e252 + 0e8eb6b commit 63523e5

File tree

21 files changed

+848
-178
lines changed

21 files changed

+848
-178
lines changed

app/code/Magento/Catalog/Model/Observer.php

Lines changed: 26 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@
55
*/
66
namespace Magento\Catalog\Model;
77

8+
/**
9+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
10+
*/
811
class Observer
912
{
1013
/**
@@ -62,16 +65,23 @@ class Observer
6265
protected $_productResourceFactory;
6366

6467
/**
65-
* @param \Magento\Catalog\Model\Resource\Category $categoryResource
66-
* @param \Magento\Catalog\Model\Resource\Product $catalogProduct
68+
* @var \Magento\Framework\Registry
69+
*/
70+
protected $_registry;
71+
72+
/**
73+
* @param \Magento\Framework\Registry $registry
74+
* @param Resource\Category $categoryResource
75+
* @param Resource\Product $catalogProduct
6776
* @param \Magento\Store\Model\StoreManagerInterface $storeManager
68-
* @param \Magento\Catalog\Model\Layer\Resolver $layerResolver
77+
* @param Layer\Resolver $layerResolver
6978
* @param \Magento\Catalog\Helper\Category $catalogCategory
7079
* @param \Magento\Catalog\Helper\Data $catalogData
7180
* @param Indexer\Category\Flat\State $categoryFlatState
72-
* @param \Magento\Catalog\Model\Resource\ProductFactory $productResourceFactory
81+
* @param Resource\ProductFactory $productResourceFactory
7382
*/
7483
public function __construct(
84+
\Magento\Framework\Registry $registry,
7585
\Magento\Catalog\Model\Resource\Category $categoryResource,
7686
\Magento\Catalog\Model\Resource\Product $catalogProduct,
7787
\Magento\Store\Model\StoreManagerInterface $storeManager,
@@ -81,6 +91,7 @@ public function __construct(
8191
\Magento\Catalog\Model\Indexer\Category\Flat\State $categoryFlatState,
8292
\Magento\Catalog\Model\Resource\ProductFactory $productResourceFactory
8393
) {
94+
$this->_registry = $registry;
8495
$this->_categoryResource = $categoryResource;
8596
$this->_catalogProduct = $catalogProduct;
8697
$this->_storeManager = $storeManager;
@@ -137,11 +148,20 @@ protected function _addCategoriesToMenu($categories, $parentCategoryNode, $block
137148
$block->addIdentity(\Magento\Catalog\Model\Category::CACHE_TAG . '_' . $category->getId());
138149

139150
$tree = $parentCategoryNode->getTree();
151+
152+
$isActiveCategory = false;
153+
/** @var \Magento\Catalog\Model\Category $currentCategory */
154+
$currentCategory = $this->_registry->registry('current_category');
155+
if ($currentCategory && $currentCategory->getId() == $category->getId()) {
156+
$isActiveCategory = true;
157+
}
158+
140159
$categoryData = [
141160
'name' => $category->getName(),
142161
'id' => $nodeId,
143162
'url' => $this->_catalogCategory->getCategoryUrl($category),
144-
'is_active' => $this->_isActiveMenuCategory($category),
163+
'has_active' => $this->hasActive($category),
164+
'is_active' => $isActiveCategory
145165
];
146166
$categoryNode = new \Magento\Framework\Data\Tree\Node($categoryData, 'id', $tree, $parentCategoryNode);
147167
$parentCategoryNode->addChild($categoryNode);
@@ -162,7 +182,7 @@ protected function _addCategoriesToMenu($categories, $parentCategoryNode, $block
162182
* @param \Magento\Framework\Data\Tree\Node $category
163183
* @return bool
164184
*/
165-
protected function _isActiveMenuCategory($category)
185+
protected function hasActive($category)
166186
{
167187
if (!$this->_catalogLayer) {
168188
return false;

app/code/Magento/Catalog/Test/Unit/Controller/Adminhtml/ProductTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ protected function initContext(array $additionalParams = [])
9393
$this->context->expects($this->any())->method('getActionFlag')->will($this->returnValue($actionFlagMock));
9494
$this->context->expects($this->any())->method('getHelper')->will($this->returnValue($helperDataMock));
9595

96-
foreach($additionalParams as $property => $object) {
96+
foreach ($additionalParams as $property => $object) {
9797
$this->context->expects($this->any())->method('get' . ucfirst($property))->willReturn($object);
9898
}
9999

app/code/Magento/Downloadable/view/adminhtml/templates/product/edit/downloadable.phtml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ require([
2626
var uploaderTemplate = '<div class="no-display" id="[[idName]]-template">' +
2727
'<div id="<%- data.id %>" class="file-row file-row-narrow">' +
2828
'<span class="file-info">' +
29-
'<span class="file-info-name"><%- data.name %></span>' +
29+
'<span class="file-info-name"><%= data.name %></span>' +
3030
' ' +
3131
'<span class="file-info-size">(<%- data.size %>)</span>' +
3232
'</span>' +
@@ -41,7 +41,7 @@ var uploaderTemplate = '<div class="no-display" id="[[idName]]-template">' +
4141
'</div>';
4242

4343
var fileListTemplate = '<span class="file-info">' +
44-
'<span class="file-info-name"><%- data.name %></span>' +
44+
'<span class="file-info-name"><%= data.name %></span>' +
4545
' ' +
4646
'<span class="file-info-size">(<%- data.size %>)</span>' +
4747
'</span>';

0 commit comments

Comments
 (0)