Completed
Push — master ( 6d27d0...5f6e40 )
by Aimeos
02:34
created
controller/frontend/src/Controller/Frontend/Basket/Decorator/Base.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -30,18 +30,18 @@  discard block
 block discarded – undo
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\Basket\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
 block discarded – undo
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
 
@@ -97,9 +97,9 @@  discard block
 block discarded – undo
97 97
 	 * @param string $type Basket type
98 98
 	 * @return \Aimeos\Controller\Frontend\Basket\Iface Basket frontend object
99 99
 	 */
100
-	public function setType( $type )
100
+	public function setType($type)
101 101
 	{
102
-		$this->controller->setType( $type );
102
+		$this->controller->setType($type);
103 103
 		return $this;
104 104
 	}
105 105
 
@@ -123,9 +123,9 @@  discard block
 block discarded – undo
123 123
 	 * @param boolean $default True to add default criteria (user logged in), false if not
124 124
 	 * @return \Aimeos\MShop\Order\Item\Base\Iface Order base object including the given parts
125 125
 	 */
126
-	public function load( $id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true )
126
+	public function load($id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true)
127 127
 	{
128
-		return $this->controller->load( $id, $parts, $default );
128
+		return $this->controller->load($id, $parts, $default);
129 129
 	}
130 130
 
131 131
 
@@ -150,9 +150,9 @@  discard block
 block discarded – undo
150 150
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
151 151
 	 * @return void
152 152
 	 */
153
-	public function addProduct( $prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(),
153
+	public function addProduct($prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(),
154 154
 		array $configAttributeIds = array(), array $hiddenAttributeIds = array(), array $customAttributeValues = array(),
155
-		$stocktype = 'default' )
155
+		$stocktype = 'default')
156 156
 	{
157 157
 		$this->controller->addProduct(
158 158
 			$prodid, $quantity, $options, $variantAttributeIds, $configAttributeIds,
@@ -167,9 +167,9 @@  discard block
 block discarded – undo
167 167
 	 * @param integer $position Position number (key) of the order product item
168 168
 	 * @return void
169 169
 	 */
170
-	public function deleteProduct( $position )
170
+	public function deleteProduct($position)
171 171
 	{
172
-		$this->controller->deleteProduct( $position );
172
+		$this->controller->deleteProduct($position);
173 173
 	}
174 174
 
175 175
 
@@ -182,9 +182,9 @@  discard block
 block discarded – undo
182 182
 	 * @param array $configAttributeCodes Codes of the product config attributes that should be REMOVED
183 183
 	 * @return void
184 184
 	 */
185
-	public function editProduct( $position, $quantity, array $options = array(), array $configAttributeCodes = array() )
185
+	public function editProduct($position, $quantity, array $options = array(), array $configAttributeCodes = array())
186 186
 	{
187
-		$this->controller->editProduct( $position, $quantity, $options, $configAttributeCodes );
187
+		$this->controller->editProduct($position, $quantity, $options, $configAttributeCodes);
188 188
 	}
189 189
 
190 190
 
@@ -195,9 +195,9 @@  discard block
 block discarded – undo
195 195
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid or not allowed
196 196
 	 * @return void
197 197
 	 */
198
-	public function addCoupon( $code )
198
+	public function addCoupon($code)
199 199
 	{
200
-		$this->controller->addCoupon( $code );
200
+		$this->controller->addCoupon($code);
201 201
 	}
202 202
 
203 203
 
@@ -208,9 +208,9 @@  discard block
 block discarded – undo
208 208
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid
209 209
 	 * @return void
210 210
 	 */
211
-	public function deleteCoupon( $code )
211
+	public function deleteCoupon($code)
212 212
 	{
213
-		$this->controller->deleteCoupon( $code );
213
+		$this->controller->deleteCoupon($code);
214 214
 	}
215 215
 
216 216
 
@@ -223,9 +223,9 @@  discard block
 block discarded – undo
223 223
 	 * 	if one of the keys is invalid when using an array with key/value pairs
224 224
 	 * @return void
225 225
 	 */
226
-	public function setAddress( $type, $value )
226
+	public function setAddress($type, $value)
227 227
 	{
228
-		$this->controller->setAddress( $type, $value );
228
+		$this->controller->setAddress($type, $value);
229 229
 	}
230 230
 
231 231
 
@@ -239,9 +239,9 @@  discard block
 block discarded – undo
239 239
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If there is no price to the service item attached
240 240
 	 * @return void
241 241
 	 */
242
-	public function setService( $type, $id, array $attributes = array() )
242
+	public function setService($type, $id, array $attributes = array())
243 243
 	{
244
-		$this->controller->setService( $type, $id, $attributes );
244
+		$this->controller->setService($type, $id, $attributes);
245 245
 	}
246 246
 
247 247
 
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Iface.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 	 * @param string $type Basket type
51 51
 	 * @return \Aimeos\Controller\Frontend\Basket\Iface Basket frontend object
52 52
 	 */
53
-	public function setType( $type );
53
+	public function setType($type);
54 54
 
55 55
 
56 56
 	/**
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 	 * @param boolean $default True to add default criteria (user logged in), false if not
70 70
 	 * @return \Aimeos\MShop\Order\Item\Base\Iface Order base object including the given parts
71 71
 	 */
72
-	public function load( $id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true );
72
+	public function load($id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true);
73 73
 
74 74
 
75 75
 	/**
@@ -89,9 +89,9 @@  discard block
 block discarded – undo
89 89
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
90 90
 	 * @return void
91 91
 	 */
92
-	public function addProduct( $prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(),
92
+	public function addProduct($prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(),
93 93
 		array $configAttributeIds = array(), array $hiddenAttributeIds = array(), array $customAttributeValues = array(),
94
-		$stocktype = 'default' );
94
+		$stocktype = 'default');
95 95
 
96 96
 
97 97
 	/**
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	 * @param integer $position Position number (key) of the order product item
101 101
 	 * @return void
102 102
 	 */
103
-	public function deleteProduct( $position );
103
+	public function deleteProduct($position);
104 104
 
105 105
 
106 106
 	/**
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 	 * @param array $configAttributeCodes Codes of the product config attributes that should be REMOVED
112 112
 	 * @return void
113 113
 	 */
114
-	public function editProduct( $position, $quantity, array $configAttributeCodes = array() );
114
+	public function editProduct($position, $quantity, array $configAttributeCodes = array());
115 115
 
116 116
 
117 117
 	/**
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid or not allowed
122 122
 	 * @return void
123 123
 	 */
124
-	public function addCoupon( $code );
124
+	public function addCoupon($code);
125 125
 
126 126
 
127 127
 	/**
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid
132 132
 	 * @return void
133 133
 	 */
134
-	public function deleteCoupon( $code );
134
+	public function deleteCoupon($code);
135 135
 
136 136
 
137 137
 	/**
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 	 * 	if one of the keys is invalid when using an array with key/value pairs
144 144
 	 * @return void
145 145
 	 */
146
-	public function setAddress( $type, $value );
146
+	public function setAddress($type, $value);
147 147
 
148 148
 
149 149
 	/**
@@ -156,5 +156,5 @@  discard block
 block discarded – undo
156 156
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If there is no price to the service item attached
157 157
 	 * @return void
158 158
 	 */
159
-	public function setService( $type, $id, array $attributes = array() );
159
+	public function setService($type, $id, array $attributes = array());
160 160
 }
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Standard.php 1 patch
Spacing   +120 added lines, -120 removed lines patch added patch discarded remove patch
@@ -33,11 +33,11 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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;
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 		$basket = $this->get();
111 111
 
112 112
 		$this->domainManager->begin();
113
-		$this->domainManager->store( $basket );
113
+		$this->domainManager->store($basket);
114 114
 		$this->domainManager->commit();
115 115
 
116 116
 		return $basket;
@@ -125,9 +125,9 @@  discard block
 block discarded – undo
125 125
 	 * @param boolean $default True to add default criteria (user logged in), false if not
126 126
 	 * @return \Aimeos\MShop\Order\Item\Base\Iface Order base object including the given parts
127 127
 	 */
128
-	public function load( $id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true )
128
+	public function load($id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true)
129 129
 	{
130
-		return $this->domainManager->load( $id, $parts, false, $default );
130
+		return $this->domainManager->load($id, $parts, false, $default);
131 131
 	}
132 132
 
133 133
 
@@ -147,36 +147,36 @@  discard block
 block discarded – undo
147 147
 	 * @param string $stocktype Unique code of the stock type to deliver the products from
148 148
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
149 149
 	 */
150
-	public function addProduct( $prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(),
150
+	public function addProduct($prodid, $quantity = 1, array $options = array(), array $variantAttributeIds = array(),
151 151
 		array $configAttributeIds = array(), array $hiddenAttributeIds = array(), array $customAttributeValues = array(),
152
-		$stocktype = 'default' )
152
+		$stocktype = 'default')
153 153
 	{
154 154
 		$context = $this->getContext();
155
-		$productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
156
-		$productItem = $productManager->getItem( $prodid, array( 'media', 'supplier', 'price', 'product', 'text' ) );
155
+		$productManager = \Aimeos\MShop\Factory::createManager($context, 'product');
156
+		$productItem = $productManager->getItem($prodid, array('media', 'supplier', 'price', 'product', 'text'));
157 157
 
158
-		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product' )->createItem();
159
-		$orderBaseProductItem->copyFrom( $productItem );
160
-		$orderBaseProductItem->setQuantity( $quantity );
161
-		$orderBaseProductItem->setStockType( $stocktype );
158
+		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager($context, 'order/base/product')->createItem();
159
+		$orderBaseProductItem->copyFrom($productItem);
160
+		$orderBaseProductItem->setQuantity($quantity);
161
+		$orderBaseProductItem->setStockType($stocktype);
162 162
 
163 163
 		$attr = array();
164
-		$prices = $productItem->getRefItems( 'price', 'default', 'default' );
164
+		$prices = $productItem->getRefItems('price', 'default', 'default');
165 165
 
166
-		$priceManager = \Aimeos\MShop\Factory::createManager( $context, 'price' );
167
-		$price = $priceManager->getLowestPrice( $prices, $quantity );
166
+		$priceManager = \Aimeos\MShop\Factory::createManager($context, 'price');
167
+		$price = $priceManager->getLowestPrice($prices, $quantity);
168 168
 
169
-		$attr = array_merge( $attr, $this->createOrderProductAttributes( $price, $prodid, $quantity, $configAttributeIds, 'config' ) );
170
-		$attr = array_merge( $attr, $this->createOrderProductAttributes( $price, $prodid, $quantity, $hiddenAttributeIds, 'hidden' ) );
171
-		$attr = array_merge( $attr, $this->createOrderProductAttributes( $price, $prodid, $quantity, array_keys( $customAttributeValues ), 'custom', $customAttributeValues ) );
169
+		$attr = array_merge($attr, $this->createOrderProductAttributes($price, $prodid, $quantity, $configAttributeIds, 'config'));
170
+		$attr = array_merge($attr, $this->createOrderProductAttributes($price, $prodid, $quantity, $hiddenAttributeIds, 'hidden'));
171
+		$attr = array_merge($attr, $this->createOrderProductAttributes($price, $prodid, $quantity, array_keys($customAttributeValues), 'custom', $customAttributeValues));
172 172
 
173 173
 		// remove product rebate of original price in favor to rebates granted for the order
174
-		$price->setRebate( '0.00' );
174
+		$price->setRebate('0.00');
175 175
 
176
-		$orderBaseProductItem->setPrice( $price );
177
-		$orderBaseProductItem->setAttributes( $attr );
176
+		$orderBaseProductItem->setPrice($price);
177
+		$orderBaseProductItem->setAttributes($attr);
178 178
 
179
-		$this->get()->addProduct( $orderBaseProductItem );
179
+		$this->get()->addProduct($orderBaseProductItem);
180 180
 		$this->save();
181 181
 	}
182 182
 
@@ -186,17 +186,17 @@  discard block
 block discarded – undo
186 186
 	 *
187 187
 	 * @param integer $position Position number (key) of the order product item
188 188
 	 */
189
-	public function deleteProduct( $position )
189
+	public function deleteProduct($position)
190 190
 	{
191
-		$product = $this->get()->getProduct( $position );
191
+		$product = $this->get()->getProduct($position);
192 192
 
193
-		if( $product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE )
193
+		if ($product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE)
194 194
 		{
195
-			$msg = sprintf( 'Basket item at position "%1$d" cannot be deleted manually', $position );
196
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
195
+			$msg = sprintf('Basket item at position "%1$d" cannot be deleted manually', $position);
196
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
197 197
 		}
198 198
 
199
-		$this->get()->deleteProduct( $position );
199
+		$this->get()->deleteProduct($position);
200 200
 		$this->save();
201 201
 	}
202 202
 
@@ -210,34 +210,34 @@  discard block
 block discarded – undo
210 210
 	 * 	The 'stock'=>false option allows adding products without being in stock.
211 211
 	 * @param string[] $configAttributeCodes Codes of the product config attributes that should be REMOVED
212 212
 	 */
213
-	public function editProduct( $position, $quantity, array $options = array(),
214
-		array $configAttributeCodes = array() )
213
+	public function editProduct($position, $quantity, array $options = array(),
214
+		array $configAttributeCodes = array())
215 215
 	{
216
-		$product = $this->get()->getProduct( $position );
216
+		$product = $this->get()->getProduct($position);
217 217
 
218
-		if( $product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE )
218
+		if ($product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE)
219 219
 		{
220
-			$msg = sprintf( 'Basket item at position "%1$d" cannot be changed', $position );
221
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
220
+			$msg = sprintf('Basket item at position "%1$d" cannot be changed', $position);
221
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
222 222
 		}
223 223
 
224
-		$product->setQuantity( $quantity );
224
+		$product->setQuantity($quantity);
225 225
 
226 226
 		$attributes = $product->getAttributes();
227
-		foreach( $attributes as $key => $attribute )
227
+		foreach ($attributes as $key => $attribute)
228 228
 		{
229
-			if( in_array( $attribute->getCode(), $configAttributeCodes ) ) {
230
-				unset( $attributes[$key] );
229
+			if (in_array($attribute->getCode(), $configAttributeCodes)) {
230
+				unset($attributes[$key]);
231 231
 			}
232 232
 		}
233
-		$product->setAttributes( $attributes );
233
+		$product->setAttributes($attributes);
234 234
 
235
-		$productItem = $this->getDomainItem( 'product', 'product.code', $product->getProductCode(), array( 'price', 'text' ) );
236
-		$prices = $productItem->getRefItems( 'price', 'default' );
237
-		$product->setPrice( $this->calcPrice( $product, $prices, $quantity ) );
235
+		$productItem = $this->getDomainItem('product', 'product.code', $product->getProductCode(), array('price', 'text'));
236
+		$prices = $productItem->getRefItems('price', 'default');
237
+		$product->setPrice($this->calcPrice($product, $prices, $quantity));
238 238
 
239
-		$this->get()->deleteProduct( $position );
240
-		$this->get()->addProduct( $product, $position );
239
+		$this->get()->deleteProduct($position);
240
+		$this->get()->addProduct($product, $position);
241 241
 
242 242
 		$this->save();
243 243
 	}
@@ -249,49 +249,49 @@  discard block
 block discarded – undo
249 249
 	 * @param string $code Coupon code entered by the user
250 250
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid or not allowed
251 251
 	 */
252
-	public function addCoupon( $code )
252
+	public function addCoupon($code)
253 253
 	{
254
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon' );
255
-		$codeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon/code' );
254
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon');
255
+		$codeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon/code');
256 256
 
257 257
 
258
-		$search = $codeManager->createSearch( true );
258
+		$search = $codeManager->createSearch(true);
259 259
 		$expr = array(
260
-			$search->compare( '==', 'coupon.code.code', $code ),
260
+			$search->compare('==', 'coupon.code.code', $code),
261 261
 			$search->getConditions(),
262 262
 		);
263
-		$search->setConditions( $search->combine( '&&', $expr ) );
264
-		$search->setSlice( 0, 1 );
263
+		$search->setConditions($search->combine('&&', $expr));
264
+		$search->setSlice(0, 1);
265 265
 
266
-		$result = $codeManager->searchItems( $search );
266
+		$result = $codeManager->searchItems($search);
267 267
 
268
-		if( ( $codeItem = reset( $result ) ) === false ) {
269
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon code "%1$s" is invalid or not available any more', $code ) );
268
+		if (($codeItem = reset($result)) === false) {
269
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon code "%1$s" is invalid or not available any more', $code));
270 270
 		}
271 271
 
272 272
 
273
-		$search = $manager->createSearch( true );
273
+		$search = $manager->createSearch(true);
274 274
 		$expr = array(
275
-			$search->compare( '==', 'coupon.id', $codeItem->getParentId() ),
275
+			$search->compare('==', 'coupon.id', $codeItem->getParentId()),
276 276
 			$search->getConditions(),
277 277
 		);
278
-		$search->setConditions( $search->combine( '&&', $expr ) );
279
-		$search->setSlice( 0, 1 );
278
+		$search->setConditions($search->combine('&&', $expr));
279
+		$search->setSlice(0, 1);
280 280
 
281
-		$result = $manager->searchItems( $search );
281
+		$result = $manager->searchItems($search);
282 282
 
283
-		if( ( $item = reset( $result ) ) === false ) {
284
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon for code "%1$s" is not available any more', $code ) );
283
+		if (($item = reset($result)) === false) {
284
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon for code "%1$s" is not available any more', $code));
285 285
 		}
286 286
 
287 287
 
288
-		$provider = $manager->getProvider( $item, $code );
288
+		$provider = $manager->getProvider($item, $code);
289 289
 
290
-		if( $provider->isAvailable( $this->get() ) !== true ) {
291
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Requirements for coupon code "%1$s" aren\'t met', $code ) );
290
+		if ($provider->isAvailable($this->get()) !== true) {
291
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Requirements for coupon code "%1$s" aren\'t met', $code));
292 292
 		}
293 293
 
294
-		$provider->addCoupon( $this->get() );
294
+		$provider->addCoupon($this->get());
295 295
 		$this->save();
296 296
 	}
297 297
 
@@ -302,21 +302,21 @@  discard block
 block discarded – undo
302 302
 	 * @param string $code Coupon code entered by the user
303 303
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid
304 304
 	 */
305
-	public function deleteCoupon( $code )
305
+	public function deleteCoupon($code)
306 306
 	{
307
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon' );
307
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon');
308 308
 
309 309
 		$search = $manager->createSearch();
310
-		$search->setConditions( $search->compare( '==', 'coupon.code.code', $code ) );
311
-		$search->setSlice( 0, 1 );
310
+		$search->setConditions($search->compare('==', 'coupon.code.code', $code));
311
+		$search->setSlice(0, 1);
312 312
 
313
-		$result = $manager->searchItems( $search );
313
+		$result = $manager->searchItems($search);
314 314
 
315
-		if( ( $item = reset( $result ) ) === false ) {
316
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon code "%1$s" is invalid', $code ) );
315
+		if (($item = reset($result)) === false) {
316
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon code "%1$s" is invalid', $code));
317 317
 		}
318 318
 
319
-		$manager->getProvider( $item, $code )->deleteCoupon( $this->get() );
319
+		$manager->getProvider($item, $code)->deleteCoupon($this->get());
320 320
 		$this->save();
321 321
 	}
322 322
 
@@ -329,28 +329,28 @@  discard block
 block discarded – undo
329 329
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the billing or delivery address is not of any required type of
330 330
 	 * 	if one of the keys is invalid when using an array with key/value pairs
331 331
 	 */
332
-	public function setAddress( $type, $value )
332
+	public function setAddress($type, $value)
333 333
 	{
334
-		$address = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/address' )->createItem();
335
-		$address->setType( $type );
334
+		$address = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/address')->createItem();
335
+		$address->setType($type);
336 336
 
337
-		if( $value instanceof \Aimeos\MShop\Common\Item\Address\Iface )
337
+		if ($value instanceof \Aimeos\MShop\Common\Item\Address\Iface)
338 338
 		{
339
-			$address->copyFrom( $value );
340
-			$this->get()->setAddress( $address, $type );
339
+			$address->copyFrom($value);
340
+			$this->get()->setAddress($address, $type);
341 341
 		}
342
-		else if( is_array( $value ) )
342
+		else if (is_array($value))
343 343
 		{
344
-			$this->setAddressFromArray( $address, $value );
345
-			$this->get()->setAddress( $address, $type );
344
+			$this->setAddressFromArray($address, $value);
345
+			$this->get()->setAddress($address, $type);
346 346
 		}
347
-		else if( $value === null )
347
+		else if ($value === null)
348 348
 		{
349
-			$this->get()->deleteAddress( $type );
349
+			$this->get()->deleteAddress($type);
350 350
 		}
351 351
 		else
352 352
 		{
353
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Invalid value for address type "%1$s"', $type ) );
353
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Invalid value for address type "%1$s"', $type));
354 354
 		}
355 355
 
356 356
 		$this->save();
@@ -366,42 +366,42 @@  discard block
 block discarded – undo
366 366
 	 * 	entered by the customer when choosing one of the delivery or payment options
367 367
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If there is no price to the service item attached
368 368
 	 */
369
-	public function setService( $type, $id, array $attributes = array() )
369
+	public function setService($type, $id, array $attributes = array())
370 370
 	{
371 371
 		$context = $this->getContext();
372 372
 
373
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $context, 'service' );
374
-		$serviceItem = $this->getDomainItem( 'service', 'service.id', $id, array( 'media', 'price', 'text' ) );
373
+		$serviceManager = \Aimeos\MShop\Factory::createManager($context, 'service');
374
+		$serviceItem = $this->getDomainItem('service', 'service.id', $id, array('media', 'price', 'text'));
375 375
 
376
-		$provider = $serviceManager->getProvider( $serviceItem );
377
-		$result = $provider->checkConfigFE( $attributes );
378
-		$unknown = array_diff_key( $attributes, $result );
376
+		$provider = $serviceManager->getProvider($serviceItem);
377
+		$result = $provider->checkConfigFE($attributes);
378
+		$unknown = array_diff_key($attributes, $result);
379 379
 
380
-		if( count( $unknown ) > 0 )
380
+		if (count($unknown) > 0)
381 381
 		{
382
-			$msg = sprintf( 'Unknown attributes "%1$s"', implode( '","', array_keys( $unknown ) ) );
383
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
382
+			$msg = sprintf('Unknown attributes "%1$s"', implode('","', array_keys($unknown)));
383
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
384 384
 		}
385 385
 
386
-		foreach( $result as $key => $value )
386
+		foreach ($result as $key => $value)
387 387
 		{
388
-			if( $value !== null ) {
389
-				throw new \Aimeos\Controller\Frontend\Basket\Exception( $value );
388
+			if ($value !== null) {
389
+				throw new \Aimeos\Controller\Frontend\Basket\Exception($value);
390 390
 			}
391 391
 		}
392 392
 
393
-		$orderBaseServiceManager = \Aimeos\MShop\Factory::createManager( $context, 'order/base/service' );
393
+		$orderBaseServiceManager = \Aimeos\MShop\Factory::createManager($context, 'order/base/service');
394 394
 		$orderServiceItem = $orderBaseServiceManager->createItem();
395
-		$orderServiceItem->copyFrom( $serviceItem );
395
+		$orderServiceItem->copyFrom($serviceItem);
396 396
 
397
-		$price = $provider->calcPrice( $this->get() );
397
+		$price = $provider->calcPrice($this->get());
398 398
 		// remove service rebate of original price
399
-		$price->setRebate( '0.00' );
400
-		$orderServiceItem->setPrice( $price );
399
+		$price->setRebate('0.00');
400
+		$orderServiceItem->setPrice($price);
401 401
 
402
-		$provider->setConfigFE( $orderServiceItem, $attributes );
402
+		$provider->setConfigFE($orderServiceItem, $attributes);
403 403
 
404
-		$this->get()->setService( $orderServiceItem, $type );
404
+		$this->get()->setService($orderServiceItem, $type);
405 405
 		$this->save();
406 406
 	}
407 407
 
@@ -414,18 +414,18 @@  discard block
 block discarded – undo
414 414
 	 * 	an address item.
415 415
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception
416 416
 	 */
417
-	protected function setAddressFromArray( \Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map )
417
+	protected function setAddressFromArray(\Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map)
418 418
 	{
419
-		foreach( $map as $key => $value ) {
420
-			$map[$key] = strip_tags( $value ); // prevent XSS
419
+		foreach ($map as $key => $value) {
420
+			$map[$key] = strip_tags($value); // prevent XSS
421 421
 		}
422 422
 
423
-		$errors = $address->fromArray( $map );
423
+		$errors = $address->fromArray($map);
424 424
 
425
-		if( count( $errors ) > 0 )
425
+		if (count($errors) > 0)
426 426
 		{
427
-			$msg = sprintf( 'Invalid address properties, please check your input' );
428
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg, 0, null, $errors );
427
+			$msg = sprintf('Invalid address properties, please check your input');
428
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg, 0, null, $errors);
429 429
 		}
430 430
 	}
431 431
 }
Please login to merge, or discard this patch.