@@ -17,237 +17,237 @@ |
||
17 | 17 | protected function setUp() |
18 | 18 | { |
19 | 19 | $this->context = \TestHelperFrontend::getContext(); |
20 | - $this->object = new \Aimeos\Controller\Frontend\Product\Standard( $this->context ); |
|
20 | + $this->object = new \Aimeos\Controller\Frontend\Product\Standard($this->context); |
|
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 testAggregate() |
31 | 31 | { |
32 | - $list = $this->object->aggregate( 'index.attribute.id' ); |
|
32 | + $list = $this->object->aggregate('index.attribute.id'); |
|
33 | 33 | |
34 | - $this->assertGreaterThan( 0, count( $list ) ); |
|
34 | + $this->assertGreaterThan(0, count($list)); |
|
35 | 35 | } |
36 | 36 | |
37 | 37 | |
38 | 38 | public function testAllOf() |
39 | 39 | { |
40 | - $manager = \Aimeos\MShop::create( $this->context, 'attribute' ); |
|
40 | + $manager = \Aimeos\MShop::create($this->context, 'attribute'); |
|
41 | 41 | |
42 | - $length = $manager->findItem( '30', [], 'product', 'length' )->getId(); |
|
43 | - $width = $manager->findItem( '29', [], 'product', 'width' )->getId(); |
|
42 | + $length = $manager->findItem('30', [], 'product', 'length')->getId(); |
|
43 | + $width = $manager->findItem('29', [], 'product', 'width')->getId(); |
|
44 | 44 | |
45 | - $this->assertEquals( 2, count( $this->object->allOf( [$length, $width] )->search() ) ); |
|
45 | + $this->assertEquals(2, count($this->object->allOf([$length, $width])->search())); |
|
46 | 46 | } |
47 | 47 | |
48 | 48 | |
49 | 49 | public function testCategory() |
50 | 50 | { |
51 | - $manager = \Aimeos\MShop::create( $this->context, 'catalog' ); |
|
52 | - $catId = $manager->findItem( 'cafe' )->getId(); |
|
51 | + $manager = \Aimeos\MShop::create($this->context, 'catalog'); |
|
52 | + $catId = $manager->findItem('cafe')->getId(); |
|
53 | 53 | |
54 | - $this->assertEquals( 2, count( $this->object->category( $catId, 'promotion' )->search() ) ); |
|
54 | + $this->assertEquals(2, count($this->object->category($catId, 'promotion')->search())); |
|
55 | 55 | } |
56 | 56 | |
57 | 57 | |
58 | 58 | public function testCategoryTree() |
59 | 59 | { |
60 | - $manager = \Aimeos\MShop::create( $this->context, 'catalog' ); |
|
60 | + $manager = \Aimeos\MShop::create($this->context, 'catalog'); |
|
61 | 61 | |
62 | - $catId = $manager->findItem( 'categories' )->getId(); |
|
63 | - $grpId = $manager->findItem( 'group' )->getId(); |
|
62 | + $catId = $manager->findItem('categories')->getId(); |
|
63 | + $grpId = $manager->findItem('group')->getId(); |
|
64 | 64 | |
65 | - $this->object->category( [$catId, $grpId], 'promotion', \Aimeos\MW\Tree\Manager\Base::LEVEL_TREE ); |
|
66 | - $this->assertEquals( 3, count( $this->object->search() ) ); |
|
65 | + $this->object->category([$catId, $grpId], 'promotion', \Aimeos\MW\Tree\Manager\Base::LEVEL_TREE); |
|
66 | + $this->assertEquals(3, count($this->object->search())); |
|
67 | 67 | } |
68 | 68 | |
69 | 69 | |
70 | 70 | public function testCompare() |
71 | 71 | { |
72 | - $this->assertEquals( 1, count( $this->object->compare( '==', 'product.type', 'bundle' )->search() ) ); |
|
72 | + $this->assertEquals(1, count($this->object->compare('==', 'product.type', 'bundle')->search())); |
|
73 | 73 | } |
74 | 74 | |
75 | 75 | |
76 | 76 | public function testGet() |
77 | 77 | { |
78 | - $item = \Aimeos\MShop::create( $this->context, 'product' )->findItem( 'CNC' ); |
|
78 | + $item = \Aimeos\MShop::create($this->context, 'product')->findItem('CNC'); |
|
79 | 79 | |
80 | - $this->assertInstanceOf( \Aimeos\MShop\Product\Item\Iface::class, $this->object->get( $item->getId() ) ); |
|
80 | + $this->assertInstanceOf(\Aimeos\MShop\Product\Item\Iface::class, $this->object->get($item->getId())); |
|
81 | 81 | } |
82 | 82 | |
83 | 83 | |
84 | 84 | public function testFind() |
85 | 85 | { |
86 | - $this->assertInstanceOf( \Aimeos\MShop\Product\Item\Iface::class, $this->object->find( 'CNC' ) ); |
|
86 | + $this->assertInstanceOf(\Aimeos\MShop\Product\Item\Iface::class, $this->object->find('CNC')); |
|
87 | 87 | } |
88 | 88 | |
89 | 89 | |
90 | 90 | public function testOneOf() |
91 | 91 | { |
92 | - $manager = \Aimeos\MShop::create( $this->context, 'attribute' ); |
|
92 | + $manager = \Aimeos\MShop::create($this->context, 'attribute'); |
|
93 | 93 | |
94 | - $length = $manager->findItem( '30', [], 'product', 'length' )->getId(); |
|
95 | - $width = $manager->findItem( '29', [], 'product', 'width' )->getId(); |
|
94 | + $length = $manager->findItem('30', [], 'product', 'length')->getId(); |
|
95 | + $width = $manager->findItem('29', [], 'product', 'width')->getId(); |
|
96 | 96 | |
97 | - $this->assertEquals( 4, count( $this->object->oneOf( [$length, $width] )->search() ) ); |
|
97 | + $this->assertEquals(4, count($this->object->oneOf([$length, $width])->search())); |
|
98 | 98 | } |
99 | 99 | |
100 | 100 | |
101 | 101 | public function testOneOfList() |
102 | 102 | { |
103 | - $manager = \Aimeos\MShop::create( $this->context, 'attribute' ); |
|
103 | + $manager = \Aimeos\MShop::create($this->context, 'attribute'); |
|
104 | 104 | |
105 | - $length = $manager->findItem( '30', [], 'product', 'length' )->getId(); |
|
106 | - $width = $manager->findItem( '29', [], 'product', 'width' )->getId(); |
|
105 | + $length = $manager->findItem('30', [], 'product', 'length')->getId(); |
|
106 | + $width = $manager->findItem('29', [], 'product', 'width')->getId(); |
|
107 | 107 | |
108 | - $this->assertEquals( 4, count( $this->object->oneOf( [[$length, $width]] )->search() ) ); |
|
108 | + $this->assertEquals(4, count($this->object->oneOf([[$length, $width]])->search())); |
|
109 | 109 | } |
110 | 110 | |
111 | 111 | |
112 | 112 | public function testParse() |
113 | 113 | { |
114 | 114 | $cond = ['&&' => [['>' => ['product.status' => 0]], ['==' => ['product.type' => 'default']]]]; |
115 | - $this->assertEquals( 4, count( $this->object->parse( $cond )->search() ) ); |
|
115 | + $this->assertEquals(4, count($this->object->parse($cond)->search())); |
|
116 | 116 | } |
117 | 117 | |
118 | 118 | |
119 | 119 | public function testProduct() |
120 | 120 | { |
121 | - $manager = \Aimeos\MShop::create( $this->context, 'product' ); |
|
121 | + $manager = \Aimeos\MShop::create($this->context, 'product'); |
|
122 | 122 | |
123 | - $cncId = $manager->findItem( 'CNC' )->getId(); |
|
124 | - $cneId = $manager->findItem( 'CNE' )->getId(); |
|
123 | + $cncId = $manager->findItem('CNC')->getId(); |
|
124 | + $cneId = $manager->findItem('CNE')->getId(); |
|
125 | 125 | |
126 | - $this->assertEquals( 2, count( $this->object->product( [$cncId, $cneId] )->search() ) ); |
|
126 | + $this->assertEquals(2, count($this->object->product([$cncId, $cneId])->search())); |
|
127 | 127 | } |
128 | 128 | |
129 | 129 | |
130 | 130 | public function testSearch() |
131 | 131 | { |
132 | 132 | $total = 0; |
133 | - $this->assertEquals( 8, count( $this->object->search( [], $total ) ) ); |
|
134 | - $this->assertEquals( 8, $total ); |
|
133 | + $this->assertEquals(8, count($this->object->search([], $total))); |
|
134 | + $this->assertEquals(8, $total); |
|
135 | 135 | } |
136 | 136 | |
137 | 137 | |
138 | 138 | public function testSlice() |
139 | 139 | { |
140 | - $this->assertEquals( 2, count( $this->object->slice( 0, 2 )->search() ) ); |
|
140 | + $this->assertEquals(2, count($this->object->slice(0, 2)->search())); |
|
141 | 141 | } |
142 | 142 | |
143 | 143 | |
144 | 144 | public function testSort() |
145 | 145 | { |
146 | - $this->assertEquals( 8, count( $this->object->sort( 'relevance' )->search() ) ); |
|
146 | + $this->assertEquals(8, count($this->object->sort('relevance')->search())); |
|
147 | 147 | } |
148 | 148 | |
149 | 149 | |
150 | 150 | public function testSortGeneric() |
151 | 151 | { |
152 | - $this->assertEquals( 8, count( $this->object->sort( 'product.status' )->search() ) ); |
|
152 | + $this->assertEquals(8, count($this->object->sort('product.status')->search())); |
|
153 | 153 | } |
154 | 154 | |
155 | 155 | |
156 | 156 | public function testSortCode() |
157 | 157 | { |
158 | - $result = $this->object->sort( 'code' )->search( [] ); |
|
159 | - $this->assertEquals( 'CNC', reset( $result )->getCode() ); |
|
158 | + $result = $this->object->sort('code')->search([]); |
|
159 | + $this->assertEquals('CNC', reset($result)->getCode()); |
|
160 | 160 | } |
161 | 161 | |
162 | 162 | |
163 | 163 | public function testSortCodeDesc() |
164 | 164 | { |
165 | - $result = $this->object->sort( '-code' )->search( [] ); |
|
166 | - $this->assertStringStartsWith( 'U:', reset( $result )->getCode() ); |
|
165 | + $result = $this->object->sort('-code')->search([]); |
|
166 | + $this->assertStringStartsWith('U:', reset($result)->getCode()); |
|
167 | 167 | } |
168 | 168 | |
169 | 169 | |
170 | 170 | public function testSortCtime() |
171 | 171 | { |
172 | - $this->assertEquals( 8, count( $this->object->sort( 'ctime' )->search( [] ) ) ); |
|
172 | + $this->assertEquals(8, count($this->object->sort('ctime')->search([]))); |
|
173 | 173 | } |
174 | 174 | |
175 | 175 | |
176 | 176 | public function testSortCtimeDesc() |
177 | 177 | { |
178 | - $this->assertEquals( 8, count( $this->object->sort( '-ctime' )->search( [] ) ) ); |
|
178 | + $this->assertEquals(8, count($this->object->sort('-ctime')->search([]))); |
|
179 | 179 | } |
180 | 180 | |
181 | 181 | |
182 | 182 | public function testSortName() |
183 | 183 | { |
184 | - $result = $this->object->sort( 'name' )->search( ['text'] ); |
|
185 | - $this->assertEquals( 'Cafe Noire Cappuccino', reset( $result )->getName() ); |
|
184 | + $result = $this->object->sort('name')->search(['text']); |
|
185 | + $this->assertEquals('Cafe Noire Cappuccino', reset($result)->getName()); |
|
186 | 186 | } |
187 | 187 | |
188 | 188 | |
189 | 189 | public function testSortNameDesc() |
190 | 190 | { |
191 | - $result = $this->object->sort( '-name' )->search( ['text'] ); |
|
192 | - $this->assertEquals( 'Unterproduct 3', reset( $result )->getName() ); |
|
191 | + $result = $this->object->sort('-name')->search(['text']); |
|
192 | + $this->assertEquals('Unterproduct 3', reset($result)->getName()); |
|
193 | 193 | } |
194 | 194 | |
195 | 195 | |
196 | 196 | public function testSortPrice() |
197 | 197 | { |
198 | - $result = $this->object->sort( 'price' )->search( ['price'] ); |
|
199 | - $prices = reset( $result )->getRefItems( 'price', 'default', 'default' ); |
|
198 | + $result = $this->object->sort('price')->search(['price']); |
|
199 | + $prices = reset($result)->getRefItems('price', 'default', 'default'); |
|
200 | 200 | |
201 | - $this->assertEquals( '12.00', reset( $prices )->getValue() ); |
|
201 | + $this->assertEquals('12.00', reset($prices)->getValue()); |
|
202 | 202 | } |
203 | 203 | |
204 | 204 | |
205 | 205 | public function testSortPriceDesc() |
206 | 206 | { |
207 | - $result = $this->object->sort( '-price' )->search( ['price'] ); |
|
208 | - $prices = reset( $result )->getRefItems( 'price', 'default', 'default' ); |
|
207 | + $result = $this->object->sort('-price')->search(['price']); |
|
208 | + $prices = reset($result)->getRefItems('price', 'default', 'default'); |
|
209 | 209 | |
210 | - $this->assertEquals( '600.00', reset( $prices )->getValue() ); |
|
210 | + $this->assertEquals('600.00', reset($prices)->getValue()); |
|
211 | 211 | } |
212 | 212 | |
213 | 213 | |
214 | 214 | public function testSortRelevanceCategory() |
215 | 215 | { |
216 | - $manager = \Aimeos\MShop::create( $this->context, 'catalog' ); |
|
217 | - $catId = $manager->findItem( 'new' )->getId(); |
|
216 | + $manager = \Aimeos\MShop::create($this->context, 'catalog'); |
|
217 | + $catId = $manager->findItem('new')->getId(); |
|
218 | 218 | |
219 | - $result = $this->object->category( $catId )->sort( 'relevance' )->search( [] ); |
|
219 | + $result = $this->object->category($catId)->sort('relevance')->search([]); |
|
220 | 220 | |
221 | - $this->assertEquals( 3, count( $result ) ); |
|
222 | - $this->assertEquals( 'CNE', reset( $result )->getCode() ); |
|
223 | - $this->assertEquals( 'U:BUNDLE', end( $result )->getCode() ); |
|
221 | + $this->assertEquals(3, count($result)); |
|
222 | + $this->assertEquals('CNE', reset($result)->getCode()); |
|
223 | + $this->assertEquals('U:BUNDLE', end($result)->getCode()); |
|
224 | 224 | } |
225 | 225 | |
226 | 226 | |
227 | 227 | public function testSortRelevanceSupplier() |
228 | 228 | { |
229 | - $manager = \Aimeos\MShop::create( $this->context, 'supplier' ); |
|
230 | - $supId = $manager->findItem( 'unitCode001' )->getId(); |
|
229 | + $manager = \Aimeos\MShop::create($this->context, 'supplier'); |
|
230 | + $supId = $manager->findItem('unitCode001')->getId(); |
|
231 | 231 | |
232 | - $result = $this->object->supplier( $supId )->sort( 'relevance' )->search( [] ); |
|
232 | + $result = $this->object->supplier($supId)->sort('relevance')->search([]); |
|
233 | 233 | |
234 | - $this->assertEquals( 2, count( $result ) ); |
|
235 | - $this->assertEquals( 'CNC', reset( $result )->getCode() ); |
|
236 | - $this->assertEquals( 'CNE', end( $result )->getCode() ); |
|
234 | + $this->assertEquals(2, count($result)); |
|
235 | + $this->assertEquals('CNC', reset($result)->getCode()); |
|
236 | + $this->assertEquals('CNE', end($result)->getCode()); |
|
237 | 237 | } |
238 | 238 | |
239 | 239 | |
240 | 240 | public function testSupplier() |
241 | 241 | { |
242 | - $manager = \Aimeos\MShop::create( $this->context, 'supplier' ); |
|
243 | - $supId = $manager->findItem( 'unitCode001' )->getId(); |
|
242 | + $manager = \Aimeos\MShop::create($this->context, 'supplier'); |
|
243 | + $supId = $manager->findItem('unitCode001')->getId(); |
|
244 | 244 | |
245 | - $this->assertEquals( 2, count( $this->object->supplier( $supId )->search( [] ) ) ); |
|
245 | + $this->assertEquals(2, count($this->object->supplier($supId)->search([]))); |
|
246 | 246 | } |
247 | 247 | |
248 | 248 | |
249 | 249 | public function testText() |
250 | 250 | { |
251 | - $this->assertEquals( 3, count( $this->object->text( 'Cafe' )->search( [] ) ) ); |
|
251 | + $this->assertEquals(3, count($this->object->text('Cafe')->search([]))); |
|
252 | 252 | } |
253 | 253 | } |
@@ -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::create( $context, 'order/base' ); |
|
40 | + $this->domainManager = \Aimeos\MShop::create($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->baskets[$this->type]->getLocale(), $this->type ); |
|
67 | + $this->baskets[$this->type] = $this->domainManager->getSession($this->type); |
|
68 | + $this->checkLocale($this->baskets[$this->type]->getLocale(), $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; |
@@ -127,7 +127,7 @@ discard block |
||
127 | 127 | * @category Developer |
128 | 128 | * @see controller/frontend/basket/limit-seconds |
129 | 129 | */ |
130 | - $count = $config->get( 'controller/frontend/basket/limit-count', 5 ); |
|
130 | + $count = $config->get('controller/frontend/basket/limit-count', 5); |
|
131 | 131 | |
132 | 132 | /** controller/frontend/basket/limit-seconds |
133 | 133 | * Order limitation time frame in seconds |
@@ -145,34 +145,34 @@ discard block |
||
145 | 145 | * @category Developer |
146 | 146 | * @see controller/frontend/basket/limit-count |
147 | 147 | */ |
148 | - $seconds = $config->get( 'controller/frontend/basket/limit-seconds', 300 ); |
|
148 | + $seconds = $config->get('controller/frontend/basket/limit-seconds', 300); |
|
149 | 149 | |
150 | 150 | $search = $this->domainManager->createSearch(); |
151 | 151 | $expr = [ |
152 | - $search->compare( '==', 'order.base.editor', $context->getEditor() ), |
|
153 | - $search->compare( '>=', 'order.base.ctime', date( 'Y-m-d H:i:s', time() - $seconds ) ), |
|
152 | + $search->compare('==', 'order.base.editor', $context->getEditor()), |
|
153 | + $search->compare('>=', 'order.base.ctime', date('Y-m-d H:i:s', time() - $seconds)), |
|
154 | 154 | ]; |
155 | - $search->setConditions( $search->combine( '&&', $expr ) ); |
|
156 | - $search->setSlice( 0, 0 ); |
|
155 | + $search->setConditions($search->combine('&&', $expr)); |
|
156 | + $search->setSlice(0, 0); |
|
157 | 157 | |
158 | - $this->domainManager->searchItems( $search, [], $total ); |
|
158 | + $this->domainManager->searchItems($search, [], $total); |
|
159 | 159 | |
160 | - if( $total > $count ) |
|
160 | + if ($total > $count) |
|
161 | 161 | { |
162 | - $msg = $context->getI18n()->dt( 'controller/frontend', 'Temporary order limit reached' ); |
|
163 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
162 | + $msg = $context->getI18n()->dt('controller/frontend', 'Temporary order limit reached'); |
|
163 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
164 | 164 | } |
165 | 165 | |
166 | 166 | |
167 | 167 | $basket = $this->get()->finish(); |
168 | - $basket->setCustomerId( (string) $context->getUserId() ); |
|
168 | + $basket->setCustomerId((string) $context->getUserId()); |
|
169 | 169 | |
170 | 170 | $this->domainManager->begin(); |
171 | - $this->domainManager->store( $basket ); |
|
171 | + $this->domainManager->store($basket); |
|
172 | 172 | $this->domainManager->commit(); |
173 | 173 | |
174 | 174 | $this->save(); // for reusing unpaid orders, might have side effects (!) |
175 | - $this->createSubscriptions( $basket ); |
|
175 | + $this->createSubscriptions($basket); |
|
176 | 176 | |
177 | 177 | return $basket; |
178 | 178 | } |
@@ -186,9 +186,9 @@ discard block |
||
186 | 186 | * @param boolean $default True to add default criteria (user logged in), false if not |
187 | 187 | * @return \Aimeos\MShop\Order\Item\Base\Iface Order base object including the given parts |
188 | 188 | */ |
189 | - public function load( $id, $parts = \Aimeos\MShop\Order\Item\Base\Base::PARTS_ALL, $default = true ) |
|
189 | + public function load($id, $parts = \Aimeos\MShop\Order\Item\Base\Base::PARTS_ALL, $default = true) |
|
190 | 190 | { |
191 | - return $this->domainManager->load( $id, $parts, false, $default ); |
|
191 | + return $this->domainManager->load($id, $parts, false, $default); |
|
192 | 192 | } |
193 | 193 | |
194 | 194 | |
@@ -207,34 +207,34 @@ discard block |
||
207 | 207 | * @param string $stocktype Unique code of the stock type to deliver the products from |
208 | 208 | * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available |
209 | 209 | */ |
210 | - public function addProduct( $prodid, $quantity = 1, $stocktype = 'default', array $variantAttributeIds = [], |
|
211 | - array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [] ) |
|
210 | + public function addProduct($prodid, $quantity = 1, $stocktype = 'default', array $variantAttributeIds = [], |
|
211 | + array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = []) |
|
212 | 212 | { |
213 | 213 | $attributeMap = [ |
214 | - 'custom' => array_keys( $customAttributeValues ), |
|
215 | - 'config' => array_keys( $configAttributeIds ), |
|
214 | + 'custom' => array_keys($customAttributeValues), |
|
215 | + 'config' => array_keys($configAttributeIds), |
|
216 | 216 | ]; |
217 | - $this->checkListRef( $prodid, 'attribute', $attributeMap ); |
|
217 | + $this->checkListRef($prodid, 'attribute', $attributeMap); |
|
218 | 218 | |
219 | 219 | |
220 | 220 | $context = $this->getContext(); |
221 | - $productManager = \Aimeos\MShop::create( $context, 'product' ); |
|
221 | + $productManager = \Aimeos\MShop::create($context, 'product'); |
|
222 | 222 | |
223 | - $productItem = $productManager->getItem( $prodid, ['attribute', 'media', 'supplier', 'price', 'product', 'text'], true ); |
|
224 | - $prices = $productItem->getRefItems( 'price', 'default', 'default' ); |
|
225 | - $hidden = $productItem->getRefItems( 'attribute', null, 'hidden' ); |
|
223 | + $productItem = $productManager->getItem($prodid, ['attribute', 'media', 'supplier', 'price', 'product', 'text'], true); |
|
224 | + $prices = $productItem->getRefItems('price', 'default', 'default'); |
|
225 | + $hidden = $productItem->getRefItems('attribute', null, 'hidden'); |
|
226 | 226 | |
227 | - $orderBaseProductItem = \Aimeos\MShop::create( $context, 'order/base/product' )->createItem(); |
|
228 | - $orderBaseProductItem->copyFrom( $productItem )->setQuantity( $quantity )->setStockType( $stocktype ); |
|
227 | + $orderBaseProductItem = \Aimeos\MShop::create($context, 'order/base/product')->createItem(); |
|
228 | + $orderBaseProductItem->copyFrom($productItem)->setQuantity($quantity)->setStockType($stocktype); |
|
229 | 229 | |
230 | - $custAttr = $this->getOrderProductAttributes( 'custom', array_keys( $customAttributeValues ), $customAttributeValues ); |
|
231 | - $confAttr = $this->getOrderProductAttributes( 'config', array_keys( $configAttributeIds ), [], $configAttributeIds ); |
|
232 | - $attr = array_merge( $custAttr, $confAttr, $this->getOrderProductAttributes( 'hidden', array_keys( $hidden ) ) ); |
|
230 | + $custAttr = $this->getOrderProductAttributes('custom', array_keys($customAttributeValues), $customAttributeValues); |
|
231 | + $confAttr = $this->getOrderProductAttributes('config', array_keys($configAttributeIds), [], $configAttributeIds); |
|
232 | + $attr = array_merge($custAttr, $confAttr, $this->getOrderProductAttributes('hidden', array_keys($hidden))); |
|
233 | 233 | |
234 | - $orderBaseProductItem->setAttributeItems( $attr ); |
|
235 | - $orderBaseProductItem->setPrice( $this->calcPrice( $orderBaseProductItem, $prices, $quantity ) ); |
|
234 | + $orderBaseProductItem->setAttributeItems($attr); |
|
235 | + $orderBaseProductItem->setPrice($this->calcPrice($orderBaseProductItem, $prices, $quantity)); |
|
236 | 236 | |
237 | - $this->get()->addProduct( $orderBaseProductItem ); |
|
237 | + $this->get()->addProduct($orderBaseProductItem); |
|
238 | 238 | $this->save(); |
239 | 239 | } |
240 | 240 | |
@@ -244,17 +244,17 @@ discard block |
||
244 | 244 | * |
245 | 245 | * @param integer $position Position number (key) of the order product item |
246 | 246 | */ |
247 | - public function deleteProduct( $position ) |
|
247 | + public function deleteProduct($position) |
|
248 | 248 | { |
249 | - $product = $this->get()->getProduct( $position ); |
|
249 | + $product = $this->get()->getProduct($position); |
|
250 | 250 | |
251 | - if( $product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE ) |
|
251 | + if ($product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE) |
|
252 | 252 | { |
253 | - $msg = $this->getContext()->getI18n()->dt( 'controller/frontend', 'Basket item at position "%1$d" cannot be deleted manually' ); |
|
254 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $position ) ); |
|
253 | + $msg = $this->getContext()->getI18n()->dt('controller/frontend', 'Basket item at position "%1$d" cannot be deleted manually'); |
|
254 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $position)); |
|
255 | 255 | } |
256 | 256 | |
257 | - $this->get()->deleteProduct( $position ); |
|
257 | + $this->get()->deleteProduct($position); |
|
258 | 258 | $this->save(); |
259 | 259 | } |
260 | 260 | |
@@ -266,32 +266,32 @@ discard block |
||
266 | 266 | * @param integer $quantity New quantiy of the product item |
267 | 267 | * @param string[] $configAttributeCodes Codes of the product config attributes that should be REMOVED |
268 | 268 | */ |
269 | - public function editProduct( $position, $quantity, array $configAttributeCodes = [] ) |
|
269 | + public function editProduct($position, $quantity, array $configAttributeCodes = []) |
|
270 | 270 | { |
271 | - $product = $this->get()->getProduct( $position ); |
|
271 | + $product = $this->get()->getProduct($position); |
|
272 | 272 | |
273 | - if( $product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE ) |
|
273 | + if ($product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE) |
|
274 | 274 | { |
275 | - $msg = $this->getContext()->getI18n()->dt( 'controller/frontend', 'Basket item at position "%1$d" cannot be changed' ); |
|
276 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $position ) ); |
|
275 | + $msg = $this->getContext()->getI18n()->dt('controller/frontend', 'Basket item at position "%1$d" cannot be changed'); |
|
276 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $position)); |
|
277 | 277 | } |
278 | 278 | |
279 | - $product->setQuantity( $quantity ); |
|
279 | + $product->setQuantity($quantity); |
|
280 | 280 | |
281 | 281 | $attributes = $product->getAttributeItems(); |
282 | - foreach( $attributes as $key => $attribute ) |
|
282 | + foreach ($attributes as $key => $attribute) |
|
283 | 283 | { |
284 | - if( in_array( $attribute->getCode(), $configAttributeCodes ) ) { |
|
285 | - unset( $attributes[$key] ); |
|
284 | + if (in_array($attribute->getCode(), $configAttributeCodes)) { |
|
285 | + unset($attributes[$key]); |
|
286 | 286 | } |
287 | 287 | } |
288 | - $product->setAttributeItems( $attributes ); |
|
288 | + $product->setAttributeItems($attributes); |
|
289 | 289 | |
290 | - $manager = \Aimeos\MShop::create( $this->getContext(), 'product' ); |
|
291 | - $productItem = $manager->findItem( $product->getProductCode(), array( 'price', 'text' ), true ); |
|
292 | - $product->setPrice( $this->calcPrice( $product, $productItem->getRefItems( 'price', 'default' ), $quantity ) ); |
|
290 | + $manager = \Aimeos\MShop::create($this->getContext(), 'product'); |
|
291 | + $productItem = $manager->findItem($product->getProductCode(), array('price', 'text'), true); |
|
292 | + $product->setPrice($this->calcPrice($product, $productItem->getRefItems('price', 'default'), $quantity)); |
|
293 | 293 | |
294 | - $this->get()->addProduct( $product, $position ); |
|
294 | + $this->get()->addProduct($product, $position); |
|
295 | 295 | |
296 | 296 | $this->save(); |
297 | 297 | } |
@@ -303,7 +303,7 @@ discard block |
||
303 | 303 | * @param string $code Coupon code entered by the user |
304 | 304 | * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid or not allowed |
305 | 305 | */ |
306 | - public function addCoupon( $code ) |
|
306 | + public function addCoupon($code) |
|
307 | 307 | { |
308 | 308 | $context = $this->getContext(); |
309 | 309 | |
@@ -323,44 +323,44 @@ discard block |
||
323 | 323 | * @category User |
324 | 324 | * @category Developer |
325 | 325 | */ |
326 | - $allowed = $context->getConfig()->get( 'controller/frontend/basket/standard/coupon/allowed', 1 ); |
|
326 | + $allowed = $context->getConfig()->get('controller/frontend/basket/standard/coupon/allowed', 1); |
|
327 | 327 | |
328 | - if( $allowed <= count( $this->get()->getCoupons() ) ) |
|
328 | + if ($allowed <= count($this->get()->getCoupons())) |
|
329 | 329 | { |
330 | - $msg = $context->getI18n()->dt( 'controller/frontend', 'Number of coupon codes exceeds the limit' ); |
|
331 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
330 | + $msg = $context->getI18n()->dt('controller/frontend', 'Number of coupon codes exceeds the limit'); |
|
331 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
332 | 332 | } |
333 | 333 | |
334 | 334 | |
335 | - $manager = \Aimeos\MShop::create( $context, 'coupon' ); |
|
336 | - $codeManager = \Aimeos\MShop::create( $context, 'coupon/code' ); |
|
335 | + $manager = \Aimeos\MShop::create($context, 'coupon'); |
|
336 | + $codeManager = \Aimeos\MShop::create($context, 'coupon/code'); |
|
337 | 337 | |
338 | - $search = $manager->createSearch( true )->setSlice( 0, 1 ); |
|
338 | + $search = $manager->createSearch(true)->setSlice(0, 1); |
|
339 | 339 | $expr = [ |
340 | - $search->compare( '==', 'coupon.code.code', $code ), |
|
341 | - $codeManager->createSearch( true )->getConditions(), |
|
340 | + $search->compare('==', 'coupon.code.code', $code), |
|
341 | + $codeManager->createSearch(true)->getConditions(), |
|
342 | 342 | $search->getConditions(), |
343 | 343 | ]; |
344 | - $search->setConditions( $search->combine( '&&', $expr ) ); |
|
344 | + $search->setConditions($search->combine('&&', $expr)); |
|
345 | 345 | |
346 | - $result = $manager->searchItems( $search ); |
|
346 | + $result = $manager->searchItems($search); |
|
347 | 347 | |
348 | - if( ( $item = reset( $result ) ) === false ) |
|
348 | + if (($item = reset($result)) === false) |
|
349 | 349 | { |
350 | - $msg = sprintf( $context->getI18n()->dt( 'controller/frontend', 'Coupon code "%1$s" is invalid or not available any more' ), $code ); |
|
351 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
350 | + $msg = sprintf($context->getI18n()->dt('controller/frontend', 'Coupon code "%1$s" is invalid or not available any more'), $code); |
|
351 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
352 | 352 | } |
353 | 353 | |
354 | 354 | |
355 | - $provider = $manager->getProvider( $item, $code ); |
|
355 | + $provider = $manager->getProvider($item, $code); |
|
356 | 356 | |
357 | - if( $provider->isAvailable( $this->get() ) !== true ) |
|
357 | + if ($provider->isAvailable($this->get()) !== true) |
|
358 | 358 | { |
359 | - $msg = sprintf( $context->getI18n()->dt( 'controller/frontend', 'Requirements for coupon code "%1$s" aren\'t met' ), $code ); |
|
360 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
359 | + $msg = sprintf($context->getI18n()->dt('controller/frontend', 'Requirements for coupon code "%1$s" aren\'t met'), $code); |
|
360 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
361 | 361 | } |
362 | 362 | |
363 | - $provider->addCoupon( $this->get() ); |
|
363 | + $provider->addCoupon($this->get()); |
|
364 | 364 | $this->save(); |
365 | 365 | } |
366 | 366 | |
@@ -371,24 +371,24 @@ discard block |
||
371 | 371 | * @param string $code Coupon code entered by the user |
372 | 372 | * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid |
373 | 373 | */ |
374 | - public function deleteCoupon( $code ) |
|
374 | + public function deleteCoupon($code) |
|
375 | 375 | { |
376 | 376 | $context = $this->getContext(); |
377 | - $manager = \Aimeos\MShop::create( $context, 'coupon' ); |
|
377 | + $manager = \Aimeos\MShop::create($context, 'coupon'); |
|
378 | 378 | |
379 | 379 | $search = $manager->createSearch(); |
380 | - $search->setConditions( $search->compare( '==', 'coupon.code.code', $code ) ); |
|
381 | - $search->setSlice( 0, 1 ); |
|
380 | + $search->setConditions($search->compare('==', 'coupon.code.code', $code)); |
|
381 | + $search->setSlice(0, 1); |
|
382 | 382 | |
383 | - $result = $manager->searchItems( $search ); |
|
383 | + $result = $manager->searchItems($search); |
|
384 | 384 | |
385 | - if( ( $item = reset( $result ) ) === false ) |
|
385 | + if (($item = reset($result)) === false) |
|
386 | 386 | { |
387 | - $msg = $context->getI18n()->dt( 'controller/frontend', 'Coupon code "%1$s" is invalid' ); |
|
388 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $code ) ); |
|
387 | + $msg = $context->getI18n()->dt('controller/frontend', 'Coupon code "%1$s" is invalid'); |
|
388 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $code)); |
|
389 | 389 | } |
390 | 390 | |
391 | - $manager->getProvider( $item, $code )->deleteCoupon( $this->get() ); |
|
391 | + $manager->getProvider($item, $code)->deleteCoupon($this->get()); |
|
392 | 392 | $this->save(); |
393 | 393 | } |
394 | 394 | |
@@ -401,28 +401,28 @@ discard block |
||
401 | 401 | * @throws \Aimeos\Controller\Frontend\Basket\Exception If the billing or delivery address is not of any required type of |
402 | 402 | * if one of the keys is invalid when using an array with key/value pairs |
403 | 403 | */ |
404 | - public function setAddress( $type, $value ) |
|
404 | + public function setAddress($type, $value) |
|
405 | 405 | { |
406 | 406 | $context = $this->getContext(); |
407 | - $address = \Aimeos\MShop::create( $context, 'order/base/address' )->createItem(); |
|
408 | - $address->setType( $type ); |
|
407 | + $address = \Aimeos\MShop::create($context, 'order/base/address')->createItem(); |
|
408 | + $address->setType($type); |
|
409 | 409 | |
410 | - if( $value instanceof \Aimeos\MShop\Common\Item\Address\Iface ) |
|
410 | + if ($value instanceof \Aimeos\MShop\Common\Item\Address\Iface) |
|
411 | 411 | { |
412 | - $this->get()->setAddress( $address->copyFrom( $value ), $type ); |
|
412 | + $this->get()->setAddress($address->copyFrom($value), $type); |
|
413 | 413 | } |
414 | - else if( is_array( $value ) ) |
|
414 | + else if (is_array($value)) |
|
415 | 415 | { |
416 | - $this->get()->setAddress( $this->setAddressFromArray( $address, $value ), $type ); |
|
416 | + $this->get()->setAddress($this->setAddressFromArray($address, $value), $type); |
|
417 | 417 | } |
418 | - else if( $value === null ) |
|
418 | + else if ($value === null) |
|
419 | 419 | { |
420 | - $this->get()->deleteAddress( $type ); |
|
420 | + $this->get()->deleteAddress($type); |
|
421 | 421 | } |
422 | 422 | else |
423 | 423 | { |
424 | - $msg = $context->getI18n()->dt( 'controller/frontend', 'Invalid value for address type "%1$s"' ); |
|
425 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $type ) ); |
|
424 | + $msg = $context->getI18n()->dt('controller/frontend', 'Invalid value for address type "%1$s"'); |
|
425 | + throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $type)); |
|
426 | 426 | } |
427 | 427 | |
428 | 428 | $this->save(); |
@@ -438,42 +438,42 @@ discard block |
||
438 | 438 | * entered by the customer when choosing one of the delivery or payment options |
439 | 439 | * @throws \Aimeos\Controller\Frontend\Basket\Exception If there is no price to the service item attached |
440 | 440 | */ |
441 | - public function addService( $type, $id, array $attributes = [] ) |
|
441 | + public function addService($type, $id, array $attributes = []) |
|
442 | 442 | { |
443 | 443 | $context = $this->getContext(); |
444 | 444 | |
445 | - $serviceManager = \Aimeos\MShop::create( $context, 'service' ); |
|
446 | - $serviceItem = $serviceManager->getItem( $id, array( 'media', 'price', 'text' ) ); |
|
445 | + $serviceManager = \Aimeos\MShop::create($context, 'service'); |
|
446 | + $serviceItem = $serviceManager->getItem($id, array('media', 'price', 'text')); |
|
447 | 447 | |
448 | - $provider = $serviceManager->getProvider( $serviceItem, $serviceItem->getType() ); |
|
449 | - $result = $provider->checkConfigFE( $attributes ); |
|
450 | - $unknown = array_diff_key( $attributes, $result ); |
|
448 | + $provider = $serviceManager->getProvider($serviceItem, $serviceItem->getType()); |
|
449 | + $result = $provider->checkConfigFE($attributes); |
|
450 | + $unknown = array_diff_key($attributes, $result); |
|
451 | 451 | |
452 | - if( count( $unknown ) > 0 ) |
|
452 | + if (count($unknown) > 0) |
|
453 | 453 | { |
454 | - $msg = $context->getI18n()->dt( 'controller/frontend', 'Unknown attributes "%1$s"' ); |
|
455 | - $msg = sprintf( $msg, implode( '","', array_keys( $unknown ) ) ); |
|
456 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg ); |
|
454 | + $msg = $context->getI18n()->dt('controller/frontend', 'Unknown attributes "%1$s"'); |
|
455 | + $msg = sprintf($msg, implode('","', array_keys($unknown))); |
|
456 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg); |
|
457 | 457 | } |
458 | 458 | |
459 | - foreach( $result as $key => $value ) |
|
459 | + foreach ($result as $key => $value) |
|
460 | 460 | { |
461 | - if( $value !== null ) { |
|
462 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $value ); |
|
461 | + if ($value !== null) { |
|
462 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($value); |
|
463 | 463 | } |
464 | 464 | } |
465 | 465 | |
466 | - $orderBaseServiceManager = \Aimeos\MShop::create( $context, 'order/base/service' ); |
|
466 | + $orderBaseServiceManager = \Aimeos\MShop::create($context, 'order/base/service'); |
|
467 | 467 | $orderServiceItem = $orderBaseServiceManager->createItem(); |
468 | - $orderServiceItem->copyFrom( $serviceItem ); |
|
468 | + $orderServiceItem->copyFrom($serviceItem); |
|
469 | 469 | |
470 | 470 | // remove service rebate of original price |
471 | - $price = $provider->calcPrice( $this->get() )->setRebate( '0.00' ); |
|
472 | - $orderServiceItem->setPrice( $price ); |
|
471 | + $price = $provider->calcPrice($this->get())->setRebate('0.00'); |
|
472 | + $orderServiceItem->setPrice($price); |
|
473 | 473 | |
474 | - $provider->setConfigFE( $orderServiceItem, $attributes ); |
|
474 | + $provider->setConfigFE($orderServiceItem, $attributes); |
|
475 | 475 | |
476 | - $this->get()->addService( $orderServiceItem, $type ); |
|
476 | + $this->get()->addService($orderServiceItem, $type); |
|
477 | 477 | $this->save(); |
478 | 478 | } |
479 | 479 | |
@@ -483,9 +483,9 @@ discard block |
||
483 | 483 | * |
484 | 484 | * @param string $type Service type code like 'payment' or 'delivery' |
485 | 485 | */ |
486 | - public function deleteService( $type ) |
|
486 | + public function deleteService($type) |
|
487 | 487 | { |
488 | - $this->get()->deleteService( $type ); |
|
488 | + $this->get()->deleteService($type); |
|
489 | 489 | $this->save(); |
490 | 490 | } |
491 | 491 | |
@@ -499,18 +499,18 @@ discard block |
||
499 | 499 | * @return \Aimeos\MShop\Order\Item\Base\Address\Iface Updated address item |
500 | 500 | * @throws \Aimeos\Controller\Frontend\Basket\Exception |
501 | 501 | */ |
502 | - protected function setAddressFromArray( \Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map ) |
|
502 | + protected function setAddressFromArray(\Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map) |
|
503 | 503 | { |
504 | - foreach( $map as $key => $value ) { |
|
505 | - $map[$key] = strip_tags( $value ); // prevent XSS |
|
504 | + foreach ($map as $key => $value) { |
|
505 | + $map[$key] = strip_tags($value); // prevent XSS |
|
506 | 506 | } |
507 | 507 | |
508 | - $address = $address->fromArray( $map ); |
|
508 | + $address = $address->fromArray($map); |
|
509 | 509 | |
510 | - if( count( $map ) > 0 ) |
|
510 | + if (count($map) > 0) |
|
511 | 511 | { |
512 | - $msg = $this->getContext()->getI18n()->dt( 'controller/frontend', 'Invalid address properties, please check your input' ); |
|
513 | - throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg, 0, null, $map ); |
|
512 | + $msg = $this->getContext()->getI18n()->dt('controller/frontend', 'Invalid address properties, please check your input'); |
|
513 | + throw new \Aimeos\Controller\Frontend\Basket\Exception($msg, 0, null, $map); |
|
514 | 514 | } |
515 | 515 | |
516 | 516 | return $address; |