Skip to content

Commit cdeb7c7

Browse files
committed
MAGETWO-70056: Get sitemap product images from image cache, if available #9082
- Coding Style updated
1 parent de706ec commit cdeb7c7

File tree

2 files changed

+6
-11
lines changed

2 files changed

+6
-11
lines changed

app/code/Magento/Sitemap/Model/ResourceModel/Catalog/Product.php

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,12 @@
1010

1111
use Magento\CatalogUrlRewrite\Model\ProductUrlRewriteGenerator;
1212
use Magento\Store\Model\Store;
13+
use Magento\Framework\App\ObjectManager;
1314

1415
/**
1516
* Sitemap resource product collection model
1617
*
17-
* @author Magento Core Team <core@magentocommerce.com>
1818
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
19-
* @SuppressWarnings(PHPMD.LongVariable)
20-
* @SuppressWarnings(PHPMD.CamelCasePropertyName)
21-
* @SuppressWarnings(PHPMD.CamelCaseMethodName)
2219
*/
2320
class Product extends \Magento\Framework\Model\ResourceModel\Db\AbstractDb
2421
{
@@ -84,12 +81,12 @@ class Product extends \Magento\Framework\Model\ResourceModel\Db\AbstractDb
8481
/**
8582
* @var \Magento\Catalog\Model\Product
8683
*/
87-
protected $productModel;
84+
private $productModel;
8885

8986
/**
9087
* @var \Magento\Catalog\Helper\Image
9188
*/
92-
protected $catalogImageHelper;
89+
private $catalogImageHelper;
9390

9491
/**
9592
* @param \Magento\Framework\Model\ResourceModel\Db\Context $context
@@ -128,8 +125,9 @@ public function __construct(
128125
$this->mediaGalleryReadHandler = $mediaGalleryReadHandler;
129126
$this->_mediaConfig = $mediaConfig;
130127
$this->_sitemapData = $sitemapData;
131-
$this->productModel = $productModel ?: \Magento\Framework\App\ObjectManager::getInstance()->get(\Magento\Catalog\Model\Product::class);
132-
$this->catalogImageHelper = $catalogImageHelper ?: \Magento\Framework\App\ObjectManager::getInstance()->get(\Magento\Catalog\Helper\Image::class);
128+
$this->productModel = $productModel ?: ObjectManager::getInstance()->get(\Magento\Catalog\Model\Product::class);
129+
$this->catalogImageHelper = $catalogImageHelper ?: ObjectManager::getInstance()
130+
->get(\Magento\Catalog\Helper\Image::class);
133131
parent::__construct($context, $connectionName);
134132
}
135133

app/code/Magento/Sitemap/Model/Sitemap.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,6 @@
2727
* @method \Magento\Sitemap\Model\Sitemap setStoreId(int $value)
2828
* @SuppressWarnings(PHPMD.TooManyFields)
2929
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
30-
* @SuppressWarnings(PHPMD.ExcessiveClassComplexity)
31-
* @SuppressWarnings(PHPMD.CamelCasePropertyName)
32-
* @SuppressWarnings(PHPMD.CamelCaseMethodName)
3330
*/
3431
class Sitemap extends \Magento\Framework\Model\AbstractModel
3532
{

0 commit comments

Comments
 (0)