@@ -17,63 +17,63 @@ discard block |
||
17 | 17 | |
18 | 18 | protected function setUp() |
19 | 19 | { |
20 | - $this->object = new \Aimeos\Controller\Frontend\Catalog\Standard( \TestHelperFrontend::getContext() ); |
|
20 | + $this->object = new \Aimeos\Controller\Frontend\Catalog\Standard(\TestHelperFrontend::getContext()); |
|
21 | 21 | } |
22 | 22 | |
23 | 23 | |
24 | 24 | protected function tearDown() |
25 | 25 | { |
26 | - unset( $this->object ); |
|
26 | + unset($this->object); |
|
27 | 27 | } |
28 | 28 | |
29 | 29 | |
30 | 30 | public function testCreateManager() |
31 | 31 | { |
32 | - $this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->createManager( 'product' ) ); |
|
32 | + $this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->createManager('product')); |
|
33 | 33 | } |
34 | 34 | |
35 | 35 | |
36 | 36 | public function testCreateCatalogFilter() |
37 | 37 | { |
38 | - $filter = $this->object->createCatalogFilter( true ); |
|
38 | + $filter = $this->object->createCatalogFilter(true); |
|
39 | 39 | |
40 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
41 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Expression\\Compare\\Iface', $filter->getConditions() ); |
|
42 | - $this->assertEquals( 'catalog.status', $filter->getConditions()->getName() ); |
|
40 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
41 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Expression\\Compare\\Iface', $filter->getConditions()); |
|
42 | + $this->assertEquals('catalog.status', $filter->getConditions()->getName()); |
|
43 | 43 | } |
44 | 44 | |
45 | 45 | |
46 | 46 | public function testGetCatalogPath() |
47 | 47 | { |
48 | - $manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager( \TestHelperFrontend::getContext() ); |
|
49 | - $item = $manager->getTree( null, array(), \Aimeos\MW\Tree\Manager\Base::LEVEL_LIST ); |
|
48 | + $manager = \Aimeos\MShop\Catalog\Manager\Factory::createManager(\TestHelperFrontend::getContext()); |
|
49 | + $item = $manager->getTree(null, array(), \Aimeos\MW\Tree\Manager\Base::LEVEL_LIST); |
|
50 | 50 | |
51 | 51 | $list = array(); |
52 | - foreach( $this->object->getCatalogPath( $item->getChild( 0 )->getId(), array( 'text' ) ) as $item ) { |
|
52 | + foreach ($this->object->getCatalogPath($item->getChild(0)->getId(), array('text')) as $item) { |
|
53 | 53 | $list[$item->getCode()] = $item; |
54 | 54 | } |
55 | 55 | |
56 | - $this->assertEquals( 2, count( $list ) ); |
|
57 | - $this->assertArrayHasKey( 'root', $list ); |
|
58 | - $this->assertArrayHasKey( 'categories', $list ); |
|
56 | + $this->assertEquals(2, count($list)); |
|
57 | + $this->assertArrayHasKey('root', $list); |
|
58 | + $this->assertArrayHasKey('categories', $list); |
|
59 | 59 | } |
60 | 60 | |
61 | 61 | |
62 | 62 | public function testGetCatalogTree() |
63 | 63 | { |
64 | - $item = $this->object->getCatalogTree( null, array( 'text' ), \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE ); |
|
64 | + $item = $this->object->getCatalogTree(null, array('text'), \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE); |
|
65 | 65 | |
66 | - $this->assertEquals( 'Root', $item->getName() ); |
|
67 | - $this->assertEquals( 0, count( $item->getChildren() ) ); |
|
66 | + $this->assertEquals('Root', $item->getName()); |
|
67 | + $this->assertEquals(0, count($item->getChildren())); |
|
68 | 68 | } |
69 | 69 | |
70 | 70 | |
71 | 71 | public function testAggregateIndex() |
72 | 72 | { |
73 | 73 | $filter = $this->object->createIndexFilter(); |
74 | - $list = $this->object->aggregateIndex( $filter, 'index.attribute.id' ); |
|
74 | + $list = $this->object->aggregateIndex($filter, 'index.attribute.id'); |
|
75 | 75 | |
76 | - $this->assertGreaterThan( 0, count( $list ) ); |
|
76 | + $this->assertGreaterThan(0, count($list)); |
|
77 | 77 | } |
78 | 78 | |
79 | 79 | |
@@ -81,286 +81,286 @@ discard block |
||
81 | 81 | { |
82 | 82 | $filter = $this->object->createIndexFilter(); |
83 | 83 | |
84 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
85 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
86 | - $this->assertEquals( 0, $filter->getSliceStart() ); |
|
87 | - $this->assertEquals( 100, $filter->getSliceSize() ); |
|
84 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
85 | + $this->assertEquals(array(), $filter->getSortations()); |
|
86 | + $this->assertEquals(0, $filter->getSliceStart()); |
|
87 | + $this->assertEquals(100, $filter->getSliceSize()); |
|
88 | 88 | } |
89 | 89 | |
90 | 90 | |
91 | 91 | public function testCreateIndexFilterCategory() |
92 | 92 | { |
93 | - $filter = $this->object->createIndexFilterCategory( 0 ); |
|
93 | + $filter = $this->object->createIndexFilterCategory(0); |
|
94 | 94 | |
95 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
95 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
96 | 96 | |
97 | 97 | $list = $filter->getConditions()->getExpressions(); |
98 | 98 | |
99 | 99 | |
100 | - if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) { |
|
101 | - throw new \RuntimeException( 'Wrong expression' ); |
|
100 | + if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) { |
|
101 | + throw new \RuntimeException('Wrong expression'); |
|
102 | 102 | } |
103 | - $this->assertEquals( 'index.catalog.id', $list[0]->getName() ); |
|
104 | - $this->assertEquals( array( 0 ), $list[0]->getValue() ); |
|
103 | + $this->assertEquals('index.catalog.id', $list[0]->getName()); |
|
104 | + $this->assertEquals(array(0), $list[0]->getValue()); |
|
105 | 105 | |
106 | 106 | |
107 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
108 | - $this->assertEquals( 0, $filter->getSliceStart() ); |
|
109 | - $this->assertEquals( 100, $filter->getSliceSize() ); |
|
107 | + $this->assertEquals(array(), $filter->getSortations()); |
|
108 | + $this->assertEquals(0, $filter->getSliceStart()); |
|
109 | + $this->assertEquals(100, $filter->getSliceSize()); |
|
110 | 110 | } |
111 | 111 | |
112 | 112 | |
113 | 113 | public function testCreateIndexFilterCategoryPosition() |
114 | 114 | { |
115 | - $filter = $this->object->createIndexFilterCategory( 0, 'relevance', '-', 1, 2, 'test' ); |
|
115 | + $filter = $this->object->createIndexFilterCategory(0, 'relevance', '-', 1, 2, 'test'); |
|
116 | 116 | |
117 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
117 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
118 | 118 | |
119 | 119 | $sort = $filter->getSortations(); |
120 | - if( ( $item = reset( $sort ) ) === false ) { |
|
121 | - throw new \RuntimeException( 'Sortation not set' ); |
|
120 | + if (($item = reset($sort)) === false) { |
|
121 | + throw new \RuntimeException('Sortation not set'); |
|
122 | 122 | } |
123 | 123 | |
124 | - $this->assertEquals( 'sort:index.catalog.position("test",["0"])', $item->getName() ); |
|
125 | - $this->assertEquals( '-', $item->getOperator() ); |
|
124 | + $this->assertEquals('sort:index.catalog.position("test",["0"])', $item->getName()); |
|
125 | + $this->assertEquals('-', $item->getOperator()); |
|
126 | 126 | |
127 | - $this->assertEquals( 1, $filter->getSliceStart() ); |
|
128 | - $this->assertEquals( 2, $filter->getSliceSize() ); |
|
127 | + $this->assertEquals(1, $filter->getSliceStart()); |
|
128 | + $this->assertEquals(2, $filter->getSliceSize()); |
|
129 | 129 | } |
130 | 130 | |
131 | 131 | |
132 | 132 | public function testCreateIndexFilterCategoryCode() |
133 | 133 | { |
134 | - $filter = $this->object->createIndexFilterCategory( 0, 'code' ); |
|
134 | + $filter = $this->object->createIndexFilterCategory(0, 'code'); |
|
135 | 135 | |
136 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
136 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
137 | 137 | |
138 | 138 | $sort = $filter->getSortations(); |
139 | - if( ( $item = reset( $sort ) ) === false ) { |
|
140 | - throw new \RuntimeException( 'Sortation not set' ); |
|
139 | + if (($item = reset($sort)) === false) { |
|
140 | + throw new \RuntimeException('Sortation not set'); |
|
141 | 141 | } |
142 | 142 | |
143 | - $this->assertStringStartsWith( 'product.code', $item->getName() ); |
|
143 | + $this->assertStringStartsWith('product.code', $item->getName()); |
|
144 | 144 | } |
145 | 145 | |
146 | 146 | |
147 | 147 | public function testCreateIndexFilterCategoryName() |
148 | 148 | { |
149 | - $filter = $this->object->createIndexFilterCategory( 0, 'name' ); |
|
149 | + $filter = $this->object->createIndexFilterCategory(0, 'name'); |
|
150 | 150 | |
151 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
151 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
152 | 152 | |
153 | 153 | $sort = $filter->getSortations(); |
154 | - if( ( $item = reset( $sort ) ) === false ) { |
|
155 | - throw new \RuntimeException( 'Sortation not set' ); |
|
154 | + if (($item = reset($sort)) === false) { |
|
155 | + throw new \RuntimeException('Sortation not set'); |
|
156 | 156 | } |
157 | 157 | |
158 | - $this->assertEquals( 'sort:index.text.value("default","de","name")', $item->getName() ); |
|
158 | + $this->assertEquals('sort:index.text.value("default","de","name")', $item->getName()); |
|
159 | 159 | } |
160 | 160 | |
161 | 161 | |
162 | 162 | public function testCreateIndexFilterCategoryPrice() |
163 | 163 | { |
164 | - $filter = $this->object->createIndexFilterCategory( 0, 'price' ); |
|
164 | + $filter = $this->object->createIndexFilterCategory(0, 'price'); |
|
165 | 165 | |
166 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
166 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
167 | 167 | |
168 | 168 | $sort = $filter->getSortations(); |
169 | - if( ( $item = reset( $sort ) ) === false ) { |
|
170 | - throw new \RuntimeException( 'Sortation not set' ); |
|
169 | + if (($item = reset($sort)) === false) { |
|
170 | + throw new \RuntimeException('Sortation not set'); |
|
171 | 171 | } |
172 | 172 | |
173 | - $this->assertStringStartsWith( 'sort:index.price.value("default","EUR","default")', $item->getName() ); |
|
173 | + $this->assertStringStartsWith('sort:index.price.value("default","EUR","default")', $item->getName()); |
|
174 | 174 | } |
175 | 175 | |
176 | 176 | |
177 | 177 | public function testCreateIndexFilterCategoryInvalidSortation() |
178 | 178 | { |
179 | - $filter = $this->object->createIndexFilterCategory( 0, 'failure' ); |
|
179 | + $filter = $this->object->createIndexFilterCategory(0, 'failure'); |
|
180 | 180 | |
181 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
182 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
181 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
182 | + $this->assertEquals(array(), $filter->getSortations()); |
|
183 | 183 | } |
184 | 184 | |
185 | 185 | |
186 | 186 | public function testAddIndexFilterCategory() |
187 | 187 | { |
188 | 188 | $filter = $this->object->createIndexFilter(); |
189 | - $filter = $this->object->addIndexFilterCategory( $filter, 0 ); |
|
189 | + $filter = $this->object->addIndexFilterCategory($filter, 0); |
|
190 | 190 | |
191 | 191 | $list = $filter->getConditions()->getExpressions(); |
192 | 192 | |
193 | - if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) { |
|
194 | - throw new \RuntimeException( 'Wrong expression' ); |
|
193 | + if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) { |
|
194 | + throw new \RuntimeException('Wrong expression'); |
|
195 | 195 | } |
196 | 196 | |
197 | - $this->assertEquals( 'index.catalog.id', $list[0]->getName() ); |
|
198 | - $this->assertEquals( array( 0 ), $list[0]->getValue() ); |
|
199 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
197 | + $this->assertEquals('index.catalog.id', $list[0]->getName()); |
|
198 | + $this->assertEquals(array(0), $list[0]->getValue()); |
|
199 | + $this->assertEquals(array(), $filter->getSortations()); |
|
200 | 200 | } |
201 | 201 | |
202 | 202 | |
203 | 203 | public function testCreateIndexFilterText() |
204 | 204 | { |
205 | - $filter = $this->object->createIndexFilterText( 'Espresso' ); |
|
205 | + $filter = $this->object->createIndexFilterText('Espresso'); |
|
206 | 206 | |
207 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
207 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
208 | 208 | |
209 | 209 | $list = $filter->getConditions()->getExpressions(); |
210 | 210 | |
211 | 211 | |
212 | - if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) { |
|
213 | - throw new \RuntimeException( 'Wrong expression' ); |
|
212 | + if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) { |
|
213 | + throw new \RuntimeException('Wrong expression'); |
|
214 | 214 | } |
215 | - $this->assertEquals( 'index.text.relevance("default","de","Espresso")', $list[0]->getName() ); |
|
216 | - $this->assertEquals( 0, $list[0]->getValue() ); |
|
215 | + $this->assertEquals('index.text.relevance("default","de","Espresso")', $list[0]->getName()); |
|
216 | + $this->assertEquals(0, $list[0]->getValue()); |
|
217 | 217 | |
218 | 218 | |
219 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
220 | - $this->assertEquals( 0, $filter->getSliceStart() ); |
|
221 | - $this->assertEquals( 100, $filter->getSliceSize() ); |
|
219 | + $this->assertEquals(array(), $filter->getSortations()); |
|
220 | + $this->assertEquals(0, $filter->getSliceStart()); |
|
221 | + $this->assertEquals(100, $filter->getSliceSize()); |
|
222 | 222 | } |
223 | 223 | |
224 | 224 | |
225 | 225 | public function testCreateIndexFilterTextRelevance() |
226 | 226 | { |
227 | - $filter = $this->object->createIndexFilterText( 'Espresso', 'relevance', '-', 1, 2, 'test' ); |
|
227 | + $filter = $this->object->createIndexFilterText('Espresso', 'relevance', '-', 1, 2, 'test'); |
|
228 | 228 | |
229 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
230 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
231 | - $this->assertEquals( 1, $filter->getSliceStart() ); |
|
232 | - $this->assertEquals( 2, $filter->getSliceSize() ); |
|
229 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
230 | + $this->assertEquals(array(), $filter->getSortations()); |
|
231 | + $this->assertEquals(1, $filter->getSliceStart()); |
|
232 | + $this->assertEquals(2, $filter->getSliceSize()); |
|
233 | 233 | } |
234 | 234 | |
235 | 235 | |
236 | 236 | public function testCreateIndexFilterTextCode() |
237 | 237 | { |
238 | - $filter = $this->object->createIndexFilterText( 'Espresso', 'code' ); |
|
238 | + $filter = $this->object->createIndexFilterText('Espresso', 'code'); |
|
239 | 239 | |
240 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
240 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
241 | 241 | |
242 | 242 | $sort = $filter->getSortations(); |
243 | - if( ( $item = reset( $sort ) ) === false ) { |
|
244 | - throw new \RuntimeException( 'Sortation not set' ); |
|
243 | + if (($item = reset($sort)) === false) { |
|
244 | + throw new \RuntimeException('Sortation not set'); |
|
245 | 245 | } |
246 | 246 | |
247 | - $this->assertEquals( 'product.code', $item->getName() ); |
|
247 | + $this->assertEquals('product.code', $item->getName()); |
|
248 | 248 | } |
249 | 249 | |
250 | 250 | |
251 | 251 | public function testCreateIndexFilterTextName() |
252 | 252 | { |
253 | - $filter = $this->object->createIndexFilterText( 'Espresso', 'name' ); |
|
253 | + $filter = $this->object->createIndexFilterText('Espresso', 'name'); |
|
254 | 254 | |
255 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
255 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
256 | 256 | |
257 | 257 | $sort = $filter->getSortations(); |
258 | - if( ( $item = reset( $sort ) ) === false ) { |
|
259 | - throw new \RuntimeException( 'Sortation not set' ); |
|
258 | + if (($item = reset($sort)) === false) { |
|
259 | + throw new \RuntimeException('Sortation not set'); |
|
260 | 260 | } |
261 | 261 | |
262 | - $this->assertEquals( 'sort:index.text.value("default","de","name")', $item->getName() ); |
|
262 | + $this->assertEquals('sort:index.text.value("default","de","name")', $item->getName()); |
|
263 | 263 | } |
264 | 264 | |
265 | 265 | |
266 | 266 | public function testCreateIndexFilterTextPrice() |
267 | 267 | { |
268 | - $filter = $this->object->createIndexFilterCategory( 'Espresso', 'price' ); |
|
268 | + $filter = $this->object->createIndexFilterCategory('Espresso', 'price'); |
|
269 | 269 | |
270 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
270 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
271 | 271 | |
272 | 272 | $sort = $filter->getSortations(); |
273 | - if( ( $item = reset( $sort ) ) === false ) { |
|
274 | - throw new \RuntimeException( 'Sortation not set' ); |
|
273 | + if (($item = reset($sort)) === false) { |
|
274 | + throw new \RuntimeException('Sortation not set'); |
|
275 | 275 | } |
276 | 276 | |
277 | - $this->assertStringStartsWith( 'sort:index.price.value("default","EUR","default")', $item->getName() ); |
|
277 | + $this->assertStringStartsWith('sort:index.price.value("default","EUR","default")', $item->getName()); |
|
278 | 278 | } |
279 | 279 | |
280 | 280 | |
281 | 281 | public function testCreateIndexFilterTextInvalidSortation() |
282 | 282 | { |
283 | - $filter = $this->object->createIndexFilterText( '', 'failure' ); |
|
283 | + $filter = $this->object->createIndexFilterText('', 'failure'); |
|
284 | 284 | |
285 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
286 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
285 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
286 | + $this->assertEquals(array(), $filter->getSortations()); |
|
287 | 287 | } |
288 | 288 | |
289 | 289 | |
290 | 290 | public function testAddIndexFilterText() |
291 | 291 | { |
292 | - $filter = $this->object->createIndexFilterText( 'Espresso' ); |
|
293 | - $filter = $this->object->addIndexFilterText( $filter, 'Espresso' ); |
|
292 | + $filter = $this->object->createIndexFilterText('Espresso'); |
|
293 | + $filter = $this->object->addIndexFilterText($filter, 'Espresso'); |
|
294 | 294 | |
295 | 295 | $list = $filter->getConditions()->getExpressions(); |
296 | 296 | |
297 | - if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) { |
|
298 | - throw new \RuntimeException( 'Wrong expression' ); |
|
297 | + if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) { |
|
298 | + throw new \RuntimeException('Wrong expression'); |
|
299 | 299 | } |
300 | 300 | |
301 | - $this->assertEquals( 'index.text.relevance("default","de","Espresso")', $list[0]->getName() ); |
|
302 | - $this->assertEquals( 0, $list[0]->getValue() ); |
|
303 | - $this->assertEquals( array(), $filter->getSortations() ); |
|
301 | + $this->assertEquals('index.text.relevance("default","de","Espresso")', $list[0]->getName()); |
|
302 | + $this->assertEquals(0, $list[0]->getValue()); |
|
303 | + $this->assertEquals(array(), $filter->getSortations()); |
|
304 | 304 | } |
305 | 305 | |
306 | 306 | |
307 | 307 | public function testGetIndexItemsCategory() |
308 | 308 | { |
309 | - $catalogManager = \Aimeos\MShop\Catalog\Manager\Factory::createManager( \TestHelperFrontend::getContext() ); |
|
309 | + $catalogManager = \Aimeos\MShop\Catalog\Manager\Factory::createManager(\TestHelperFrontend::getContext()); |
|
310 | 310 | $search = $catalogManager->createSearch(); |
311 | 311 | |
312 | - $search->setConditions( $search->compare( '==', 'catalog.code', 'new' ) ); |
|
313 | - $search->setSlice( 0, 1 ); |
|
314 | - $items = $catalogManager->searchItems( $search ); |
|
312 | + $search->setConditions($search->compare('==', 'catalog.code', 'new')); |
|
313 | + $search->setSlice(0, 1); |
|
314 | + $items = $catalogManager->searchItems($search); |
|
315 | 315 | |
316 | - if( ( $item = reset( $items ) ) === false ) { |
|
317 | - throw new \RuntimeException( 'Catalog item not found' ); |
|
316 | + if (($item = reset($items)) === false) { |
|
317 | + throw new \RuntimeException('Catalog item not found'); |
|
318 | 318 | } |
319 | 319 | |
320 | 320 | $total = 0; |
321 | - $filter = $this->object->createIndexFilterCategory( $item->getId(), 'position', '+', 1, 1 ); |
|
322 | - $results = $this->object->getIndexItems( $filter, array(), $total ); |
|
321 | + $filter = $this->object->createIndexFilterCategory($item->getId(), 'position', '+', 1, 1); |
|
322 | + $results = $this->object->getIndexItems($filter, array(), $total); |
|
323 | 323 | |
324 | - $this->assertEquals( 3, $total ); |
|
325 | - $this->assertEquals( 1, count( $results ) ); |
|
324 | + $this->assertEquals(3, $total); |
|
325 | + $this->assertEquals(1, count($results)); |
|
326 | 326 | } |
327 | 327 | |
328 | 328 | |
329 | 329 | public function testGetIndexItemsText() |
330 | 330 | { |
331 | 331 | $total = 0; |
332 | - $filter = $this->object->createIndexFilterText( 'Expresso', 'relevance', '+', 0, 1, 'unittype13' ); |
|
333 | - $results = $this->object->getIndexItems( $filter, array(), $total ); |
|
332 | + $filter = $this->object->createIndexFilterText('Expresso', 'relevance', '+', 0, 1, 'unittype13'); |
|
333 | + $results = $this->object->getIndexItems($filter, array(), $total); |
|
334 | 334 | |
335 | - $this->assertEquals( 2, $total ); |
|
336 | - $this->assertEquals( 1, count( $results ) ); |
|
335 | + $this->assertEquals(2, $total); |
|
336 | + $this->assertEquals(1, count($results)); |
|
337 | 337 | } |
338 | 338 | |
339 | 339 | |
340 | 340 | public function testGetProductItems() |
341 | 341 | { |
342 | - $list = $this->object->getProductItems( [-1] ); |
|
343 | - $this->assertEquals( 0, count( $list ) ); |
|
342 | + $list = $this->object->getProductItems([-1]); |
|
343 | + $this->assertEquals(0, count($list)); |
|
344 | 344 | } |
345 | 345 | |
346 | 346 | |
347 | 347 | public function testCreateTextFilter() |
348 | 348 | { |
349 | - $filter = $this->object->createTextFilter( 'Expresso', 'name', '+', 0, 1 ); |
|
349 | + $filter = $this->object->createTextFilter('Expresso', 'name', '+', 0, 1); |
|
350 | 350 | |
351 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
352 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Expression\\Combine\\Iface', $filter->getConditions() ); |
|
353 | - $this->assertEquals( 3, count( $filter->getConditions()->getExpressions() ) ); |
|
351 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
352 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Expression\\Combine\\Iface', $filter->getConditions()); |
|
353 | + $this->assertEquals(3, count($filter->getConditions()->getExpressions())); |
|
354 | 354 | } |
355 | 355 | |
356 | 356 | |
357 | 357 | public function testGetTextListName() |
358 | 358 | { |
359 | - $filter = $this->object->createTextFilter( 'Cafe Noire', 'relevance', '-', 0, 25, 'unittype19', 'name' ); |
|
360 | - $results = $this->object->getTextList( $filter ); |
|
359 | + $filter = $this->object->createTextFilter('Cafe Noire', 'relevance', '-', 0, 25, 'unittype19', 'name'); |
|
360 | + $results = $this->object->getTextList($filter); |
|
361 | 361 | |
362 | - $this->assertEquals( 1, count( $results ) ); |
|
363 | - $this->assertContains( 'Cafe Noire Cappuccino', $results ); |
|
362 | + $this->assertEquals(1, count($results)); |
|
363 | + $this->assertContains('Cafe Noire Cappuccino', $results); |
|
364 | 364 | } |
365 | 365 | |
366 | 366 | } |
@@ -104,7 +104,7 @@ |
||
104 | 104 | /** |
105 | 105 | * Returns the frontend controller |
106 | 106 | * |
107 | - * @return \Aimeos\Controller\Frontend\Locale\Iface Frontend controller object |
|
107 | + * @return \Aimeos\Controller\Frontend\Iface Frontend controller object |
|
108 | 108 | */ |
109 | 109 | protected function getController() |
110 | 110 | { |
@@ -30,18 +30,18 @@ discard block |
||
30 | 30 | * @param \Aimeos\Controller\Frontend\Iface $controller Controller object |
31 | 31 | * @param \Aimeos\MShop\Context\Item\Iface $context Context object with required objects |
32 | 32 | */ |
33 | - public function __construct( \Aimeos\Controller\Frontend\Iface $controller, \Aimeos\MShop\Context\Item\Iface $context ) |
|
33 | + public function __construct(\Aimeos\Controller\Frontend\Iface $controller, \Aimeos\MShop\Context\Item\Iface $context) |
|
34 | 34 | { |
35 | 35 | $iface = '\Aimeos\Controller\Frontend\Locale\Iface'; |
36 | - if( !( $controller instanceof $iface ) ) |
|
36 | + if (!($controller instanceof $iface)) |
|
37 | 37 | { |
38 | - $msg = sprintf( 'Class "%1$s" does not implement interface "%2$s"', get_class( $controller ), $iface ); |
|
39 | - throw new \Aimeos\Controller\Frontend\Exception( $msg ); |
|
38 | + $msg = sprintf('Class "%1$s" does not implement interface "%2$s"', get_class($controller), $iface); |
|
39 | + throw new \Aimeos\Controller\Frontend\Exception($msg); |
|
40 | 40 | } |
41 | 41 | |
42 | 42 | $this->controller = $controller; |
43 | 43 | |
44 | - parent::__construct( $context ); |
|
44 | + parent::__construct($context); |
|
45 | 45 | } |
46 | 46 | |
47 | 47 | |
@@ -53,9 +53,9 @@ discard block |
||
53 | 53 | * @return mixed Returns the value of the called method |
54 | 54 | * @throws \Aimeos\Controller\Frontend\Exception If method call failed |
55 | 55 | */ |
56 | - public function __call( $name, array $param ) |
|
56 | + public function __call($name, array $param) |
|
57 | 57 | { |
58 | - return @call_user_func_array( array( $this->controller, $name ), $param ); |
|
58 | + return @call_user_func_array(array($this->controller, $name), $param); |
|
59 | 59 | } |
60 | 60 | |
61 | 61 | |
@@ -80,9 +80,9 @@ discard block |
||
80 | 80 | * @return \Aimeos\MShop\Locale\Item\Iface Locale item including the referenced domains items |
81 | 81 | * @since 2017.03 |
82 | 82 | */ |
83 | - public function getItem( $id, array $domains = array() ) |
|
83 | + public function getItem($id, array $domains = array()) |
|
84 | 84 | { |
85 | - return $this->controller->getItem( $id, $domains ); |
|
85 | + return $this->controller->getItem($id, $domains); |
|
86 | 86 | } |
87 | 87 | |
88 | 88 | |
@@ -95,9 +95,9 @@ discard block |
||
95 | 95 | * @return array Ordered list of locale items implementing \Aimeos\MShop\Locale\Item\Iface |
96 | 96 | * @since 2017.03 |
97 | 97 | */ |
98 | - public function searchItems( \Aimeos\MW\Criteria\Iface $filter, array $domains = array(), &$total = null ) |
|
98 | + public function searchItems(\Aimeos\MW\Criteria\Iface $filter, array $domains = array(), &$total = null) |
|
99 | 99 | { |
100 | - return $this->controller->searchItems( $filter, $domains, $total ); |
|
100 | + return $this->controller->searchItems($filter, $domains, $total); |
|
101 | 101 | } |
102 | 102 | |
103 | 103 |
@@ -15,31 +15,31 @@ |
||
15 | 15 | { |
16 | 16 | $target = '\\Aimeos\\Controller\\Frontend\\Locale\\Iface'; |
17 | 17 | |
18 | - $controller = \Aimeos\Controller\Frontend\Locale\Factory::createController( \TestHelperFrontend::getContext() ); |
|
19 | - $this->assertInstanceOf( $target, $controller ); |
|
18 | + $controller = \Aimeos\Controller\Frontend\Locale\Factory::createController(\TestHelperFrontend::getContext()); |
|
19 | + $this->assertInstanceOf($target, $controller); |
|
20 | 20 | |
21 | - $controller = \Aimeos\Controller\Frontend\Locale\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' ); |
|
22 | - $this->assertInstanceOf( $target, $controller ); |
|
21 | + $controller = \Aimeos\Controller\Frontend\Locale\Factory::createController(\TestHelperFrontend::getContext(), 'Standard'); |
|
22 | + $this->assertInstanceOf($target, $controller); |
|
23 | 23 | } |
24 | 24 | |
25 | 25 | |
26 | 26 | public function testCreateControllerInvalidImplementation() |
27 | 27 | { |
28 | - $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' ); |
|
29 | - \Aimeos\Controller\Frontend\Locale\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' ); |
|
28 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
29 | + \Aimeos\Controller\Frontend\Locale\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid'); |
|
30 | 30 | } |
31 | 31 | |
32 | 32 | |
33 | 33 | public function testCreateControllerInvalidName() |
34 | 34 | { |
35 | - $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' ); |
|
36 | - \Aimeos\Controller\Frontend\Locale\Factory::createController( \TestHelperFrontend::getContext(), '%^' ); |
|
35 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
36 | + \Aimeos\Controller\Frontend\Locale\Factory::createController(\TestHelperFrontend::getContext(), '%^'); |
|
37 | 37 | } |
38 | 38 | |
39 | 39 | |
40 | 40 | public function testCreateControllerNotExisting() |
41 | 41 | { |
42 | - $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' ); |
|
43 | - \Aimeos\Controller\Frontend\Locale\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' ); |
|
42 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
43 | + \Aimeos\Controller\Frontend\Locale\Factory::createController(\TestHelperFrontend::getContext(), 'notexist'); |
|
44 | 44 | } |
45 | 45 | } |
@@ -16,13 +16,13 @@ discard block |
||
16 | 16 | |
17 | 17 | protected function setUp() |
18 | 18 | { |
19 | - $this->object = new \Aimeos\Controller\Frontend\Locale\Standard( \TestHelperFrontend::getContext() ); |
|
19 | + $this->object = new \Aimeos\Controller\Frontend\Locale\Standard(\TestHelperFrontend::getContext()); |
|
20 | 20 | } |
21 | 21 | |
22 | 22 | |
23 | 23 | protected function tearDown() |
24 | 24 | { |
25 | - unset( $this->object ); |
|
25 | + unset($this->object); |
|
26 | 26 | } |
27 | 27 | |
28 | 28 | |
@@ -30,29 +30,29 @@ discard block |
||
30 | 30 | { |
31 | 31 | $filter = $this->object->createFilter(); |
32 | 32 | |
33 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
34 | - $this->assertEquals( 1, count( $filter->getSortations() ) ); |
|
35 | - $this->assertEquals( 0, $filter->getSliceStart() ); |
|
36 | - $this->assertEquals( 100, $filter->getSliceSize() ); |
|
33 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
34 | + $this->assertEquals(1, count($filter->getSortations())); |
|
35 | + $this->assertEquals(0, $filter->getSliceStart()); |
|
36 | + $this->assertEquals(100, $filter->getSliceSize()); |
|
37 | 37 | } |
38 | 38 | |
39 | 39 | |
40 | 40 | public function testGetItem() |
41 | 41 | { |
42 | - $localeManager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'locale' ); |
|
42 | + $localeManager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'locale'); |
|
43 | 43 | $search = $localeManager->createSearch(); |
44 | - $search->setSortations( [$search->sort( '+', 'locale.position' )] ); |
|
45 | - $search->setSlice( 0, 1 ); |
|
46 | - $localeItems = $localeManager->searchItems( $search ); |
|
44 | + $search->setSortations([$search->sort('+', 'locale.position')]); |
|
45 | + $search->setSlice(0, 1); |
|
46 | + $localeItems = $localeManager->searchItems($search); |
|
47 | 47 | |
48 | - if( ( $localeItem = reset( $localeItems ) ) === false ) { |
|
49 | - throw new \Exception( 'No locale item found' ); |
|
48 | + if (($localeItem = reset($localeItems)) === false) { |
|
49 | + throw new \Exception('No locale item found'); |
|
50 | 50 | } |
51 | 51 | |
52 | 52 | |
53 | - $result = $this->object->getItem( $localeItem->getId() ); |
|
53 | + $result = $this->object->getItem($localeItem->getId()); |
|
54 | 54 | |
55 | - $this->assertInstanceOf( '\Aimeos\MShop\Locale\Item\Iface', $result ); |
|
55 | + $this->assertInstanceOf('\Aimeos\MShop\Locale\Item\Iface', $result); |
|
56 | 56 | } |
57 | 57 | |
58 | 58 | |
@@ -60,9 +60,9 @@ discard block |
||
60 | 60 | { |
61 | 61 | $total = 0; |
62 | 62 | $filter = $this->object->createFilter(); |
63 | - $results = $this->object->searchItems( $filter, [], $total ); |
|
63 | + $results = $this->object->searchItems($filter, [], $total); |
|
64 | 64 | |
65 | - $this->assertEquals( 1, $total ); |
|
66 | - $this->assertEquals( 1, count( $results ) ); |
|
65 | + $this->assertEquals(1, $total); |
|
66 | + $this->assertEquals(1, count($results)); |
|
67 | 67 | } |
68 | 68 | } |
@@ -20,97 +20,97 @@ |
||
20 | 20 | { |
21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
22 | 22 | |
23 | - $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Locale\Standard' ) |
|
23 | + $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Locale\Standard') |
|
24 | 24 | ->disableOriginalConstructor() |
25 | 25 | ->getMock(); |
26 | 26 | |
27 | - $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Locale\Decorator\Base' ) |
|
28 | - ->setConstructorArgs( [$this->stub, $this->context] ) |
|
27 | + $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Locale\Decorator\Base') |
|
28 | + ->setConstructorArgs([$this->stub, $this->context]) |
|
29 | 29 | ->getMockForAbstractClass(); |
30 | 30 | } |
31 | 31 | |
32 | 32 | |
33 | 33 | protected function tearDown() |
34 | 34 | { |
35 | - unset( $this->context, $this->object, $this->stub ); |
|
35 | + unset($this->context, $this->object, $this->stub); |
|
36 | 36 | } |
37 | 37 | |
38 | 38 | |
39 | 39 | public function testConstructException() |
40 | 40 | { |
41 | - $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock(); |
|
41 | + $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock(); |
|
42 | 42 | |
43 | - $this->setExpectedException( '\Aimeos\Controller\Frontend\Exception' ); |
|
43 | + $this->setExpectedException('\Aimeos\Controller\Frontend\Exception'); |
|
44 | 44 | |
45 | - $this->getMockBuilder( '\Aimeos\Controller\Frontend\Locale\Decorator\Base' ) |
|
46 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
45 | + $this->getMockBuilder('\Aimeos\Controller\Frontend\Locale\Decorator\Base') |
|
46 | + ->setConstructorArgs([$stub, $this->context]) |
|
47 | 47 | ->getMockForAbstractClass(); |
48 | 48 | } |
49 | 49 | |
50 | 50 | |
51 | 51 | public function testCall() |
52 | 52 | { |
53 | - $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Locale\Standard' ) |
|
53 | + $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Locale\Standard') |
|
54 | 54 | ->disableOriginalConstructor() |
55 | - ->setMethods( ['invalid'] ) |
|
55 | + ->setMethods(['invalid']) |
|
56 | 56 | ->getMock(); |
57 | 57 | |
58 | - $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Locale\Decorator\Base' ) |
|
59 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
58 | + $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Locale\Decorator\Base') |
|
59 | + ->setConstructorArgs([$stub, $this->context]) |
|
60 | 60 | ->getMockForAbstractClass(); |
61 | 61 | |
62 | - $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) ); |
|
62 | + $stub->expects($this->once())->method('invalid')->will($this->returnValue(true)); |
|
63 | 63 | |
64 | - $this->assertTrue( $object->invalid() ); |
|
64 | + $this->assertTrue($object->invalid()); |
|
65 | 65 | } |
66 | 66 | |
67 | 67 | |
68 | 68 | public function testCreateFilter() |
69 | 69 | { |
70 | - $search = \Aimeos\MShop\Factory::createManager( $this->context, 'locale' )->createSearch(); |
|
70 | + $search = \Aimeos\MShop\Factory::createManager($this->context, 'locale')->createSearch(); |
|
71 | 71 | |
72 | - $this->stub->expects( $this->once() )->method( 'createFilter' ) |
|
73 | - ->will( $this->returnValue( $search ) ); |
|
72 | + $this->stub->expects($this->once())->method('createFilter') |
|
73 | + ->will($this->returnValue($search)); |
|
74 | 74 | |
75 | - $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() ); |
|
75 | + $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter()); |
|
76 | 76 | } |
77 | 77 | |
78 | 78 | |
79 | 79 | public function testGetItem() |
80 | 80 | { |
81 | - $item = \Aimeos\MShop\Factory::createManager( $this->context, 'locale' )->createItem(); |
|
81 | + $item = \Aimeos\MShop\Factory::createManager($this->context, 'locale')->createItem(); |
|
82 | 82 | |
83 | - $this->stub->expects( $this->once() )->method( 'getItem' ) |
|
84 | - ->will( $this->returnValue( $item ) ); |
|
83 | + $this->stub->expects($this->once())->method('getItem') |
|
84 | + ->will($this->returnValue($item)); |
|
85 | 85 | |
86 | - $this->assertInstanceOf( '\Aimeos\MShop\Locale\Item\Iface', $this->object->getItem( -1 ) ); |
|
86 | + $this->assertInstanceOf('\Aimeos\MShop\Locale\Item\Iface', $this->object->getItem( -1 )); |
|
87 | 87 | } |
88 | 88 | |
89 | 89 | |
90 | 90 | public function testSearchItems() |
91 | 91 | { |
92 | - $filter = \Aimeos\MShop\Factory::createManager( $this->context, 'locale' )->createSearch(); |
|
92 | + $filter = \Aimeos\MShop\Factory::createManager($this->context, 'locale')->createSearch(); |
|
93 | 93 | |
94 | - $this->stub->expects( $this->once() )->method( 'searchItems' ) |
|
95 | - ->will( $this->returnValue( [] ) ); |
|
94 | + $this->stub->expects($this->once())->method('searchItems') |
|
95 | + ->will($this->returnValue([])); |
|
96 | 96 | |
97 | - $this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) ); |
|
97 | + $this->assertEquals([], $this->object->searchItems($filter, ['media'])); |
|
98 | 98 | } |
99 | 99 | |
100 | 100 | |
101 | 101 | public function testGetController() |
102 | 102 | { |
103 | - $result = $this->access( 'getController' )->invokeArgs( $this->object, [] ); |
|
103 | + $result = $this->access('getController')->invokeArgs($this->object, []); |
|
104 | 104 | |
105 | - $this->assertSame( $this->stub, $result ); |
|
105 | + $this->assertSame($this->stub, $result); |
|
106 | 106 | } |
107 | 107 | |
108 | 108 | |
109 | - protected function access( $name ) |
|
109 | + protected function access($name) |
|
110 | 110 | { |
111 | - $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Locale\Decorator\Base' ); |
|
112 | - $method = $class->getMethod( $name ); |
|
113 | - $method->setAccessible( true ); |
|
111 | + $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Locale\Decorator\Base'); |
|
112 | + $method = $class->getMethod($name); |
|
113 | + $method->setAccessible(true); |
|
114 | 114 | |
115 | 115 | return $method; |
116 | 116 | } |
@@ -36,7 +36,7 @@ discard block |
||
36 | 36 | * @return \Aimeos\MShop\Locale\Item\Iface Locale item including the referenced domains items |
37 | 37 | * @since 2017.03 |
38 | 38 | */ |
39 | - public function getItem( $id, array $domains = array() ); |
|
39 | + public function getItem($id, array $domains = array()); |
|
40 | 40 | |
41 | 41 | |
42 | 42 | /** |
@@ -48,5 +48,5 @@ discard block |
||
48 | 48 | * @return array Ordered list of locale items implementing \Aimeos\MShop\Locale\Item\Iface |
49 | 49 | * @since 2017.03 |
50 | 50 | */ |
51 | - public function searchItems( \Aimeos\MW\Criteria\Iface $filter, array $domains = array(), &$total = null ); |
|
51 | + public function searchItems(\Aimeos\MW\Criteria\Iface $filter, array $domains = array(), &$total = null); |
|
52 | 52 | } |
@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | * @param string|null $name Name of the controller implementaton (default: "Standard") |
29 | 29 | * @return \Aimeos\Controller\Frontend\Locale\Iface Controller object |
30 | 30 | */ |
31 | - public static function createController( \Aimeos\MShop\Context\Item\Iface $context, $name = null ) |
|
31 | + public static function createController(\Aimeos\MShop\Context\Item\Iface $context, $name = null) |
|
32 | 32 | { |
33 | 33 | /** controller/frontend/locale/name |
34 | 34 | * Class name of the used locale frontend controller implementation |
@@ -63,20 +63,20 @@ discard block |
||
63 | 63 | * @since 2014.03 |
64 | 64 | * @category Developer |
65 | 65 | */ |
66 | - if( $name === null ) { |
|
67 | - $name = $context->getConfig()->get( 'controller/frontend/locale/name', 'Standard' ); |
|
66 | + if ($name === null) { |
|
67 | + $name = $context->getConfig()->get('controller/frontend/locale/name', 'Standard'); |
|
68 | 68 | } |
69 | 69 | |
70 | - if( ctype_alnum( $name ) === false ) |
|
70 | + if (ctype_alnum($name) === false) |
|
71 | 71 | { |
72 | - $classname = is_string( $name ) ? '\\Aimeos\\Controller\\Frontend\\Locale\\' . $name : '<not a string>'; |
|
73 | - throw new \Aimeos\Controller\Frontend\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) ); |
|
72 | + $classname = is_string($name) ? '\\Aimeos\\Controller\\Frontend\\Locale\\' . $name : '<not a string>'; |
|
73 | + throw new \Aimeos\Controller\Frontend\Exception(sprintf('Invalid characters in class name "%1$s"', $classname)); |
|
74 | 74 | } |
75 | 75 | |
76 | 76 | $iface = '\\Aimeos\\Controller\\Frontend\\Locale\\Iface'; |
77 | 77 | $classname = '\\Aimeos\\Controller\\Frontend\\Locale\\' . $name; |
78 | 78 | |
79 | - $manager = self::createControllerBase( $context, $classname, $iface ); |
|
79 | + $manager = self::createControllerBase($context, $classname, $iface); |
|
80 | 80 | |
81 | 81 | /** controller/frontend/locale/decorators/excludes |
82 | 82 | * Excludes decorators added by the "common" option from the locale frontend controllers |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | * @see controller/frontend/locale/decorators/excludes |
153 | 153 | * @see controller/frontend/locale/decorators/global |
154 | 154 | */ |
155 | - return self::addControllerDecorators( $context, $manager, 'locale' ); |
|
155 | + return self::addControllerDecorators($context, $manager, 'locale'); |
|
156 | 156 | } |
157 | 157 | |
158 | 158 | } |
@@ -31,15 +31,15 @@ discard block |
||
31 | 31 | public function createFilter() |
32 | 32 | { |
33 | 33 | $context = $this->getContext(); |
34 | - $filter = \Aimeos\MShop\Factory::createManager( $context, 'locale' )->createSearch( true ); |
|
34 | + $filter = \Aimeos\MShop\Factory::createManager($context, 'locale')->createSearch(true); |
|
35 | 35 | |
36 | 36 | $expr = array( |
37 | - $filter->compare( '==', 'locale.siteid', $context->getLocale()->getSitePath() ), |
|
37 | + $filter->compare('==', 'locale.siteid', $context->getLocale()->getSitePath()), |
|
38 | 38 | $filter->getConditions(), |
39 | 39 | ); |
40 | 40 | |
41 | - $filter->setConditions( $filter->combine( '&&', $expr ) ); |
|
42 | - $filter->setSortations( array( $filter->sort( '+', 'locale.position' ) ) ); |
|
41 | + $filter->setConditions($filter->combine('&&', $expr)); |
|
42 | + $filter->setSortations(array($filter->sort('+', 'locale.position'))); |
|
43 | 43 | |
44 | 44 | return $filter; |
45 | 45 | } |
@@ -53,9 +53,9 @@ discard block |
||
53 | 53 | * @return \Aimeos\MShop\Locale\Item\Iface Locale item including the referenced domains items |
54 | 54 | * @since 2017.03 |
55 | 55 | */ |
56 | - public function getItem( $id, array $domains = array() ) |
|
56 | + public function getItem($id, array $domains = array()) |
|
57 | 57 | { |
58 | - return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'locale' )->getItem( $id ); |
|
58 | + return \Aimeos\MShop\Factory::createManager($this->getContext(), 'locale')->getItem($id); |
|
59 | 59 | } |
60 | 60 | |
61 | 61 | |
@@ -68,8 +68,8 @@ discard block |
||
68 | 68 | * @return array Ordered list of locale items implementing \Aimeos\MShop\Locale\Item\Iface |
69 | 69 | * @since 2017.03 |
70 | 70 | */ |
71 | - public function searchItems( \Aimeos\MW\Criteria\Iface $filter, array $domains = array(), &$total = null ) |
|
71 | + public function searchItems(\Aimeos\MW\Criteria\Iface $filter, array $domains = array(), &$total = null) |
|
72 | 72 | { |
73 | - return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'locale' )->searchItems( $filter, [], $total ); |
|
73 | + return \Aimeos\MShop\Factory::createManager($this->getContext(), 'locale')->searchItems($filter, [], $total); |
|
74 | 74 | } |
75 | 75 | } |
@@ -33,11 +33,11 @@ discard block |
||
33 | 33 | * @param \Aimeos\MShop\Context\Item\Iface $context Object storing the required instances for manaing databases |
34 | 34 | * connections, logger, session, etc. |
35 | 35 | */ |
36 | - public function __construct( \Aimeos\MShop\Context\Item\Iface $context ) |
|
36 | + public function __construct(\Aimeos\MShop\Context\Item\Iface $context) |
|
37 | 37 | { |
38 | - parent::__construct( $context ); |
|
38 | + parent::__construct($context); |
|
39 | 39 | |
40 | - $this->domainManager = \Aimeos\MShop\Factory::createManager( $context, 'order/base' ); |
|
40 | + $this->domainManager = \Aimeos\MShop\Factory::createManager($context, 'order/base'); |
|
41 | 41 | } |
42 | 42 | |
43 | 43 | |
@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | public function clear() |
50 | 50 | { |
51 | 51 | $this->baskets[$this->type] = $this->domainManager->createItem(); |
52 | - $this->domainManager->setSession( $this->baskets[$this->type], $this->type ); |
|
52 | + $this->domainManager->setSession($this->baskets[$this->type], $this->type); |
|
53 | 53 | |
54 | 54 | return $this; |
55 | 55 | } |
@@ -62,10 +62,10 @@ discard block |
||
62 | 62 | */ |
63 | 63 | public function get() |
64 | 64 | { |
65 | - if( !isset( $this->baskets[$this->type] ) ) |
|
65 | + if (!isset($this->baskets[$this->type])) |
|
66 | 66 | { |
67 | - $this->baskets[$this->type] = $this->domainManager->getSession( $this->type ); |
|
68 | - $this->checkLocale( $this->type ); |
|
67 | + $this->baskets[$this->type] = $this->domainManager->getSession($this->type); |
|
68 | + $this->checkLocale($this->type); |
|
69 | 69 | } |
70 | 70 | |
71 | 71 | return $this->baskets[$this->type]; |
@@ -79,8 +79,8 @@ discard block |
||
79 | 79 | */ |
80 | 80 | public function save() |
81 | 81 | { |
82 | - if( isset( $this->baskets[$this->type] ) && $this->baskets[$this->type]->isModified() ) { |
|
83 | - $this->domainManager->setSession( $this->baskets[$this->type], $this->type ); |
|
82 | + if (isset($this->baskets[$this->type]) && $this->baskets[$this->type]->isModified()) { |
|
83 | + $this->domainManager->setSession($this->baskets[$this->type], $this->type); |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | return $this; |
@@ -93,7 +93,7 @@ discard block |
||
93 | 93 | * @param string $type Basket type |
94 | 94 | * @return \Aimeos\Controller\Frontend\Basket\Iface Basket frontend object |
95 | 95 | */ |
96 | - public function setType( $type ) |
|
96 | + public function setType($type) |
|
97 | 97 | { |
98 | 98 | $this->type = $type; |
99 | 99 | return $this; |
@@ -120,36 +120,36 @@ discard block |
||
120 | 120 | * @param string $stocktype Unique code of the stock type to deliver the products from |
121 | 121 | * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available |
122 | 122 | */ |
123 | - public function addProduct( $prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(), |
|
123 | + public function addProduct($prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(), |
|
124 | 124 | array $configAttributeIds = array(), array $hiddenAttributeIds = array(), array $customAttributeValues = array(), |
125 | - $stocktype = 'default' ) |
|
125 | + $stocktype = 'default') |
|
126 | 126 | { |
127 | 127 | $context = $this->getContext(); |
128 | - $productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' ); |
|
129 | - $productItem = $productManager->getItem( $prodid, array( 'media', 'supplier', 'price', 'product', 'text' ) ); |
|
128 | + $productManager = \Aimeos\MShop\Factory::createManager($context, 'product'); |
|
129 | + $productItem = $productManager->getItem($prodid, array('media', 'supplier', 'price', 'product', 'text')); |
|
130 | 130 | |
131 | - $orderBaseProductItem = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product' )->createItem(); |
|
132 | - $orderBaseProductItem->copyFrom( $productItem ); |
|
133 | - $orderBaseProductItem->setQuantity( $quantity ); |
|
134 | - $orderBaseProductItem->setStockType( $stocktype ); |
|
131 | + $orderBaseProductItem = \Aimeos\MShop\Factory::createManager($context, 'order/base/product')->createItem(); |
|
132 | + $orderBaseProductItem->copyFrom($productItem); |
|
133 | + $orderBaseProductItem->setQuantity($quantity); |
|
134 | + $orderBaseProductItem->setStockType($stocktype); |
|
135 | 135 | |
136 | 136 | $attr = array(); |
137 | - $prices = $productItem->getRefItems( 'price', 'default', 'default' ); |
|
137 | + $prices = $productItem->getRefItems('price', 'default', 'default'); |
|
138 | 138 | |
139 | - $priceManager = \Aimeos\MShop\Factory::createManager( $context, 'price' ); |
|
140 | - $price = $priceManager->getLowestPrice( $prices, $quantity ); |
|
139 | + $priceManager = \Aimeos\MShop\Factory::createManager($context, 'price'); |
|
140 | + $price = $priceManager->getLowestPrice($prices, $quantity); |
|
141 | 141 | |
142 | - $attr = array_merge( $attr, $this->createOrderProductAttributes( $price, $prodid, $quantity, $configAttributeIds, 'config' ) ); |
|
143 | - $attr = array_merge( $attr, $this->createOrderProductAttributes( $price, $prodid, $quantity, $hiddenAttributeIds, 'hidden' ) ); |
|
144 | - $attr = array_merge( $attr, $this->createOrderProductAttributes( $price, $prodid, $quantity, array_keys( $customAttributeValues ), 'custom', $customAttributeValues ) ); |
|
142 | + $attr = array_merge($attr, $this->createOrderProductAttributes($price, $prodid, $quantity, $configAttributeIds, 'config')); |
|
143 | + $attr = array_merge($attr, $this->createOrderProductAttributes($price, $prodid, $quantity, $hiddenAttributeIds, 'hidden')); |
|
144 | + $attr = array_merge($attr, $this->createOrderProductAttributes($price, $prodid, $quantity, array_keys($customAttributeValues), 'custom', $customAttributeValues)); |
|
145 | 145 | |
146 | 146 | // remove product rebate of original price in favor to rebates granted for the order |
147 | - $price->setRebate( '0.00' ); |
|
147 | + $price->setRebate('0.00'); |
|
148 | 148 | |
149 | - $orderBaseProductItem->setPrice( $price ); |
|
150 | - $orderBaseProductItem->setAttributes( $attr ); |
|
149 | + $orderBaseProductItem->setPrice($price); |
|
150 | + $orderBaseProductItem->setAttributes($attr); |
|
151 | 151 | |
152 | - $this->get()->addProduct( $orderBaseProductItem ); |
|
152 | + $this->get()->addProduct($orderBaseProductItem); |
|
153 | 153 | $this->save(); |
154 | 154 | } |
155 | 155 | |
@@ -159,17 +159,17 @@ discard block |
||
159 | 159 | * |
160 | 160 | * @param integer $position Position number (key) of the order product item |
161 | 161 | */ |
162 | - public function deleteProduct( $position ) |
|
162 | + public function deleteProduct($position) |
|
163 | 163 | { |
164 | - $product = $this->get()->getProduct( $position ); |
|
164 | + $product = $this->get()->getProduct($position); |
|
165 | 165 | |
166 | - if( $product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE ) |
|
166 | + if ($product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE) |
|
167 | 167 | { |
168 | - $msg = sprintf( 'Basket item at position "%1$d" cannot be deleted manually', $position ); |
|
169 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
168 | + $msg = sprintf('Basket item at position "%1$d" cannot be deleted manually', $position); |
|
169 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
170 | 170 | } |
171 | 171 | |
172 | - $this->get()->deleteProduct( $position ); |
|
172 | + $this->get()->deleteProduct($position); |
|
173 | 173 | $this->save(); |
174 | 174 | } |
175 | 175 | |
@@ -183,34 +183,34 @@ discard block |
||
183 | 183 | * The 'stock'=>false option allows adding products without being in stock. |
184 | 184 | * @param string[] $configAttributeCodes Codes of the product config attributes that should be REMOVED |
185 | 185 | */ |
186 | - public function editProduct( $position, $quantity, array $options = array(), |
|
187 | - array $configAttributeCodes = array() ) |
|
186 | + public function editProduct($position, $quantity, array $options = array(), |
|
187 | + array $configAttributeCodes = array()) |
|
188 | 188 | { |
189 | - $product = $this->get()->getProduct( $position ); |
|
189 | + $product = $this->get()->getProduct($position); |
|
190 | 190 | |
191 | - if( $product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE ) |
|
191 | + if ($product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE) |
|
192 | 192 | { |
193 | - $msg = sprintf( 'Basket item at position "%1$d" cannot be changed', $position ); |
|
194 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
193 | + $msg = sprintf('Basket item at position "%1$d" cannot be changed', $position); |
|
194 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
195 | 195 | } |
196 | 196 | |
197 | - $product->setQuantity( $quantity ); |
|
197 | + $product->setQuantity($quantity); |
|
198 | 198 | |
199 | 199 | $attributes = $product->getAttributes(); |
200 | - foreach( $attributes as $key => $attribute ) |
|
200 | + foreach ($attributes as $key => $attribute) |
|
201 | 201 | { |
202 | - if( in_array( $attribute->getCode(), $configAttributeCodes ) ) { |
|
203 | - unset( $attributes[$key] ); |
|
202 | + if (in_array($attribute->getCode(), $configAttributeCodes)) { |
|
203 | + unset($attributes[$key]); |
|
204 | 204 | } |
205 | 205 | } |
206 | - $product->setAttributes( $attributes ); |
|
206 | + $product->setAttributes($attributes); |
|
207 | 207 | |
208 | - $productItem = $this->getDomainItem( 'product', 'product.code', $product->getProductCode(), array( 'price', 'text' ) ); |
|
209 | - $prices = $productItem->getRefItems( 'price', 'default' ); |
|
210 | - $product->setPrice( $this->calcPrice( $product, $prices, $quantity ) ); |
|
208 | + $productItem = $this->getDomainItem('product', 'product.code', $product->getProductCode(), array('price', 'text')); |
|
209 | + $prices = $productItem->getRefItems('price', 'default'); |
|
210 | + $product->setPrice($this->calcPrice($product, $prices, $quantity)); |
|
211 | 211 | |
212 | - $this->get()->deleteProduct( $position ); |
|
213 | - $this->get()->addProduct( $product, $position ); |
|
212 | + $this->get()->deleteProduct($position); |
|
213 | + $this->get()->addProduct($product, $position); |
|
214 | 214 | |
215 | 215 | $this->save(); |
216 | 216 | } |
@@ -222,49 +222,49 @@ discard block |
||
222 | 222 | * @param string $code Coupon code entered by the user |
223 | 223 | * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid or not allowed |
224 | 224 | */ |
225 | - public function addCoupon( $code ) |
|
225 | + public function addCoupon($code) |
|
226 | 226 | { |
227 | - $manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon' ); |
|
228 | - $codeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon/code' ); |
|
227 | + $manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon'); |
|
228 | + $codeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon/code'); |
|
229 | 229 | |
230 | 230 | |
231 | - $search = $codeManager->createSearch( true ); |
|
231 | + $search = $codeManager->createSearch(true); |
|
232 | 232 | $expr = array( |
233 | - $search->compare( '==', 'coupon.code.code', $code ), |
|
233 | + $search->compare('==', 'coupon.code.code', $code), |
|
234 | 234 | $search->getConditions(), |
235 | 235 | ); |
236 | - $search->setConditions( $search->combine( '&&', $expr ) ); |
|
237 | - $search->setSlice( 0, 1 ); |
|
236 | + $search->setConditions($search->combine('&&', $expr)); |
|
237 | + $search->setSlice(0, 1); |
|
238 | 238 | |
239 | - $result = $codeManager->searchItems( $search ); |
|
239 | + $result = $codeManager->searchItems($search); |
|
240 | 240 | |
241 | - if( ( $codeItem = reset( $result ) ) === false ) { |
|
242 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon code "%1$s" is invalid or not available any more', $code ) ); |
|
241 | + if (($codeItem = reset($result)) === false) { |
|
242 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon code "%1$s" is invalid or not available any more', $code)); |
|
243 | 243 | } |
244 | 244 | |
245 | 245 | |
246 | - $search = $manager->createSearch( true ); |
|
246 | + $search = $manager->createSearch(true); |
|
247 | 247 | $expr = array( |
248 | - $search->compare( '==', 'coupon.id', $codeItem->getParentId() ), |
|
248 | + $search->compare('==', 'coupon.id', $codeItem->getParentId()), |
|
249 | 249 | $search->getConditions(), |
250 | 250 | ); |
251 | - $search->setConditions( $search->combine( '&&', $expr ) ); |
|
252 | - $search->setSlice( 0, 1 ); |
|
251 | + $search->setConditions($search->combine('&&', $expr)); |
|
252 | + $search->setSlice(0, 1); |
|
253 | 253 | |
254 | - $result = $manager->searchItems( $search ); |
|
254 | + $result = $manager->searchItems($search); |
|
255 | 255 | |
256 | - if( ( $item = reset( $result ) ) === false ) { |
|
257 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon for code "%1$s" is not available any more', $code ) ); |
|
256 | + if (($item = reset($result)) === false) { |
|
257 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon for code "%1$s" is not available any more', $code)); |
|
258 | 258 | } |
259 | 259 | |
260 | 260 | |
261 | - $provider = $manager->getProvider( $item, $code ); |
|
261 | + $provider = $manager->getProvider($item, $code); |
|
262 | 262 | |
263 | - if( $provider->isAvailable( $this->get() ) !== true ) { |
|
264 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Requirements for coupon code "%1$s" aren\'t met', $code ) ); |
|
263 | + if ($provider->isAvailable($this->get()) !== true) { |
|
264 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Requirements for coupon code "%1$s" aren\'t met', $code)); |
|
265 | 265 | } |
266 | 266 | |
267 | - $provider->addCoupon( $this->get() ); |
|
267 | + $provider->addCoupon($this->get()); |
|
268 | 268 | $this->save(); |
269 | 269 | } |
270 | 270 | |
@@ -275,21 +275,21 @@ discard block |
||
275 | 275 | * @param string $code Coupon code entered by the user |
276 | 276 | * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid |
277 | 277 | */ |
278 | - public function deleteCoupon( $code ) |
|
278 | + public function deleteCoupon($code) |
|
279 | 279 | { |
280 | - $manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon' ); |
|
280 | + $manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon'); |
|
281 | 281 | |
282 | 282 | $search = $manager->createSearch(); |
283 | - $search->setConditions( $search->compare( '==', 'coupon.code.code', $code ) ); |
|
284 | - $search->setSlice( 0, 1 ); |
|
283 | + $search->setConditions($search->compare('==', 'coupon.code.code', $code)); |
|
284 | + $search->setSlice(0, 1); |
|
285 | 285 | |
286 | - $result = $manager->searchItems( $search ); |
|
286 | + $result = $manager->searchItems($search); |
|
287 | 287 | |
288 | - if( ( $item = reset( $result ) ) === false ) { |
|
289 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon code "%1$s" is invalid', $code ) ); |
|
288 | + if (($item = reset($result)) === false) { |
|
289 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon code "%1$s" is invalid', $code)); |
|
290 | 290 | } |
291 | 291 | |
292 | - $manager->getProvider( $item, $code )->deleteCoupon( $this->get() ); |
|
292 | + $manager->getProvider($item, $code)->deleteCoupon($this->get()); |
|
293 | 293 | $this->save(); |
294 | 294 | } |
295 | 295 | |
@@ -302,28 +302,28 @@ discard block |
||
302 | 302 | * @throws \Aimeos\Controller\Frontend\Basket\Exception If the billing or delivery address is not of any required type of |
303 | 303 | * if one of the keys is invalid when using an array with key/value pairs |
304 | 304 | */ |
305 | - public function setAddress( $type, $value ) |
|
305 | + public function setAddress($type, $value) |
|
306 | 306 | { |
307 | - $address = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/address' )->createItem(); |
|
308 | - $address->setType( $type ); |
|
307 | + $address = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/address')->createItem(); |
|
308 | + $address->setType($type); |
|
309 | 309 | |
310 | - if( $value instanceof \Aimeos\MShop\Common\Item\Address\Iface ) |
|
310 | + if ($value instanceof \Aimeos\MShop\Common\Item\Address\Iface) |
|
311 | 311 | { |
312 | - $address->copyFrom( $value ); |
|
313 | - $this->get()->setAddress( $address, $type ); |
|
312 | + $address->copyFrom($value); |
|
313 | + $this->get()->setAddress($address, $type); |
|
314 | 314 | } |
315 | - else if( is_array( $value ) ) |
|
315 | + else if (is_array($value)) |
|
316 | 316 | { |
317 | - $this->setAddressFromArray( $address, $value ); |
|
318 | - $this->get()->setAddress( $address, $type ); |
|
317 | + $this->setAddressFromArray($address, $value); |
|
318 | + $this->get()->setAddress($address, $type); |
|
319 | 319 | } |
320 | - else if( $value === null ) |
|
320 | + else if ($value === null) |
|
321 | 321 | { |
322 | - $this->get()->deleteAddress( $type ); |
|
322 | + $this->get()->deleteAddress($type); |
|
323 | 323 | } |
324 | 324 | else |
325 | 325 | { |
326 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Invalid value for address type "%1$s"', $type ) ); |
|
326 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Invalid value for address type "%1$s"', $type)); |
|
327 | 327 | } |
328 | 328 | |
329 | 329 | $this->save(); |
@@ -339,42 +339,42 @@ discard block |
||
339 | 339 | * entered by the customer when choosing one of the delivery or payment options |
340 | 340 | * @throws \Aimeos\Controller\Frontend\Basket\Exception If there is no price to the service item attached |
341 | 341 | */ |
342 | - public function setService( $type, $id, array $attributes = array() ) |
|
342 | + public function setService($type, $id, array $attributes = array()) |
|
343 | 343 | { |
344 | 344 | $context = $this->getContext(); |
345 | 345 | |
346 | - $serviceManager = \Aimeos\MShop\Factory::createManager( $context, 'service' ); |
|
347 | - $serviceItem = $this->getDomainItem( 'service', 'service.id', $id, array( 'media', 'price', 'text' ) ); |
|
346 | + $serviceManager = \Aimeos\MShop\Factory::createManager($context, 'service'); |
|
347 | + $serviceItem = $this->getDomainItem('service', 'service.id', $id, array('media', 'price', 'text')); |
|
348 | 348 | |
349 | - $provider = $serviceManager->getProvider( $serviceItem ); |
|
350 | - $result = $provider->checkConfigFE( $attributes ); |
|
351 | - $unknown = array_diff_key( $attributes, $result ); |
|
349 | + $provider = $serviceManager->getProvider($serviceItem); |
|
350 | + $result = $provider->checkConfigFE($attributes); |
|
351 | + $unknown = array_diff_key($attributes, $result); |
|
352 | 352 | |
353 | - if( count( $unknown ) > 0 ) |
|
353 | + if (count($unknown) > 0) |
|
354 | 354 | { |
355 | - $msg = sprintf( 'Unknown attributes "%1$s"', implode( '","', array_keys( $unknown ) ) ); |
|
356 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
355 | + $msg = sprintf('Unknown attributes "%1$s"', implode('","', array_keys($unknown))); |
|
356 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
357 | 357 | } |
358 | 358 | |
359 | - foreach( $result as $key => $value ) |
|
359 | + foreach ($result as $key => $value) |
|
360 | 360 | { |
361 | - if( $value !== null ) { |
|
362 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $value ); |
|
361 | + if ($value !== null) { |
|
362 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($value); |
|
363 | 363 | } |
364 | 364 | } |
365 | 365 | |
366 | - $orderBaseServiceManager = \Aimeos\MShop\Factory::createManager( $context, 'order/base/service' ); |
|
366 | + $orderBaseServiceManager = \Aimeos\MShop\Factory::createManager($context, 'order/base/service'); |
|
367 | 367 | $orderServiceItem = $orderBaseServiceManager->createItem(); |
368 | - $orderServiceItem->copyFrom( $serviceItem ); |
|
368 | + $orderServiceItem->copyFrom($serviceItem); |
|
369 | 369 | |
370 | - $price = $provider->calcPrice( $this->get() ); |
|
370 | + $price = $provider->calcPrice($this->get()); |
|
371 | 371 | // remove service rebate of original price |
372 | - $price->setRebate( '0.00' ); |
|
373 | - $orderServiceItem->setPrice( $price ); |
|
372 | + $price->setRebate('0.00'); |
|
373 | + $orderServiceItem->setPrice($price); |
|
374 | 374 | |
375 | - $provider->setConfigFE( $orderServiceItem, $attributes ); |
|
375 | + $provider->setConfigFE($orderServiceItem, $attributes); |
|
376 | 376 | |
377 | - $this->get()->setService( $orderServiceItem, $type ); |
|
377 | + $this->get()->setService($orderServiceItem, $type); |
|
378 | 378 | $this->save(); |
379 | 379 | } |
380 | 380 | |
@@ -387,18 +387,18 @@ discard block |
||
387 | 387 | * an address item. |
388 | 388 | * @throws \Aimeos\Controller\Frontend\Basket\Exception |
389 | 389 | */ |
390 | - protected function setAddressFromArray( \Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map ) |
|
390 | + protected function setAddressFromArray(\Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map) |
|
391 | 391 | { |
392 | - foreach( $map as $key => $value ) { |
|
393 | - $map[$key] = strip_tags( $value ); // prevent XSS |
|
392 | + foreach ($map as $key => $value) { |
|
393 | + $map[$key] = strip_tags($value); // prevent XSS |
|
394 | 394 | } |
395 | 395 | |
396 | - $errors = $address->fromArray( $map ); |
|
396 | + $errors = $address->fromArray($map); |
|
397 | 397 | |
398 | - if( count( $errors ) > 0 ) |
|
398 | + if (count($errors) > 0) |
|
399 | 399 | { |
400 | - $msg = sprintf( 'Invalid address properties, please check your input' ); |
|
401 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg, 0, null, $errors ); |
|
400 | + $msg = sprintf('Invalid address properties, please check your input'); |
|
401 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg, 0, null, $errors); |
|
402 | 402 | } |
403 | 403 | } |
404 | 404 | } |
@@ -311,17 +311,14 @@ |
||
311 | 311 | { |
312 | 312 | $address->copyFrom( $value ); |
313 | 313 | $this->get()->setAddress( $address, $type ); |
314 | - } |
|
315 | - else if( is_array( $value ) ) |
|
314 | + } else if( is_array( $value ) ) |
|
316 | 315 | { |
317 | 316 | $this->setAddressFromArray( $address, $value ); |
318 | 317 | $this->get()->setAddress( $address, $type ); |
319 | - } |
|
320 | - else if( $value === null ) |
|
318 | + } else if( $value === null ) |
|
321 | 319 | { |
322 | 320 | $this->get()->deleteAddress( $type ); |
323 | - } |
|
324 | - else |
|
321 | + } else |
|
325 | 322 | { |
326 | 323 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Invalid value for address type "%1$s"', $type ) ); |
327 | 324 | } |