aimeos / aimeos-core

lib/mwlib/src/MW/Criteria/MySQL.php

Spacing +2 added lines, -2 removed lines

@@ -44,9 +44,9 @@
 						$len = strlen( $part );
 
 						if( $len > 3 ) {
-							$str .= ' +' . $part . '*';
+							$str .= ' +'.$part.'*';
 						} else if( $len > 0 ) {
-							$str .= ' ' . $part . '*';
+							$str .= ' '.$part.'*';
 						}
 					}
 

lib/mwlib/src/MW/Criteria/PgSQL.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 					$regex = '/(\s|\&|\%|\?|\#|\=|\{|\}|\||\\\\|\~|\[|\]|\`|\^|\/|\-|\+|\>|\<|\(|\)|\*|\:|\"|\!|\§|\$|\'|\;|\.|\,|\@)+/';
 					$search = trim( preg_replace( $regex, ' ', $params[2] ) );
 
-					$params[2] = implode( ':* & ', explode( ' ', $search ) ) . ':*';
+					$params[2] = implode( ':* & ', explode( ' ', $search ) ).':*';
 				}
 				break;
 		}

lib/mshoplib/src/MShop/Service/Provider/Payment/DirectDebit.php

Spacing +3 added lines, -3 removed lines

@@ -78,10 +78,10 @@ 
 			$address = $basket->getAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
 
 			if( ( $fn = $address->getFirstname() ) !== '' && ( $ln = $address->getLastname() ) !== '' ) {
-				$feconfig['directdebit.accountowner']['default'] = $fn . ' ' . $ln;
+				$feconfig['directdebit.accountowner']['default'] = $fn.' '.$ln;
 			}
 		}
-		catch( \Aimeos\MShop\Order\Exception $e ) { ; } // If address isn't available
+		catch( \Aimeos\MShop\Order\Exception $e ) {; } // If address isn't available
 
 		return $this->getConfigItems( $feconfig );
 	}
@@ -119,7 +119,7 @@ 
 			$len = strlen( $value );
 			$xstr = ( $len > 3 ? str_repeat( 'X', $len - 3 ) : '' );
 
-			$attrItem->setValue( $xstr . substr( $value, -3 ) );
+			$attrItem->setValue( $xstr.substr( $value, -3 ) );
 			$orderServiceItem->setAttributeItem( $attrItem );
 		}
 	}

Braces +3 added lines, -1 removed lines

@@ -81,7 +81,9 @@
 				$feconfig['directdebit.accountowner']['default'] = $fn . ' ' . $ln;
 			}
 		}
-		catch( \Aimeos\MShop\Order\Exception $e ) { ; } // If address isn't available
+		catch( \Aimeos\MShop\Order\Exception $e )
+		{
+; } // If address isn't available
 
 		return $this->getConfigItems( $feconfig );
 	}

lib/mshoplib/src/MShop/Service/Provider/Decorator/Supplier.php

Braces +3 added lines, -1 removed lines

@@ -134,7 +134,9 @@
 				$feconfig['supplier.code']['default'] = array( $value => $address ) + $feconfig['supplier.code']['default'];
 			}
 		}
-		catch( \Aimeos\MShop\Order\Exception $e ) {} // If service isn't available
+		catch( \Aimeos\MShop\Order\Exception $e )
+		{
+} // If service isn't available
 
 		return array_merge( $this->getProvider()->getConfigFE( $basket ), $this->getConfigItems( $feconfig ) );
 	}

lib/mshoplib/src/MShop/Service/Provider/Payment/PayPalExpress.php

Doc Comments +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
 	 * Returns the configuration attribute definitions of the provider to generate a list of available fields and
 	 * rules for the value of each field in the administration interface.
 	 *
-	 * @return array List of attribute definitions implementing \Aimeos\MW\Common\Critera\Attribute\Iface
+	 * @return \Aimeos\MW\Criteria\Attribute\Iface[] List of attribute definitions implementing \Aimeos\MW\Common\Critera\Attribute\Iface
 	 */
 	public function getConfigBE()
 	{

Spacing +14 added lines, -14 removed lines

@@ -506,7 +506,7 @@ 
 
 		if( $rvals['ACK'] !== 'Success' )
 		{
-			$msg = 'PayPal Express: method = ' . $method . ', order ID = ' . $orderid . ', response = ' . print_r( $rvals, true );
+			$msg = 'PayPal Express: method = '.$method.', order ID = '.$orderid.', response = '.print_r( $rvals, true );
 			$this->getContext()->getLogger()->log( $msg, \Aimeos\MW\Logger\Base::WARN, 'core/service/payment' );
 
 			if( $rvals['ACK'] !== 'SuccessWithWarning' )
@@ -585,7 +585,7 @@ 
 						break;
 					}
 
-					$str = 'PayPal Express: order ID = ' . $invoice->getId() . ', PENDINGREASON = ' . $response['PENDINGREASON'];
+					$str = 'PayPal Express: order ID = '.$invoice->getId().', PENDINGREASON = '.$response['PENDINGREASON'];
 					$this->getContext()->getLogger()->log( $str, \Aimeos\MW\Logger\Base::INFO, 'core/service/payment' );
 				}
 
@@ -619,7 +619,7 @@ 
 				break;
 
 			default:
-				$str = 'PayPal Express: order ID = ' . $invoice->getId() . ', response = ' . print_r( $response, true );
+				$str = 'PayPal Express: order ID = '.$invoice->getId().', response = '.print_r( $response, true );
 				$this->getContext()->getLogger()->log( $str, \Aimeos\MW\Logger\Base::INFO, 'core/service/payment' );
 		}
 	}
@@ -647,14 +647,14 @@ 
 				/* setting up the address details */
 				$values['NOSHIPPING'] = $this->getConfigValue( array( 'paypalexpress.NoShipping' ), 1 );
 				$values['ADDROVERRIDE'] = $this->getConfigValue( array( 'paypalexpress.AddrOverride' ), 0 );
-				$values['PAYMENTREQUEST_0_SHIPTONAME'] = $orderAddressDelivery->getFirstName() . ' ' . $orderAddressDelivery->getLastName();
-				$values['PAYMENTREQUEST_0_SHIPTOSTREET'] = $orderAddressDelivery->getAddress1() . ' ' . $orderAddressDelivery->getAddress2() . ' ' . $orderAddressDelivery->getAddress3();
+				$values['PAYMENTREQUEST_0_SHIPTONAME'] = $orderAddressDelivery->getFirstName().' '.$orderAddressDelivery->getLastName();
+				$values['PAYMENTREQUEST_0_SHIPTOSTREET'] = $orderAddressDelivery->getAddress1().' '.$orderAddressDelivery->getAddress2().' '.$orderAddressDelivery->getAddress3();
 				$values['PAYMENTREQUEST_0_SHIPTOCITY'] = $orderAddressDelivery->getCity();
 				$values['PAYMENTREQUEST_0_SHIPTOSTATE'] = $orderAddressDelivery->getState();
 				$values['PAYMENTREQUEST_0_SHIPTOCOUNTRYCODE'] = $orderAddressDelivery->getCountryId();
 				$values['PAYMENTREQUEST_0_SHIPTOZIP'] = $orderAddressDelivery->getPostal();
 			}
-			catch( \Exception $e ) { ; } // If no address is available
+			catch( \Exception $e ) {; } // If no address is available
 		}
 
 
@@ -669,10 +669,10 @@ 
 				$deliveryPrice = clone $price;
 				$deliveryPrices = $this->addPrice( $deliveryPrices, $deliveryPrice->setValue( '0.00' ), $product->getQuantity() );
 
-				$values['L_PAYMENTREQUEST_0_NUMBER' . $lastPos] = $product->getId();
-				$values['L_PAYMENTREQUEST_0_NAME' . $lastPos] = $product->getName();
-				$values['L_PAYMENTREQUEST_0_QTY' . $lastPos] = $product->getQuantity();
-				$values['L_PAYMENTREQUEST_0_AMT' . $lastPos] = $this->getAmount( $price, false );
+				$values['L_PAYMENTREQUEST_0_NUMBER'.$lastPos] = $product->getId();
+				$values['L_PAYMENTREQUEST_0_NAME'.$lastPos] = $product->getName();
+				$values['L_PAYMENTREQUEST_0_QTY'.$lastPos] = $product->getQuantity();
+				$values['L_PAYMENTREQUEST_0_AMT'.$lastPos] = $this->getAmount( $price, false );
 			}
 		}
 
@@ -683,9 +683,9 @@ 
 			if( ( $paymentCosts = $this->getAmount( $price ) ) > '0.00' )
 			{
 				$lastPos++;
-				$values['L_PAYMENTREQUEST_0_NAME' . $lastPos] = $this->getContext()->getI18n()->dt( 'mshop', 'Payment costs' );
-				$values['L_PAYMENTREQUEST_0_QTY' . $lastPos] = '1';
-				$values['L_PAYMENTREQUEST_0_AMT' . $lastPos] = $paymentCosts;
+				$values['L_PAYMENTREQUEST_0_NAME'.$lastPos] = $this->getContext()->getI18n()->dt( 'mshop', 'Payment costs' );
+				$values['L_PAYMENTREQUEST_0_QTY'.$lastPos] = '1';
+				$values['L_PAYMENTREQUEST_0_AMT'.$lastPos] = $paymentCosts;
 			}
 
 			try
@@ -703,7 +703,7 @@ 
 				$values['L_SHIPPINGOPTIONNAME0'] = $orderServiceDeliveryItem->getName();
 				$values['L_SHIPPINGOPTIONISDEFAULT0'] = 'true';
 			}
-			catch( \Exception $e ) { ; } // If no delivery service is available
+			catch( \Exception $e ) {; } // If no delivery service is available
 		}
 
 

Braces +6 added lines, -2 removed lines

@@ -654,7 +654,9 @@ 
 				$values['PAYMENTREQUEST_0_SHIPTOCOUNTRYCODE'] = $orderAddressDelivery->getCountryId();
 				$values['PAYMENTREQUEST_0_SHIPTOZIP'] = $orderAddressDelivery->getPostal();
 			}
-			catch( \Exception $e ) { ; } // If no address is available
+			catch( \Exception $e )
+			{
+; } // If no address is available
 		}
 
 
@@ -703,7 +705,9 @@ 
 				$values['L_SHIPPINGOPTIONNAME0'] = $orderServiceDeliveryItem->getName();
 				$values['L_SHIPPINGOPTIONISDEFAULT0'] = 'true';
 			}
-			catch( \Exception $e ) { ; } // If no delivery service is available
+			catch( \Exception $e )
+			{
+; } // If no delivery service is available
 		}
 
 

lib/mshoplib/setup/default/DemoAddSupplierData.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 		if( $value === '1' )
 		{
 			$ds = DIRECTORY_SEPARATOR;
-			$path = __DIR__ . $ds . 'data' . $ds . 'demo-supplier.php';
+			$path = __DIR__.$ds.'data'.$ds.'demo-supplier.php';
 
 			if( ( $data = include( $path ) ) == false ) {
 				throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for supplier domain', $path ) );

Braces +1 added lines, -2 removed lines

@@ -75,8 +75,7 @@
 			$this->saveCustomerItems( $data );
 
 			$this->status( 'added' );
-		}
-		else
+		} else
 		{
 			$this->status( 'removed' );
 		}

lib/mshoplib/setup/unitperf/ServiceAddBasePerfData.php

Spacing +4 added lines, -4 removed lines

@@ -80,11 +80,11 @@ 
 
 		for( $i = 0; $i < 100; $i++ )
 		{
-			$code = 'perf-pay-' . str_pad( $i, 3, '0', STR_PAD_LEFT );
+			$code = 'perf-pay-'.str_pad( $i, 3, '0', STR_PAD_LEFT );
 
 			$item->setId( null );
 			$item->setCode( $code );
-			$item->setLabel( 'Payment service ' . $code );
+			$item->setLabel( 'Payment service '.$code );
 			$item->setPosition( $i );
 
 			$manager->saveItem( $item, false );
@@ -114,11 +114,11 @@ 
 
 		for( $i = 0; $i < 100; $i++ )
 		{
-			$code = 'perf-ship-' . str_pad( $i, 3, '0', STR_PAD_LEFT );
+			$code = 'perf-ship-'.str_pad( $i, 3, '0', STR_PAD_LEFT );
 
 			$item->setId( null );
 			$item->setCode( $code );
-			$item->setLabel( 'Delivery service ' . $code );
+			$item->setLabel( 'Delivery service '.$code );
 			$item->setPosition( $i );
 
 			$manager->saveItem( $item, false );

lib/mshoplib/tests/MShop/Coupon/Provider/Decorator/CategoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 		$this->couponItem = \Aimeos\MShop\Coupon\Manager\Factory::createManager( $this->context )->createItem();
 
 		$provider = new \Aimeos\MShop\Coupon\Provider\Example( $this->context, $this->couponItem, 'abcd' );
-		$this->object = new \Aimeos\MShop\Coupon\Provider\Decorator\Category( $provider, $this->context, $this->couponItem, 'abcd');
+		$this->object = new \Aimeos\MShop\Coupon\Provider\Decorator\Category( $provider, $this->context, $this->couponItem, 'abcd' );
 		$this->object->setObject( $this->object );
 
 		$priceManager = \Aimeos\MShop\Factory::createManager( $this->context, 'price' );

lib/mshoplib/src/MShop/Customer/Manager/Base.php

Spacing +4 added lines, -4 removed lines

@@ -123,7 +123,7 @@ 
 		$search->setSlice( 0, 0x7fffffff );
 
 		foreach( $manager->searchItems( $search ) as $listid => $listItem ) {
-			$listMap[ $listItem->getRefId() ] = $listid;
+			$listMap[$listItem->getRefId()] = $listid;
 		}
 
 
@@ -138,7 +138,7 @@ 
 			$listItem->setTypeId( $typeId );
 			$listItem->setStatus( 1 );
 
-			$pos = count( $listMap ) ;
+			$pos = count( $listMap );
 
 			foreach( $item->getGroups() as $gid )
 			{
@@ -253,12 +253,12 @@ 
 
 		if( ctype_alnum( $name ) === false )
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\MShop\\Common\\Item\\Helper\\Password\\' . $name : '<not a string>';
+			$classname = is_string( $name ) ? '\\Aimeos\\MShop\\Common\\Item\\Helper\\Password\\'.$name : '<not a string>';
 			throw new \Aimeos\MShop\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 		}
 
 		$iface = '\\Aimeos\\MShop\\Common\\Item\\Helper\\Password\\Iface';
-		$classname = '\\Aimeos\\MShop\\Common\\Item\\Helper\\Password\\' . $name;
+		$classname = '\\Aimeos\\MShop\\Common\\Item\\Helper\\Password\\'.$name;
 
 		if( class_exists( $classname ) === false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'Class "%1$s" not available', $classname ) );