Completed
Push — master ( 7a3ba1...08fedb )
by Aimeos
02:02
created
controller/jobs/src/Controller/Jobs/Product/Export/Sitemap/Standard.php 1 patch
Spacing   +50 added lines, -50 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
Controller/Common/Product/Import/Csv/Processor/Property/StandardTest.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -17,16 +17,16 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.
tests/Controller/Common/Product/Import/Csv/Processor/Media/StandardTest.php 1 patch
Spacing   +63 added lines, -63 removed lines patch added patch discarded remove patch
@@ -17,16 +17,16 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.
Controller/Common/Product/Import/Csv/Processor/Product/StandardTest.php 1 patch
Spacing   +57 added lines, -57 removed lines patch added patch discarded remove patch
@@ -18,17 +18,17 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
tests/Controller/Common/Product/Import/Csv/Processor/Price/StandardTest.php 1 patch
Spacing   +58 added lines, -58 removed lines patch added patch discarded remove patch
@@ -17,16 +17,16 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.
tests/Controller/Common/Product/Import/Csv/Processor/Text/StandardTest.php 1 patch
Spacing   +61 added lines, -61 removed lines patch added patch discarded remove patch
@@ -17,16 +17,16 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.
Controller/Common/Product/Import/Csv/Processor/Attribute/StandardTest.php 1 patch
Spacing   +64 added lines, -64 removed lines patch added patch discarded remove patch
@@ -17,16 +17,16 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.
src/Controller/Common/Product/Import/Csv/Processor/Property/Standard.php 2 patches
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -40,44 +40,44 @@
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -65,8 +65,7 @@
 block discarded – undo
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
 			}
Please login to merge, or discard this patch.
src/Controller/Common/Product/Import/Csv/Processor/Media/Standard.php 2 patches
Spacing   +36 added lines, -36 removed lines patch added patch discarded remove patch
@@ -42,10 +42,10 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 		}
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -114,8 +114,7 @@
 block discarded – undo
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' );
Please login to merge, or discard this patch.