@@ -14,92 +14,92 @@ discard block |
||
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 | 20 | $aimeos = \TestHelperCntl::getAimeos(); |
21 | 21 | |
22 | - $this->object = new TestAbstract( $context, $aimeos ); |
|
22 | + $this->object = new TestAbstract($context, $aimeos); |
|
23 | 23 | } |
24 | 24 | |
25 | 25 | |
26 | 26 | protected function tearDown() |
27 | 27 | { |
28 | - \Aimeos\MShop\Factory::setCache( false ); |
|
28 | + \Aimeos\MShop\Factory::setCache(false); |
|
29 | 29 | \Aimeos\MShop\Factory::clear(); |
30 | 30 | } |
31 | 31 | |
32 | 32 | |
33 | 33 | public function testGetCache() |
34 | 34 | { |
35 | - $cache = $this->object->getCachePublic( 'attribute' ); |
|
35 | + $cache = $this->object->getCachePublic('attribute'); |
|
36 | 36 | |
37 | - $this->assertInstanceOf( '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\Iface', $cache ); |
|
37 | + $this->assertInstanceOf('\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\Iface', $cache); |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | |
41 | 41 | public function testGetCacheInvalidType() |
42 | 42 | { |
43 | - $this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' ); |
|
44 | - $this->object->getCachePublic( '$' ); |
|
43 | + $this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception'); |
|
44 | + $this->object->getCachePublic('$'); |
|
45 | 45 | } |
46 | 46 | |
47 | 47 | |
48 | 48 | public function testGetCacheInvalidClass() |
49 | 49 | { |
50 | - $this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' ); |
|
51 | - $this->object->getCachePublic( 'unknown' ); |
|
50 | + $this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception'); |
|
51 | + $this->object->getCachePublic('unknown'); |
|
52 | 52 | } |
53 | 53 | |
54 | 54 | |
55 | 55 | public function testGetCacheInvalidInterface() |
56 | 56 | { |
57 | - $this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' ); |
|
58 | - $this->object->getCachePublic( 'attribute', 'unknown' ); |
|
57 | + $this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception'); |
|
58 | + $this->object->getCachePublic('attribute', 'unknown'); |
|
59 | 59 | } |
60 | 60 | |
61 | 61 | |
62 | 62 | public function testGetProcessors() |
63 | 63 | { |
64 | - $processor = $this->object->getProcessorsPublic( array( 'attribute' => [] ) ); |
|
64 | + $processor = $this->object->getProcessorsPublic(array('attribute' => [])); |
|
65 | 65 | |
66 | - $this->assertInstanceOf( '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\Iface', $processor ); |
|
66 | + $this->assertInstanceOf('\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\Iface', $processor); |
|
67 | 67 | } |
68 | 68 | |
69 | 69 | |
70 | 70 | public function testGetProcessorsInvalidType() |
71 | 71 | { |
72 | - $this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' ); |
|
73 | - $this->object->getProcessorsPublic( array( '$' => [] ) ); |
|
72 | + $this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception'); |
|
73 | + $this->object->getProcessorsPublic(array('$' => [])); |
|
74 | 74 | } |
75 | 75 | |
76 | 76 | |
77 | 77 | public function testGetProcessorsInvalidClass() |
78 | 78 | { |
79 | - $this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' ); |
|
80 | - $this->object->getProcessorsPublic( array( 'unknown' => [] ) ); |
|
79 | + $this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception'); |
|
80 | + $this->object->getProcessorsPublic(array('unknown' => [])); |
|
81 | 81 | } |
82 | 82 | |
83 | 83 | |
84 | 84 | public function testGetProcessorsInvalidInterface() |
85 | 85 | { |
86 | - $this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' ); |
|
87 | - $this->object->getProcessorsPublic( array( 'unknown' => [] ) ); |
|
86 | + $this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception'); |
|
87 | + $this->object->getProcessorsPublic(array('unknown' => [])); |
|
88 | 88 | } |
89 | 89 | |
90 | 90 | |
91 | 91 | public function testGetTypeId() |
92 | 92 | { |
93 | - $typeid = $this->object->getTypeIdPublic( 'attribute/type', 'product', 'color' ); |
|
93 | + $typeid = $this->object->getTypeIdPublic('attribute/type', 'product', 'color'); |
|
94 | 94 | |
95 | - $this->assertNotEquals( null, $typeid ); |
|
95 | + $this->assertNotEquals(null, $typeid); |
|
96 | 96 | } |
97 | 97 | |
98 | 98 | |
99 | 99 | public function testGetTypeIdUnknown() |
100 | 100 | { |
101 | - $this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' ); |
|
102 | - $this->object->getTypeIdPublic( 'attribute/type', 'product', 'unknown' ); |
|
101 | + $this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception'); |
|
102 | + $this->object->getTypeIdPublic('attribute/type', 'product', 'unknown'); |
|
103 | 103 | } |
104 | 104 | } |
105 | 105 | |
@@ -107,21 +107,21 @@ discard block |
||
107 | 107 | class TestAbstract |
108 | 108 | extends \Aimeos\Controller\Common\Product\Import\Csv\Base |
109 | 109 | { |
110 | - public function getCachePublic( $type, $name = null ) |
|
110 | + public function getCachePublic($type, $name = null) |
|
111 | 111 | { |
112 | - return $this->getCache( $type, $name ); |
|
112 | + return $this->getCache($type, $name); |
|
113 | 113 | } |
114 | 114 | |
115 | 115 | |
116 | - public function getProcessorsPublic( array $mappings ) |
|
116 | + public function getProcessorsPublic(array $mappings) |
|
117 | 117 | { |
118 | - return $this->getProcessors( $mappings ); |
|
118 | + return $this->getProcessors($mappings); |
|
119 | 119 | } |
120 | 120 | |
121 | 121 | |
122 | - public function getTypeIdPublic( $path, $domain, $code ) |
|
122 | + public function getTypeIdPublic($path, $domain, $code) |
|
123 | 123 | { |
124 | - return $this->getTypeId( $path, $domain, $code ); |
|
124 | + return $this->getTypeId($path, $domain, $code); |
|
125 | 125 | } |
126 | 126 | } |
127 | 127 |
@@ -40,15 +40,15 @@ discard block |
||
40 | 40 | * |
41 | 41 | * @param \Aimeos\MShop\Context\Item\Iface $context Context object |
42 | 42 | */ |
43 | - public function __construct( \Aimeos\MShop\Context\Item\Iface $context ) |
|
43 | + public function __construct(\Aimeos\MShop\Context\Item\Iface $context) |
|
44 | 44 | { |
45 | - parent::__construct( $context ); |
|
45 | + parent::__construct($context); |
|
46 | 46 | |
47 | - $manager = \Aimeos\MShop\Factory::createManager( $context, 'attribute' ); |
|
48 | - $result = $manager->searchItems( $manager->createSearch() ); |
|
47 | + $manager = \Aimeos\MShop\Factory::createManager($context, 'attribute'); |
|
48 | + $result = $manager->searchItems($manager->createSearch()); |
|
49 | 49 | |
50 | - foreach( $result as $id => $item ) { |
|
51 | - $this->attributes[ $item->getCode() ][ $item->getType() ] = $item; |
|
50 | + foreach ($result as $id => $item) { |
|
51 | + $this->attributes[$item->getCode()][$item->getType()] = $item; |
|
52 | 52 | } |
53 | 53 | } |
54 | 54 | |
@@ -60,24 +60,24 @@ discard block |
||
60 | 60 | * @param string|null $type Attribute type |
61 | 61 | * @return \Aimeos\MShop\Attribute\Item\Iface|null Attribute object or null if not found |
62 | 62 | */ |
63 | - public function get( $code, $type = null ) |
|
63 | + public function get($code, $type = null) |
|
64 | 64 | { |
65 | - if( isset( $this->attributes[$code] ) && isset( $this->attributes[$code][$type] ) ) { |
|
65 | + if (isset($this->attributes[$code]) && isset($this->attributes[$code][$type])) { |
|
66 | 66 | return $this->attributes[$code][$type]; |
67 | 67 | } |
68 | 68 | |
69 | - $manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' ); |
|
69 | + $manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute'); |
|
70 | 70 | |
71 | 71 | $search = $manager->createSearch(); |
72 | 72 | $expr = array( |
73 | - $search->compare( '==', 'attribute.code', $code ), |
|
74 | - $search->compare( '==', 'attribute.type.code', $type ), |
|
73 | + $search->compare('==', 'attribute.code', $code), |
|
74 | + $search->compare('==', 'attribute.type.code', $type), |
|
75 | 75 | ); |
76 | - $search->setConditions( $search->combine( '&&', $expr ) ); |
|
76 | + $search->setConditions($search->combine('&&', $expr)); |
|
77 | 77 | |
78 | - $result = $manager->searchItems( $search ); |
|
78 | + $result = $manager->searchItems($search); |
|
79 | 79 | |
80 | - if( ( $item = reset( $result ) ) !== false ) |
|
80 | + if (($item = reset($result)) !== false) |
|
81 | 81 | { |
82 | 82 | $this->attributes[$code][$type] = $item; |
83 | 83 | return $item; |
@@ -90,14 +90,14 @@ discard block |
||
90 | 90 | * |
91 | 91 | * @param \Aimeos\MShop\Common\Item\Iface $item Attribute object |
92 | 92 | */ |
93 | - public function set( \Aimeos\MShop\Common\Item\Iface $item ) |
|
93 | + public function set(\Aimeos\MShop\Common\Item\Iface $item) |
|
94 | 94 | { |
95 | 95 | $code = $item->getCode(); |
96 | 96 | |
97 | - if( !isset( $this->attributes[$code] ) || !is_array( $this->attributes[$code] ) ) { |
|
97 | + if (!isset($this->attributes[$code]) || !is_array($this->attributes[$code])) { |
|
98 | 98 | $this->attributes[$code] = []; |
99 | 99 | } |
100 | 100 | |
101 | - $this->attributes[$code][ $item->getType() ] = $item; |
|
101 | + $this->attributes[$code][$item->getType()] = $item; |
|
102 | 102 | } |
103 | 103 | } |
104 | 104 | \ No newline at end of file |
@@ -30,14 +30,14 @@ discard block |
||
30 | 30 | * @param array $data Associative list of product codes and lists of CSV field indexes and their data |
31 | 31 | * @return array Associative list of CSV field indexes and their converted data |
32 | 32 | */ |
33 | - protected function convertData( array $convlist, array $data ) |
|
33 | + protected function convertData(array $convlist, array $data) |
|
34 | 34 | { |
35 | - foreach( $convlist as $idx => $converter ) |
|
35 | + foreach ($convlist as $idx => $converter) |
|
36 | 36 | { |
37 | - foreach( $data as $code => $list ) |
|
37 | + foreach ($data as $code => $list) |
|
38 | 38 | { |
39 | - if( isset( $list[$idx] ) ) { |
|
40 | - $data[$code][$idx] = $converter->translate( $list[$idx] ); |
|
39 | + if (isset($list[$idx])) { |
|
40 | + $data[$code][$idx] = $converter->translate($list[$idx]); |
|
41 | 41 | } |
42 | 42 | } |
43 | 43 | } |
@@ -53,38 +53,38 @@ discard block |
||
53 | 53 | * @param string|null Name of the cache implementation |
54 | 54 | * @return \Aimeos\Controller\Common\Product\Import\Csv\Cache\Iface Cache object |
55 | 55 | */ |
56 | - protected function getCache( $type, $name = null ) |
|
56 | + protected function getCache($type, $name = null) |
|
57 | 57 | { |
58 | 58 | $context = $this->getContext(); |
59 | 59 | $config = $context->getConfig(); |
60 | 60 | $iface = '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\Iface'; |
61 | 61 | |
62 | - if( ctype_alnum( $type ) === false ) |
|
62 | + if (ctype_alnum($type) === false) |
|
63 | 63 | { |
64 | 64 | $classname = is_string($name) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . $type : '<not a string>'; |
65 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) ); |
|
65 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname)); |
|
66 | 66 | } |
67 | 67 | |
68 | - if( $name === null ) { |
|
69 | - $name = $config->get( 'controller/common/product/import/csv/cache/' . $type . '/name', 'Standard' ); |
|
68 | + if ($name === null) { |
|
69 | + $name = $config->get('controller/common/product/import/csv/cache/' . $type . '/name', 'Standard'); |
|
70 | 70 | } |
71 | 71 | |
72 | - if( ctype_alnum( $name ) === false ) |
|
72 | + if (ctype_alnum($name) === false) |
|
73 | 73 | { |
74 | 74 | $classname = is_string($name) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . $type . '\\' . $name : '<not a string>'; |
75 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) ); |
|
75 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname)); |
|
76 | 76 | } |
77 | 77 | |
78 | - $classname = '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . ucfirst( $type ) . '\\' . $name; |
|
78 | + $classname = '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . ucfirst($type) . '\\' . $name; |
|
79 | 79 | |
80 | - if( class_exists( $classname ) === false ) { |
|
81 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Class "%1$s" not found', $classname ) ); |
|
80 | + if (class_exists($classname) === false) { |
|
81 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Class "%1$s" not found', $classname)); |
|
82 | 82 | } |
83 | 83 | |
84 | - $object = new $classname( $context ); |
|
84 | + $object = new $classname($context); |
|
85 | 85 | |
86 | - if( !( $object instanceof $iface ) ) { |
|
87 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Class "%1$s" does not implement interface "%2$s"', $classname, $iface ) ); |
|
86 | + if (!($object instanceof $iface)) { |
|
87 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Class "%1$s" does not implement interface "%2$s"', $classname, $iface)); |
|
88 | 88 | } |
89 | 89 | |
90 | 90 | return $object; |
@@ -97,12 +97,12 @@ discard block |
||
97 | 97 | * @param array $convmap List of converter names for the values at the position in the CSV file |
98 | 98 | * @return array Associative list of positions and converter objects |
99 | 99 | */ |
100 | - protected function getConverterList( array $convmap ) |
|
100 | + protected function getConverterList(array $convmap) |
|
101 | 101 | { |
102 | 102 | $convlist = []; |
103 | 103 | |
104 | - foreach( $convmap as $idx => $name ) { |
|
105 | - $convlist[$idx] = \Aimeos\MW\Convert\Factory::createConverter( $name ); |
|
104 | + foreach ($convmap as $idx => $name) { |
|
105 | + $convlist[$idx] = \Aimeos\MW\Convert\Factory::createConverter($name); |
|
106 | 106 | } |
107 | 107 | |
108 | 108 | return $convlist; |
@@ -117,15 +117,15 @@ discard block |
||
117 | 117 | * @param integer $codePos Column position which contains the unique product code (starting from 0) |
118 | 118 | * @return array List of arrays with product codes as keys and list of values from the CSV file |
119 | 119 | */ |
120 | - protected function getData( \Aimeos\MW\Container\Content\Iface $content, $maxcnt, $codePos ) |
|
120 | + protected function getData(\Aimeos\MW\Container\Content\Iface $content, $maxcnt, $codePos) |
|
121 | 121 | { |
122 | 122 | $count = 0; |
123 | 123 | $data = []; |
124 | 124 | |
125 | - while( $content->valid() && $count++ < $maxcnt ) |
|
125 | + while ($content->valid() && $count++ < $maxcnt) |
|
126 | 126 | { |
127 | 127 | $row = $content->current(); |
128 | - $data[ $row[$codePos] ] = $row; |
|
128 | + $data[$row[$codePos]] = $row; |
|
129 | 129 | $content->next(); |
130 | 130 | } |
131 | 131 | |
@@ -222,21 +222,21 @@ discard block |
||
222 | 222 | * @param array $mapping List of domain item keys with the CSV field position as key |
223 | 223 | * @return array List of associative arrays containing the chunked properties |
224 | 224 | */ |
225 | - protected function getMappedChunk( array &$data, array $mapping ) |
|
225 | + protected function getMappedChunk(array &$data, array $mapping) |
|
226 | 226 | { |
227 | 227 | $idx = 0; |
228 | 228 | $map = []; |
229 | 229 | |
230 | - foreach( $mapping as $pos => $key ) |
|
230 | + foreach ($mapping as $pos => $key) |
|
231 | 231 | { |
232 | - if( isset( $map[$idx][$key] ) ) { |
|
232 | + if (isset($map[$idx][$key])) { |
|
233 | 233 | $idx++; |
234 | 234 | } |
235 | 235 | |
236 | - if( isset( $data[$pos] ) ) |
|
236 | + if (isset($data[$pos])) |
|
237 | 237 | { |
238 | 238 | $map[$idx][$key] = $data[$pos]; |
239 | - unset( $data[$pos] ); |
|
239 | + unset($data[$pos]); |
|
240 | 240 | } |
241 | 241 | } |
242 | 242 | |
@@ -250,39 +250,39 @@ discard block |
||
250 | 250 | * @param array $mappings Associative list of processor types as keys and index/data mappings as values |
251 | 251 | * @return \Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface Processor object |
252 | 252 | */ |
253 | - protected function getProcessors( array $mappings ) |
|
253 | + protected function getProcessors(array $mappings) |
|
254 | 254 | { |
255 | 255 | $context = $this->getContext(); |
256 | 256 | $config = $context->getConfig(); |
257 | 257 | $iface = '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\Iface'; |
258 | - $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $context, [] ); |
|
258 | + $object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($context, []); |
|
259 | 259 | |
260 | - foreach( $mappings as $type => $mapping ) |
|
260 | + foreach ($mappings as $type => $mapping) |
|
261 | 261 | { |
262 | - if( ctype_alnum( $type ) === false ) |
|
262 | + if (ctype_alnum($type) === false) |
|
263 | 263 | { |
264 | 264 | $classname = is_string($type) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . $type : '<not a string>'; |
265 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) ); |
|
265 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname)); |
|
266 | 266 | } |
267 | 267 | |
268 | - $name = $config->get( 'controller/common/product/import/csv/processor/' . $type . '/name', 'Standard' ); |
|
268 | + $name = $config->get('controller/common/product/import/csv/processor/' . $type . '/name', 'Standard'); |
|
269 | 269 | |
270 | - if( ctype_alnum( $name ) === false ) |
|
270 | + if (ctype_alnum($name) === false) |
|
271 | 271 | { |
272 | 272 | $classname = is_string($name) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . $type . '\\' . $name : '<not a string>'; |
273 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) ); |
|
273 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname)); |
|
274 | 274 | } |
275 | 275 | |
276 | - $classname = '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . ucfirst( $type ) . '\\' . $name; |
|
276 | + $classname = '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . ucfirst($type) . '\\' . $name; |
|
277 | 277 | |
278 | - if( class_exists( $classname ) === false ) { |
|
279 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Class "%1$s" not found', $classname ) ); |
|
278 | + if (class_exists($classname) === false) { |
|
279 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Class "%1$s" not found', $classname)); |
|
280 | 280 | } |
281 | 281 | |
282 | - $object = new $classname( $context, $mapping, $object ); |
|
282 | + $object = new $classname($context, $mapping, $object); |
|
283 | 283 | |
284 | - if( !( $object instanceof $iface ) ) { |
|
285 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Class "%1$s" does not implement interface "%2$s"', $classname, $iface ) ); |
|
284 | + if (!($object instanceof $iface)) { |
|
285 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('Class "%1$s" does not implement interface "%2$s"', $classname, $iface)); |
|
286 | 286 | } |
287 | 287 | } |
288 | 288 | |
@@ -297,17 +297,17 @@ discard block |
||
297 | 297 | * @param array $domains List of domains whose items should be fetched too |
298 | 298 | * @return array Associative list of product codes as key and product items as value |
299 | 299 | */ |
300 | - protected function getProducts( array $codes, array $domains ) |
|
300 | + protected function getProducts(array $codes, array $domains) |
|
301 | 301 | { |
302 | 302 | $result = []; |
303 | - $manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' ); |
|
303 | + $manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product'); |
|
304 | 304 | |
305 | 305 | $search = $manager->createSearch(); |
306 | - $search->setConditions( $search->compare( '==', 'product.code', $codes ) ); |
|
307 | - $search->setSlice( 0, count( $codes ) ); |
|
306 | + $search->setConditions($search->compare('==', 'product.code', $codes)); |
|
307 | + $search->setSlice(0, count($codes)); |
|
308 | 308 | |
309 | - foreach( $manager->searchItems( $search, $domains ) as $item ) { |
|
310 | - $result[ $item->getCode() ] = $item; |
|
309 | + foreach ($manager->searchItems($search, $domains) as $item) { |
|
310 | + $result[$item->getCode()] = $item; |
|
311 | 311 | } |
312 | 312 | |
313 | 313 | return $result; |
@@ -322,23 +322,23 @@ discard block |
||
322 | 322 | * @param string $code Unique code of the type item |
323 | 323 | * @return string Unique ID of the type item |
324 | 324 | */ |
325 | - protected function getTypeId( $path, $domain, $code ) |
|
325 | + protected function getTypeId($path, $domain, $code) |
|
326 | 326 | { |
327 | - if( !isset( self::$types[$path][$domain] ) ) |
|
327 | + if (!isset(self::$types[$path][$domain])) |
|
328 | 328 | { |
329 | - $manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), $path ); |
|
330 | - $key = str_replace( '/', '.', $path ); |
|
329 | + $manager = \Aimeos\MShop\Factory::createManager($this->getContext(), $path); |
|
330 | + $key = str_replace('/', '.', $path); |
|
331 | 331 | |
332 | 332 | $search = $manager->createSearch(); |
333 | - $search->setConditions( $search->compare( '==', $key . '.domain', $domain ) ); |
|
333 | + $search->setConditions($search->compare('==', $key . '.domain', $domain)); |
|
334 | 334 | |
335 | - foreach( $manager->searchItems( $search ) as $id => $item ) { |
|
336 | - self::$types[$path][$domain][ $item->getCode() ] = $id; |
|
335 | + foreach ($manager->searchItems($search) as $id => $item) { |
|
336 | + self::$types[$path][$domain][$item->getCode()] = $id; |
|
337 | 337 | } |
338 | 338 | } |
339 | 339 | |
340 | - if( !isset( self::$types[$path][$domain][$code] ) ) { |
|
341 | - throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'No type item for "%1$s/%2$s" in "%3$s" found', $domain, $code, $path ) ); |
|
340 | + if (!isset(self::$types[$path][$domain][$code])) { |
|
341 | + throw new \Aimeos\Controller\Jobs\Exception(sprintf('No type item for "%1$s/%2$s" in "%3$s" found', $domain, $code, $path)); |
|
342 | 342 | } |
343 | 343 | |
344 | 344 | return self::$types[$path][$domain][$code]; |
@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | */ |
29 | 29 | public function getName() |
30 | 30 | { |
31 | - return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Product site map' ); |
|
31 | + return $this->getContext()->getI18n()->dt('controller/jobs', 'Product site map'); |
|
32 | 32 | } |
33 | 33 | |
34 | 34 | |
@@ -39,7 +39,7 @@ discard block |
||
39 | 39 | */ |
40 | 40 | public function getDescription() |
41 | 41 | { |
42 | - return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Creates a product site map for search engines' ); |
|
42 | + return $this->getContext()->getI18n()->dt('controller/jobs', 'Creates a product site map for search engines'); |
|
43 | 43 | } |
44 | 44 | |
45 | 45 | |
@@ -52,8 +52,8 @@ discard block |
||
52 | 52 | { |
53 | 53 | $container = $this->createContainer(); |
54 | 54 | |
55 | - $files = $this->export( $container ); |
|
56 | - $this->createSitemapIndex( $container, $files ); |
|
55 | + $files = $this->export($container); |
|
56 | + $this->createSitemapIndex($container, $files); |
|
57 | 57 | |
58 | 58 | $container->close(); |
59 | 59 | } |
@@ -65,7 +65,7 @@ discard block |
||
65 | 65 | * @param \Aimeos\MW\Container\Content\Iface $content File content object |
66 | 66 | * @param \Aimeos\MShop\Product\Item\Iface[] $items List of product items |
67 | 67 | */ |
68 | - protected function addItems( \Aimeos\MW\Container\Content\Iface $content, array $items ) |
|
68 | + protected function addItems(\Aimeos\MW\Container\Content\Iface $content, array $items) |
|
69 | 69 | { |
70 | 70 | $config = $this->getContext()->getConfig(); |
71 | 71 | |
@@ -96,7 +96,7 @@ discard block |
||
96 | 96 | * @see controller/jobs/product/export/sitemap/max-items |
97 | 97 | * @see controller/jobs/product/export/sitemap/max-query |
98 | 98 | */ |
99 | - $changefreq = $config->get( 'controller/jobs/product/export/sitemap/changefreq', 'daily' ); |
|
99 | + $changefreq = $config->get('controller/jobs/product/export/sitemap/changefreq', 'daily'); |
|
100 | 100 | |
101 | 101 | /** controller/jobs/product/export/sitemap/standard/template-items |
102 | 102 | * Relative path to the XML items template of the product site map job controller. |
@@ -129,7 +129,7 @@ discard block |
||
129 | 129 | $view->siteItems = $items; |
130 | 130 | $view->siteFreq = $changefreq; |
131 | 131 | |
132 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
132 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
133 | 133 | } |
134 | 134 | |
135 | 135 | |
@@ -175,7 +175,7 @@ discard block |
||
175 | 175 | * @see controller/jobs/product/export/sitemap/max-query |
176 | 176 | * @see controller/jobs/product/export/sitemap/changefreq |
177 | 177 | */ |
178 | - $location = $config->get( 'controller/jobs/product/export/sitemap/location', sys_get_temp_dir() ); |
|
178 | + $location = $config->get('controller/jobs/product/export/sitemap/location', sys_get_temp_dir()); |
|
179 | 179 | |
180 | 180 | /** controller/jobs/product/export/sitemap/container/options |
181 | 181 | * List of file container options for the site map files |
@@ -196,10 +196,10 @@ discard block |
||
196 | 196 | * @see controller/jobs/product/export/sitemap/max-query |
197 | 197 | * @see controller/jobs/product/export/sitemap/changefreq |
198 | 198 | */ |
199 | - $default = array( 'gzip-mode' => 'wb' ); |
|
200 | - $options = $config->get( 'controller/jobs/product/export/sitemap/container/options', $default ); |
|
199 | + $default = array('gzip-mode' => 'wb'); |
|
200 | + $options = $config->get('controller/jobs/product/export/sitemap/container/options', $default); |
|
201 | 201 | |
202 | - return \Aimeos\MW\Container\Factory::getContainer( $location, 'Directory', 'Gzip', $options ); |
|
202 | + return \Aimeos\MW\Container\Factory::getContainer($location, 'Directory', 'Gzip', $options); |
|
203 | 203 | } |
204 | 204 | |
205 | 205 | |
@@ -210,7 +210,7 @@ discard block |
||
210 | 210 | * @param integer $filenum New file number |
211 | 211 | * @return \Aimeos\MW\Container\Content\Iface New content object |
212 | 212 | */ |
213 | - protected function createContent( \Aimeos\MW\Container\Iface $container, $filenum ) |
|
213 | + protected function createContent(\Aimeos\MW\Container\Iface $container, $filenum) |
|
214 | 214 | { |
215 | 215 | /** controller/jobs/product/export/sitemap/standard/template-header |
216 | 216 | * Relative path to the XML site map header template of the product site map job controller. |
@@ -240,9 +240,9 @@ discard block |
||
240 | 240 | $context = $this->getContext(); |
241 | 241 | $view = $context->getView(); |
242 | 242 | |
243 | - $content = $container->create( $this->getFilename( $filenum ) ); |
|
244 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
245 | - $container->add( $content ); |
|
243 | + $content = $container->create($this->getFilename($filenum)); |
|
244 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
245 | + $container->add($content); |
|
246 | 246 | |
247 | 247 | return $content; |
248 | 248 | } |
@@ -253,7 +253,7 @@ discard block |
||
253 | 253 | * |
254 | 254 | * @param \Aimeos\MW\Container\Content\Iface $content |
255 | 255 | */ |
256 | - protected function closeContent( \Aimeos\MW\Container\Content\Iface $content ) |
|
256 | + protected function closeContent(\Aimeos\MW\Container\Content\Iface $content) |
|
257 | 257 | { |
258 | 258 | /** controller/jobs/product/export/sitemap/standard/template-footer |
259 | 259 | * Relative path to the XML site map footer template of the product site map job controller. |
@@ -283,7 +283,7 @@ discard block |
||
283 | 283 | $context = $this->getContext(); |
284 | 284 | $view = $context->getView(); |
285 | 285 | |
286 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
286 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
287 | 287 | } |
288 | 288 | |
289 | 289 | |
@@ -293,7 +293,7 @@ discard block |
||
293 | 293 | * @param \Aimeos\MW\Container\Iface $container File container object |
294 | 294 | * @param array $files List of generated site map file names |
295 | 295 | */ |
296 | - protected function createSitemapIndex( \Aimeos\MW\Container\Iface $container, array $files ) |
|
296 | + protected function createSitemapIndex(\Aimeos\MW\Container\Iface $container, array $files) |
|
297 | 297 | { |
298 | 298 | /** controller/jobs/product/export/sitemap/standard/template-index |
299 | 299 | * Relative path to the XML site map index template of the product site map job controller. |
@@ -325,9 +325,9 @@ discard block |
||
325 | 325 | |
326 | 326 | $view->siteFiles = $files; |
327 | 327 | |
328 | - $content = $container->create( 'aimeos-sitemap-index.xml' ); |
|
329 | - $content->add( $view->render( $context->getConfig()->get( $tplconf, $default ) ) ); |
|
330 | - $container->add( $content ); |
|
328 | + $content = $container->create('aimeos-sitemap-index.xml'); |
|
329 | + $content->add($view->render($context->getConfig()->get($tplconf, $default))); |
|
330 | + $container->add($content); |
|
331 | 331 | } |
332 | 332 | |
333 | 333 | |
@@ -338,11 +338,11 @@ discard block |
||
338 | 338 | * @param mixed $default Default value if name is unknown |
339 | 339 | * @return mixed Configuration value |
340 | 340 | */ |
341 | - protected function getConfig( $name, $default = null ) |
|
341 | + protected function getConfig($name, $default = null) |
|
342 | 342 | { |
343 | 343 | $config = $this->getContext()->getConfig(); |
344 | 344 | |
345 | - switch( $name ) |
|
345 | + switch ($name) |
|
346 | 346 | { |
347 | 347 | case 'domain': |
348 | 348 | return []; |
@@ -369,7 +369,7 @@ discard block |
||
369 | 369 | * @see controller/jobs/product/export/sitemap/max-query |
370 | 370 | * @see controller/jobs/product/export/sitemap/changefreq |
371 | 371 | */ |
372 | - return $config->get( 'controller/jobs/product/export/sitemap/max-items', 50000 ); |
|
372 | + return $config->get('controller/jobs/product/export/sitemap/max-items', 50000); |
|
373 | 373 | |
374 | 374 | case 'max-query': |
375 | 375 | /** controller/jobs/product/export/sitemap/max-query |
@@ -389,7 +389,7 @@ discard block |
||
389 | 389 | * @see controller/jobs/product/export/sitemap/max-items |
390 | 390 | * @see controller/jobs/product/export/sitemap/changefreq |
391 | 391 | */ |
392 | - return $config->get( 'controller/jobs/product/export/sitemap/max-query', 1000 ); |
|
392 | + return $config->get('controller/jobs/product/export/sitemap/max-query', 1000); |
|
393 | 393 | } |
394 | 394 | |
395 | 395 | return $default; |
@@ -402,8 +402,8 @@ discard block |
||
402 | 402 | * @param integer $number Current file number |
403 | 403 | * @return string New file name |
404 | 404 | */ |
405 | - protected function getFilename( $number ) |
|
405 | + protected function getFilename($number) |
|
406 | 406 | { |
407 | - return sprintf( 'aimeos-sitemap-%d.xml', $number ); |
|
407 | + return sprintf('aimeos-sitemap-%d.xml', $number); |
|
408 | 408 | } |
409 | 409 | } |