@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | */ |
29 | 29 | public function getName() |
30 | 30 | { |
31 | - return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Product site map' ); |
|
31 | + return $this->getContext()->getI18n()->dt('controller/jobs', 'Product site map'); |
|
32 | 32 | } |
33 | 33 | |
34 | 34 | |
@@ -39,7 +39,7 @@ discard block |
||
39 | 39 | */ |
40 | 40 | public function getDescription() |
41 | 41 | { |
42 | - return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Creates a product site map for search engines' ); |
|
42 | + return $this->getContext()->getI18n()->dt('controller/jobs', 'Creates a product site map for search engines'); |
|
43 | 43 | } |
44 | 44 | |
45 | 45 | |
@@ -52,8 +52,8 @@ discard block |
||
52 | 52 | { |
53 | 53 | $container = $this->createContainer(); |
54 | 54 | |
55 | - $files = $this->export( $container ); |
|
56 | - $this->createSitemapIndex( $container, $files ); |
|
55 | + $files = $this->export($container); |
|
56 | + $this->createSitemapIndex($container, $files); |
|
57 | 57 | |
58 | 58 | $container->close(); |
59 | 59 | } |
@@ -65,7 +65,7 @@ discard block |
||
65 | 65 | * @param \Aimeos\MW\Container\Content\Iface $content File content object |
66 | 66 | * @param \Aimeos\MShop\Product\Item\Iface[] $items List of product items |
67 | 67 | */ |
68 | - protected function addItems( \Aimeos\MW\Container\Content\Iface $content, array $items ) |
|
68 | + protected function addItems(\Aimeos\MW\Container\Content\Iface $content, array $items) |
|
69 | 69 | { |
70 | 70 | $config = $this->getContext()->getConfig(); |
71 | 71 | |
@@ -96,7 +96,7 @@ discard block |
||
96 | 96 | * @see controller/jobs/product/export/sitemap/max-items |
97 | 97 | * @see controller/jobs/product/export/sitemap/max-query |
98 | 98 | */ |
99 | - $changefreq = $config->get( 'controller/jobs/product/export/sitemap/changefreq', 'daily' ); |
|
99 | + $changefreq = $config->get('controller/jobs/product/export/sitemap/changefreq', 'daily'); |
|
100 | 100 | |
101 | 101 | /** controller/jobs/product/export/sitemap/standard/template-items |
102 | 102 | * Relative path to the XML items template of the product site map job controller. |
@@ -129,7 +129,7 @@ discard block |
||
129 | 129 | $view->siteItems = $items; |
130 | 130 | $view->siteFreq = $changefreq; |
131 | 131 | |
132 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
132 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
133 | 133 | } |
134 | 134 | |
135 | 135 | |
@@ -175,7 +175,7 @@ discard block |
||
175 | 175 | * @see controller/jobs/product/export/sitemap/max-query |
176 | 176 | * @see controller/jobs/product/export/sitemap/changefreq |
177 | 177 | */ |
178 | - $location = $config->get( 'controller/jobs/product/export/sitemap/location' ); |
|
178 | + $location = $config->get('controller/jobs/product/export/sitemap/location'); |
|
179 | 179 | |
180 | 180 | /** controller/jobs/product/export/sitemap/container/options |
181 | 181 | * List of file container options for the site map files |
@@ -196,16 +196,16 @@ discard block |
||
196 | 196 | * @see controller/jobs/product/export/sitemap/max-query |
197 | 197 | * @see controller/jobs/product/export/sitemap/changefreq |
198 | 198 | */ |
199 | - $default = array( 'gzip-mode' => 'wb' ); |
|
200 | - $options = $config->get( 'controller/jobs/product/export/sitemap/container/options', $default ); |
|
199 | + $default = array('gzip-mode' => 'wb'); |
|
200 | + $options = $config->get('controller/jobs/product/export/sitemap/container/options', $default); |
|
201 | 201 | |
202 | - if( $location === null ) |
|
202 | + if ($location === null) |
|
203 | 203 | { |
204 | - $msg = sprintf( 'Required configuration for "%1$s" is missing', 'controller/jobs/product/export/sitemap/location' ); |
|
205 | - throw new \Aimeos\Controller\Jobs\Exception( $msg ); |
|
204 | + $msg = sprintf('Required configuration for "%1$s" is missing', 'controller/jobs/product/export/sitemap/location'); |
|
205 | + throw new \Aimeos\Controller\Jobs\Exception($msg); |
|
206 | 206 | } |
207 | 207 | |
208 | - return \Aimeos\MW\Container\Factory::getContainer( $location, 'Directory', 'Gzip', $options ); |
|
208 | + return \Aimeos\MW\Container\Factory::getContainer($location, 'Directory', 'Gzip', $options); |
|
209 | 209 | } |
210 | 210 | |
211 | 211 | |
@@ -216,7 +216,7 @@ discard block |
||
216 | 216 | * @param integer $filenum New file number |
217 | 217 | * @return \Aimeos\MW\Container\Content\Iface New content object |
218 | 218 | */ |
219 | - protected function createContent( \Aimeos\MW\Container\Iface $container, $filenum ) |
|
219 | + protected function createContent(\Aimeos\MW\Container\Iface $container, $filenum) |
|
220 | 220 | { |
221 | 221 | /** controller/jobs/product/export/sitemap/standard/template-header |
222 | 222 | * Relative path to the XML site map header template of the product site map job controller. |
@@ -246,9 +246,9 @@ discard block |
||
246 | 246 | $context = $this->getContext(); |
247 | 247 | $view = $context->getView(); |
248 | 248 | |
249 | - $content = $container->create( $this->getFilename( $filenum ) ); |
|
250 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
251 | - $container->add( $content ); |
|
249 | + $content = $container->create($this->getFilename($filenum)); |
|
250 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
251 | + $container->add($content); |
|
252 | 252 | |
253 | 253 | return $content; |
254 | 254 | } |
@@ -259,7 +259,7 @@ discard block |
||
259 | 259 | * |
260 | 260 | * @param \Aimeos\MW\Container\Content\Iface $content |
261 | 261 | */ |
262 | - protected function closeContent( \Aimeos\MW\Container\Content\Iface $content ) |
|
262 | + protected function closeContent(\Aimeos\MW\Container\Content\Iface $content) |
|
263 | 263 | { |
264 | 264 | /** controller/jobs/product/export/sitemap/standard/template-footer |
265 | 265 | * Relative path to the XML site map footer template of the product site map job controller. |
@@ -289,7 +289,7 @@ discard block |
||
289 | 289 | $context = $this->getContext(); |
290 | 290 | $view = $context->getView(); |
291 | 291 | |
292 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
292 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
293 | 293 | } |
294 | 294 | |
295 | 295 | |
@@ -299,7 +299,7 @@ discard block |
||
299 | 299 | * @param \Aimeos\MW\Container\Iface $container File container object |
300 | 300 | * @param array $files List of generated site map file names |
301 | 301 | */ |
302 | - protected function createSitemapIndex( \Aimeos\MW\Container\Iface $container, array $files ) |
|
302 | + protected function createSitemapIndex(\Aimeos\MW\Container\Iface $container, array $files) |
|
303 | 303 | { |
304 | 304 | /** controller/jobs/product/export/sitemap/standard/template-index |
305 | 305 | * Relative path to the XML site map index template of the product site map job controller. |
@@ -331,9 +331,9 @@ discard block |
||
331 | 331 | |
332 | 332 | $view->siteFiles = $files; |
333 | 333 | |
334 | - $content = $container->create( 'aimeos-sitemap-index.xml' ); |
|
335 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
336 | - $container->add( $content ); |
|
334 | + $content = $container->create('aimeos-sitemap-index.xml'); |
|
335 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
336 | + $container->add($content); |
|
337 | 337 | } |
338 | 338 | |
339 | 339 | |
@@ -344,44 +344,44 @@ discard block |
||
344 | 344 | * @param boolean $default True to filter exported products by default criteria |
345 | 345 | * @return array List of content (file) names |
346 | 346 | */ |
347 | - protected function export( \Aimeos\MW\Container\Iface $container, $default = true ) |
|
347 | + protected function export(\Aimeos\MW\Container\Iface $container, $default = true) |
|
348 | 348 | { |
349 | - $domains = $this->getConfig( 'domains', [] ); |
|
350 | - $maxItems = $this->getConfig( 'max-items', 10000 ); |
|
351 | - $maxQuery = $this->getConfig( 'max-query', 1000 ); |
|
349 | + $domains = $this->getConfig('domains', []); |
|
350 | + $maxItems = $this->getConfig('max-items', 10000); |
|
351 | + $maxQuery = $this->getConfig('max-query', 1000); |
|
352 | 352 | |
353 | 353 | $start = 0; $filenum = 1; |
354 | 354 | $names = []; |
355 | 355 | |
356 | - $manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'index' ); |
|
356 | + $manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'index'); |
|
357 | 357 | |
358 | - $search = $manager->createSearch( $default ); |
|
359 | - $search->setConditions( $search->compare( '!=', 'index.catalog.id', null ) ); |
|
360 | - $search->setSortations( array( $search->sort( '+', 'product.id' ) ) ); |
|
361 | - $search->setSlice( 0, $maxQuery ); |
|
358 | + $search = $manager->createSearch($default); |
|
359 | + $search->setConditions($search->compare('!=', 'index.catalog.id', null)); |
|
360 | + $search->setSortations(array($search->sort('+', 'product.id'))); |
|
361 | + $search->setSlice(0, $maxQuery); |
|
362 | 362 | |
363 | - $content = $this->createContent( $container, $filenum ); |
|
363 | + $content = $this->createContent($container, $filenum); |
|
364 | 364 | $names[] = $content->getResource(); |
365 | 365 | |
366 | 366 | do |
367 | 367 | { |
368 | - $items = $manager->searchItems( $search, $domains ); |
|
369 | - $this->addItems( $content, $items ); |
|
368 | + $items = $manager->searchItems($search, $domains); |
|
369 | + $this->addItems($content, $items); |
|
370 | 370 | |
371 | - $count = count( $items ); |
|
371 | + $count = count($items); |
|
372 | 372 | $start += $count; |
373 | - $search->setSlice( $start, $maxQuery ); |
|
373 | + $search->setSlice($start, $maxQuery); |
|
374 | 374 | |
375 | - if( $start + $maxQuery > $maxItems * $filenum ) |
|
375 | + if ($start + $maxQuery > $maxItems * $filenum) |
|
376 | 376 | { |
377 | - $this->closeContent( $content ); |
|
378 | - $content = $this->createContent( $container, ++$filenum ); |
|
377 | + $this->closeContent($content); |
|
378 | + $content = $this->createContent($container, ++$filenum); |
|
379 | 379 | $names[] = $content->getResource(); |
380 | 380 | } |
381 | 381 | } |
382 | - while( $count >= $search->getSliceSize() ); |
|
382 | + while ($count >= $search->getSliceSize()); |
|
383 | 383 | |
384 | - $this->closeContent( $content ); |
|
384 | + $this->closeContent($content); |
|
385 | 385 | |
386 | 386 | return $names; |
387 | 387 | } |
@@ -394,11 +394,11 @@ discard block |
||
394 | 394 | * @param mixed $default Default value if name is unknown |
395 | 395 | * @return mixed Configuration value |
396 | 396 | */ |
397 | - protected function getConfig( $name, $default = null ) |
|
397 | + protected function getConfig($name, $default = null) |
|
398 | 398 | { |
399 | 399 | $config = $this->getContext()->getConfig(); |
400 | 400 | |
401 | - switch( $name ) |
|
401 | + switch ($name) |
|
402 | 402 | { |
403 | 403 | case 'domain': |
404 | 404 | /** controller/jobs/product/export/sitemap/domains |
@@ -419,7 +419,7 @@ discard block |
||
419 | 419 | * @see controller/jobs/product/export/sitemap/max-query |
420 | 420 | * @see controller/jobs/product/export/sitemap/changefreq |
421 | 421 | */ |
422 | - return $config->get( 'controller/jobs/product/export/sitemap/domains', $default ); |
|
422 | + return $config->get('controller/jobs/product/export/sitemap/domains', $default); |
|
423 | 423 | |
424 | 424 | case 'max-items': |
425 | 425 | /** controller/jobs/product/export/sitemap/max-items |
@@ -444,7 +444,7 @@ discard block |
||
444 | 444 | * @see controller/jobs/product/export/sitemap/changefreq |
445 | 445 | * @see controller/jobs/product/export/sitemap/domains |
446 | 446 | */ |
447 | - return $config->get( 'controller/jobs/product/export/sitemap/max-items', 50000 ); |
|
447 | + return $config->get('controller/jobs/product/export/sitemap/max-items', 50000); |
|
448 | 448 | |
449 | 449 | case 'max-query': |
450 | 450 | /** controller/jobs/product/export/sitemap/max-query |
@@ -465,7 +465,7 @@ discard block |
||
465 | 465 | * @see controller/jobs/product/export/sitemap/changefreq |
466 | 466 | * @see controller/jobs/product/export/sitemap/domains |
467 | 467 | */ |
468 | - return $config->get( 'controller/jobs/product/export/sitemap/max-query', 1000 ); |
|
468 | + return $config->get('controller/jobs/product/export/sitemap/max-query', 1000); |
|
469 | 469 | } |
470 | 470 | |
471 | 471 | return $default; |
@@ -478,8 +478,8 @@ discard block |
||
478 | 478 | * @param integer $number Current file number |
479 | 479 | * @return string New file name |
480 | 480 | */ |
481 | - protected function getFilename( $number ) |
|
481 | + protected function getFilename($number) |
|
482 | 482 | { |
483 | - return sprintf( 'aimeos-sitemap-%d.xml', $number ); |
|
483 | + return sprintf('aimeos-sitemap-%d.xml', $number); |
|
484 | 484 | } |
485 | 485 | } |
@@ -17,16 +17,16 @@ discard block |
||
17 | 17 | |
18 | 18 | protected function setUp() |
19 | 19 | { |
20 | - \Aimeos\MShop\Factory::setCache( true ); |
|
20 | + \Aimeos\MShop\Factory::setCache(true); |
|
21 | 21 | |
22 | 22 | $this->context = \TestHelperCntl::getContext(); |
23 | - $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, [] ); |
|
23 | + $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, []); |
|
24 | 24 | } |
25 | 25 | |
26 | 26 | |
27 | 27 | protected function tearDown() |
28 | 28 | { |
29 | - \Aimeos\MShop\Factory::setCache( false ); |
|
29 | + \Aimeos\MShop\Factory::setCache(false); |
|
30 | 30 | \Aimeos\MShop\Factory::clear(); |
31 | 31 | } |
32 | 32 | |
@@ -49,26 +49,26 @@ discard block |
||
49 | 49 | 4 => '50', |
50 | 50 | ); |
51 | 51 | |
52 | - $product = $this->create( 'job_csv_test' ); |
|
52 | + $product = $this->create('job_csv_test'); |
|
53 | 53 | |
54 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard( $this->context, $mapping, $this->endpoint ); |
|
55 | - $object->process( $product, $data ); |
|
54 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard($this->context, $mapping, $this->endpoint); |
|
55 | + $object->process($product, $data); |
|
56 | 56 | |
57 | 57 | |
58 | 58 | $pos = 0; |
59 | 59 | $expected = array( |
60 | - array( 'package-weight', '3.00', 'de' ), |
|
61 | - array( 'package-width', '50', null ), |
|
60 | + array('package-weight', '3.00', 'de'), |
|
61 | + array('package-width', '50', null), |
|
62 | 62 | ); |
63 | 63 | |
64 | 64 | $items = $product->getPropertyItems(); |
65 | - $this->assertEquals( 2, count( $items ) ); |
|
65 | + $this->assertEquals(2, count($items)); |
|
66 | 66 | |
67 | - foreach( $items as $item ) |
|
67 | + foreach ($items as $item) |
|
68 | 68 | { |
69 | - $this->assertEquals( $expected[$pos][0], $item->getType() ); |
|
70 | - $this->assertEquals( $expected[$pos][1], $item->getValue() ); |
|
71 | - $this->assertEquals( $expected[$pos][2], $item->getLanguageId() ); |
|
69 | + $this->assertEquals($expected[$pos][0], $item->getType()); |
|
70 | + $this->assertEquals($expected[$pos][1], $item->getValue()); |
|
71 | + $this->assertEquals($expected[$pos][2], $item->getLanguageId()); |
|
72 | 72 | $pos++; |
73 | 73 | } |
74 | 74 | } |
@@ -91,21 +91,21 @@ discard block |
||
91 | 91 | 1 => '10', |
92 | 92 | ); |
93 | 93 | |
94 | - $product = $this->create( 'job_csv_test' ); |
|
94 | + $product = $this->create('job_csv_test'); |
|
95 | 95 | |
96 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard( $this->context, $mapping, $this->endpoint ); |
|
97 | - $object->process( $product, $data ); |
|
98 | - $object->process( $product, $dataUpdate ); |
|
96 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard($this->context, $mapping, $this->endpoint); |
|
97 | + $object->process($product, $data); |
|
98 | + $object->process($product, $dataUpdate); |
|
99 | 99 | |
100 | 100 | |
101 | 101 | $items = $product->getPropertyItems(); |
102 | - $item = reset( $items ); |
|
102 | + $item = reset($items); |
|
103 | 103 | |
104 | - $this->assertEquals( 1, count( $items ) ); |
|
105 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Property\\Iface', $item ); |
|
104 | + $this->assertEquals(1, count($items)); |
|
105 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Property\\Iface', $item); |
|
106 | 106 | |
107 | - $this->assertEquals( 'package-height', $item->getType() ); |
|
108 | - $this->assertEquals( '10', $item->getValue() ); |
|
107 | + $this->assertEquals('package-height', $item->getType()); |
|
108 | + $this->assertEquals('10', $item->getValue()); |
|
109 | 109 | } |
110 | 110 | |
111 | 111 | |
@@ -121,18 +121,18 @@ discard block |
||
121 | 121 | 1 => '3.00', |
122 | 122 | ); |
123 | 123 | |
124 | - $product = $this->create( 'job_csv_test' ); |
|
124 | + $product = $this->create('job_csv_test'); |
|
125 | 125 | |
126 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard( $this->context, $mapping, $this->endpoint ); |
|
127 | - $object->process( $product, $data ); |
|
126 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard($this->context, $mapping, $this->endpoint); |
|
127 | + $object->process($product, $data); |
|
128 | 128 | |
129 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard( $this->context, [], $this->endpoint ); |
|
130 | - $object->process( $product, [] ); |
|
129 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard($this->context, [], $this->endpoint); |
|
130 | + $object->process($product, []); |
|
131 | 131 | |
132 | 132 | |
133 | 133 | $items = $product->getPropertyItems(); |
134 | 134 | |
135 | - $this->assertEquals( 0, count( $items ) ); |
|
135 | + $this->assertEquals(0, count($items)); |
|
136 | 136 | } |
137 | 137 | |
138 | 138 | |
@@ -152,23 +152,23 @@ discard block |
||
152 | 152 | 3 => '3.00', |
153 | 153 | ); |
154 | 154 | |
155 | - $product = $this->create( 'job_csv_test' ); |
|
155 | + $product = $this->create('job_csv_test'); |
|
156 | 156 | |
157 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard( $this->context, $mapping, $this->endpoint ); |
|
158 | - $object->process( $product, $data ); |
|
157 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Property\Standard($this->context, $mapping, $this->endpoint); |
|
158 | + $object->process($product, $data); |
|
159 | 159 | |
160 | 160 | $items = $product->getPropertyItems(); |
161 | 161 | |
162 | - $this->assertEquals( 1, count( $items ) ); |
|
162 | + $this->assertEquals(1, count($items)); |
|
163 | 163 | } |
164 | 164 | |
165 | 165 | |
166 | 166 | /** |
167 | 167 | * @param string $code |
168 | 168 | */ |
169 | - protected function create( $code ) |
|
169 | + protected function create($code) |
|
170 | 170 | { |
171 | - $manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context ); |
|
172 | - return $manager->createItem()->setCode( $code ); |
|
171 | + $manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context); |
|
172 | + return $manager->createItem()->setCode($code); |
|
173 | 173 | } |
174 | 174 | } |
175 | 175 | \ No newline at end of file |
@@ -17,16 +17,16 @@ discard block |
||
17 | 17 | |
18 | 18 | protected function setUp() |
19 | 19 | { |
20 | - \Aimeos\MShop\Factory::setCache( true ); |
|
20 | + \Aimeos\MShop\Factory::setCache(true); |
|
21 | 21 | |
22 | 22 | $this->context = \TestHelperCntl::getContext(); |
23 | - $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, [] ); |
|
23 | + $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, []); |
|
24 | 24 | } |
25 | 25 | |
26 | 26 | |
27 | 27 | protected function tearDown() |
28 | 28 | { |
29 | - \Aimeos\MShop\Factory::setCache( false ); |
|
29 | + \Aimeos\MShop\Factory::setCache(false); |
|
30 | 30 | \Aimeos\MShop\Factory::clear(); |
31 | 31 | } |
32 | 32 | |
@@ -51,31 +51,31 @@ discard block |
||
51 | 51 | 5 => 1, |
52 | 52 | ); |
53 | 53 | |
54 | - $product = $this->create( 'job_csv_test' ); |
|
54 | + $product = $this->create('job_csv_test'); |
|
55 | 55 | |
56 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, $mapping, $this->endpoint ); |
|
57 | - $object->process( $product, $data ); |
|
56 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, $mapping, $this->endpoint); |
|
57 | + $object->process($product, $data); |
|
58 | 58 | |
59 | 59 | |
60 | 60 | $listItems = $product->getListItems(); |
61 | - $listItem = reset( $listItems ); |
|
61 | + $listItem = reset($listItems); |
|
62 | 62 | |
63 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
64 | - $this->assertEquals( 1, count( $listItems ) ); |
|
63 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
64 | + $this->assertEquals(1, count($listItems)); |
|
65 | 65 | |
66 | - $this->assertEquals( 1, $listItem->getStatus() ); |
|
67 | - $this->assertEquals( 0, $listItem->getPosition() ); |
|
68 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
66 | + $this->assertEquals(1, $listItem->getStatus()); |
|
67 | + $this->assertEquals(0, $listItem->getPosition()); |
|
68 | + $this->assertEquals('default', $listItem->getType()); |
|
69 | 69 | |
70 | 70 | $refItem = $listItem->getRefItem(); |
71 | 71 | |
72 | - $this->assertEquals( 1, $refItem->getStatus() ); |
|
73 | - $this->assertEquals( 'default', $refItem->getType() ); |
|
74 | - $this->assertEquals( 'test image', $refItem->getLabel() ); |
|
75 | - $this->assertEquals( 'image/jpeg', $refItem->getMimetype() ); |
|
76 | - $this->assertEquals( 'path/to/preview', $refItem->getPreview() ); |
|
77 | - $this->assertEquals( 'path/to/file', $refItem->getUrl() ); |
|
78 | - $this->assertEquals( 'de', $refItem->getLanguageId() ); |
|
72 | + $this->assertEquals(1, $refItem->getStatus()); |
|
73 | + $this->assertEquals('default', $refItem->getType()); |
|
74 | + $this->assertEquals('test image', $refItem->getLabel()); |
|
75 | + $this->assertEquals('image/jpeg', $refItem->getMimetype()); |
|
76 | + $this->assertEquals('path/to/preview', $refItem->getPreview()); |
|
77 | + $this->assertEquals('path/to/file', $refItem->getUrl()); |
|
78 | + $this->assertEquals('de', $refItem->getLanguageId()); |
|
79 | 79 | } |
80 | 80 | |
81 | 81 | |
@@ -89,21 +89,21 @@ discard block |
||
89 | 89 | 0 => "path/to/0\npath/to/1\npath/to/2\npath/to/3", |
90 | 90 | ); |
91 | 91 | |
92 | - $product = $this->create( 'job_csv_test' ); |
|
92 | + $product = $this->create('job_csv_test'); |
|
93 | 93 | |
94 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, $mapping, $this->endpoint ); |
|
95 | - $object->process( $product, $data ); |
|
94 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, $mapping, $this->endpoint); |
|
95 | + $object->process($product, $data); |
|
96 | 96 | |
97 | 97 | |
98 | 98 | $pos = 0; |
99 | 99 | $listItems = $product->getListItems(); |
100 | - $expected = array( 'path/to/0', 'path/to/1', 'path/to/2', 'path/to/3' ); |
|
100 | + $expected = array('path/to/0', 'path/to/1', 'path/to/2', 'path/to/3'); |
|
101 | 101 | |
102 | - $this->assertEquals( 4, count( $listItems ) ); |
|
102 | + $this->assertEquals(4, count($listItems)); |
|
103 | 103 | |
104 | - foreach( $listItems as $listItem ) |
|
104 | + foreach ($listItems as $listItem) |
|
105 | 105 | { |
106 | - $this->assertEquals( $expected[$pos], $listItem->getRefItem()->getUrl() ); |
|
106 | + $this->assertEquals($expected[$pos], $listItem->getRefItem()->getUrl()); |
|
107 | 107 | $pos++; |
108 | 108 | } |
109 | 109 | } |
@@ -125,20 +125,20 @@ discard block |
||
125 | 125 | 3 => 'path/to/3', |
126 | 126 | ); |
127 | 127 | |
128 | - $product = $this->create( 'job_csv_test' ); |
|
128 | + $product = $this->create('job_csv_test'); |
|
129 | 129 | |
130 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, $mapping, $this->endpoint ); |
|
131 | - $object->process( $product, $data ); |
|
130 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, $mapping, $this->endpoint); |
|
131 | + $object->process($product, $data); |
|
132 | 132 | |
133 | 133 | |
134 | 134 | $pos = 0; |
135 | 135 | $listItems = $product->getListItems(); |
136 | 136 | |
137 | - $this->assertEquals( 4, count( $listItems ) ); |
|
137 | + $this->assertEquals(4, count($listItems)); |
|
138 | 138 | |
139 | - foreach( $listItems as $listItem ) |
|
139 | + foreach ($listItems as $listItem) |
|
140 | 140 | { |
141 | - $this->assertEquals( $data[$pos], $listItem->getRefItem()->getUrl() ); |
|
141 | + $this->assertEquals($data[$pos], $listItem->getRefItem()->getUrl()); |
|
142 | 142 | $pos++; |
143 | 143 | } |
144 | 144 | } |
@@ -161,21 +161,21 @@ discard block |
||
161 | 161 | 1 => '', |
162 | 162 | ); |
163 | 163 | |
164 | - $product = $this->create( 'job_csv_test' ); |
|
164 | + $product = $this->create('job_csv_test'); |
|
165 | 165 | |
166 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, $mapping, $this->endpoint ); |
|
167 | - $object->process( $product, $data ); |
|
168 | - $object->process( $product, $dataUpdate ); |
|
166 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, $mapping, $this->endpoint); |
|
167 | + $object->process($product, $data); |
|
168 | + $object->process($product, $dataUpdate); |
|
169 | 169 | |
170 | 170 | |
171 | 171 | $listItems = $product->getListItems(); |
172 | - $listItem = reset( $listItems ); |
|
172 | + $listItem = reset($listItems); |
|
173 | 173 | |
174 | - $this->assertEquals( 1, count( $listItems ) ); |
|
175 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
174 | + $this->assertEquals(1, count($listItems)); |
|
175 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
176 | 176 | |
177 | - $this->assertEquals( 'path/to/new', $listItem->getRefItem()->getUrl() ); |
|
178 | - $this->assertEquals( null, $listItem->getRefItem()->getLanguageId() ); |
|
177 | + $this->assertEquals('path/to/new', $listItem->getRefItem()->getUrl()); |
|
178 | + $this->assertEquals(null, $listItem->getRefItem()->getLanguageId()); |
|
179 | 179 | } |
180 | 180 | |
181 | 181 | |
@@ -189,18 +189,18 @@ discard block |
||
189 | 189 | 0 => '/path/to/file', |
190 | 190 | ); |
191 | 191 | |
192 | - $product = $this->create( 'job_csv_test' ); |
|
192 | + $product = $this->create('job_csv_test'); |
|
193 | 193 | |
194 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, $mapping, $this->endpoint ); |
|
195 | - $object->process( $product, $data ); |
|
194 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, $mapping, $this->endpoint); |
|
195 | + $object->process($product, $data); |
|
196 | 196 | |
197 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, [], $this->endpoint ); |
|
198 | - $object->process( $product, [] ); |
|
197 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, [], $this->endpoint); |
|
198 | + $object->process($product, []); |
|
199 | 199 | |
200 | 200 | |
201 | 201 | $listItems = $product->getListItems(); |
202 | 202 | |
203 | - $this->assertEquals( 0, count( $listItems ) ); |
|
203 | + $this->assertEquals(0, count($listItems)); |
|
204 | 204 | } |
205 | 205 | |
206 | 206 | |
@@ -216,15 +216,15 @@ discard block |
||
216 | 216 | 1 => '', |
217 | 217 | ); |
218 | 218 | |
219 | - $product = $this->create( 'job_csv_test' ); |
|
219 | + $product = $this->create('job_csv_test'); |
|
220 | 220 | |
221 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, $mapping, $this->endpoint ); |
|
222 | - $object->process( $product, $data ); |
|
221 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, $mapping, $this->endpoint); |
|
222 | + $object->process($product, $data); |
|
223 | 223 | |
224 | 224 | |
225 | 225 | $listItems = $product->getListItems(); |
226 | 226 | |
227 | - $this->assertEquals( 1, count( $listItems ) ); |
|
227 | + $this->assertEquals(1, count($listItems)); |
|
228 | 228 | } |
229 | 229 | |
230 | 230 | |
@@ -244,31 +244,31 @@ discard block |
||
244 | 244 | 3 => 'default', |
245 | 245 | ); |
246 | 246 | |
247 | - $this->context->getConfig()->set( 'controller/common/product/import/csv/processor/media/listtypes', array( 'default' ) ); |
|
247 | + $this->context->getConfig()->set('controller/common/product/import/csv/processor/media/listtypes', array('default')); |
|
248 | 248 | |
249 | - $product = $this->create( 'job_csv_test' ); |
|
249 | + $product = $this->create('job_csv_test'); |
|
250 | 250 | |
251 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard( $this->context, $mapping, $this->endpoint ); |
|
252 | - $object->process( $product, $data ); |
|
251 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Media\Standard($this->context, $mapping, $this->endpoint); |
|
252 | + $object->process($product, $data); |
|
253 | 253 | |
254 | 254 | |
255 | 255 | $listItems = $product->getListItems(); |
256 | - $listItem = reset( $listItems ); |
|
256 | + $listItem = reset($listItems); |
|
257 | 257 | |
258 | - $this->assertEquals( 1, count( $listItems ) ); |
|
259 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
258 | + $this->assertEquals(1, count($listItems)); |
|
259 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
260 | 260 | |
261 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
262 | - $this->assertEquals( 'path/to/file2', $listItem->getRefItem()->getUrl() ); |
|
261 | + $this->assertEquals('default', $listItem->getType()); |
|
262 | + $this->assertEquals('path/to/file2', $listItem->getRefItem()->getUrl()); |
|
263 | 263 | } |
264 | 264 | |
265 | 265 | |
266 | 266 | /** |
267 | 267 | * @param string $code |
268 | 268 | */ |
269 | - protected function create( $code ) |
|
269 | + protected function create($code) |
|
270 | 270 | { |
271 | - $manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context ); |
|
272 | - return $manager->createItem()->setCode( $code ); |
|
271 | + $manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context); |
|
272 | + return $manager->createItem()->setCode($code); |
|
273 | 273 | } |
274 | 274 | } |
275 | 275 | \ No newline at end of file |
@@ -18,17 +18,17 @@ discard block |
||
18 | 18 | |
19 | 19 | protected function setUp() |
20 | 20 | { |
21 | - \Aimeos\MShop\Factory::setCache( true ); |
|
21 | + \Aimeos\MShop\Factory::setCache(true); |
|
22 | 22 | |
23 | 23 | $this->context = \TestHelperCntl::getContext(); |
24 | - $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, [] ); |
|
24 | + $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, []); |
|
25 | 25 | |
26 | - $manager = \Aimeos\MShop\Factory::createManager( $this->context, 'product' ); |
|
26 | + $manager = \Aimeos\MShop\Factory::createManager($this->context, 'product'); |
|
27 | 27 | $search = $manager->createSearch(); |
28 | - $search->setConditions( $search->compare( '==', 'product.code', ['CNC', 'CNE'] ) ); |
|
28 | + $search->setConditions($search->compare('==', 'product.code', ['CNC', 'CNE'])); |
|
29 | 29 | |
30 | 30 | $this->products = []; |
31 | - foreach( $manager->searchItems( $search ) as $id => $item ) { |
|
31 | + foreach ($manager->searchItems($search) as $id => $item) { |
|
32 | 32 | $this->products[$item->getCode()] = $id; |
33 | 33 | } |
34 | 34 | } |
@@ -36,7 +36,7 @@ discard block |
||
36 | 36 | |
37 | 37 | protected function tearDown() |
38 | 38 | { |
39 | - \Aimeos\MShop\Factory::setCache( false ); |
|
39 | + \Aimeos\MShop\Factory::setCache(false); |
|
40 | 40 | \Aimeos\MShop\Factory::clear(); |
41 | 41 | } |
42 | 42 | |
@@ -57,23 +57,23 @@ discard block |
||
57 | 57 | 3 => 'CNE', |
58 | 58 | ); |
59 | 59 | |
60 | - $product = $this->create( 'job_csv_test' ); |
|
60 | + $product = $this->create('job_csv_test'); |
|
61 | 61 | |
62 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard( $this->context, $mapping, $this->endpoint ); |
|
63 | - $object->process( $product, $data ); |
|
62 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard($this->context, $mapping, $this->endpoint); |
|
63 | + $object->process($product, $data); |
|
64 | 64 | |
65 | 65 | |
66 | - $listItems1 = $product->getListItems( 'product', 'default' ); |
|
67 | - $listItems2 = $product->getListItems( 'product', 'suggestion' ); |
|
66 | + $listItems1 = $product->getListItems('product', 'default'); |
|
67 | + $listItems2 = $product->getListItems('product', 'suggestion'); |
|
68 | 68 | |
69 | - $this->assertEquals( 1, count( $listItems1 ) ); |
|
70 | - $this->assertEquals( 1, count( $listItems2 ) ); |
|
69 | + $this->assertEquals(1, count($listItems1)); |
|
70 | + $this->assertEquals(1, count($listItems2)); |
|
71 | 71 | |
72 | - $this->assertEquals( 1, reset( $listItems1 )->getStatus() ); |
|
73 | - $this->assertEquals( 1, reset( $listItems2 )->getStatus() ); |
|
72 | + $this->assertEquals(1, reset($listItems1)->getStatus()); |
|
73 | + $this->assertEquals(1, reset($listItems2)->getStatus()); |
|
74 | 74 | |
75 | - $this->assertEquals( $this->products['CNC'], reset( $listItems1 )->getRefId() ); |
|
76 | - $this->assertEquals( $this->products['CNE'], reset( $listItems2 )->getRefId() ); |
|
75 | + $this->assertEquals($this->products['CNC'], reset($listItems1)->getRefId()); |
|
76 | + $this->assertEquals($this->products['CNE'], reset($listItems2)->getRefId()); |
|
77 | 77 | } |
78 | 78 | |
79 | 79 | |
@@ -89,24 +89,24 @@ discard block |
||
89 | 89 | 1 => "CNC\nCNE", |
90 | 90 | ); |
91 | 91 | |
92 | - $product = $this->create( 'job_csv_test' ); |
|
92 | + $product = $this->create('job_csv_test'); |
|
93 | 93 | |
94 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard( $this->context, $mapping, $this->endpoint ); |
|
95 | - $object->process( $product, $data ); |
|
94 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard($this->context, $mapping, $this->endpoint); |
|
95 | + $object->process($product, $data); |
|
96 | 96 | |
97 | 97 | |
98 | 98 | $pos = 0; |
99 | 99 | $listItems = $product->getListItems(); |
100 | - $prodIds = array( $this->products['CNC'], $this->products['CNE'] ); |
|
100 | + $prodIds = array($this->products['CNC'], $this->products['CNE']); |
|
101 | 101 | |
102 | - $this->assertEquals( 2, count( $listItems ) ); |
|
102 | + $this->assertEquals(2, count($listItems)); |
|
103 | 103 | |
104 | - foreach( $listItems as $listItem ) |
|
104 | + foreach ($listItems as $listItem) |
|
105 | 105 | { |
106 | - $this->assertEquals( 1, $listItem->getStatus() ); |
|
107 | - $this->assertEquals( 'product', $listItem->getDomain() ); |
|
108 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
109 | - $this->assertEquals( $prodIds[$pos], $listItem->getRefId() ); |
|
106 | + $this->assertEquals(1, $listItem->getStatus()); |
|
107 | + $this->assertEquals('product', $listItem->getDomain()); |
|
108 | + $this->assertEquals('default', $listItem->getType()); |
|
109 | + $this->assertEquals($prodIds[$pos], $listItem->getRefId()); |
|
110 | 110 | $pos++; |
111 | 111 | } |
112 | 112 | } |
@@ -129,20 +129,20 @@ discard block |
||
129 | 129 | 1 => 'CNE', |
130 | 130 | ); |
131 | 131 | |
132 | - $product = $this->create( 'job_csv_test' ); |
|
132 | + $product = $this->create('job_csv_test'); |
|
133 | 133 | |
134 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard( $this->context, $mapping, $this->endpoint ); |
|
135 | - $object->process( $product, $data ); |
|
136 | - $object->process( $product, $dataUpdate ); |
|
134 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard($this->context, $mapping, $this->endpoint); |
|
135 | + $object->process($product, $data); |
|
136 | + $object->process($product, $dataUpdate); |
|
137 | 137 | |
138 | 138 | |
139 | 139 | $listItems = $product->getListItems(); |
140 | - $listItem = reset( $listItems ); |
|
140 | + $listItem = reset($listItems); |
|
141 | 141 | |
142 | - $this->assertEquals( 1, count( $listItems ) ); |
|
143 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
142 | + $this->assertEquals(1, count($listItems)); |
|
143 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
144 | 144 | |
145 | - $this->assertEquals( $this->products['CNE'], $listItem->getRefId() ); |
|
145 | + $this->assertEquals($this->products['CNE'], $listItem->getRefId()); |
|
146 | 146 | } |
147 | 147 | |
148 | 148 | |
@@ -158,16 +158,16 @@ discard block |
||
158 | 158 | 1 => 'CNC', |
159 | 159 | ); |
160 | 160 | |
161 | - $product = $this->create( 'job_csv_test' ); |
|
161 | + $product = $this->create('job_csv_test'); |
|
162 | 162 | |
163 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard( $this->context, $mapping, $this->endpoint ); |
|
164 | - $object->process( $product, $data ); |
|
163 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard($this->context, $mapping, $this->endpoint); |
|
164 | + $object->process($product, $data); |
|
165 | 165 | |
166 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard( $this->context, [], $this->endpoint ); |
|
167 | - $object->process( $product, [] ); |
|
166 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard($this->context, [], $this->endpoint); |
|
167 | + $object->process($product, []); |
|
168 | 168 | |
169 | 169 | |
170 | - $this->assertEquals( 0, count( $product->getListItems() ) ); |
|
170 | + $this->assertEquals(0, count($product->getListItems())); |
|
171 | 171 | } |
172 | 172 | |
173 | 173 | |
@@ -187,15 +187,15 @@ discard block |
||
187 | 187 | 3 => 'CNE', |
188 | 188 | ); |
189 | 189 | |
190 | - $product = $this->create( 'job_csv_test' ); |
|
190 | + $product = $this->create('job_csv_test'); |
|
191 | 191 | |
192 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard( $this->context, $mapping, $this->endpoint ); |
|
193 | - $object->process( $product, $data ); |
|
192 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard($this->context, $mapping, $this->endpoint); |
|
193 | + $object->process($product, $data); |
|
194 | 194 | |
195 | 195 | |
196 | 196 | $listItems = $product->getListItems(); |
197 | 197 | |
198 | - $this->assertEquals( 1, count( $listItems ) ); |
|
198 | + $this->assertEquals(1, count($listItems)); |
|
199 | 199 | } |
200 | 200 | |
201 | 201 | |
@@ -215,31 +215,31 @@ discard block |
||
215 | 215 | 3 => 'CNE', |
216 | 216 | ); |
217 | 217 | |
218 | - $this->context->getConfig()->set( 'controller/common/product/import/csv/processor/product/listtypes', array( 'default' ) ); |
|
218 | + $this->context->getConfig()->set('controller/common/product/import/csv/processor/product/listtypes', array('default')); |
|
219 | 219 | |
220 | - $product = $this->create( 'job_csv_test' ); |
|
220 | + $product = $this->create('job_csv_test'); |
|
221 | 221 | |
222 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard( $this->context, $mapping, $this->endpoint ); |
|
223 | - $object->process( $product, $data ); |
|
222 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Product\Standard($this->context, $mapping, $this->endpoint); |
|
223 | + $object->process($product, $data); |
|
224 | 224 | |
225 | 225 | |
226 | 226 | $listItems = $product->getListItems(); |
227 | - $listItem = reset( $listItems ); |
|
227 | + $listItem = reset($listItems); |
|
228 | 228 | |
229 | - $this->assertEquals( 1, count( $listItems ) ); |
|
230 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
229 | + $this->assertEquals(1, count($listItems)); |
|
230 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
231 | 231 | |
232 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
233 | - $this->assertEquals( $this->products['CNE'], $listItem->getRefId() ); |
|
232 | + $this->assertEquals('default', $listItem->getType()); |
|
233 | + $this->assertEquals($this->products['CNE'], $listItem->getRefId()); |
|
234 | 234 | } |
235 | 235 | |
236 | 236 | |
237 | 237 | /** |
238 | 238 | * @param string $code |
239 | 239 | */ |
240 | - protected function create( $code ) |
|
240 | + protected function create($code) |
|
241 | 241 | { |
242 | - $manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context ); |
|
243 | - return $manager->createItem()->setCode( $code ); |
|
242 | + $manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context); |
|
243 | + return $manager->createItem()->setCode($code); |
|
244 | 244 | } |
245 | 245 | } |
@@ -17,16 +17,16 @@ discard block |
||
17 | 17 | |
18 | 18 | protected function setUp() |
19 | 19 | { |
20 | - \Aimeos\MShop\Factory::setCache( true ); |
|
20 | + \Aimeos\MShop\Factory::setCache(true); |
|
21 | 21 | |
22 | 22 | $this->context = \TestHelperCntl::getContext(); |
23 | - $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, [] ); |
|
23 | + $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, []); |
|
24 | 24 | } |
25 | 25 | |
26 | 26 | |
27 | 27 | protected function tearDown() |
28 | 28 | { |
29 | - \Aimeos\MShop\Factory::setCache( false ); |
|
29 | + \Aimeos\MShop\Factory::setCache(false); |
|
30 | 30 | \Aimeos\MShop\Factory::clear(); |
31 | 31 | } |
32 | 32 | |
@@ -57,33 +57,33 @@ discard block |
||
57 | 57 | 8 => 1, |
58 | 58 | ); |
59 | 59 | |
60 | - $product = $this->create( 'job_csv_test' ); |
|
60 | + $product = $this->create('job_csv_test'); |
|
61 | 61 | |
62 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard( $this->context, $mapping, $this->endpoint ); |
|
63 | - $object->process( $product, $data ); |
|
62 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard($this->context, $mapping, $this->endpoint); |
|
63 | + $object->process($product, $data); |
|
64 | 64 | |
65 | 65 | |
66 | 66 | $listItems = $product->getListItems(); |
67 | - $listItem = reset( $listItems ); |
|
67 | + $listItem = reset($listItems); |
|
68 | 68 | |
69 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
70 | - $this->assertEquals( 1, count( $listItems ) ); |
|
69 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
70 | + $this->assertEquals(1, count($listItems)); |
|
71 | 71 | |
72 | - $this->assertEquals( 1, $listItem->getStatus() ); |
|
73 | - $this->assertEquals( 0, $listItem->getPosition() ); |
|
74 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
72 | + $this->assertEquals(1, $listItem->getStatus()); |
|
73 | + $this->assertEquals(0, $listItem->getPosition()); |
|
74 | + $this->assertEquals('default', $listItem->getType()); |
|
75 | 75 | |
76 | 76 | $refItem = $listItem->getRefItem(); |
77 | 77 | |
78 | - $this->assertEquals( 1, $refItem->getStatus() ); |
|
79 | - $this->assertEquals( 'default', $refItem->getType() ); |
|
80 | - $this->assertEquals( 'EUR 1.00', $refItem->getLabel() ); |
|
81 | - $this->assertEquals( 5, $refItem->getQuantity() ); |
|
82 | - $this->assertEquals( '1.00', $refItem->getValue() ); |
|
83 | - $this->assertEquals( '0.20', $refItem->getCosts() ); |
|
84 | - $this->assertEquals( '0.10', $refItem->getRebate() ); |
|
85 | - $this->assertEquals( '20.00', $refItem->getTaxrate() ); |
|
86 | - $this->assertEquals( 1, $refItem->getStatus() ); |
|
78 | + $this->assertEquals(1, $refItem->getStatus()); |
|
79 | + $this->assertEquals('default', $refItem->getType()); |
|
80 | + $this->assertEquals('EUR 1.00', $refItem->getLabel()); |
|
81 | + $this->assertEquals(5, $refItem->getQuantity()); |
|
82 | + $this->assertEquals('1.00', $refItem->getValue()); |
|
83 | + $this->assertEquals('0.20', $refItem->getCosts()); |
|
84 | + $this->assertEquals('0.10', $refItem->getRebate()); |
|
85 | + $this->assertEquals('20.00', $refItem->getTaxrate()); |
|
86 | + $this->assertEquals(1, $refItem->getStatus()); |
|
87 | 87 | } |
88 | 88 | |
89 | 89 | |
@@ -111,21 +111,21 @@ discard block |
||
111 | 111 | 7 => '4.00', |
112 | 112 | ); |
113 | 113 | |
114 | - $product = $this->create( 'job_csv_test' ); |
|
114 | + $product = $this->create('job_csv_test'); |
|
115 | 115 | |
116 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard( $this->context, $mapping, $this->endpoint ); |
|
117 | - $object->process( $product, $data ); |
|
116 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard($this->context, $mapping, $this->endpoint); |
|
117 | + $object->process($product, $data); |
|
118 | 118 | |
119 | 119 | |
120 | 120 | $pos = 0; |
121 | 121 | $listItems = $product->getListItems(); |
122 | 122 | |
123 | - $this->assertEquals( 4, count( $listItems ) ); |
|
123 | + $this->assertEquals(4, count($listItems)); |
|
124 | 124 | |
125 | - foreach( $listItems as $listItem ) |
|
125 | + foreach ($listItems as $listItem) |
|
126 | 126 | { |
127 | - $this->assertEquals( $data[$pos++], $listItem->getRefItem()->getCurrencyId() ); |
|
128 | - $this->assertEquals( $data[$pos++], $listItem->getRefItem()->getValue() ); |
|
127 | + $this->assertEquals($data[$pos++], $listItem->getRefItem()->getCurrencyId()); |
|
128 | + $this->assertEquals($data[$pos++], $listItem->getRefItem()->getValue()); |
|
129 | 129 | } |
130 | 130 | } |
131 | 131 | |
@@ -147,20 +147,20 @@ discard block |
||
147 | 147 | 1 => '2.00', |
148 | 148 | ); |
149 | 149 | |
150 | - $product = $this->create( 'job_csv_test' ); |
|
150 | + $product = $this->create('job_csv_test'); |
|
151 | 151 | |
152 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard( $this->context, $mapping, $this->endpoint ); |
|
153 | - $object->process( $product, $data ); |
|
154 | - $object->process( $product, $dataUpdate ); |
|
152 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard($this->context, $mapping, $this->endpoint); |
|
153 | + $object->process($product, $data); |
|
154 | + $object->process($product, $dataUpdate); |
|
155 | 155 | |
156 | 156 | |
157 | 157 | $listItems = $product->getListItems(); |
158 | - $listItem = reset( $listItems ); |
|
158 | + $listItem = reset($listItems); |
|
159 | 159 | |
160 | - $this->assertEquals( 1, count( $listItems ) ); |
|
161 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
160 | + $this->assertEquals(1, count($listItems)); |
|
161 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
162 | 162 | |
163 | - $this->assertEquals( '2.00', $listItem->getRefItem()->getValue() ); |
|
163 | + $this->assertEquals('2.00', $listItem->getRefItem()->getValue()); |
|
164 | 164 | } |
165 | 165 | |
166 | 166 | |
@@ -176,18 +176,18 @@ discard block |
||
176 | 176 | 1 => '1.00', |
177 | 177 | ); |
178 | 178 | |
179 | - $product = $this->create( 'job_csv_test' ); |
|
179 | + $product = $this->create('job_csv_test'); |
|
180 | 180 | |
181 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard( $this->context, $mapping, $this->endpoint ); |
|
182 | - $object->process( $product, $data ); |
|
181 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard($this->context, $mapping, $this->endpoint); |
|
182 | + $object->process($product, $data); |
|
183 | 183 | |
184 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard( $this->context, [], $this->endpoint ); |
|
185 | - $object->process( $product, [] ); |
|
184 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard($this->context, [], $this->endpoint); |
|
185 | + $object->process($product, []); |
|
186 | 186 | |
187 | 187 | |
188 | 188 | $listItems = $product->getListItems(); |
189 | 189 | |
190 | - $this->assertEquals( 0, count( $listItems ) ); |
|
190 | + $this->assertEquals(0, count($listItems)); |
|
191 | 191 | } |
192 | 192 | |
193 | 193 | |
@@ -207,15 +207,15 @@ discard block |
||
207 | 207 | 3 => '', |
208 | 208 | ); |
209 | 209 | |
210 | - $product = $this->create( 'job_csv_test' ); |
|
210 | + $product = $this->create('job_csv_test'); |
|
211 | 211 | |
212 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard( $this->context, $mapping, $this->endpoint ); |
|
213 | - $object->process( $product, $data ); |
|
212 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard($this->context, $mapping, $this->endpoint); |
|
213 | + $object->process($product, $data); |
|
214 | 214 | |
215 | 215 | |
216 | 216 | $listItems = $product->getListItems(); |
217 | 217 | |
218 | - $this->assertEquals( 1, count( $listItems ) ); |
|
218 | + $this->assertEquals(1, count($listItems)); |
|
219 | 219 | } |
220 | 220 | |
221 | 221 | |
@@ -239,31 +239,31 @@ discard block |
||
239 | 239 | 5 => 'default', |
240 | 240 | ); |
241 | 241 | |
242 | - $this->context->getConfig()->set( 'controller/common/product/import/csv/processor/price/listtypes', array( 'default' ) ); |
|
242 | + $this->context->getConfig()->set('controller/common/product/import/csv/processor/price/listtypes', array('default')); |
|
243 | 243 | |
244 | - $product = $this->create( 'job_csv_test' ); |
|
244 | + $product = $this->create('job_csv_test'); |
|
245 | 245 | |
246 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard( $this->context, $mapping, $this->endpoint ); |
|
247 | - $object->process( $product, $data ); |
|
246 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Price\Standard($this->context, $mapping, $this->endpoint); |
|
247 | + $object->process($product, $data); |
|
248 | 248 | |
249 | 249 | |
250 | 250 | $listItems = $product->getListItems(); |
251 | - $listItem = reset( $listItems ); |
|
251 | + $listItem = reset($listItems); |
|
252 | 252 | |
253 | - $this->assertEquals( 1, count( $listItems ) ); |
|
254 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
253 | + $this->assertEquals(1, count($listItems)); |
|
254 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
255 | 255 | |
256 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
257 | - $this->assertEquals( '2.00', $listItem->getRefItem()->getValue() ); |
|
256 | + $this->assertEquals('default', $listItem->getType()); |
|
257 | + $this->assertEquals('2.00', $listItem->getRefItem()->getValue()); |
|
258 | 258 | } |
259 | 259 | |
260 | 260 | |
261 | 261 | /** |
262 | 262 | * @param string $code |
263 | 263 | */ |
264 | - protected function create( $code ) |
|
264 | + protected function create($code) |
|
265 | 265 | { |
266 | - $manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context ); |
|
267 | - return $manager->createItem()->setCode( $code ); |
|
266 | + $manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context); |
|
267 | + return $manager->createItem()->setCode($code); |
|
268 | 268 | } |
269 | 269 | } |
270 | 270 | \ No newline at end of file |
@@ -17,16 +17,16 @@ discard block |
||
17 | 17 | |
18 | 18 | protected function setUp() |
19 | 19 | { |
20 | - \Aimeos\MShop\Factory::setCache( true ); |
|
20 | + \Aimeos\MShop\Factory::setCache(true); |
|
21 | 21 | |
22 | 22 | $this->context = \TestHelperCntl::getContext(); |
23 | - $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, [] ); |
|
23 | + $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, []); |
|
24 | 24 | } |
25 | 25 | |
26 | 26 | |
27 | 27 | protected function tearDown() |
28 | 28 | { |
29 | - \Aimeos\MShop\Factory::setCache( false ); |
|
29 | + \Aimeos\MShop\Factory::setCache(false); |
|
30 | 30 | \Aimeos\MShop\Factory::clear(); |
31 | 31 | } |
32 | 32 | |
@@ -49,30 +49,30 @@ discard block |
||
49 | 49 | 4 => 1, |
50 | 50 | ); |
51 | 51 | |
52 | - $product = $this->create( 'job_csv_test' ); |
|
52 | + $product = $this->create('job_csv_test'); |
|
53 | 53 | |
54 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint ); |
|
55 | - $object->process( $product, $data ); |
|
54 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint); |
|
55 | + $object->process($product, $data); |
|
56 | 56 | |
57 | 57 | |
58 | 58 | $listItems = $product->getListItems(); |
59 | - $listItem = reset( $listItems ); |
|
59 | + $listItem = reset($listItems); |
|
60 | 60 | |
61 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
62 | - $this->assertEquals( 1, count( $listItems ) ); |
|
61 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
62 | + $this->assertEquals(1, count($listItems)); |
|
63 | 63 | |
64 | - $this->assertEquals( 1, $listItem->getStatus() ); |
|
65 | - $this->assertEquals( 0, $listItem->getPosition() ); |
|
66 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
64 | + $this->assertEquals(1, $listItem->getStatus()); |
|
65 | + $this->assertEquals(0, $listItem->getPosition()); |
|
66 | + $this->assertEquals('default', $listItem->getType()); |
|
67 | 67 | |
68 | 68 | $refItem = $listItem->getRefItem(); |
69 | 69 | |
70 | - $this->assertEquals( 1, $refItem->getStatus() ); |
|
71 | - $this->assertEquals( 'name', $refItem->getType() ); |
|
72 | - $this->assertEquals( 'test text', $refItem->getLabel() ); |
|
73 | - $this->assertEquals( 'Job CSV test', $refItem->getContent() ); |
|
74 | - $this->assertEquals( 'de', $refItem->getLanguageId() ); |
|
75 | - $this->assertEquals( 1, $refItem->getStatus() ); |
|
70 | + $this->assertEquals(1, $refItem->getStatus()); |
|
71 | + $this->assertEquals('name', $refItem->getType()); |
|
72 | + $this->assertEquals('test text', $refItem->getLabel()); |
|
73 | + $this->assertEquals('Job CSV test', $refItem->getContent()); |
|
74 | + $this->assertEquals('de', $refItem->getLanguageId()); |
|
75 | + $this->assertEquals(1, $refItem->getStatus()); |
|
76 | 76 | } |
77 | 77 | |
78 | 78 | |
@@ -100,27 +100,27 @@ discard block |
||
100 | 100 | 7 => 'Long: Job CSV test 2', |
101 | 101 | ); |
102 | 102 | |
103 | - $product = $this->create( 'job_csv_test' ); |
|
103 | + $product = $this->create('job_csv_test'); |
|
104 | 104 | |
105 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint ); |
|
106 | - $object->process( $product, $data ); |
|
105 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint); |
|
106 | + $object->process($product, $data); |
|
107 | 107 | |
108 | 108 | |
109 | 109 | $pos = 0; |
110 | 110 | $listItems = $product->getListItems(); |
111 | 111 | $expected = array( |
112 | - 0 => array( 'name', 'Job CSV test' ), |
|
113 | - 1 => array( 'short', 'Short: Job CSV test' ), |
|
114 | - 2 => array( 'long', 'Long: Job CSV test' ), |
|
115 | - 3 => array( 'long', 'Long: Job CSV test 2' ), |
|
112 | + 0 => array('name', 'Job CSV test'), |
|
113 | + 1 => array('short', 'Short: Job CSV test'), |
|
114 | + 2 => array('long', 'Long: Job CSV test'), |
|
115 | + 3 => array('long', 'Long: Job CSV test 2'), |
|
116 | 116 | ); |
117 | 117 | |
118 | - $this->assertEquals( 4, count( $listItems ) ); |
|
118 | + $this->assertEquals(4, count($listItems)); |
|
119 | 119 | |
120 | - foreach( $listItems as $listItem ) |
|
120 | + foreach ($listItems as $listItem) |
|
121 | 121 | { |
122 | - $this->assertEquals( $expected[$pos][0], $listItem->getRefItem()->getType() ); |
|
123 | - $this->assertEquals( $expected[$pos][1], $listItem->getRefItem()->getContent() ); |
|
122 | + $this->assertEquals($expected[$pos][0], $listItem->getRefItem()->getType()); |
|
123 | + $this->assertEquals($expected[$pos][1], $listItem->getRefItem()->getContent()); |
|
124 | 124 | $pos++; |
125 | 125 | } |
126 | 126 | } |
@@ -143,21 +143,21 @@ discard block |
||
143 | 143 | 1 => 'Short: Job CSV test', |
144 | 144 | ); |
145 | 145 | |
146 | - $product = $this->create( 'job_csv_test' ); |
|
146 | + $product = $this->create('job_csv_test'); |
|
147 | 147 | |
148 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint ); |
|
149 | - $object->process( $product, $data ); |
|
150 | - $object->process( $product, $dataUpdate ); |
|
148 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint); |
|
149 | + $object->process($product, $data); |
|
150 | + $object->process($product, $dataUpdate); |
|
151 | 151 | |
152 | 152 | |
153 | 153 | $listItems = $product->getListItems(); |
154 | - $listItem = reset( $listItems ); |
|
154 | + $listItem = reset($listItems); |
|
155 | 155 | |
156 | - $this->assertEquals( 1, count( $listItems ) ); |
|
157 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
156 | + $this->assertEquals(1, count($listItems)); |
|
157 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
158 | 158 | |
159 | - $this->assertEquals( 'short', $listItem->getRefItem()->getType() ); |
|
160 | - $this->assertEquals( 'Short: Job CSV test', $listItem->getRefItem()->getContent() ); |
|
159 | + $this->assertEquals('short', $listItem->getRefItem()->getType()); |
|
160 | + $this->assertEquals('Short: Job CSV test', $listItem->getRefItem()->getContent()); |
|
161 | 161 | } |
162 | 162 | |
163 | 163 | |
@@ -173,18 +173,18 @@ discard block |
||
173 | 173 | 1 => 'Job CSV test', |
174 | 174 | ); |
175 | 175 | |
176 | - $product = $this->create( 'job_csv_test' ); |
|
176 | + $product = $this->create('job_csv_test'); |
|
177 | 177 | |
178 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint ); |
|
179 | - $object->process( $product, $data ); |
|
178 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint); |
|
179 | + $object->process($product, $data); |
|
180 | 180 | |
181 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, [], $this->endpoint ); |
|
182 | - $object->process( $product, [] ); |
|
181 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, [], $this->endpoint); |
|
182 | + $object->process($product, []); |
|
183 | 183 | |
184 | 184 | |
185 | 185 | $listItems = $product->getListItems(); |
186 | 186 | |
187 | - $this->assertEquals( 0, count( $listItems ) ); |
|
187 | + $this->assertEquals(0, count($listItems)); |
|
188 | 188 | } |
189 | 189 | |
190 | 190 | |
@@ -204,15 +204,15 @@ discard block |
||
204 | 204 | 3 => '', |
205 | 205 | ); |
206 | 206 | |
207 | - $product = $this->create( 'job_csv_test' ); |
|
207 | + $product = $this->create('job_csv_test'); |
|
208 | 208 | |
209 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint ); |
|
210 | - $object->process( $product, $data ); |
|
209 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint); |
|
210 | + $object->process($product, $data); |
|
211 | 211 | |
212 | 212 | |
213 | 213 | $listItems = $product->getListItems(); |
214 | 214 | |
215 | - $this->assertEquals( 1, count( $listItems ) ); |
|
215 | + $this->assertEquals(1, count($listItems)); |
|
216 | 216 | } |
217 | 217 | |
218 | 218 | |
@@ -236,32 +236,32 @@ discard block |
||
236 | 236 | 5 => 'default', |
237 | 237 | ); |
238 | 238 | |
239 | - $this->context->getConfig()->set( 'controller/common/product/import/csv/processor/text/listtypes', array( 'default' ) ); |
|
239 | + $this->context->getConfig()->set('controller/common/product/import/csv/processor/text/listtypes', array('default')); |
|
240 | 240 | |
241 | - $product = $this->create( 'job_csv_test' ); |
|
241 | + $product = $this->create('job_csv_test'); |
|
242 | 242 | |
243 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint ); |
|
244 | - $object->process( $product, $data ); |
|
243 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint); |
|
244 | + $object->process($product, $data); |
|
245 | 245 | |
246 | 246 | |
247 | 247 | $listItems = $product->getListItems(); |
248 | - $listItem = reset( $listItems ); |
|
248 | + $listItem = reset($listItems); |
|
249 | 249 | |
250 | - $this->assertEquals( 1, count( $listItems ) ); |
|
251 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
250 | + $this->assertEquals(1, count($listItems)); |
|
251 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
252 | 252 | |
253 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
254 | - $this->assertEquals( 'short', $listItem->getRefItem()->getType() ); |
|
255 | - $this->assertEquals( 'test short', $listItem->getRefItem()->getContent() ); |
|
253 | + $this->assertEquals('default', $listItem->getType()); |
|
254 | + $this->assertEquals('short', $listItem->getRefItem()->getType()); |
|
255 | + $this->assertEquals('test short', $listItem->getRefItem()->getContent()); |
|
256 | 256 | } |
257 | 257 | |
258 | 258 | |
259 | 259 | /** |
260 | 260 | * @param string $code |
261 | 261 | */ |
262 | - protected function create( $code ) |
|
262 | + protected function create($code) |
|
263 | 263 | { |
264 | - $manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context ); |
|
265 | - return $manager->createItem()->setCode( $code ); |
|
264 | + $manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context); |
|
265 | + return $manager->createItem()->setCode($code); |
|
266 | 266 | } |
267 | 267 | } |
268 | 268 | \ No newline at end of file |
@@ -17,16 +17,16 @@ discard block |
||
17 | 17 | |
18 | 18 | protected function setUp() |
19 | 19 | { |
20 | - \Aimeos\MShop\Factory::setCache( true ); |
|
20 | + \Aimeos\MShop\Factory::setCache(true); |
|
21 | 21 | |
22 | 22 | $this->context = \TestHelperCntl::getContext(); |
23 | - $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, [] ); |
|
23 | + $this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, []); |
|
24 | 24 | } |
25 | 25 | |
26 | 26 | |
27 | 27 | protected function tearDown() |
28 | 28 | { |
29 | - \Aimeos\MShop\Factory::setCache( false ); |
|
29 | + \Aimeos\MShop\Factory::setCache(false); |
|
30 | 30 | \Aimeos\MShop\Factory::clear(); |
31 | 31 | } |
32 | 32 | |
@@ -57,29 +57,29 @@ discard block |
||
57 | 57 | 8 => 'variant', |
58 | 58 | ); |
59 | 59 | |
60 | - $product = $this->create( 'job_csv_test' ); |
|
60 | + $product = $this->create('job_csv_test'); |
|
61 | 61 | |
62 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint ); |
|
63 | - $object->process( $product, $data ); |
|
62 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint); |
|
63 | + $object->process($product, $data); |
|
64 | 64 | |
65 | 65 | |
66 | 66 | $pos = 0; |
67 | 67 | $listItems = $product->getListItems(); |
68 | 68 | $expected = array( |
69 | - array( 'variant', 'length', '30' ), |
|
70 | - array( 'variant', 'width', '29' ), |
|
71 | - array( 'variant', 'color', 'white' ), |
|
69 | + array('variant', 'length', '30'), |
|
70 | + array('variant', 'width', '29'), |
|
71 | + array('variant', 'color', 'white'), |
|
72 | 72 | ); |
73 | 73 | |
74 | - $this->assertEquals( 3, count( $listItems ) ); |
|
74 | + $this->assertEquals(3, count($listItems)); |
|
75 | 75 | |
76 | - foreach( $listItems as $listItem ) |
|
76 | + foreach ($listItems as $listItem) |
|
77 | 77 | { |
78 | - $this->assertEquals( 1, $listItem->getStatus() ); |
|
79 | - $this->assertEquals( 'attribute', $listItem->getDomain() ); |
|
80 | - $this->assertEquals( $expected[$pos][0], $listItem->getType() ); |
|
81 | - $this->assertEquals( $expected[$pos][1], $listItem->getRefItem()->getType() ); |
|
82 | - $this->assertEquals( $expected[$pos][2], $listItem->getRefItem()->getCode() ); |
|
78 | + $this->assertEquals(1, $listItem->getStatus()); |
|
79 | + $this->assertEquals('attribute', $listItem->getDomain()); |
|
80 | + $this->assertEquals($expected[$pos][0], $listItem->getType()); |
|
81 | + $this->assertEquals($expected[$pos][1], $listItem->getRefItem()->getType()); |
|
82 | + $this->assertEquals($expected[$pos][2], $listItem->getRefItem()->getCode()); |
|
83 | 83 | $pos++; |
84 | 84 | } |
85 | 85 | } |
@@ -99,25 +99,25 @@ discard block |
||
99 | 99 | 2 => 'variant', |
100 | 100 | ); |
101 | 101 | |
102 | - $product = $this->create( 'job_csv_test' ); |
|
102 | + $product = $this->create('job_csv_test'); |
|
103 | 103 | |
104 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, $mapping, $this->endpoint ); |
|
105 | - $object->process( $product, $data ); |
|
104 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint); |
|
105 | + $object->process($product, $data); |
|
106 | 106 | |
107 | 107 | |
108 | 108 | $pos = 0; |
109 | - $codes = array( 'white', 'black', 'aimeos' ); |
|
109 | + $codes = array('white', 'black', 'aimeos'); |
|
110 | 110 | |
111 | - foreach( $product->getListItems() as $listItems ) |
|
111 | + foreach ($product->getListItems() as $listItems) |
|
112 | 112 | { |
113 | - $this->assertEquals( 3, count( $listItems ) ); |
|
113 | + $this->assertEquals(3, count($listItems)); |
|
114 | 114 | |
115 | - foreach( $listItems as $listItem ) |
|
115 | + foreach ($listItems as $listItem) |
|
116 | 116 | { |
117 | - $this->assertEquals( 1, $listItem->getStatus() ); |
|
118 | - $this->assertEquals( 'attribute', $listItem->getDomain() ); |
|
119 | - $this->assertEquals( 'variant', $listItem->getType() ); |
|
120 | - $this->assertEquals( $codes[$pos], $listItem->getRefItem()->getCode() ); |
|
117 | + $this->assertEquals(1, $listItem->getStatus()); |
|
118 | + $this->assertEquals('attribute', $listItem->getDomain()); |
|
119 | + $this->assertEquals('variant', $listItem->getType()); |
|
120 | + $this->assertEquals($codes[$pos], $listItem->getRefItem()->getCode()); |
|
121 | 121 | $pos++; |
122 | 122 | } |
123 | 123 | } |
@@ -141,21 +141,21 @@ discard block |
||
141 | 141 | 1 => '29', |
142 | 142 | ); |
143 | 143 | |
144 | - $product = $this->create( 'job_csv_test' ); |
|
144 | + $product = $this->create('job_csv_test'); |
|
145 | 145 | |
146 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint ); |
|
147 | - $object->process( $product, $data ); |
|
148 | - $object->process( $product, $dataUpdate ); |
|
146 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint); |
|
147 | + $object->process($product, $data); |
|
148 | + $object->process($product, $dataUpdate); |
|
149 | 149 | |
150 | 150 | |
151 | 151 | $listItems = $product->getListItems(); |
152 | - $listItem = reset( $listItems ); |
|
152 | + $listItem = reset($listItems); |
|
153 | 153 | |
154 | - $this->assertEquals( 1, count( $listItems ) ); |
|
155 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
154 | + $this->assertEquals(1, count($listItems)); |
|
155 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
156 | 156 | |
157 | - $this->assertEquals( 'width', $listItem->getRefItem()->getType() ); |
|
158 | - $this->assertEquals( '29', $listItem->getRefItem()->getCode() ); |
|
157 | + $this->assertEquals('width', $listItem->getRefItem()->getType()); |
|
158 | + $this->assertEquals('29', $listItem->getRefItem()->getCode()); |
|
159 | 159 | } |
160 | 160 | |
161 | 161 | |
@@ -171,18 +171,18 @@ discard block |
||
171 | 171 | 1 => '30', |
172 | 172 | ); |
173 | 173 | |
174 | - $product = $this->create( 'job_csv_test' ); |
|
174 | + $product = $this->create('job_csv_test'); |
|
175 | 175 | |
176 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint ); |
|
177 | - $object->process( $product, $data ); |
|
176 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint); |
|
177 | + $object->process($product, $data); |
|
178 | 178 | |
179 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, [], $this->endpoint ); |
|
180 | - $object->process( $product, [] ); |
|
179 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, [], $this->endpoint); |
|
180 | + $object->process($product, []); |
|
181 | 181 | |
182 | 182 | |
183 | 183 | $listItems = $product->getListItems(); |
184 | 184 | |
185 | - $this->assertEquals( 0, count( $listItems ) ); |
|
185 | + $this->assertEquals(0, count($listItems)); |
|
186 | 186 | } |
187 | 187 | |
188 | 188 | |
@@ -202,15 +202,15 @@ discard block |
||
202 | 202 | 3 => '30', |
203 | 203 | ); |
204 | 204 | |
205 | - $product = $this->create( 'job_csv_test' ); |
|
205 | + $product = $this->create('job_csv_test'); |
|
206 | 206 | |
207 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint ); |
|
208 | - $object->process( $product, $data ); |
|
207 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint); |
|
208 | + $object->process($product, $data); |
|
209 | 209 | |
210 | 210 | |
211 | 211 | $listItems = $product->getListItems(); |
212 | 212 | |
213 | - $this->assertEquals( 1, count( $listItems ) ); |
|
213 | + $this->assertEquals(1, count($listItems)); |
|
214 | 214 | } |
215 | 215 | |
216 | 216 | |
@@ -226,17 +226,17 @@ discard block |
||
226 | 226 | 1 => '30', |
227 | 227 | ); |
228 | 228 | |
229 | - $product = $this->create( 'job_csv_test' ); |
|
229 | + $product = $this->create('job_csv_test'); |
|
230 | 230 | |
231 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint ); |
|
231 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint); |
|
232 | 232 | |
233 | 233 | try { |
234 | - $object->process( $product, $data ); |
|
235 | - } catch( \Exception $e ) { |
|
234 | + $object->process($product, $data); |
|
235 | + } catch (\Exception $e) { |
|
236 | 236 | return; |
237 | 237 | } |
238 | 238 | |
239 | - $this->markTestFail( 'No exception thrown' ); |
|
239 | + $this->markTestFail('No exception thrown'); |
|
240 | 240 | } |
241 | 241 | |
242 | 242 | |
@@ -260,32 +260,32 @@ discard block |
||
260 | 260 | 5 => 'default', |
261 | 261 | ); |
262 | 262 | |
263 | - $this->context->getConfig()->set( 'controller/common/product/import/csv/processor/attribute/listtypes', array( 'default' ) ); |
|
263 | + $this->context->getConfig()->set('controller/common/product/import/csv/processor/attribute/listtypes', array('default')); |
|
264 | 264 | |
265 | - $product = $this->create( 'job_csv_test' ); |
|
265 | + $product = $this->create('job_csv_test'); |
|
266 | 266 | |
267 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint ); |
|
268 | - $object->process( $product, $data ); |
|
267 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint); |
|
268 | + $object->process($product, $data); |
|
269 | 269 | |
270 | 270 | |
271 | 271 | $listItems = $product->getListItems(); |
272 | - $listItem = reset( $listItems ); |
|
272 | + $listItem = reset($listItems); |
|
273 | 273 | |
274 | - $this->assertEquals( 1, count( $listItems ) ); |
|
275 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem ); |
|
274 | + $this->assertEquals(1, count($listItems)); |
|
275 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem); |
|
276 | 276 | |
277 | - $this->assertEquals( 'default', $listItem->getType() ); |
|
278 | - $this->assertEquals( 'width', $listItem->getRefItem()->getType() ); |
|
279 | - $this->assertEquals( '30', $listItem->getRefItem()->getCode() ); |
|
277 | + $this->assertEquals('default', $listItem->getType()); |
|
278 | + $this->assertEquals('width', $listItem->getRefItem()->getType()); |
|
279 | + $this->assertEquals('30', $listItem->getRefItem()->getCode()); |
|
280 | 280 | } |
281 | 281 | |
282 | 282 | |
283 | 283 | /** |
284 | 284 | * @param string $code |
285 | 285 | */ |
286 | - protected function create( $code ) |
|
286 | + protected function create($code) |
|
287 | 287 | { |
288 | - $manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context ); |
|
289 | - return $manager->createItem()->setCode( $code ); |
|
288 | + $manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context); |
|
289 | + return $manager->createItem()->setCode($code); |
|
290 | 290 | } |
291 | 291 | } |
292 | 292 | \ No newline at end of file |
@@ -40,44 +40,44 @@ |
||
40 | 40 | * @param array $data List of CSV fields with position as key and data as value |
41 | 41 | * @return array List of data which hasn't been imported |
42 | 42 | */ |
43 | - public function process( \Aimeos\MShop\Product\Item\Iface $product, array $data ) |
|
43 | + public function process(\Aimeos\MShop\Product\Item\Iface $product, array $data) |
|
44 | 44 | { |
45 | - $manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product/property' ); |
|
45 | + $manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product/property'); |
|
46 | 46 | |
47 | 47 | $propMap = []; |
48 | - $items = $product->getPropertyItems( null, false ); |
|
49 | - $map = $this->getMappedChunk( $data, $this->getMapping() ); |
|
48 | + $items = $product->getPropertyItems(null, false); |
|
49 | + $map = $this->getMappedChunk($data, $this->getMapping()); |
|
50 | 50 | |
51 | - foreach( $items as $item ) { |
|
52 | - $propMap[ $item->getValue() ][ $item->getType() ] = $item; |
|
51 | + foreach ($items as $item) { |
|
52 | + $propMap[$item->getValue()][$item->getType()] = $item; |
|
53 | 53 | } |
54 | 54 | |
55 | - foreach( $map as $list ) |
|
55 | + foreach ($map as $list) |
|
56 | 56 | { |
57 | - $typecode = trim( $list['product.property.type'] ); |
|
58 | - $value = trim( $list['product.property.value'] ); |
|
57 | + $typecode = trim($list['product.property.type']); |
|
58 | + $value = trim($list['product.property.value']); |
|
59 | 59 | |
60 | - if( $typecode == '' || $value == '' ) { |
|
60 | + if ($typecode == '' || $value == '') { |
|
61 | 61 | continue; |
62 | 62 | } |
63 | 63 | |
64 | - if( isset( $propMap[$value][$typecode] ) ) |
|
64 | + if (isset($propMap[$value][$typecode])) |
|
65 | 65 | { |
66 | 66 | $item = $propMap[$value][$typecode]; |
67 | - unset( $items[ $item->getId() ] ); |
|
67 | + unset($items[$item->getId()]); |
|
68 | 68 | } |
69 | 69 | else |
70 | 70 | { |
71 | - $item = $manager->createItem( $typecode, 'product' ); |
|
71 | + $item = $manager->createItem($typecode, 'product'); |
|
72 | 72 | } |
73 | 73 | |
74 | - $item->fromArray( $list ); |
|
74 | + $item->fromArray($list); |
|
75 | 75 | |
76 | - $product->addPropertyItem( $item ); |
|
76 | + $product->addPropertyItem($item); |
|
77 | 77 | } |
78 | 78 | |
79 | - $product->deletePropertyItems( $items ); |
|
79 | + $product->deletePropertyItems($items); |
|
80 | 80 | |
81 | - return $this->getObject()->process( $product, $data ); |
|
81 | + return $this->getObject()->process($product, $data); |
|
82 | 82 | } |
83 | 83 | } |
@@ -65,8 +65,7 @@ |
||
65 | 65 | { |
66 | 66 | $item = $propMap[$value][$typecode]; |
67 | 67 | unset( $items[ $item->getId() ] ); |
68 | - } |
|
69 | - else |
|
68 | + } else |
|
70 | 69 | { |
71 | 70 | $item = $manager->createItem( $typecode, 'product' ); |
72 | 71 | } |
@@ -42,10 +42,10 @@ discard block |
||
42 | 42 | * @param array $mapping Associative list of field position in CSV as key and domain item key as value |
43 | 43 | * @param \Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object Decorated processor |
44 | 44 | */ |
45 | - public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping, |
|
46 | - \Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null ) |
|
45 | + public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping, |
|
46 | + \Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null) |
|
47 | 47 | { |
48 | - parent::__construct( $context, $mapping, $object ); |
|
48 | + parent::__construct($context, $mapping, $object); |
|
49 | 49 | |
50 | 50 | /** controller/common/product/import/csv/processor/media/listtypes |
51 | 51 | * Names of the product list types for media that are updated or removed |
@@ -66,7 +66,7 @@ discard block |
||
66 | 66 | * @see controller/common/product/import/csv/processor/price/listtypes |
67 | 67 | * @see controller/common/product/import/csv/processor/text/listtypes |
68 | 68 | */ |
69 | - $this->listTypes = $context->getConfig()->get( 'controller/common/product/import/csv/processor/media/listtypes' ); |
|
69 | + $this->listTypes = $context->getConfig()->get('controller/common/product/import/csv/processor/media/listtypes'); |
|
70 | 70 | } |
71 | 71 | |
72 | 72 | |
@@ -77,62 +77,62 @@ discard block |
||
77 | 77 | * @param array $data List of CSV fields with position as key and data as value |
78 | 78 | * @return array List of data which hasn't been imported |
79 | 79 | */ |
80 | - public function process( \Aimeos\MShop\Product\Item\Iface $product, array $data ) |
|
80 | + public function process(\Aimeos\MShop\Product\Item\Iface $product, array $data) |
|
81 | 81 | { |
82 | 82 | $context = $this->getContext(); |
83 | - $manager = \Aimeos\MShop\Factory::createManager( $context, 'media' ); |
|
84 | - $listManager = \Aimeos\MShop\Factory::createManager( $context, 'product/lists' ); |
|
85 | - $separator = $context->getConfig()->get( 'controller/common/product/import/csv/separator', "\n" ); |
|
83 | + $manager = \Aimeos\MShop\Factory::createManager($context, 'media'); |
|
84 | + $listManager = \Aimeos\MShop\Factory::createManager($context, 'product/lists'); |
|
85 | + $separator = $context->getConfig()->get('controller/common/product/import/csv/separator', "\n"); |
|
86 | 86 | |
87 | 87 | $listMap = []; |
88 | - $map = $this->getMappedChunk( $data, $this->getMapping() ); |
|
89 | - $listItems = $product->getListItems( 'media', $this->listTypes ); |
|
88 | + $map = $this->getMappedChunk($data, $this->getMapping()); |
|
89 | + $listItems = $product->getListItems('media', $this->listTypes); |
|
90 | 90 | |
91 | - foreach( $listItems as $listItem ) |
|
91 | + foreach ($listItems as $listItem) |
|
92 | 92 | { |
93 | - if( ( $refItem = $listItem->getRefItem() ) !== null ) { |
|
94 | - $listMap[ $refItem->getUrl() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem; |
|
93 | + if (($refItem = $listItem->getRefItem()) !== null) { |
|
94 | + $listMap[$refItem->getUrl()][$refItem->getType()][$listItem->getType()] = $listItem; |
|
95 | 95 | } |
96 | 96 | } |
97 | 97 | |
98 | - foreach( $map as $pos => $list ) |
|
98 | + foreach ($map as $pos => $list) |
|
99 | 99 | { |
100 | - if( $this->checkEntry( $list ) === false ) { |
|
100 | + if ($this->checkEntry($list) === false) { |
|
101 | 101 | continue; |
102 | 102 | } |
103 | 103 | |
104 | - $urls = explode( $separator, trim( $list['media.url'] ) ); |
|
105 | - $type = trim( $this->getValue( $list, 'media.type', 'default' ) ); |
|
106 | - $typecode = trim( $this->getValue( $list, 'product.lists.type', 'default' ) ); |
|
104 | + $urls = explode($separator, trim($list['media.url'])); |
|
105 | + $type = trim($this->getValue($list, 'media.type', 'default')); |
|
106 | + $typecode = trim($this->getValue($list, 'product.lists.type', 'default')); |
|
107 | 107 | |
108 | - foreach( $urls as $url ) |
|
108 | + foreach ($urls as $url) |
|
109 | 109 | { |
110 | - $url = trim( $url ); |
|
110 | + $url = trim($url); |
|
111 | 111 | |
112 | - if( isset( $listMap[$url][$type][$typecode] ) ) |
|
112 | + if (isset($listMap[$url][$type][$typecode])) |
|
113 | 113 | { |
114 | 114 | $listItem = $listMap[$url][$type][$typecode]; |
115 | 115 | $refItem = $listItem->getRefItem(); |
116 | - unset( $listItems[ $listItem->getId() ] ); |
|
116 | + unset($listItems[$listItem->getId()]); |
|
117 | 117 | } |
118 | 118 | else |
119 | 119 | { |
120 | - $listItem = $listManager->createItem( $typecode, 'media' ); |
|
121 | - $refItem = $manager->createItem( $type, 'product' ); |
|
120 | + $listItem = $listManager->createItem($typecode, 'media'); |
|
121 | + $refItem = $manager->createItem($type, 'product'); |
|
122 | 122 | } |
123 | 123 | |
124 | 124 | $list['media.url'] = $url; |
125 | 125 | |
126 | - $refItem->fromArray( $this->addItemDefaults( $list ) ); |
|
127 | - $listItem->fromArray( $this->addListItemDefaults( $list, $pos++ ) ); |
|
126 | + $refItem->fromArray($this->addItemDefaults($list)); |
|
127 | + $listItem->fromArray($this->addListItemDefaults($list, $pos++)); |
|
128 | 128 | |
129 | - $product->addListItem( 'media', $listItem, $refItem ); |
|
129 | + $product->addListItem('media', $listItem, $refItem); |
|
130 | 130 | } |
131 | 131 | } |
132 | 132 | |
133 | - $product->deleteListItems( $listItems, true ); |
|
133 | + $product->deleteListItems($listItems, true); |
|
134 | 134 | |
135 | - return $this->getObject()->process( $product, $data ); |
|
135 | + return $this->getObject()->process($product, $data); |
|
136 | 136 | } |
137 | 137 | |
138 | 138 | |
@@ -142,17 +142,17 @@ discard block |
||
142 | 142 | * @param array $list Associative list of domain item keys and their values, e.g. "media.status" => 1 |
143 | 143 | * @return array Given associative list enriched by default values if they were not already set |
144 | 144 | */ |
145 | - protected function addItemDefaults( array $list ) |
|
145 | + protected function addItemDefaults(array $list) |
|
146 | 146 | { |
147 | - if( !isset( $list['media.label'] ) ) { |
|
147 | + if (!isset($list['media.label'])) { |
|
148 | 148 | $list['media.label'] = $list['media.url']; |
149 | 149 | } |
150 | 150 | |
151 | - if( !isset( $list['media.preview'] ) ) { |
|
151 | + if (!isset($list['media.preview'])) { |
|
152 | 152 | $list['media.preview'] = $list['media.url']; |
153 | 153 | } |
154 | 154 | |
155 | - if( !isset( $list['media.status'] ) ) { |
|
155 | + if (!isset($list['media.status'])) { |
|
156 | 156 | $list['media.status'] = 1; |
157 | 157 | } |
158 | 158 | |
@@ -166,10 +166,10 @@ discard block |
||
166 | 166 | * @param array $list Associative list of key/value pairs from the mapping |
167 | 167 | * @return boolean True if valid, false if not |
168 | 168 | */ |
169 | - protected function checkEntry( array $list ) |
|
169 | + protected function checkEntry(array $list) |
|
170 | 170 | { |
171 | - if( !isset( $list['media.url'] ) || trim( $list['media.url'] ) === '' || isset( $list['product.lists.type'] ) |
|
172 | - && $this->listTypes !== null && !in_array( trim( $list['product.lists.type'] ), (array) $this->listTypes ) |
|
171 | + if (!isset($list['media.url']) || trim($list['media.url']) === '' || isset($list['product.lists.type']) |
|
172 | + && $this->listTypes !== null && !in_array(trim($list['product.lists.type']), (array) $this->listTypes) |
|
173 | 173 | ) { |
174 | 174 | return false; |
175 | 175 | } |
@@ -114,8 +114,7 @@ |
||
114 | 114 | $listItem = $listMap[$url][$type][$typecode]; |
115 | 115 | $refItem = $listItem->getRefItem(); |
116 | 116 | unset( $listItems[ $listItem->getId() ] ); |
117 | - } |
|
118 | - else |
|
117 | + } else |
|
119 | 118 | { |
120 | 119 | $listItem = $listManager->createItem( $typecode, 'media' ); |
121 | 120 | $refItem = $manager->createItem( $type, 'product' ); |