Completed
Push — master ( 8fed4a...ad140d )
by Aimeos
03:06
created
tests/Controller/Common/Product/Import/Csv/Processor/Text/StandardTest.php 1 patch
Spacing   +98 added lines, -98 removed lines patch added patch discarded remove patch
@@ -21,10 +21,10 @@  discard block
 block discarded – undo
21 21
 	 */
22 22
 	protected function setUp()
23 23
 	{
24
-		\Aimeos\MShop\Factory::setCache( true );
24
+		\Aimeos\MShop\Factory::setCache(true);
25 25
 
26 26
 		$this->context = \TestHelperCntl::getContext();
27
-		$this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, array() );
27
+		$this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, array());
28 28
 	}
29 29
 
30 30
 
@@ -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
 
@@ -59,35 +59,35 @@  discard block
 block discarded – undo
59 59
 			4 => 1,
60 60
 		);
61 61
 
62
-		$product = $this->create( 'job_csv_test' );
62
+		$product = $this->create('job_csv_test');
63 63
 
64
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint );
65
-		$object->process( $product, $data );
64
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint);
65
+		$object->process($product, $data);
66 66
 
67
-		$product = $this->get( 'job_csv_test' );
68
-		$this->delete( $product );
67
+		$product = $this->get('job_csv_test');
68
+		$this->delete($product);
69 69
 
70 70
 
71 71
 		$listItems = $product->getListItems();
72
-		$listItem = reset( $listItems );
72
+		$listItem = reset($listItems);
73 73
 
74
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem );
75
-		$this->assertEquals( 1, count( $listItems ) );
74
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem);
75
+		$this->assertEquals(1, count($listItems));
76 76
 
77
-		$this->assertEquals( 1, $listItem->getStatus() );
78
-		$this->assertEquals( 0, $listItem->getPosition() );
79
-		$this->assertEquals( 'text', $listItem->getDomain() );
80
-		$this->assertEquals( 'default', $listItem->getType() );
77
+		$this->assertEquals(1, $listItem->getStatus());
78
+		$this->assertEquals(0, $listItem->getPosition());
79
+		$this->assertEquals('text', $listItem->getDomain());
80
+		$this->assertEquals('default', $listItem->getType());
81 81
 
82 82
 		$refItem = $listItem->getRefItem();
83 83
 
84
-		$this->assertEquals( 1, $refItem->getStatus() );
85
-		$this->assertEquals( 'name', $refItem->getType() );
86
-		$this->assertEquals( 'product', $refItem->getDomain() );
87
-		$this->assertEquals( 'test text', $refItem->getLabel() );
88
-		$this->assertEquals( 'Job CSV test', $refItem->getContent() );
89
-		$this->assertEquals( 'de', $refItem->getLanguageId() );
90
-		$this->assertEquals( 1, $refItem->getStatus() );
84
+		$this->assertEquals(1, $refItem->getStatus());
85
+		$this->assertEquals('name', $refItem->getType());
86
+		$this->assertEquals('product', $refItem->getDomain());
87
+		$this->assertEquals('test text', $refItem->getLabel());
88
+		$this->assertEquals('Job CSV test', $refItem->getContent());
89
+		$this->assertEquals('de', $refItem->getLanguageId());
90
+		$this->assertEquals(1, $refItem->getStatus());
91 91
 	}
92 92
 
93 93
 
@@ -115,30 +115,30 @@  discard block
 block discarded – undo
115 115
 			7 => 'Long: Job CSV test 2',
116 116
 		);
117 117
 
118
-		$product = $this->create( 'job_csv_test' );
118
+		$product = $this->create('job_csv_test');
119 119
 
120
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint );
121
-		$object->process( $product, $data );
120
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint);
121
+		$object->process($product, $data);
122 122
 
123
-		$product = $this->get( 'job_csv_test' );
124
-		$this->delete( $product );
123
+		$product = $this->get('job_csv_test');
124
+		$this->delete($product);
125 125
 
126 126
 
127 127
 		$pos = 0;
128 128
 		$listItems = $product->getListItems();
129 129
 		$expected = array(
130
-			0 => array( 'name', 'Job CSV test' ),
131
-			1 => array( 'short', 'Short: Job CSV test' ),
132
-			2 => array( 'long', 'Long: Job CSV test' ),
133
-			3 => array( 'long', 'Long: Job CSV test 2' ),
130
+			0 => array('name', 'Job CSV test'),
131
+			1 => array('short', 'Short: Job CSV test'),
132
+			2 => array('long', 'Long: Job CSV test'),
133
+			3 => array('long', 'Long: Job CSV test 2'),
134 134
 		);
135 135
 
136
-		$this->assertEquals( 4, count( $listItems ) );
136
+		$this->assertEquals(4, count($listItems));
137 137
 
138
-		foreach( $listItems as $listItem )
138
+		foreach ($listItems as $listItem)
139 139
 		{
140
-			$this->assertEquals( $expected[$pos][0], $listItem->getRefItem()->getType() );
141
-			$this->assertEquals( $expected[$pos][1], $listItem->getRefItem()->getContent() );
140
+			$this->assertEquals($expected[$pos][0], $listItem->getRefItem()->getType());
141
+			$this->assertEquals($expected[$pos][1], $listItem->getRefItem()->getContent());
142 142
 			$pos++;
143 143
 		}
144 144
 	}
@@ -161,27 +161,27 @@  discard block
 block discarded – undo
161 161
 			1 => 'Short: Job CSV test',
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\Text\Standard( $this->context, $mapping, $this->endpoint );
167
-		$object->process( $product, $data );
166
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint);
167
+		$object->process($product, $data);
168 168
 
169
-		$product = $this->get( 'job_csv_test' );
169
+		$product = $this->get('job_csv_test');
170 170
 
171
-		$object->process( $product, $dataUpdate );
171
+		$object->process($product, $dataUpdate);
172 172
 
173
-		$product = $this->get( 'job_csv_test' );
174
-		$this->delete( $product );
173
+		$product = $this->get('job_csv_test');
174
+		$this->delete($product);
175 175
 
176 176
 
177 177
 		$listItems = $product->getListItems();
178
-		$listItem = reset( $listItems );
178
+		$listItem = reset($listItems);
179 179
 
180
-		$this->assertEquals( 1, count( $listItems ) );
181
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem );
180
+		$this->assertEquals(1, count($listItems));
181
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem);
182 182
 
183
-		$this->assertEquals( 'short', $listItem->getRefItem()->getType() );
184
-		$this->assertEquals( 'Short: Job CSV test', $listItem->getRefItem()->getContent() );
183
+		$this->assertEquals('short', $listItem->getRefItem()->getType());
184
+		$this->assertEquals('Short: Job CSV test', $listItem->getRefItem()->getContent());
185 185
 	}
186 186
 
187 187
 
@@ -197,23 +197,23 @@  discard block
 block discarded – undo
197 197
 			1 => 'Job CSV test',
198 198
 		);
199 199
 
200
-		$product = $this->create( 'job_csv_test' );
200
+		$product = $this->create('job_csv_test');
201 201
 
202
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint );
203
-		$object->process( $product, $data );
202
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint);
203
+		$object->process($product, $data);
204 204
 
205
-		$product = $this->get( 'job_csv_test' );
205
+		$product = $this->get('job_csv_test');
206 206
 
207
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, array(), $this->endpoint );
208
-		$object->process( $product, array() );
207
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, array(), $this->endpoint);
208
+		$object->process($product, array());
209 209
 
210
-		$product = $this->get( 'job_csv_test' );
211
-		$this->delete( $product );
210
+		$product = $this->get('job_csv_test');
211
+		$this->delete($product);
212 212
 
213 213
 
214 214
 		$listItems = $product->getListItems();
215 215
 
216
-		$this->assertEquals( 0, count( $listItems ) );
216
+		$this->assertEquals(0, count($listItems));
217 217
 	}
218 218
 
219 219
 
@@ -233,18 +233,18 @@  discard block
 block discarded – undo
233 233
 			3 => '',
234 234
 		);
235 235
 
236
-		$product = $this->create( 'job_csv_test' );
236
+		$product = $this->create('job_csv_test');
237 237
 
238
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint );
239
-		$object->process( $product, $data );
238
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint);
239
+		$object->process($product, $data);
240 240
 
241
-		$product = $this->get( 'job_csv_test' );
242
-		$this->delete( $product );
241
+		$product = $this->get('job_csv_test');
242
+		$this->delete($product);
243 243
 
244 244
 
245 245
 		$listItems = $product->getListItems();
246 246
 
247
-		$this->assertEquals( 1, count( $listItems ) );
247
+		$this->assertEquals(1, count($listItems));
248 248
 	}
249 249
 
250 250
 
@@ -268,85 +268,85 @@  discard block
 block discarded – undo
268 268
 			5 => 'default',
269 269
 		);
270 270
 
271
-		$this->context->getConfig()->set( 'controller/common/product/import/csv/processor/text/listtypes', array( 'default' ) );
271
+		$this->context->getConfig()->set('controller/common/product/import/csv/processor/text/listtypes', array('default'));
272 272
 
273
-		$product = $this->create( 'job_csv_test' );
273
+		$product = $this->create('job_csv_test');
274 274
 
275
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard( $this->context, $mapping, $this->endpoint );
276
-		$object->process( $product, $data );
275
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Text\Standard($this->context, $mapping, $this->endpoint);
276
+		$object->process($product, $data);
277 277
 
278
-		$product = $this->get( 'job_csv_test' );
279
-		$this->delete( $product );
278
+		$product = $this->get('job_csv_test');
279
+		$this->delete($product);
280 280
 
281 281
 
282 282
 		$listItems = $product->getListItems();
283
-		$listItem = reset( $listItems );
283
+		$listItem = reset($listItems);
284 284
 
285
-		$this->assertEquals( 1, count( $listItems ) );
286
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem );
285
+		$this->assertEquals(1, count($listItems));
286
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem);
287 287
 
288
-		$this->assertEquals( 'default', $listItem->getType() );
289
-		$this->assertEquals( 'short', $listItem->getRefItem()->getType() );
290
-		$this->assertEquals( 'test short', $listItem->getRefItem()->getContent() );
288
+		$this->assertEquals('default', $listItem->getType());
289
+		$this->assertEquals('short', $listItem->getRefItem()->getType());
290
+		$this->assertEquals('test short', $listItem->getRefItem()->getContent());
291 291
 	}
292 292
 
293 293
 
294 294
 	/**
295 295
 	 * @param string $code
296 296
 	 */
297
-	protected function create( $code )
297
+	protected function create($code)
298 298
 	{
299
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context );
300
-		$typeManager = $manager->getSubManager( 'type' );
299
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context);
300
+		$typeManager = $manager->getSubManager('type');
301 301
 
302 302
 		$typeSearch = $typeManager->createSearch();
303
-		$typeSearch->setConditions( $typeSearch->compare( '==', 'product.type.code', 'default' ) );
304
-		$typeResult = $typeManager->searchItems( $typeSearch );
303
+		$typeSearch->setConditions($typeSearch->compare('==', 'product.type.code', 'default'));
304
+		$typeResult = $typeManager->searchItems($typeSearch);
305 305
 
306
-		if( ( $typeItem = reset( $typeResult ) ) === false ) {
307
-			throw new \Exception( 'No product type "default" found' );
306
+		if (($typeItem = reset($typeResult)) === false) {
307
+			throw new \Exception('No product type "default" found');
308 308
 		}
309 309
 
310 310
 		$item = $manager->createItem();
311
-		$item->setTypeid( $typeItem->getId() );
312
-		$item->setCode( $code );
311
+		$item->setTypeid($typeItem->getId());
312
+		$item->setCode($code);
313 313
 
314
-		$manager->saveItem( $item );
314
+		$manager->saveItem($item);
315 315
 
316 316
 		return $item;
317 317
 	}
318 318
 
319 319
 
320
-	protected function delete( \Aimeos\MShop\Product\Item\Iface $product )
320
+	protected function delete(\Aimeos\MShop\Product\Item\Iface $product)
321 321
 	{
322
-		$textManager = \Aimeos\MShop\Text\Manager\Factory::createManager( $this->context );
323
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context );
324
-		$listManager = $manager->getSubManager( 'lists' );
322
+		$textManager = \Aimeos\MShop\Text\Manager\Factory::createManager($this->context);
323
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context);
324
+		$listManager = $manager->getSubManager('lists');
325 325
 
326
-		foreach( $product->getListItems('text') as $listItem )
326
+		foreach ($product->getListItems('text') as $listItem)
327 327
 		{
328
-			$textManager->deleteItem( $listItem->getRefItem()->getId() );
329
-			$listManager->deleteItem( $listItem->getId() );
328
+			$textManager->deleteItem($listItem->getRefItem()->getId());
329
+			$listManager->deleteItem($listItem->getId());
330 330
 		}
331 331
 
332
-		$manager->deleteItem( $product->getId() );
332
+		$manager->deleteItem($product->getId());
333 333
 	}
334 334
 
335 335
 
336 336
 	/**
337 337
 	 * @param string $code
338 338
 	 */
339
-	protected function get( $code )
339
+	protected function get($code)
340 340
 	{
341
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context );
341
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context);
342 342
 
343 343
 		$search = $manager->createSearch();
344
-		$search->setConditions( $search->compare( '==', 'product.code', $code ) );
344
+		$search->setConditions($search->compare('==', 'product.code', $code));
345 345
 
346
-		$result = $manager->searchItems( $search, array('text') );
346
+		$result = $manager->searchItems($search, array('text'));
347 347
 
348
-		if( ( $item = reset( $result ) ) === false ) {
349
-			throw new \Exception( sprintf( 'No product item for code "%1$s"', $code ) );
348
+		if (($item = reset($result)) === false) {
349
+			throw new \Exception(sprintf('No product item for code "%1$s"', $code));
350 350
 		}
351 351
 
352 352
 		return $item;
Please login to merge, or discard this patch.
Controller/Common/Product/Import/Csv/Processor/Attribute/StandardTest.php 1 patch
Spacing   +99 added lines, -99 removed lines patch added patch discarded remove patch
@@ -15,16 +15,16 @@  discard block
 block discarded – undo
15 15
 
16 16
 	protected function setUp()
17 17
 	{
18
-		\Aimeos\MShop\Factory::setCache( true );
18
+		\Aimeos\MShop\Factory::setCache(true);
19 19
 
20 20
 		$this->context = \TestHelperCntl::getContext();
21
-		$this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, array() );
21
+		$this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, array());
22 22
 	}
23 23
 
24 24
 
25 25
 	protected function tearDown()
26 26
 	{
27
-		\Aimeos\MShop\Factory::setCache( false );
27
+		\Aimeos\MShop\Factory::setCache(false);
28 28
 		\Aimeos\MShop\Factory::clear();
29 29
 	}
30 30
 
@@ -55,32 +55,32 @@  discard block
 block discarded – undo
55 55
 			8 => 'variant',
56 56
 		);
57 57
 
58
-		$product = $this->create( 'job_csv_test' );
58
+		$product = $this->create('job_csv_test');
59 59
 
60
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint );
61
-		$object->process( $product, $data );
60
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint);
61
+		$object->process($product, $data);
62 62
 
63
-		$product = $this->get( 'job_csv_test' );
64
-		$this->delete( $product );
63
+		$product = $this->get('job_csv_test');
64
+		$this->delete($product);
65 65
 
66 66
 
67 67
 		$pos = 0;
68 68
 		$listItems = $product->getListItems();
69 69
 		$expected = array(
70
-			array( 'variant', 'length', '30' ),
71
-			array( 'variant', 'width', '29' ),
72
-			array( 'variant', 'color', 'white' ),
70
+			array('variant', 'length', '30'),
71
+			array('variant', 'width', '29'),
72
+			array('variant', 'color', 'white'),
73 73
 		);
74 74
 
75
-		$this->assertEquals( 3, count( $listItems ) );
75
+		$this->assertEquals(3, count($listItems));
76 76
 
77
-		foreach( $listItems as $listItem )
77
+		foreach ($listItems as $listItem)
78 78
 		{
79
-			$this->assertEquals( 1, $listItem->getStatus() );
80
-			$this->assertEquals( 'attribute', $listItem->getDomain() );
81
-			$this->assertEquals( $expected[$pos][0], $listItem->getType() );
82
-			$this->assertEquals( $expected[$pos][1], $listItem->getRefItem()->getType() );
83
-			$this->assertEquals( $expected[$pos][2], $listItem->getRefItem()->getCode() );
79
+			$this->assertEquals(1, $listItem->getStatus());
80
+			$this->assertEquals('attribute', $listItem->getDomain());
81
+			$this->assertEquals($expected[$pos][0], $listItem->getType());
82
+			$this->assertEquals($expected[$pos][1], $listItem->getRefItem()->getType());
83
+			$this->assertEquals($expected[$pos][2], $listItem->getRefItem()->getCode());
84 84
 			$pos++;
85 85
 		}
86 86
 	}
@@ -100,29 +100,29 @@  discard block
 block discarded – undo
100 100
 			2 => 'variant',
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\Catalog\Standard( $this->context, $mapping, $this->endpoint );
106
-		$object->process( $product, $data );
105
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint);
106
+		$object->process($product, $data);
107 107
 
108
-		$product = $this->get( 'job_csv_test' );
108
+		$product = $this->get('job_csv_test');
109 109
 
110
-		$this->delete( $product );
110
+		$this->delete($product);
111 111
 
112 112
 
113 113
 		$pos = 0;
114
-		$codes = array( 'white', 'black', 'aimeos' );
114
+		$codes = array('white', 'black', 'aimeos');
115 115
 
116
-		foreach( $product->getListItems() as $listItems )
116
+		foreach ($product->getListItems() as $listItems)
117 117
 		{
118
-			$this->assertEquals( 3, count( $listItems ) );
118
+			$this->assertEquals(3, count($listItems));
119 119
 
120
-			foreach( $listItems as $listItem )
120
+			foreach ($listItems as $listItem)
121 121
 			{
122
-				$this->assertEquals( 1, $listItem->getStatus() );
123
-				$this->assertEquals( 'attribute', $listItem->getDomain() );
124
-				$this->assertEquals( 'variant', $listItem->getType() );
125
-				$this->assertEquals( $codes[$pos], $listItem->getRefItem()->getCode() );
122
+				$this->assertEquals(1, $listItem->getStatus());
123
+				$this->assertEquals('attribute', $listItem->getDomain());
124
+				$this->assertEquals('variant', $listItem->getType());
125
+				$this->assertEquals($codes[$pos], $listItem->getRefItem()->getCode());
126 126
 				$pos++;
127 127
 			}
128 128
 		}
@@ -146,27 +146,27 @@  discard block
 block discarded – undo
146 146
 			1 => '29',
147 147
 		);
148 148
 
149
-		$product = $this->create( 'job_csv_test' );
149
+		$product = $this->create('job_csv_test');
150 150
 
151
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint );
152
-		$object->process( $product, $data );
151
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint);
152
+		$object->process($product, $data);
153 153
 
154
-		$product = $this->get( 'job_csv_test' );
154
+		$product = $this->get('job_csv_test');
155 155
 
156
-		$object->process( $product, $dataUpdate );
156
+		$object->process($product, $dataUpdate);
157 157
 
158
-		$product = $this->get( 'job_csv_test' );
159
-		$this->delete( $product );
158
+		$product = $this->get('job_csv_test');
159
+		$this->delete($product);
160 160
 
161 161
 
162 162
 		$listItems = $product->getListItems();
163
-		$listItem = reset( $listItems );
163
+		$listItem = reset($listItems);
164 164
 
165
-		$this->assertEquals( 1, count( $listItems ) );
166
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem );
165
+		$this->assertEquals(1, count($listItems));
166
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem);
167 167
 
168
-		$this->assertEquals( 'width', $listItem->getRefItem()->getType() );
169
-		$this->assertEquals( '29', $listItem->getRefItem()->getCode() );
168
+		$this->assertEquals('width', $listItem->getRefItem()->getType());
169
+		$this->assertEquals('29', $listItem->getRefItem()->getCode());
170 170
 	}
171 171
 
172 172
 
@@ -182,23 +182,23 @@  discard block
 block discarded – undo
182 182
 			1 => '30',
183 183
 		);
184 184
 
185
-		$product = $this->create( 'job_csv_test' );
185
+		$product = $this->create('job_csv_test');
186 186
 
187
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint );
188
-		$object->process( $product, $data );
187
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint);
188
+		$object->process($product, $data);
189 189
 
190
-		$product = $this->get( 'job_csv_test' );
190
+		$product = $this->get('job_csv_test');
191 191
 
192
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, array(), $this->endpoint );
193
-		$object->process( $product, array() );
192
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, array(), $this->endpoint);
193
+		$object->process($product, array());
194 194
 
195
-		$product = $this->get( 'job_csv_test' );
196
-		$this->delete( $product );
195
+		$product = $this->get('job_csv_test');
196
+		$this->delete($product);
197 197
 
198 198
 
199 199
 		$listItems = $product->getListItems();
200 200
 
201
-		$this->assertEquals( 0, count( $listItems ) );
201
+		$this->assertEquals(0, count($listItems));
202 202
 	}
203 203
 
204 204
 
@@ -218,18 +218,18 @@  discard block
 block discarded – undo
218 218
 			3 => '30',
219 219
 		);
220 220
 
221
-		$product = $this->create( 'job_csv_test' );
221
+		$product = $this->create('job_csv_test');
222 222
 
223
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint );
224
-		$object->process( $product, $data );
223
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint);
224
+		$object->process($product, $data);
225 225
 
226
-		$product = $this->get( 'job_csv_test' );
227
-		$this->delete( $product );
226
+		$product = $this->get('job_csv_test');
227
+		$this->delete($product);
228 228
 
229 229
 
230 230
 		$listItems = $product->getListItems();
231 231
 
232
-		$this->assertEquals( 1, count( $listItems ) );
232
+		$this->assertEquals(1, count($listItems));
233 233
 	}
234 234
 
235 235
 
@@ -245,19 +245,19 @@  discard block
 block discarded – undo
245 245
 			1 => '30',
246 246
 		);
247 247
 
248
-		$product = $this->create( 'job_csv_test' );
248
+		$product = $this->create('job_csv_test');
249 249
 
250
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint );
250
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint);
251 251
 
252 252
 		try {
253
-			$object->process( $product, $data );
254
-		} catch( \Exception $e ) {
255
-			$this->delete( $product );
253
+			$object->process($product, $data);
254
+		} catch (\Exception $e) {
255
+			$this->delete($product);
256 256
 			return;
257 257
 		}
258 258
 
259
-		$this->delete( $product );
260
-		$this->markTestFail( 'No exception thrown' );
259
+		$this->delete($product);
260
+		$this->markTestFail('No exception thrown');
261 261
 	}
262 262
 
263 263
 
@@ -281,82 +281,82 @@  discard block
 block discarded – undo
281 281
 			5 => 'default',
282 282
 		);
283 283
 
284
-		$this->context->getConfig()->set( 'controller/common/product/import/csv/processor/attribute/listtypes', array( 'default' ) );
284
+		$this->context->getConfig()->set('controller/common/product/import/csv/processor/attribute/listtypes', array('default'));
285 285
 
286
-		$product = $this->create( 'job_csv_test' );
286
+		$product = $this->create('job_csv_test');
287 287
 
288
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard( $this->context, $mapping, $this->endpoint );
289
-		$object->process( $product, $data );
288
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Attribute\Standard($this->context, $mapping, $this->endpoint);
289
+		$object->process($product, $data);
290 290
 
291
-		$product = $this->get( 'job_csv_test' );
292
-		$this->delete( $product );
291
+		$product = $this->get('job_csv_test');
292
+		$this->delete($product);
293 293
 
294 294
 
295 295
 		$listItems = $product->getListItems();
296
-		$listItem = reset( $listItems );
296
+		$listItem = reset($listItems);
297 297
 
298
-		$this->assertEquals( 1, count( $listItems ) );
299
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem );
298
+		$this->assertEquals(1, count($listItems));
299
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem);
300 300
 
301
-		$this->assertEquals( 'default', $listItem->getType() );
302
-		$this->assertEquals( 'width', $listItem->getRefItem()->getType() );
303
-		$this->assertEquals( '30', $listItem->getRefItem()->getCode() );
301
+		$this->assertEquals('default', $listItem->getType());
302
+		$this->assertEquals('width', $listItem->getRefItem()->getType());
303
+		$this->assertEquals('30', $listItem->getRefItem()->getCode());
304 304
 	}
305 305
 
306 306
 
307 307
 	/**
308 308
 	 * @param string $code
309 309
 	 */
310
-	protected function create( $code )
310
+	protected function create($code)
311 311
 	{
312
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context );
313
-		$typeManager = $manager->getSubManager( 'type' );
312
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context);
313
+		$typeManager = $manager->getSubManager('type');
314 314
 
315 315
 		$typeSearch = $typeManager->createSearch();
316
-		$typeSearch->setConditions( $typeSearch->compare( '==', 'product.type.code', 'default' ) );
317
-		$typeResult = $typeManager->searchItems( $typeSearch );
316
+		$typeSearch->setConditions($typeSearch->compare('==', 'product.type.code', 'default'));
317
+		$typeResult = $typeManager->searchItems($typeSearch);
318 318
 
319
-		if( ( $typeItem = reset( $typeResult ) ) === false ) {
320
-			throw new \Exception( 'No product type "default" found' );
319
+		if (($typeItem = reset($typeResult)) === false) {
320
+			throw new \Exception('No product type "default" found');
321 321
 		}
322 322
 
323 323
 		$item = $manager->createItem();
324
-		$item->setTypeid( $typeItem->getId() );
325
-		$item->setCode( $code );
324
+		$item->setTypeid($typeItem->getId());
325
+		$item->setCode($code);
326 326
 
327
-		$manager->saveItem( $item );
327
+		$manager->saveItem($item);
328 328
 
329 329
 		return $item;
330 330
 	}
331 331
 
332 332
 
333
-	protected function delete( \Aimeos\MShop\Product\Item\Iface $product )
333
+	protected function delete(\Aimeos\MShop\Product\Item\Iface $product)
334 334
 	{
335
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context );
336
-		$listManager = $manager->getSubManager( 'lists' );
335
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context);
336
+		$listManager = $manager->getSubManager('lists');
337 337
 
338
-		foreach( $product->getListItems('attribute') as $listItem ) {
339
-			$listManager->deleteItem( $listItem->getId() );
338
+		foreach ($product->getListItems('attribute') as $listItem) {
339
+			$listManager->deleteItem($listItem->getId());
340 340
 		}
341 341
 
342
-		$manager->deleteItem( $product->getId() );
342
+		$manager->deleteItem($product->getId());
343 343
 	}
344 344
 
345 345
 
346 346
 	/**
347 347
 	 * @param string $code
348 348
 	 */
349
-	protected function get( $code )
349
+	protected function get($code)
350 350
 	{
351
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( $this->context );
351
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager($this->context);
352 352
 
353 353
 		$search = $manager->createSearch();
354
-		$search->setConditions( $search->compare( '==', 'product.code', $code ) );
354
+		$search->setConditions($search->compare('==', 'product.code', $code));
355 355
 
356
-		$result = $manager->searchItems( $search, array('attribute') );
356
+		$result = $manager->searchItems($search, array('attribute'));
357 357
 
358
-		if( ( $item = reset( $result ) ) === false ) {
359
-			throw new \Exception( sprintf( 'No product item for code "%1$s"', $code ) );
358
+		if (($item = reset($result)) === false) {
359
+			throw new \Exception(sprintf('No product item for code "%1$s"', $code));
360 360
 		}
361 361
 
362 362
 		return $item;
Please login to merge, or discard this patch.
Controller/Common/Product/Import/Csv/Processor/Catalog/StandardTest.php 1 patch
Spacing   +95 added lines, -95 removed lines patch added patch discarded remove patch
@@ -16,23 +16,23 @@  discard block
 block discarded – undo
16 16
 
17 17
 	public static function setUpBeforeClass()
18 18
 	{
19
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( \TestHelperCntl::getContext() );
20
-		$typeManager = $manager->getSubmanager( 'type' );
19
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager(\TestHelperCntl::getContext());
20
+		$typeManager = $manager->getSubmanager('type');
21 21
 
22 22
 		$typeSearch = $typeManager->createSearch();
23
-		$typeSearch->setConditions( $typeSearch->compare( '==', 'product.type.code', 'default' ) );
24
-		$result = $typeManager->searchItems( $typeSearch );
23
+		$typeSearch->setConditions($typeSearch->compare('==', 'product.type.code', 'default'));
24
+		$result = $typeManager->searchItems($typeSearch);
25 25
 
26
-		if( ( $typeItem = reset( $result ) ) === false ) {
27
-			throw new \Exception( 'Product type "default" not found' );
26
+		if (($typeItem = reset($result)) === false) {
27
+			throw new \Exception('Product type "default" not found');
28 28
 		}
29 29
 
30 30
 		$item = $manager->createItem();
31
-		$item->setCode( 'job_csv_prod' );
32
-		$item->setTypeId( $typeItem->getId() );
33
-		$item->setStatus( 1 );
31
+		$item->setCode('job_csv_prod');
32
+		$item->setTypeId($typeItem->getId());
33
+		$item->setStatus(1);
34 34
 
35
-		$manager->saveItem( $item );
35
+		$manager->saveItem($item);
36 36
 
37 37
 		self::$product = $item;
38 38
 	}
@@ -40,23 +40,23 @@  discard block
 block discarded – undo
40 40
 
41 41
 	public static function tearDownAfterClass()
42 42
 	{
43
-		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( \TestHelperCntl::getContext() );
44
-		$manager->deleteItem( self::$product->getId() );
43
+		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager(\TestHelperCntl::getContext());
44
+		$manager->deleteItem(self::$product->getId());
45 45
 	}
46 46
 
47 47
 
48 48
 	protected function setUp()
49 49
 	{
50
-		\Aimeos\MShop\Factory::setCache( true );
50
+		\Aimeos\MShop\Factory::setCache(true);
51 51
 
52 52
 		$this->context = \TestHelperCntl::getContext();
53
-		$this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, array() );
53
+		$this->endpoint = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, array());
54 54
 	}
55 55
 
56 56
 
57 57
 	protected function tearDown()
58 58
 	{
59
-		\Aimeos\MShop\Factory::setCache( false );
59
+		\Aimeos\MShop\Factory::setCache(false);
60 60
 		\Aimeos\MShop\Factory::clear();
61 61
 	}
62 62
 
@@ -77,30 +77,30 @@  discard block
 block discarded – undo
77 77
 			3 => 'job_csv_test',
78 78
 		);
79 79
 
80
-		$this->create( 'job_csv_test' );
80
+		$this->create('job_csv_test');
81 81
 
82
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, $mapping, $this->endpoint );
83
-		$object->process( self::$product, $data );
82
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint);
83
+		$object->process(self::$product, $data);
84 84
 
85
-		$category = $this->get( 'job_csv_test' );
86
-		$this->delete( $category );
85
+		$category = $this->get('job_csv_test');
86
+		$this->delete($category);
87 87
 
88 88
 
89 89
 		$pos = 0;
90 90
 		$listItems = $category->getListItems();
91 91
 		$expected = array(
92
-			array( 'default', 'job_csv_prod' ),
93
-			array( 'promotion', 'job_csv_prod' ),
92
+			array('default', 'job_csv_prod'),
93
+			array('promotion', 'job_csv_prod'),
94 94
 		);
95 95
 
96
-		$this->assertEquals( 2, count( $listItems ) );
96
+		$this->assertEquals(2, count($listItems));
97 97
 
98
-		foreach( $listItems as $listItem )
98
+		foreach ($listItems as $listItem)
99 99
 		{
100
-			$this->assertEquals( 1, $listItem->getStatus() );
101
-			$this->assertEquals( 'product', $listItem->getDomain() );
102
-			$this->assertEquals( $expected[$pos][0], $listItem->getType() );
103
-			$this->assertEquals( $expected[$pos][1], $listItem->getRefItem()->getCode() );
100
+			$this->assertEquals(1, $listItem->getStatus());
101
+			$this->assertEquals('product', $listItem->getDomain());
102
+			$this->assertEquals($expected[$pos][0], $listItem->getType());
103
+			$this->assertEquals($expected[$pos][1], $listItem->getRefItem()->getCode());
104 104
 			$pos++;
105 105
 		}
106 106
 	}
@@ -122,32 +122,32 @@  discard block
 block discarded – undo
122 122
 			3 => "job_csv_test\njob_csv_test2",
123 123
 		);
124 124
 
125
-		$this->create( 'job_csv_test' );
126
-		$this->create( 'job_csv_test2' );
125
+		$this->create('job_csv_test');
126
+		$this->create('job_csv_test2');
127 127
 
128
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, $mapping, $this->endpoint );
129
-		$object->process( self::$product, $data );
128
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint);
129
+		$object->process(self::$product, $data);
130 130
 
131
-		$category = $this->get( 'job_csv_test' );
132
-		$category2 = $this->get( 'job_csv_test2' );
131
+		$category = $this->get('job_csv_test');
132
+		$category2 = $this->get('job_csv_test2');
133 133
 
134
-		$this->delete( $category );
135
-		$this->delete( $category2 );
134
+		$this->delete($category);
135
+		$this->delete($category2);
136 136
 
137 137
 
138 138
 		$pos = 0;
139
-		$types = array( 'default', 'promotion', 'default', 'promotion' );
139
+		$types = array('default', 'promotion', 'default', 'promotion');
140 140
 
141
-		foreach( array( $category->getListItems(), $category2->getListItems() ) as $listItems )
141
+		foreach (array($category->getListItems(), $category2->getListItems()) as $listItems)
142 142
 		{
143
-			$this->assertEquals( 2, count( $listItems ) );
143
+			$this->assertEquals(2, count($listItems));
144 144
 
145
-			foreach( $listItems as $listItem )
145
+			foreach ($listItems as $listItem)
146 146
 			{
147
-				$this->assertEquals( 1, $listItem->getStatus() );
148
-				$this->assertEquals( 'product', $listItem->getDomain() );
149
-				$this->assertEquals( $types[$pos], $listItem->getType() );
150
-				$this->assertEquals( 'job_csv_prod', $listItem->getRefItem()->getCode() );
147
+				$this->assertEquals(1, $listItem->getStatus());
148
+				$this->assertEquals('product', $listItem->getDomain());
149
+				$this->assertEquals($types[$pos], $listItem->getType());
150
+				$this->assertEquals('job_csv_prod', $listItem->getRefItem()->getCode());
151 151
 				$pos++;
152 152
 			}
153 153
 		}
@@ -171,23 +171,23 @@  discard block
 block discarded – undo
171 171
 			1 => 'job_csv_test',
172 172
 		);
173 173
 
174
-		$this->create( 'job_csv_test' );
174
+		$this->create('job_csv_test');
175 175
 
176
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, $mapping, $this->endpoint );
177
-		$object->process( self::$product, $data );
178
-		$object->process( self::$product, $dataUpdate );
176
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint);
177
+		$object->process(self::$product, $data);
178
+		$object->process(self::$product, $dataUpdate);
179 179
 
180
-		$category = $this->get( 'job_csv_test' );
181
-		$this->delete( $category );
180
+		$category = $this->get('job_csv_test');
181
+		$this->delete($category);
182 182
 
183 183
 
184 184
 		$listItems = $category->getListItems();
185
-		$listItem = reset( $listItems );
185
+		$listItem = reset($listItems);
186 186
 
187
-		$this->assertEquals( 1, count( $listItems ) );
188
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem );
187
+		$this->assertEquals(1, count($listItems));
188
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem);
189 189
 
190
-		$this->assertEquals( 'job_csv_prod', $listItem->getRefItem()->getCode() );
190
+		$this->assertEquals('job_csv_prod', $listItem->getRefItem()->getCode());
191 191
 	}
192 192
 
193 193
 
@@ -203,21 +203,21 @@  discard block
 block discarded – undo
203 203
 			1 => 'job_csv_test',
204 204
 		);
205 205
 
206
-		$this->create( 'job_csv_test' );
206
+		$this->create('job_csv_test');
207 207
 
208
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, $mapping, $this->endpoint );
209
-		$object->process( self::$product, $data );
208
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint);
209
+		$object->process(self::$product, $data);
210 210
 
211
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, array(), $this->endpoint );
212
-		$object->process( self::$product, array() );
211
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, array(), $this->endpoint);
212
+		$object->process(self::$product, array());
213 213
 
214
-		$category = $this->get( 'job_csv_test' );
215
-		$this->delete( $category );
214
+		$category = $this->get('job_csv_test');
215
+		$this->delete($category);
216 216
 
217 217
 
218 218
 		$listItems = $category->getListItems();
219 219
 
220
-		$this->assertEquals( 0, count( $listItems ) );
220
+		$this->assertEquals(0, count($listItems));
221 221
 	}
222 222
 
223 223
 
@@ -237,18 +237,18 @@  discard block
 block discarded – undo
237 237
 			3 => 'job_csv_test',
238 238
 		);
239 239
 
240
-		$this->create( 'job_csv_test' );
240
+		$this->create('job_csv_test');
241 241
 
242
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, $mapping, $this->endpoint );
243
-		$object->process( self::$product, $data );
242
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint);
243
+		$object->process(self::$product, $data);
244 244
 
245
-		$category = $this->get( 'job_csv_test' );
246
-		$this->delete( $category );
245
+		$category = $this->get('job_csv_test');
246
+		$this->delete($category);
247 247
 
248 248
 
249 249
 		$listItems = $category->getListItems();
250 250
 
251
-		$this->assertEquals( 1, count( $listItems ) );
251
+		$this->assertEquals(1, count($listItems));
252 252
 	}
253 253
 
254 254
 
@@ -268,71 +268,71 @@  discard block
 block discarded – undo
268 268
 			3 => 'job_csv_test',
269 269
 		);
270 270
 
271
-		$this->context->getConfig()->set( 'controller/common/product/import/csv/processor/catalog/listtypes', array( 'default' ) );
271
+		$this->context->getConfig()->set('controller/common/product/import/csv/processor/catalog/listtypes', array('default'));
272 272
 
273
-		$this->create( 'job_csv_test' );
273
+		$this->create('job_csv_test');
274 274
 
275
-		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard( $this->context, $mapping, $this->endpoint );
276
-		$object->process( self::$product, $data );
275
+		$object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Catalog\Standard($this->context, $mapping, $this->endpoint);
276
+		$object->process(self::$product, $data);
277 277
 
278
-		$category = $this->get( 'job_csv_test' );
279
-		$this->delete( $category );
278
+		$category = $this->get('job_csv_test');
279
+		$this->delete($category);
280 280
 
281 281
 
282 282
 		$listItems = $category->getListItems();
283
-		$listItem = reset( $listItems );
283
+		$listItem = reset($listItems);
284 284
 
285
-		$this->assertEquals( 1, count( $listItems ) );
286
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem );
285
+		$this->assertEquals(1, count($listItems));
286
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $listItem);
287 287
 
288
-		$this->assertEquals( 'default', $listItem->getType() );
289
-		$this->assertEquals( 'job_csv_prod', $listItem->getRefItem()->getCode() );
288
+		$this->assertEquals('default', $listItem->getType());
289
+		$this->assertEquals('job_csv_prod', $listItem->getRefItem()->getCode());
290 290
 	}
291 291
 
292 292
 
293 293
 	/**
294 294
 	 * @param string $code
295 295
 	 */
296
-	protected function create( $code )
296
+	protected function create($code)
297 297
 	{
298
-		$manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager( $this->context );
298
+		$manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager($this->context);
299 299
 
300 300
 		$item = $manager->createItem();
301
-		$item->setCode( $code );
301
+		$item->setCode($code);
302 302
 
303
-		$manager->insertItem( $item );
303
+		$manager->insertItem($item);
304 304
 
305 305
 		return $item;
306 306
 	}
307 307
 
308 308
 
309
-	protected function delete( \Aimeos\MShop\Catalog\Item\Iface $catItem )
309
+	protected function delete(\Aimeos\MShop\Catalog\Item\Iface $catItem)
310 310
 	{
311
-		$manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager( $this->context );
312
-		$listManager = $manager->getSubManager( 'lists' );
311
+		$manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager($this->context);
312
+		$listManager = $manager->getSubManager('lists');
313 313
 
314
-		foreach( $catItem->getListItems('product') as $listItem ) {
315
-			$listManager->deleteItem( $listItem->getId() );
314
+		foreach ($catItem->getListItems('product') as $listItem) {
315
+			$listManager->deleteItem($listItem->getId());
316 316
 		}
317 317
 
318
-		$manager->deleteItem( $catItem->getId() );
318
+		$manager->deleteItem($catItem->getId());
319 319
 	}
320 320
 
321 321
 
322 322
 	/**
323 323
 	 * @param string $code
324 324
 	 */
325
-	protected function get( $code )
325
+	protected function get($code)
326 326
 	{
327
-		$manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager( $this->context );
327
+		$manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager($this->context);
328 328
 
329 329
 		$search = $manager->createSearch();
330
-		$search->setConditions( $search->compare( '==', 'catalog.code', $code ) );
330
+		$search->setConditions($search->compare('==', 'catalog.code', $code));
331 331
 
332
-		$result = $manager->searchItems( $search, array('product') );
332
+		$result = $manager->searchItems($search, array('product'));
333 333
 
334
-		if( ( $item = reset( $result ) ) === false ) {
335
-			throw new \Exception( sprintf( 'No catalog item for code "%1$s"', $code ) );
334
+		if (($item = reset($result)) === false) {
335
+			throw new \Exception(sprintf('No catalog item for code "%1$s"', $code));
336 336
 		}
337 337
 
338 338
 		return $item;
Please login to merge, or discard this patch.
controller/jobs/tests/TestHelperJobs.php 1 patch
Spacing   +48 added lines, -48 removed lines patch added patch discarded remove patch
@@ -15,14 +15,14 @@  discard block
 block discarded – undo
15 15
 	public static function bootstrap()
16 16
 	{
17 17
 		self::getAimeos();
18
-		\Aimeos\MShop\Factory::setCache( false );
18
+		\Aimeos\MShop\Factory::setCache(false);
19 19
 	}
20 20
 
21 21
 
22
-	public static function getContext( $site = 'unittest' )
22
+	public static function getContext($site = 'unittest')
23 23
 	{
24
-		if( !isset( self::$context[$site] ) ) {
25
-			self::$context[$site] = self::createContext( $site );
24
+		if (!isset(self::$context[$site])) {
25
+			self::$context[$site] = self::createContext($site);
26 26
 		}
27 27
 
28 28
 		return clone self::$context[$site];
@@ -31,13 +31,13 @@  discard block
 block discarded – undo
31 31
 
32 32
 	public static function getAimeos()
33 33
 	{
34
-		if( !isset( self::$aimeos ) )
34
+		if (!isset(self::$aimeos))
35 35
 		{
36 36
 			require_once 'Bootstrap.php';
37
-			spl_autoload_register( 'Aimeos\\Bootstrap::autoload' );
37
+			spl_autoload_register('Aimeos\\Bootstrap::autoload');
38 38
 
39
-			$extdir = dirname( dirname( dirname( dirname( dirname( __FILE__ ) ) ) ) );
40
-			self::$aimeos = new \Aimeos\Bootstrap( array( $extdir ), true );
39
+			$extdir = dirname(dirname(dirname(dirname(dirname(__FILE__)))));
40
+			self::$aimeos = new \Aimeos\Bootstrap(array($extdir), true);
41 41
 		}
42 42
 
43 43
 		return self::$aimeos;
@@ -46,93 +46,93 @@  discard block
 block discarded – undo
46 46
 
47 47
 	public static function getControllerPaths()
48 48
 	{
49
-		return self::getAimeos()->getCustomPaths( 'controller/jobs' );
49
+		return self::getAimeos()->getCustomPaths('controller/jobs');
50 50
 	}
51 51
 
52 52
 
53 53
 	/**
54 54
 	 * @param string $site
55 55
 	 */
56
-	private static function createContext( $site )
56
+	private static function createContext($site)
57 57
 	{
58 58
 		$ctx = new \Aimeos\MShop\Context\Item\Standard();
59 59
 		$aimeos = self::getAimeos();
60 60
 
61 61
 
62
-		$paths = $aimeos->getConfigPaths( 'mysql' );
62
+		$paths = $aimeos->getConfigPaths('mysql');
63 63
 		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
64 64
 		$file = __DIR__ . DIRECTORY_SEPARATOR . 'confdoc.ser';
65 65
 
66
-		$conf = new \Aimeos\MW\Config\PHPArray( array(), $paths );
67
-		$conf = new \Aimeos\MW\Config\Decorator\Memory( $conf );
68
-		$conf = new \Aimeos\MW\Config\Decorator\Documentor( $conf, $file );
69
-		$ctx->setConfig( $conf );
66
+		$conf = new \Aimeos\MW\Config\PHPArray(array(), $paths);
67
+		$conf = new \Aimeos\MW\Config\Decorator\Memory($conf);
68
+		$conf = new \Aimeos\MW\Config\Decorator\Documentor($conf, $file);
69
+		$ctx->setConfig($conf);
70 70
 
71 71
 
72
-		$logger = new \Aimeos\MW\Logger\File( $site . '.log', \Aimeos\MW\Logger\Base::DEBUG );
73
-		$ctx->setLogger( $logger );
72
+		$logger = new \Aimeos\MW\Logger\File($site . '.log', \Aimeos\MW\Logger\Base::DEBUG);
73
+		$ctx->setLogger($logger);
74 74
 
75 75
 
76
-		$dbm = new \Aimeos\MW\DB\Manager\PDO( $conf );
77
-		$ctx->setDatabaseManager( $dbm );
76
+		$dbm = new \Aimeos\MW\DB\Manager\PDO($conf);
77
+		$ctx->setDatabaseManager($dbm);
78 78
 
79 79
 
80
-		$fs = new \Aimeos\MW\Filesystem\Manager\Standard( $conf );
81
-		$ctx->setFilesystemManager( $fs );
80
+		$fs = new \Aimeos\MW\Filesystem\Manager\Standard($conf);
81
+		$ctx->setFilesystemManager($fs);
82 82
 
83 83
 
84
-		$mq = new \Aimeos\MW\MQueue\Manager\Standard( $conf );
85
-		$ctx->setMessageQueueManager( $mq );
84
+		$mq = new \Aimeos\MW\MQueue\Manager\Standard($conf);
85
+		$ctx->setMessageQueueManager($mq);
86 86
 
87 87
 
88 88
 		$session = new \Aimeos\MW\Session\None();
89
-		$ctx->setSession( $session );
89
+		$ctx->setSession($session);
90 90
 
91 91
 
92
-		$i18n = new \Aimeos\MW\Translation\None( 'de' );
93
-		$ctx->setI18n( array( 'de' => $i18n ) );
92
+		$i18n = new \Aimeos\MW\Translation\None('de');
93
+		$ctx->setI18n(array('de' => $i18n));
94 94
 
95 95
 
96
-		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::createManager( $ctx );
97
-		$locale = $localeManager->bootstrap( $site, 'de', '', false );
98
-		$ctx->setLocale( $locale );
96
+		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::createManager($ctx);
97
+		$locale = $localeManager->bootstrap($site, 'de', '', false);
98
+		$ctx->setLocale($locale);
99 99
 
100 100
 
101
-		$view = self::createView( $conf );
102
-		$ctx->setView( $view );
101
+		$view = self::createView($conf);
102
+		$ctx->setView($view);
103 103
 
104 104
 
105
-		$ctx->setEditor( 'core:controller/jobs' );
105
+		$ctx->setEditor('core:controller/jobs');
106 106
 
107 107
 		return $ctx;
108 108
 	}
109 109
 
110 110
 
111
-	protected static function createView( \Aimeos\MW\Config\Iface $config )
111
+	protected static function createView(\Aimeos\MW\Config\Iface $config)
112 112
 	{
113 113
 		$tmplpaths = array_merge_recursive(
114
-			self::getAimeos()->getCustomPaths( 'client/html/templates' ),
115
-			self::getAimeos()->getCustomPaths( 'controller/jobs/templates' )
114
+			self::getAimeos()->getCustomPaths('client/html/templates'),
115
+			self::getAimeos()->getCustomPaths('controller/jobs/templates')
116 116
 		);
117 117
 
118
-		$view = new \Aimeos\MW\View\Standard( $tmplpaths );
118
+		$view = new \Aimeos\MW\View\Standard($tmplpaths);
119 119
 
120
-		$trans = new \Aimeos\MW\Translation\None( 'de_DE' );
121
-		$helper = new \Aimeos\MW\View\Helper\Translate\Standard( $view, $trans );
122
-		$view->addHelper( 'translate', $helper );
120
+		$trans = new \Aimeos\MW\Translation\None('de_DE');
121
+		$helper = new \Aimeos\MW\View\Helper\Translate\Standard($view, $trans);
122
+		$view->addHelper('translate', $helper);
123 123
 
124
-		$helper = new \Aimeos\MW\View\Helper\Url\Standard( $view, 'http://baseurl' );
125
-		$view->addHelper( 'url', $helper );
124
+		$helper = new \Aimeos\MW\View\Helper\Url\Standard($view, 'http://baseurl');
125
+		$view->addHelper('url', $helper);
126 126
 
127
-		$helper = new \Aimeos\MW\View\Helper\Number\Standard( $view, '.', '' );
128
-		$view->addHelper( 'number', $helper );
127
+		$helper = new \Aimeos\MW\View\Helper\Number\Standard($view, '.', '');
128
+		$view->addHelper('number', $helper);
129 129
 
130
-		$helper = new \Aimeos\MW\View\Helper\Date\Standard( $view, 'Y-m-d' );
131
-		$view->addHelper( 'date', $helper );
130
+		$helper = new \Aimeos\MW\View\Helper\Date\Standard($view, 'Y-m-d');
131
+		$view->addHelper('date', $helper);
132 132
 
133
-		$config = new \Aimeos\MW\Config\Decorator\Protect( $config, array( 'controller/jobs', 'client/html' ) );
134
-		$helper = new \Aimeos\MW\View\Helper\Config\Standard( $view, $config );
135
-		$view->addHelper( 'config', $helper );
133
+		$config = new \Aimeos\MW\Config\Decorator\Protect($config, array('controller/jobs', 'client/html'));
134
+		$helper = new \Aimeos\MW\View\Helper\Config\Standard($view, $config);
135
+		$view->addHelper('config', $helper);
136 136
 
137 137
 		return $view;
138 138
 	}
Please login to merge, or discard this patch.
tests/Controller/Common/Product/Import/Csv/Cache/Attribute/StandardTest.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -14,49 +14,49 @@
 block discarded – undo
14 14
 
15 15
 	protected function setUp()
16 16
 	{
17
-		\Aimeos\MShop\Factory::setCache( true );
17
+		\Aimeos\MShop\Factory::setCache(true);
18 18
 
19 19
 		$context = \TestHelperCntl::getContext();
20
-		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Attribute\Standard( $context );
20
+		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Attribute\Standard($context);
21 21
 	}
22 22
 
23 23
 
24 24
 	protected function tearDown()
25 25
 	{
26
-		\Aimeos\MShop\Factory::setCache( false );
26
+		\Aimeos\MShop\Factory::setCache(false);
27 27
 		\Aimeos\MShop\Factory::clear();
28 28
 	}
29 29
 
30 30
 
31 31
 	public function testGet()
32 32
 	{
33
-		$item = $this->object->get( 'black', 'color' );
33
+		$item = $this->object->get('black', 'color');
34 34
 
35
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item );
36
-		$this->assertEquals( 'black', $item->getCode() );
37
-		$this->assertEquals( 'color', $item->getType() );
35
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item);
36
+		$this->assertEquals('black', $item->getCode());
37
+		$this->assertEquals('color', $item->getType());
38 38
 	}
39 39
 
40 40
 
41 41
 	public function testGetUnknown()
42 42
 	{
43
-		$this->assertEquals( null, $this->object->get( 'cache-test', 'color' ) );
43
+		$this->assertEquals(null, $this->object->get('cache-test', 'color'));
44 44
 	}
45 45
 
46 46
 
47 47
 	public function testSet()
48 48
 	{
49
-		$item = $this->object->get( 'black', 'color' );
49
+		$item = $this->object->get('black', 'color');
50 50
 
51
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item );
51
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item);
52 52
 
53
-		$item->setCode( 'cache-test' );
53
+		$item->setCode('cache-test');
54 54
 
55
-		$this->object->set( $item );
56
-		$item = $this->object->get( 'cache-test', 'color' );
55
+		$this->object->set($item);
56
+		$item = $this->object->get('cache-test', 'color');
57 57
 
58
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item );
59
-		$this->assertEquals( 'cache-test', $item->getCode() );
60
-		$this->assertEquals( 'color', $item->getType() );
58
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item);
59
+		$this->assertEquals('cache-test', $item->getCode());
60
+		$this->assertEquals('color', $item->getType());
61 61
 	}
62 62
 }
63 63
\ No newline at end of file
Please login to merge, or discard this patch.
src/Controller/Common/Product/Import/Csv/Processor/Media/Standard.php 1 patch
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,72 +77,72 @@  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
 		$manager->begin();
88 88
 
89 89
 		try
90 90
 		{
91
-			$map = $this->getMappedChunk( $data );
92
-			$listItems = $product->getListItems( 'media' );
91
+			$map = $this->getMappedChunk($data);
92
+			$listItems = $product->getListItems('media');
93 93
 
94
-			foreach( $map as $pos => $list )
94
+			foreach ($map as $pos => $list)
95 95
 			{
96
-				if( $this->checkEntry( $list ) === false ) {
96
+				if ($this->checkEntry($list) === false) {
97 97
 					continue;
98 98
 				}
99 99
 
100
-				$urls = explode( $separator, $list['media.url'] );
101
-				$type = $this->getValue( $list, 'media.type', 'default' );
102
-				$typecode = $this->getValue( $list, 'product.lists.type', 'default' );
100
+				$urls = explode($separator, $list['media.url']);
101
+				$type = $this->getValue($list, 'media.type', 'default');
102
+				$typecode = $this->getValue($list, 'product.lists.type', 'default');
103 103
 
104
-				if( ( $langid = $this->getValue( $list, 'media.languageid', null ) ) === '' ) {
104
+				if (($langid = $this->getValue($list, 'media.languageid', null)) === '') {
105 105
 					$langid = null;
106 106
 				}
107 107
 
108
-				foreach( $urls as $url )
108
+				foreach ($urls as $url)
109 109
 				{
110
-					if( ( $listItem = array_shift( $listItems ) ) !== null ) {
110
+					if (($listItem = array_shift($listItems)) !== null) {
111 111
 						$refItem = $listItem->getRefItem();
112 112
 					} else {
113 113
 						$listItem = $listManager->createItem();
114 114
 						$refItem = $manager->createItem();
115 115
 					}
116 116
 
117
-					$list['media.typeid'] = $this->getTypeId( 'media/type', 'product', $type );
117
+					$list['media.typeid'] = $this->getTypeId('media/type', 'product', $type);
118 118
 					$list['media.languageid'] = $langid;
119 119
 					$list['media.domain'] = 'product';
120 120
 					$list['media.url'] = $url;
121 121
 
122
-					$refItem->fromArray( $this->addItemDefaults( $list ) );
123
-					$manager->saveItem( $refItem );
122
+					$refItem->fromArray($this->addItemDefaults($list));
123
+					$manager->saveItem($refItem);
124 124
 
125
-					$list['product.lists.typeid'] = $this->getTypeId( 'product/lists/type', 'media', $typecode );
125
+					$list['product.lists.typeid'] = $this->getTypeId('product/lists/type', 'media', $typecode);
126 126
 					$list['product.lists.parentid'] = $product->getId();
127 127
 					$list['product.lists.refid'] = $refItem->getId();
128 128
 					$list['product.lists.domain'] = 'media';
129 129
 
130
-					$listItem->fromArray( $this->addListItemDefaults( $list, $pos++ ) );
131
-					$listManager->saveItem( $listItem );
130
+					$listItem->fromArray($this->addListItemDefaults($list, $pos++));
131
+					$listManager->saveItem($listItem);
132 132
 				}
133 133
 			}
134 134
 
135
-			foreach( $listItems as $listItem )
135
+			foreach ($listItems as $listItem)
136 136
 			{
137
-				$manager->deleteItem( $listItem->getRefItem()->getId() );
138
-				$listManager->deleteItem( $listItem->getId() );
137
+				$manager->deleteItem($listItem->getRefItem()->getId());
138
+				$listManager->deleteItem($listItem->getId());
139 139
 			}
140 140
 
141
-			$remaining = $this->getObject()->process( $product, $data );
141
+			$remaining = $this->getObject()->process($product, $data);
142 142
 
143 143
 			$manager->commit();
144 144
 		}
145
-		catch( \Exception $e )
145
+		catch (\Exception $e)
146 146
 		{
147 147
 			$manager->rollback();
148 148
 			throw $e;
@@ -158,17 +158,17 @@  discard block
 block discarded – undo
158 158
 	 * @param array $list Associative list of domain item keys and their values, e.g. "media.status" => 1
159 159
 	 * @return array Given associative list enriched by default values if they were not already set
160 160
 	 */
161
-	protected function addItemDefaults( array $list )
161
+	protected function addItemDefaults(array $list)
162 162
 	{
163
-		if( !isset( $list['media.label'] ) ) {
163
+		if (!isset($list['media.label'])) {
164 164
 			$list['media.label'] = $list['media.url'];
165 165
 		}
166 166
 
167
-		if( !isset( $list['media.preview'] ) ) {
167
+		if (!isset($list['media.preview'])) {
168 168
 			$list['media.preview'] = $list['media.url'];
169 169
 		}
170 170
 
171
-		if( !isset( $list['media.status'] ) ) {
171
+		if (!isset($list['media.status'])) {
172 172
 			$list['media.status'] = 1;
173 173
 		}
174 174
 
@@ -182,10 +182,10 @@  discard block
 block discarded – undo
182 182
 	 * @param array $list Associative list of key/value pairs from the mapping
183 183
 	 * @return boolean True if valid, false if not
184 184
 	 */
185
-	protected function checkEntry( array $list )
185
+	protected function checkEntry(array $list)
186 186
 	{
187
-		if( !isset( $list['media.url'] ) || $list['media.url'] === '' || isset( $list['product.lists.type'] )
188
-				&& $this->listTypes !== null && !in_array( $list['product.lists.type'], (array) $this->listTypes )
187
+		if (!isset($list['media.url']) || $list['media.url'] === '' || isset($list['product.lists.type'])
188
+				&& $this->listTypes !== null && !in_array($list['product.lists.type'], (array) $this->listTypes)
189 189
 		) {
190 190
 			return false;
191 191
 		}
Please login to merge, or discard this patch.
common/src/Controller/Common/Product/Import/Csv/Processor/Base.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -32,8 +32,8 @@  discard block
 block discarded – undo
32 32
 	 * @param array $mapping Associative list of field position in CSV as key and domain item key as value
33 33
 	 * @param \Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object Decorated processor
34 34
 	 */
35
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping,
36
-		\Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null )
35
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping,
36
+		\Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null)
37 37
 	{
38 38
 		$this->context = $context;
39 39
 		$this->mapping = $mapping;
@@ -48,13 +48,13 @@  discard block
 block discarded – undo
48 48
 	 * @param integer $pos Computed position of the list item in the associated list of items
49 49
 	 * @return array Given associative list enriched by default values if they were not already set
50 50
 	 */
51
-	protected function addListItemDefaults( array $list, $pos )
51
+	protected function addListItemDefaults(array $list, $pos)
52 52
 	{
53
-		if( !isset( $list['product.lists.position'] ) ) {
53
+		if (!isset($list['product.lists.position'])) {
54 54
 			$list['product.lists.position'] = $pos;
55 55
 		}
56 56
 
57
-		if( !isset( $list['product.lists.status'] ) ) {
57
+		if (!isset($list['product.lists.status'])) {
58 58
 			$list['product.lists.status'] = 1;
59 59
 		}
60 60
 
@@ -92,8 +92,8 @@  discard block
 block discarded – undo
92 92
 	 */
93 93
 	protected function getObject()
94 94
 	{
95
-		if( $this->object === null ) {
96
-			throw new \Aimeos\Controller\Jobs\Exception( 'No processor object available' );
95
+		if ($this->object === null) {
96
+			throw new \Aimeos\Controller\Jobs\Exception('No processor object available');
97 97
 		}
98 98
 
99 99
 		return $this->object;
@@ -106,21 +106,21 @@  discard block
 block discarded – undo
106 106
 	 * @param array $data List of CSV fields with position as key and domain item key as value
107 107
 	 * @return array List of associative arrays containing the chunked properties
108 108
 	 */
109
-	protected function getMappedChunk( array &$data )
109
+	protected function getMappedChunk(array &$data)
110 110
 	{
111 111
 		$idx = 0;
112 112
 		$map = array();
113 113
 
114
-		foreach( $this->getMapping() as $pos => $key )
114
+		foreach ($this->getMapping() as $pos => $key)
115 115
 		{
116
-			if( isset( $map[$idx][$key] ) ) {
116
+			if (isset($map[$idx][$key])) {
117 117
 				$idx++;
118 118
 			}
119 119
 
120
-			if( isset( $data[$pos] ) )
120
+			if (isset($data[$pos]))
121 121
 			{
122 122
 				$map[$idx][$key] = $data[$pos];
123
-				unset( $data[$pos] );
123
+				unset($data[$pos]);
124 124
 			}
125 125
 		}
126 126
 
@@ -136,8 +136,8 @@  discard block
 block discarded – undo
136 136
 	 * @param mixed $default Default value if key isn't found
137 137
 	 * @param mixed Value for the key in the list of the default value
138 138
 	 */
139
-	protected function getValue( array $list, $key, $default )
139
+	protected function getValue(array $list, $key, $default)
140 140
 	{
141
-		return ( isset( $list[$key] ) ? $list[$key] : $default );
141
+		return (isset($list[$key]) ? $list[$key] : $default);
142 142
 	}
143 143
 }
Please login to merge, or discard this patch.
src/Controller/Common/Product/Import/Csv/Processor/Attribute/Standard.php 1 patch
Spacing   +49 added lines, -49 removed lines patch added patch discarded remove patch
@@ -43,10 +43,10 @@  discard block
 block discarded – undo
43 43
 	 * @param array $mapping Associative list of field position in CSV as key and domain item key as value
44 44
 	 * @param \Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object Decorated processor
45 45
 	 */
46
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping,
47
-		\Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null )
46
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping,
47
+		\Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null)
48 48
 	{
49
-		parent::__construct( $context, $mapping, $object );
49
+		parent::__construct($context, $mapping, $object);
50 50
 
51 51
 		/** controller/common/product/import/csv/processor/attribute/listtypes
52 52
 		 * Names of the product list types for attributes that are updated or removed
@@ -67,9 +67,9 @@  discard block
 block discarded – undo
67 67
 		 * @see controller/common/product/import/csv/processor/price/listtypes
68 68
 		 * @see controller/common/product/import/csv/processor/text/listtypes
69 69
 		 */
70
-		$this->listTypes = $context->getConfig()->get( 'controller/common/product/import/csv/processor/attribute/listtypes');
70
+		$this->listTypes = $context->getConfig()->get('controller/common/product/import/csv/processor/attribute/listtypes');
71 71
 
72
-		$this->cache = $this->getCache( 'attribute' );
72
+		$this->cache = $this->getCache('attribute');
73 73
 	}
74 74
 
75 75
 
@@ -80,12 +80,12 @@  discard block
 block discarded – undo
80 80
 	 * @param array $data List of CSV fields with position as key and data as value
81 81
 	 * @return array List of data which hasn't been imported
82 82
 	 */
83
-	public function process( \Aimeos\MShop\Product\Item\Iface $product, array $data )
83
+	public function process(\Aimeos\MShop\Product\Item\Iface $product, array $data)
84 84
 	{
85 85
 		$context = $this->getContext();
86
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'attribute' );
87
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'product/lists' );
88
-		$separator = $context->getConfig()->get( 'controller/common/product/import/csv/separator', "\n" );
86
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'attribute');
87
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'product/lists');
88
+		$separator = $context->getConfig()->get('controller/common/product/import/csv/separator', "\n");
89 89
 
90 90
 		$manager->begin();
91 91
 
@@ -93,20 +93,20 @@  discard block
 block discarded – undo
93 93
 		{
94 94
 			$pos = 0;
95 95
 			$delete = array();
96
-			$map = $this->getMappedChunk( $data );
97
-			$listItems = $product->getListItems( 'attribute', $this->listTypes );
96
+			$map = $this->getMappedChunk($data);
97
+			$listItems = $product->getListItems('attribute', $this->listTypes);
98 98
 
99
-			foreach( $listItems as $listId => $listItem )
99
+			foreach ($listItems as $listId => $listItem)
100 100
 			{
101
-				if( isset( $map[$pos] ) )
101
+				if (isset($map[$pos]))
102 102
 				{
103
-					if( !isset( $map[$pos]['attribute.code'] ) || !isset( $map[$pos]['attribute.type'] ) )
103
+					if (!isset($map[$pos]['attribute.code']) || !isset($map[$pos]['attribute.type']))
104 104
 					{
105
-						unset( $map[$pos] );
105
+						unset($map[$pos]);
106 106
 						continue;
107 107
 					}
108 108
 
109
-					if( $this->checkMatch( $listItem, $map[$pos] ) === true )
109
+					if ($this->checkMatch($listItem, $map[$pos]) === true)
110 110
 					{
111 111
 						$pos++;
112 112
 						continue;
@@ -118,43 +118,43 @@  discard block
 block discarded – undo
118 118
 				$pos++;
119 119
 			}
120 120
 
121
-			$listManager->deleteItems( $delete );
121
+			$listManager->deleteItems($delete);
122 122
 
123
-			foreach( $map as $pos => $list )
123
+			foreach ($map as $pos => $list)
124 124
 			{
125
-				if( $this->checkEntry( $list ) === false ) {
125
+				if ($this->checkEntry($list) === false) {
126 126
 					continue;
127 127
 				}
128 128
 
129
-				$codes = explode( $separator, $list['attribute.code'] );
129
+				$codes = explode($separator, $list['attribute.code']);
130 130
 
131
-				foreach( $codes as $code )
131
+				foreach ($codes as $code)
132 132
 				{
133
-					$attrItem = $this->getAttributeItem( $code, $list['attribute.type'] );
134
-					$attrItem->fromArray( $list );
135
-					$attrItem->setCode( $code );
136
-					$manager->saveItem( $attrItem );
133
+					$attrItem = $this->getAttributeItem($code, $list['attribute.type']);
134
+					$attrItem->fromArray($list);
135
+					$attrItem->setCode($code);
136
+					$manager->saveItem($attrItem);
137 137
 
138
-					if( ( $listItem = array_shift( $listItems ) ) === null ) {
138
+					if (($listItem = array_shift($listItems)) === null) {
139 139
 						$listItem = $listManager->createItem();
140 140
 					}
141 141
 
142
-					$typecode = $this->getValue( $list, 'product.lists.type', 'default' );
143
-					$list['product.lists.typeid'] = $this->getTypeId( 'product/lists/type', 'attribute', $typecode );
142
+					$typecode = $this->getValue($list, 'product.lists.type', 'default');
143
+					$list['product.lists.typeid'] = $this->getTypeId('product/lists/type', 'attribute', $typecode);
144 144
 					$list['product.lists.refid'] = $attrItem->getId();
145 145
 					$list['product.lists.parentid'] = $product->getId();
146 146
 					$list['product.lists.domain'] = 'attribute';
147 147
 
148
-					$listItem->fromArray( $this->addListItemDefaults( $list, $pos ) );
149
-					$listManager->saveItem( $listItem );
148
+					$listItem->fromArray($this->addListItemDefaults($list, $pos));
149
+					$listManager->saveItem($listItem);
150 150
 				}
151 151
 			}
152 152
 
153
-			$remaining = $this->getObject()->process( $product, $data );
153
+			$remaining = $this->getObject()->process($product, $data);
154 154
 
155 155
 			$manager->commit();
156 156
 		}
157
-		catch( \Exception $e )
157
+		catch (\Exception $e)
158 158
 		{
159 159
 			$manager->rollback();
160 160
 			throw $e;
@@ -170,11 +170,11 @@  discard block
 block discarded – undo
170 170
 	 * @param array $list Associative list of key/value pairs from the mapped data
171 171
 	 * @return boolean True if the entry is valid, false if not
172 172
 	 */
173
-	protected function checkEntry( array $list )
173
+	protected function checkEntry(array $list)
174 174
 	{
175
-		if( !isset( $list['attribute.code'] ) || $list['attribute.code'] === '' || $list['attribute.type'] === ''
176
-			|| isset( $list['product.lists.type'] ) && $this->listTypes !== null
177
-			&& !in_array( $list['product.lists.type'], (array) $this->listTypes )
175
+		if (!isset($list['attribute.code']) || $list['attribute.code'] === '' || $list['attribute.type'] === ''
176
+			|| isset($list['product.lists.type']) && $this->listTypes !== null
177
+			&& !in_array($list['product.lists.type'], (array) $this->listTypes)
178 178
 		) {
179 179
 			return false;
180 180
 		}
@@ -190,14 +190,14 @@  discard block
 block discarded – undo
190 190
 	 * @param array $list Associative list of key/value pairs from the mapped data
191 191
 	 * @return boolean True if the list item matches the values in the list, false if not
192 192
 	 */
193
-	protected function checkMatch( \Aimeos\MShop\Common\Item\Lists\Iface $listItem, array $list )
193
+	protected function checkMatch(\Aimeos\MShop\Common\Item\Lists\Iface $listItem, array $list)
194 194
 	{
195 195
 		$refItem = $listItem->getRefItem();
196 196
 
197
-		if( $refItem !== null && $list['attribute.code'] === $refItem->getCode()
197
+		if ($refItem !== null && $list['attribute.code'] === $refItem->getCode()
198 198
 			&& $list['attribute.type'] === $refItem->getType()
199
-			&& ( !isset( $list['product.lists.type'] ) || isset( $list['product.lists.type'] )
200
-			&& $list['product.lists.type'] === $listItem->getType() )
199
+			&& (!isset($list['product.lists.type']) || isset($list['product.lists.type'])
200
+			&& $list['product.lists.type'] === $listItem->getType())
201 201
 		) {
202 202
 			return true;
203 203
 		}
@@ -213,21 +213,21 @@  discard block
 block discarded – undo
213 213
 	 * @param string $type Attribute type
214 214
 	 * @return \Aimeos\MShop\Attribute\Item\Iface Attribute item object
215 215
 	 */
216
-	protected function getAttributeItem( $code, $type )
216
+	protected function getAttributeItem($code, $type)
217 217
 	{
218
-		if( ( $item = $this->cache->get( $code, $type ) ) === null )
218
+		if (($item = $this->cache->get($code, $type)) === null)
219 219
 		{
220
-			$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' );
220
+			$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute');
221 221
 
222 222
 			$item = $manager->createItem();
223
-			$item->setTypeId( $this->getTypeId( 'attribute/type', 'product', $type ) );
224
-			$item->setCode( $code );
225
-			$item->setLabel( $type . ' ' . $code );
226
-			$item->setStatus( 1 );
223
+			$item->setTypeId($this->getTypeId('attribute/type', 'product', $type));
224
+			$item->setCode($code);
225
+			$item->setLabel($type . ' ' . $code);
226
+			$item->setStatus(1);
227 227
 
228
-			$manager->saveItem( $item );
228
+			$manager->saveItem($item);
229 229
 
230
-			$this->cache->set( $item );
230
+			$this->cache->set($item);
231 231
 		}
232 232
 
233 233
 		return $item;
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Order/Service/Payment/Standard.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 */
30 30
 	public function getName()
31 31
 	{
32
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Capture authorized payments' );
32
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Capture authorized payments');
33 33
 	}
34 34
 
35 35
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	public function getDescription()
42 42
 	{
43
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Authorized payments of orders will be captured after dispatching or after a configurable amount of time' );
43
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Authorized payments of orders will be captured after dispatching or after a configurable amount of time');
44 44
 	}
45 45
 
46 46
 
@@ -68,8 +68,8 @@  discard block
 block discarded – undo
68 68
 		 * @category User
69 69
 		 * @category Developer
70 70
 		 */
71
-		$days = $config->get( 'controller/jobs/order/service/payment/limit-days', 90 );
72
-		$date = date( 'Y-m-d 00:00:00', time() - 86400 * $days );
71
+		$days = $config->get('controller/jobs/order/service/payment/limit-days', 90);
72
+		$date = date('Y-m-d 00:00:00', time() - 86400 * $days);
73 73
 
74 74
 		/** controller/jobs/order/service/payment/capture-days
75 75
 		 * Automatically capture payments after the configured amount of days
@@ -84,90 +84,90 @@  discard block
 block discarded – undo
84 84
 		 * @category User
85 85
 		 * @category Developer
86 86
 		 */
87
-		$capDays = $config->get( 'controller/jobs/order/service/payment/capture-days', null );
87
+		$capDays = $config->get('controller/jobs/order/service/payment/capture-days', null);
88 88
 
89 89
 
90
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $context, 'service' );
90
+		$serviceManager = \Aimeos\MShop\Factory::createManager($context, 'service');
91 91
 		$serviceSearch = $serviceManager->createSearch();
92
-		$serviceSearch->setConditions( $serviceSearch->compare( '==', 'service.type.code', 'payment' ) );
92
+		$serviceSearch->setConditions($serviceSearch->compare('==', 'service.type.code', 'payment'));
93 93
 
94
-		$orderManager = \Aimeos\MShop\Factory::createManager( $context, 'order' );
94
+		$orderManager = \Aimeos\MShop\Factory::createManager($context, 'order');
95 95
 		$orderSearch = $orderManager->createSearch();
96 96
 
97
-		$status = array( \Aimeos\MShop\Order\Item\Base::STAT_DISPATCHED, \Aimeos\MShop\Order\Item\Base::STAT_DELIVERED );
97
+		$status = array(\Aimeos\MShop\Order\Item\Base::STAT_DISPATCHED, \Aimeos\MShop\Order\Item\Base::STAT_DELIVERED);
98 98
 		$start = 0;
99 99
 
100 100
 		do
101 101
 		{
102
-			$serviceItems = $serviceManager->searchItems( $serviceSearch );
102
+			$serviceItems = $serviceManager->searchItems($serviceSearch);
103 103
 
104
-			foreach( $serviceItems as $serviceItem )
104
+			foreach ($serviceItems as $serviceItem)
105 105
 			{
106 106
 				try
107 107
 				{
108
-					$serviceProvider = $serviceManager->getProvider( $serviceItem );
108
+					$serviceProvider = $serviceManager->getProvider($serviceItem);
109 109
 
110
-					if( !$serviceProvider->isImplemented( \Aimeos\MShop\Service\Provider\Payment\Base::FEAT_CAPTURE ) ) {
110
+					if (!$serviceProvider->isImplemented(\Aimeos\MShop\Service\Provider\Payment\Base::FEAT_CAPTURE)) {
111 111
 						continue;
112 112
 					}
113 113
 
114 114
 
115 115
 					$expr = array();
116
-					$expr[] = $orderSearch->compare( '>', 'order.datepayment', $date );
116
+					$expr[] = $orderSearch->compare('>', 'order.datepayment', $date);
117 117
 
118
-					if( $capDays !== null )
118
+					if ($capDays !== null)
119 119
 					{
120
-						$capdate = date( 'Y-m-d 00:00:00', time() - 86400 * $capDays );
121
-						$expr[] = $orderSearch->compare( '<=', 'order.datepayment', $capdate );
120
+						$capdate = date('Y-m-d 00:00:00', time() - 86400 * $capDays);
121
+						$expr[] = $orderSearch->compare('<=', 'order.datepayment', $capdate);
122 122
 					}
123 123
 					else
124 124
 					{
125
-						$expr[] = $orderSearch->compare( '==', 'order.statusdelivery', $status );
125
+						$expr[] = $orderSearch->compare('==', 'order.statusdelivery', $status);
126 126
 					}
127 127
 
128
-					$expr[] = $orderSearch->compare( '==', 'order.statuspayment', \Aimeos\MShop\Order\Item\Base::PAY_AUTHORIZED );
129
-					$expr[] = $orderSearch->compare( '==', 'order.base.service.code', $serviceItem->getCode() );
130
-					$expr[] = $orderSearch->compare( '==', 'order.base.service.type', 'payment' );
128
+					$expr[] = $orderSearch->compare('==', 'order.statuspayment', \Aimeos\MShop\Order\Item\Base::PAY_AUTHORIZED);
129
+					$expr[] = $orderSearch->compare('==', 'order.base.service.code', $serviceItem->getCode());
130
+					$expr[] = $orderSearch->compare('==', 'order.base.service.type', 'payment');
131 131
 
132
-					$orderSearch->setConditions( $orderSearch->combine( '&&', $expr ) );
132
+					$orderSearch->setConditions($orderSearch->combine('&&', $expr));
133 133
 
134 134
 
135 135
 					$orderStart = 0;
136 136
 
137 137
 					do
138 138
 					{
139
-						$orderItems = $orderManager->searchItems( $orderSearch );
139
+						$orderItems = $orderManager->searchItems($orderSearch);
140 140
 
141
-						foreach( $orderItems as $orderItem )
141
+						foreach ($orderItems as $orderItem)
142 142
 						{
143 143
 							try
144 144
 							{
145
-								$serviceProvider->capture( $orderItem );
145
+								$serviceProvider->capture($orderItem);
146 146
 							}
147
-							catch( \Exception $e )
147
+							catch (\Exception $e)
148 148
 							{
149 149
 								$str = 'Error while capturing payment for order with ID "%1$s": %2$s';
150
-								$context->getLogger()->log( sprintf( $str, $orderItem->getId(), $e->getMessage() ) );
150
+								$context->getLogger()->log(sprintf($str, $orderItem->getId(), $e->getMessage()));
151 151
 							}
152 152
 						}
153 153
 
154
-						$orderCount = count( $orderItems );
154
+						$orderCount = count($orderItems);
155 155
 						$orderStart += $orderCount;
156
-						$orderSearch->setSlice( $orderStart );
156
+						$orderSearch->setSlice($orderStart);
157 157
 					}
158
-					while( $orderCount >= $orderSearch->getSliceSize() );
158
+					while ($orderCount >= $orderSearch->getSliceSize());
159 159
 				}
160
-				catch( \Exception $e )
160
+				catch (\Exception $e)
161 161
 				{
162 162
 					$str = 'Error while capturing payments for service with ID "%1$s": %2$s';
163
-					$context->getLogger()->log( sprintf( $str, $serviceItem->getId(), $e->getMessage() ) );
163
+					$context->getLogger()->log(sprintf($str, $serviceItem->getId(), $e->getMessage()));
164 164
 				}
165 165
 			}
166 166
 
167
-			$count = count( $serviceItems );
167
+			$count = count($serviceItems);
168 168
 			$start += $count;
169
-			$serviceSearch->setSlice( $start );
169
+			$serviceSearch->setSlice($start);
170 170
 		}
171
-		while( $count >= $serviceSearch->getSliceSize() );
171
+		while ($count >= $serviceSearch->getSliceSize());
172 172
 	}
173 173
 }
Please login to merge, or discard this patch.