1
|
|
|
<?php |
2
|
|
|
|
3
|
|
|
namespace Stockbase\Integration\Cron; |
4
|
|
|
|
5
|
|
|
use Psr\Log\LoggerInterface; |
6
|
|
|
use Magento\Catalog\Model\ResourceModel\Product\CollectionFactory as ProductCollection; |
7
|
|
|
use Stockbase\Integration\StockbaseApi\Client\StockbaseClientFactory; |
8
|
|
|
use Stockbase\Integration\Model\Config\StockbaseConfiguration; |
9
|
|
|
use Stockbase\Integration\Model\ResourceModel\ProductImage as ProductImageResource; |
10
|
|
|
use Stockbase\Integration\Helper\Images as ImagesHelper; |
11
|
|
|
|
12
|
|
|
/** |
13
|
|
|
* Stockbase images synchronization cron job. |
14
|
|
|
*/ |
15
|
|
|
class Images |
16
|
|
|
{ |
17
|
|
|
|
18
|
|
|
/** |
19
|
|
|
* @var LoggerInterface |
20
|
|
|
*/ |
21
|
|
|
private $logger; |
22
|
|
|
/** |
23
|
|
|
* @var StockbaseClientFactory |
24
|
|
|
*/ |
25
|
|
|
private $stockbaseClientFactory; |
26
|
|
|
/** |
27
|
|
|
* @var StockbaseConfiguration |
28
|
|
|
*/ |
29
|
|
|
private $config; |
30
|
|
|
/** |
31
|
|
|
* @var ProductImageResource |
32
|
|
|
*/ |
33
|
|
|
private $productImageResource; |
34
|
|
|
/** |
35
|
|
|
* @var ImagesHelper |
36
|
|
|
*/ |
37
|
|
|
private $imagesHelper; |
38
|
|
|
/** |
39
|
|
|
* @var ProductCollection |
40
|
|
|
*/ |
41
|
|
|
private $productCollection; |
42
|
|
|
/** |
43
|
|
|
* @var array |
44
|
|
|
*/ |
45
|
|
|
private $eans = array(); |
46
|
|
|
|
47
|
|
|
/** |
48
|
|
|
* Images constructor. |
49
|
|
|
* @param LoggerInterface $logger |
50
|
|
|
* @param StockbaseClientFactory $stockbaseClientFactory |
51
|
|
|
* @param StockbaseConfiguration $config |
52
|
|
|
* @param ProductCollection $productCollection |
53
|
|
|
* @param ProductImageResource $productImageResource |
54
|
|
|
* @param ImagesHelper $imagesHelper |
55
|
|
|
*/ |
56
|
|
|
public function __construct( |
57
|
|
|
LoggerInterface $logger, |
58
|
|
|
StockbaseClientFactory $stockbaseClientFactory, |
59
|
|
|
StockbaseConfiguration $config, |
60
|
|
|
ProductCollection $productCollection, |
61
|
|
|
ProductImageResource $productImageResource, |
62
|
|
|
ImagesHelper $imagesHelper |
63
|
|
|
) { |
64
|
|
|
$this->logger = $logger; |
65
|
|
|
$this->stockbaseClientFactory = $stockbaseClientFactory; |
66
|
|
|
$this->config = $config; |
67
|
|
|
$this->productCollection = $productCollection; |
68
|
|
|
$this->productImageResource = $productImageResource; |
69
|
|
|
$this->imagesHelper = $imagesHelper; |
70
|
|
|
} |
71
|
|
|
|
72
|
|
|
/** |
73
|
|
|
* Executes the job. |
74
|
|
|
*/ |
75
|
|
|
public function execute() |
76
|
|
|
{ |
77
|
|
|
// validate configuration: |
78
|
|
|
if (!$this->config->isModuleEnabled() || !$this->config->isImagesSyncCronEnabled()) { |
79
|
|
|
return; |
80
|
|
|
} |
81
|
|
|
// start process: |
82
|
|
|
$this->logger->info('Synchronizing Stockbase images...'); |
83
|
|
|
// get all the eans: |
84
|
|
|
$eans = $this->getEansToProcess(); |
85
|
|
|
try { |
86
|
|
|
// if still need to process eans: |
87
|
|
|
if(count($eans) > 0) { |
88
|
|
|
$client = $this->stockbaseClientFactory->create(); |
89
|
|
|
$images = $client->getImages($eans); |
90
|
|
|
// validate returned images: |
91
|
|
View Code Duplication |
if(is_array($images->{'Items'}) && count($images->{'Items'}) > 0) { |
|
|
|
|
92
|
|
|
// download and save the images locally: |
93
|
|
|
$newImagesCount = $this->imagesHelper->saveProductImages($images->{'Items'}); |
94
|
|
|
$this->logger->info('New synchronized images: '.$newImagesCount); |
95
|
|
|
} |
96
|
|
|
} |
97
|
|
|
} catch (Exception $e) { |
|
|
|
|
98
|
|
|
$this->logger->info('Cron runImageImport error: '.$e->getMessage()); |
99
|
|
|
return false; |
100
|
|
|
} |
101
|
|
|
$this->logger->info('Stockbase images synchronization complete.'); |
102
|
|
|
} |
103
|
|
|
|
104
|
|
|
/** |
105
|
|
|
* @return array |
106
|
|
|
*/ |
107
|
|
|
private function getEansToProcess() |
108
|
|
|
{ |
109
|
|
|
// clean eans list: |
110
|
|
|
$this->eans = array(); |
111
|
|
|
// start process: |
112
|
|
|
$this->logger->info('Get All EANs'); |
113
|
|
|
// get ean attribute: |
114
|
|
|
$attribute = $this->config->getEanFieldName(); |
115
|
|
|
// validate attribute: |
116
|
|
|
if($attribute) { |
117
|
|
|
// create collection and apply filters: |
118
|
|
|
$collection = $this->productCollection->create() |
119
|
|
|
->addAttributeToSelect($attribute) |
120
|
|
|
->addAttributeToSelect('stockbase_product') |
121
|
|
|
->addAttributeToFilter('stockbase_product', array('eq' => '1')) // only stockbase products |
122
|
|
|
->addAttributeToFilter($attribute, array('notnull' => true, 'neq' => '')) // not null and not empty ean |
123
|
|
|
; |
124
|
|
|
// if the filter is active then exclude the eans processed: |
125
|
|
|
if($this->config->filterProcessedProducts()) { |
126
|
|
|
// get eans list: |
127
|
|
|
$processedEans = $this->productImageResource->getProcessedEans(); |
128
|
|
|
// add eans filter: |
129
|
|
|
if(count($processedEans) > 0) { |
130
|
|
|
$this->logger->info('Filtered EANs: '.count($processedEans)); |
131
|
|
|
$collection->addAttributeToFilter($attribute, array('nin' => $processedEans)); |
132
|
|
|
} |
133
|
|
|
} |
134
|
|
|
// walk collection and save eans in the object: |
135
|
|
|
$collection->walk(array($this, 'getProductEan')); |
136
|
|
|
// log eans count: |
137
|
|
|
$this->logger->info('EANs to process: '.count($this->eans)); |
138
|
|
|
} else { |
139
|
|
|
// missing ean attribute: |
140
|
|
|
$this->logger->info('Please setup the EAN attribute.'); |
141
|
|
|
} |
142
|
|
|
return $this->eans; |
143
|
|
|
} |
144
|
|
|
|
145
|
|
|
/** |
146
|
|
|
* @param $product |
147
|
|
|
*/ |
148
|
|
|
public function getProductEan($product) |
149
|
|
|
{ |
150
|
|
|
// get ean attribute: |
151
|
|
|
$attribute = $this->config->getEanFieldName(); |
152
|
|
|
// get ean: |
153
|
|
|
$ean = $product->getData($attribute); |
154
|
|
|
// if the ean is not empty: |
155
|
|
|
if ($ean) { |
156
|
|
|
// add the ean if this product has one: |
157
|
|
|
$this->eans[] = $ean; |
158
|
|
|
} |
159
|
|
|
} |
160
|
|
|
|
161
|
|
|
} |
162
|
|
|
|
Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.
You can also find more detailed suggestions in the “Code” section of your repository.