aimeos / ai-laravel

lib/custom/src/MW/View/Helper/Url/Laravel5.php

Spacing +9 added lines, -9 removed lines

@@ -32,9 +32,9 @@ 
 	 * @param \Illuminate\Contracts\Routing\UrlGenerator $builder Laravel URL builder object
 	 * @param array Associative list of fixed parameters that should be available for all routes
 	 */
-	public function __construct( \Aimeos\MW\View\Iface $view, \Illuminate\Contracts\Routing\UrlGenerator $builder, array $fixed )
+	public function __construct(\Aimeos\MW\View\Iface $view, \Illuminate\Contracts\Routing\UrlGenerator $builder, array $fixed)
 	{
-		parent::__construct( $view );
+		parent::__construct($view);
 
 		$this->builder = $builder;
 		$this->fixed = $fixed;
@@ -52,13 +52,13 @@ 
 	 * @param array $config Additional configuration parameter per URL
 	 * @return string Complete URL that can be used in the template
 	 */
-	public function transform( $target = null, $controller = null, $action = null, array $params = [], array $trailing = [], array $config = [] )
+	public function transform($target = null, $controller = null, $action = null, array $params = [], array $trailing = [], array $config = [])
 	{
-		$params = $this->sanitize( $params );
-		$values = $this->getValues( $config );
-		$fragment = ( !empty( $trailing ) ? '#' . implode( '/', $trailing ) : '' );
+		$params = $this->sanitize($params);
+		$values = $this->getValues($config);
+		$fragment = (!empty($trailing) ? '#'.implode('/', $trailing) : '');
 
-		return $this->builder->route( $target, $params + $this->fixed, $values['absoluteUri'] ) . $fragment;
+		return $this->builder->route($target, $params + $this->fixed, $values['absoluteUri']).$fragment;
 	}
 
 
@@ -68,13 +68,13 @@ 
 	 * @param array $config Associative list of key/value pairs
 	 * @return array Associative list of sanitized key/value pairs
 	 */
-	protected function getValues( array $config )
+	protected function getValues(array $config)
 	{
 		$values = array(
 			'absoluteUri' => false,
 		);
 
-		if( isset( $config['absoluteUri'] ) ) {
+		if (isset($config['absoluteUri'])) {
 			$values['absoluteUri'] = (bool) $config['absoluteUri'];
 		}
 

lib/custom/tests/MW/View/Helper/Url/Laravel5Test.php

Spacing +12 added lines, -12 removed lines

@@ -17,37 +17,37 @@
 
 	protected function setUp()
 	{
-		if( !interface_exists( '\\Illuminate\\Contracts\\Routing\\UrlGenerator' ) ) {
-			$this->markTestSkipped( '\\Illuminate\\Contracts\\Routing\\UrlGenerator is not available' );
+		if (!interface_exists('\\Illuminate\\Contracts\\Routing\\UrlGenerator')) {
+			$this->markTestSkipped('\\Illuminate\\Contracts\\Routing\\UrlGenerator is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Routing\\UrlGenerator' )->getMock();
-		$this->object = new \Aimeos\MW\View\Helper\Url\Laravel5( $view, $this->mock, [] );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Routing\\UrlGenerator')->getMock();
+		$this->object = new \Aimeos\MW\View\Helper\Url\Laravel5($view, $this->mock, []);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testTransform()
 	{
-		$this->mock->expects( $this->once() )->method( 'route' )
-			->with( $this->equalTo( 'route'), $this->equalTo( array( 'key' => 'value' ) ), $this->equalTo( false ) );
+		$this->mock->expects($this->once())->method('route')
+			->with($this->equalTo('route'), $this->equalTo(array('key' => 'value')), $this->equalTo(false));
 
-		$this->object->transform( 'route', 'catalog', 'lists', array( 'key' => 'value' ) );
+		$this->object->transform('route', 'catalog', 'lists', array('key' => 'value'));
 	}
 
 
 	public function testTransformAbsolute()
 	{
-		$this->mock->expects( $this->once() )->method( 'route' )
-			->with( $this->equalTo( 'route'), $this->equalTo( [] ), $this->equalTo( true ) );
+		$this->mock->expects($this->once())->method('route')
+			->with($this->equalTo('route'), $this->equalTo([]), $this->equalTo(true));
 
-		$options = array( 'absoluteUri' => true );
-		$this->object->transform( 'route', 'catalog', 'lists', [], [], $options );
+		$options = array('absoluteUri' => true);
+		$this->object->transform('route', 'catalog', 'lists', [], [], $options);
 	}
 }

lib/custom/tests/MW/Session/Laravel5Test.php

Spacing +10 added lines, -10 removed lines

@@ -17,32 +17,32 @@
 
 	protected function setUp()
 	{
-		if( interface_exists( '\\Illuminate\\Session\\Store' ) === false ) {
-			$this->markTestSkipped( 'Class \\Illuminate\\Session\\Store not found' );
+		if (interface_exists('\\Illuminate\\Session\\Store') === false) {
+			$this->markTestSkipped('Class \\Illuminate\\Session\\Store not found');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Session\\Store' )->getMock();
-		$this->object = new \Aimeos\MW\Session\Laravel5( $this->mock );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Session\\Store')->getMock();
+		$this->object = new \Aimeos\MW\Session\Laravel5($this->mock);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testGetDefault()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )->with( $this->equalTo( 'notexist' ) );
-		$this->object->get( 'notexist' );
+		$this->mock->expects($this->once())->method('get')->with($this->equalTo('notexist'));
+		$this->object->get('notexist');
 	}
 
 
 	public function testSet()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->with( $this->equalTo( 'key' ), $this->equalTo( 'value' ) );
-		$this->object->set( 'key', 'value' );
+		$this->mock->expects($this->once())->method('put')
+			->with($this->equalTo('key'), $this->equalTo('value'));
+		$this->object->set('key', 'value');
 	}
 }

lib/custom/tests/MW/Logger/Laravel5Test.php

Spacing +21 added lines, -21 removed lines

@@ -17,53 +17,53 @@
 
 	protected function setUp()
 	{
-		if( interface_exists( '\\Illuminate\\Contracts\\Logging\\Log' ) === false ) {
-			$this->markTestSkipped( 'Class \\Illuminate\\Contracts\\Logging\\Log not found' );
+		if (interface_exists('\\Illuminate\\Contracts\\Logging\\Log') === false) {
+			$this->markTestSkipped('Class \\Illuminate\\Contracts\\Logging\\Log not found');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Logging\\Log' )->getMock();
-		$this->object = new \Aimeos\MW\Logger\Laravel5( $this->mock );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Logging\\Log')->getMock();
+		$this->object = new \Aimeos\MW\Logger\Laravel5($this->mock);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->mock, $this->object );
+		unset($this->mock, $this->object);
 	}
 
 
 	public function testLog()
 	{
-		$this->mock->expects( $this->once() )->method( 'log' )
-			->with( $this->equalTo( 'msg' ), $this->equalTo( 'error' ) );
-		$this->object->log( 'msg' );
+		$this->mock->expects($this->once())->method('log')
+			->with($this->equalTo('msg'), $this->equalTo('error'));
+		$this->object->log('msg');
 	}
 
 
 	public function testNonScalarLog()
 	{
-		$this->mock->expects( $this->once() )->method( 'log' )
-			->with( $this->equalTo( '["error","error2",2]' ), $this->equalTo( 'error' ) );
-		$this->object->log( array( 'error', 'error2', 2 ) );
+		$this->mock->expects($this->once())->method('log')
+			->with($this->equalTo('["error","error2",2]'), $this->equalTo('error'));
+		$this->object->log(array('error', 'error2', 2));
 	}
 
 
 	public function testLogDebug()
 	{
-		$this->object->log( 'emergency', \Aimeos\MW\Logger\Base::EMERG );
-		$this->object->log( 'alert', \Aimeos\MW\Logger\Base::ALERT );
-		$this->object->log( 'critical', \Aimeos\MW\Logger\Base::CRIT );
-		$this->object->log( 'error', \Aimeos\MW\Logger\Base::ERR );
-		$this->object->log( 'warning', \Aimeos\MW\Logger\Base::WARN );
-		$this->object->log( 'notice', \Aimeos\MW\Logger\Base::NOTICE );
-		$this->object->log( 'info', \Aimeos\MW\Logger\Base::INFO );
-		$this->object->log( 'debug', \Aimeos\MW\Logger\Base::DEBUG );
+		$this->object->log('emergency', \Aimeos\MW\Logger\Base::EMERG);
+		$this->object->log('alert', \Aimeos\MW\Logger\Base::ALERT);
+		$this->object->log('critical', \Aimeos\MW\Logger\Base::CRIT);
+		$this->object->log('error', \Aimeos\MW\Logger\Base::ERR);
+		$this->object->log('warning', \Aimeos\MW\Logger\Base::WARN);
+		$this->object->log('notice', \Aimeos\MW\Logger\Base::NOTICE);
+		$this->object->log('info', \Aimeos\MW\Logger\Base::INFO);
+		$this->object->log('debug', \Aimeos\MW\Logger\Base::DEBUG);
 	}
 
 
 	public function testBadPriority()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Logger\\Exception' );
-		$this->object->log( 'error', -1 );
+		$this->setExpectedException('\\Aimeos\\MW\\Logger\\Exception');
+		$this->object->log('error', -1);
 	}
 }

lib/custom/setup/CustomerChangeAddressRefidParentidLaravel.php

Spacing +11 added lines, -11 removed lines

@@ -40,7 +40,7 @@ 
 	 */
 	public function getPostDependencies()
 	{
-		return array( 'TablesCreateLaravel' );
+		return array('TablesCreateLaravel');
 	}
 
 
@@ -49,7 +49,7 @@ 
 	 */
 	public function migrate()
 	{
-		$this->process( $this->mysql );
+		$this->process($this->mysql);
 	}
 
 
@@ -58,22 +58,22 @@ 
 	 *
 	 * array string $stmts List of SQL statements for changing the columns
 	 */
-	protected function process( array $stmts )
+	protected function process(array $stmts)
 	{
 		$table = 'users_address';
-		$this->msg( sprintf( 'Rename "refid" to "parentid" in table "%1$s"', $table ), 0 ); $this->status( '' );
+		$this->msg(sprintf('Rename "refid" to "parentid" in table "%1$s"', $table), 0); $this->status('');
 
-		foreach( $stmts as $column => $stmts )
+		foreach ($stmts as $column => $stmts)
 		{
-			$this->msg( sprintf( 'Checking column "%1$s"', $column ), 1 );
+			$this->msg(sprintf('Checking column "%1$s"', $column), 1);
 
-			if( $this->schema->tableExists( $table )
-				&& $this->schema->columnExists( $table, $column ) === true
+			if ($this->schema->tableExists($table)
+				&& $this->schema->columnExists($table, $column) === true
 			) {
-				$this->executeList( $stmts );
-				$this->status( 'done' );
+				$this->executeList($stmts);
+				$this->status('done');
 			} else {
-				$this->status( 'OK' );
+				$this->status('OK');
 			}
 		}
 	}

lib/custom/src/MShop/Customer/Manager/Laravel.php

Spacing +73 added lines, -73 removed lines

@@ -241,11 +241,11 @@ 
 	 *
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, array( 'address', 'lists' ) ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, array('address', 'lists')) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 	}
 
@@ -257,7 +257,7 @@ 
 	 */
 	public function createItem()
 	{
-		return $this->createItemBase( ['customer.siteid' => $this->getContext()->getLocale()->getSiteId()] );
+		return $this->createItemBase(['customer.siteid' => $this->getContext()->getLocale()->getSiteId()]);
 	}
 
 
@@ -266,10 +266,10 @@ 
 	 *
 	 * @param array $ids List of IDs
 	 */
-	public function deleteItems( array $ids )
+	public function deleteItems(array $ids)
 	{
 		$path = 'mshop/customer/manager/laravel/delete';
-		$this->deleteItemsBase( $ids, $path, false );
+		$this->deleteItemsBase($ids, $path, false);
 	}
 
 
@@ -279,11 +279,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array List of attribute items implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, array( 'address', 'lists' ), $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, array('address', 'lists'), $withsub);
 	}
 
 
@@ -294,29 +294,29 @@ 
 	 * @param boolean $fetch True if the new ID should be returned in the item
 	 * @return \Aimeos\MShop\Common\Item\Iface $item Updated item including the generated ID
 	 */
-	public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true )
+	public function saveItem(\Aimeos\MShop\Common\Item\Iface $item, $fetch = true)
 	{
 		$iface = '\\Aimeos\\MShop\\Customer\\Item\\Iface';
-		if( !( $item instanceof $iface ) ) {
-			throw new \Aimeos\MShop\Customer\Exception( sprintf( 'Object is not of required type "%1$s"', $iface ) );
+		if (!($item instanceof $iface)) {
+			throw new \Aimeos\MShop\Customer\Exception(sprintf('Object is not of required type "%1$s"', $iface));
 		}
 
-		if( !$item->isModified() ) {
-			return $this->saveRefItems( $item, 'customer' );
+		if (!$item->isModified()) {
+			return $this->saveRefItems($item, 'customer');
 		}
 
 		$context = $this->getContext();
 		$dbm = $context->getDatabaseManager();
 		$dbname = $this->getResourceName();
-		$conn = $dbm->acquire( $dbname );
+		$conn = $dbm->acquire($dbname);
 
 		try
 		{
 			$id = $item->getId();
-			$date = date( 'Y-m-d H:i:s' );
+			$date = date('Y-m-d H:i:s');
 			$billingAddress = $item->getPaymentAddress();
 
-			if( $id === null )
+			if ($id === null)
 			{
 				/** mshop/customer/manager/laravel/insert
 				 * Inserts a new customer record into the database table
@@ -380,48 +380,48 @@ 
 				$path = 'mshop/customer/manager/laravel/update';
 			}
 
-			$stmt = $this->getCachedStatement( $conn, $path );
-
-			$stmt->bind( 1, $context->getLocale()->getSiteId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-			$stmt->bind( 2, $item->getCode() );
-			$stmt->bind( 3, $billingAddress->getCompany() );
-			$stmt->bind( 4, $billingAddress->getVatID() );
-			$stmt->bind( 5, $billingAddress->getSalutation() );
-			$stmt->bind( 6, $billingAddress->getTitle() );
-			$stmt->bind( 7, $billingAddress->getFirstname() );
-			$stmt->bind( 8, $billingAddress->getLastname() );
-			$stmt->bind( 9, $billingAddress->getAddress1() );
-			$stmt->bind( 10, $billingAddress->getAddress2() );
-			$stmt->bind( 11, $billingAddress->getAddress3() );
-			$stmt->bind( 12, $billingAddress->getPostal() );
-			$stmt->bind( 13, $billingAddress->getCity() );
-			$stmt->bind( 14, $billingAddress->getState() );
-			$stmt->bind( 15, $billingAddress->getCountryId() );
-			$stmt->bind( 16, $billingAddress->getLanguageId() );
-			$stmt->bind( 17, $billingAddress->getTelephone() );
-			$stmt->bind( 18, $billingAddress->getTelefax() );
-			$stmt->bind( 19, $billingAddress->getWebsite() );
-			$stmt->bind( 20, $billingAddress->getEmail() );
-			$stmt->bind( 21, $billingAddress->getLongitude() );
-			$stmt->bind( 22, $billingAddress->getLatitude() );
-			$stmt->bind( 23, $item->getLabel() );
-			$stmt->bind( 24, $item->getBirthday() );
-			$stmt->bind( 25, $item->getStatus(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-			$stmt->bind( 26, $item->getDateVerified() );
-			$stmt->bind( 27, $item->getPassword() );
-			$stmt->bind( 28, $date ); // Modification time
-			$stmt->bind( 29, $context->getEditor() );
-
-			if( $id !== null ) {
-				$stmt->bind( 30, $id, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-				$item->setId( $id );
+			$stmt = $this->getCachedStatement($conn, $path);
+
+			$stmt->bind(1, $context->getLocale()->getSiteId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+			$stmt->bind(2, $item->getCode());
+			$stmt->bind(3, $billingAddress->getCompany());
+			$stmt->bind(4, $billingAddress->getVatID());
+			$stmt->bind(5, $billingAddress->getSalutation());
+			$stmt->bind(6, $billingAddress->getTitle());
+			$stmt->bind(7, $billingAddress->getFirstname());
+			$stmt->bind(8, $billingAddress->getLastname());
+			$stmt->bind(9, $billingAddress->getAddress1());
+			$stmt->bind(10, $billingAddress->getAddress2());
+			$stmt->bind(11, $billingAddress->getAddress3());
+			$stmt->bind(12, $billingAddress->getPostal());
+			$stmt->bind(13, $billingAddress->getCity());
+			$stmt->bind(14, $billingAddress->getState());
+			$stmt->bind(15, $billingAddress->getCountryId());
+			$stmt->bind(16, $billingAddress->getLanguageId());
+			$stmt->bind(17, $billingAddress->getTelephone());
+			$stmt->bind(18, $billingAddress->getTelefax());
+			$stmt->bind(19, $billingAddress->getWebsite());
+			$stmt->bind(20, $billingAddress->getEmail());
+			$stmt->bind(21, $billingAddress->getLongitude());
+			$stmt->bind(22, $billingAddress->getLatitude());
+			$stmt->bind(23, $item->getLabel());
+			$stmt->bind(24, $item->getBirthday());
+			$stmt->bind(25, $item->getStatus(), \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+			$stmt->bind(26, $item->getDateVerified());
+			$stmt->bind(27, $item->getPassword());
+			$stmt->bind(28, $date); // Modification time
+			$stmt->bind(29, $context->getEditor());
+
+			if ($id !== null) {
+				$stmt->bind(30, $id, \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+				$item->setId($id);
 			} else {
-				$stmt->bind( 30, $date ); // Creation time
+				$stmt->bind(30, $date); // Creation time
 			}
 
 			$stmt->execute()->finish();
 
-			if( $id === null && $fetch === true )
+			if ($id === null && $fetch === true)
 			{
 				/** mshop/customer/manager/laravel/newid
 				 * Retrieves the ID generated by the database when inserting a new record
@@ -454,20 +454,20 @@ 
 				 * @see mshop/customer/manager/laravel/count
 				 */
 				$path = 'mshop/customer/manager/laravel/newid';
-				$item->setId( $this->newId( $conn, $path ) );
+				$item->setId($this->newId($conn, $path));
 			}
 
-			$dbm->release( $conn, $dbname );
+			$dbm->release($conn, $dbname);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$dbm->release( $conn, $dbname );
+			$dbm->release($conn, $dbname);
 			throw $e;
 		}
 
-		$this->addGroups( $item );
+		$this->addGroups($item);
 
-		return $this->saveRefItems( $item, 'customer' );
+		return $this->saveRefItems($item, 'customer');
 	}
 
 
@@ -479,11 +479,11 @@ 
 	 * @return array List of items implementing \Aimeos\MShop\Customer\Item\Iface
 	 * @throws \Aimeos\MShop\Customer\Exception If creating items failed
 	 */
-	public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null )
+	public function searchItems(\Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null)
 	{
 		$dbm = $this->getContext()->getDatabaseManager();
 		$dbname = $this->getResourceName();
-		$conn = $dbm->acquire( $dbname );
+		$conn = $dbm->acquire($dbname);
 		$map = [];
 
 		try
@@ -491,27 +491,27 @@ 
 			$level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL;
 			$cfgPathSearch = 'mshop/customer/manager/laravel/search';
 			$cfgPathCount = 'mshop/customer/manager/laravel/count';
-			$required = array( 'customer' );
+			$required = array('customer');
 
-			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
-			while( ( $row = $results->fetch() ) !== false ) {
-				$map[ $row['customer.id'] ] = $row;
+			$results = $this->searchItemsBase($conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level);
+			while (($row = $results->fetch()) !== false) {
+				$map[$row['customer.id']] = $row;
 			}
 
-			$dbm->release( $conn, $dbname );
+			$dbm->release($conn, $dbname);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$dbm->release( $conn, $dbname  );
+			$dbm->release($conn, $dbname);
 			throw $e;
 		}
 
 		$addrItems = [];
-		if( in_array( 'customer/address', $ref, true ) ) {
-			$addrItems = $this->getAddressItems( array_keys( $map ) );
+		if (in_array('customer/address', $ref, true)) {
+			$addrItems = $this->getAddressItems(array_keys($map));
 		}
 
-		return $this->buildItems( $map, $ref, 'customer', $addrItems );
+		return $this->buildItems($map, $ref, 'customer', $addrItems);
 	}
 
 
@@ -522,8 +522,8 @@ 
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
 	 * @return mixed Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', $manager, ($name === null ? 'Laravel' : $name));
 	}
 }

lib/custom/src/MShop/Customer/Manager/Property/Type/Laravel.php

Spacing +14 added lines, -14 removed lines

@@ -25,7 +25,7 @@ 
 		'customer.property.type.id' => array(
 			'code' => 'customer.property.type.id',
 			'internalcode' => 'lvuprty."id"',
-			'internaldeps' => array( 'LEFT JOIN "users_property_type" AS lvuprty ON ( lvupr."typeid" = lvuprty."id" )' ),
+			'internaldeps' => array('LEFT JOIN "users_property_type" AS lvuprty ON ( lvupr."typeid" = lvuprty."id" )'),
 			'label' => 'Property type ID',
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -99,10 +99,10 @@ 
 	 *
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context object
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context)
 	{
-		parent::__construct( $context );
-		$this->setResourceName( 'db-customer' );
+		parent::__construct($context);
+		$this->setResourceName('db-customer');
 	}
 
 
@@ -111,14 +111,14 @@ 
 	 *
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/property/type/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/property/type/laravel/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/type/laravel/delete');
 	}
 
 
@@ -128,11 +128,11 @@ 
 	 * @param boolean $withsub Return also the resource type of sub-managers if true
 	 * @return array Type of the manager and submanagers, subtypes are separated by slashes
 	 */
-	public function getResourceType( $withsub = true )
+	public function getResourceType($withsub = true)
 	{
 		$path = 'mshop/customer/manager/property/type/submanagers';
 
-		return $this->getResourceTypeBase( 'customer/property/type', $path, [], $withsub );
+		return $this->getResourceTypeBase('customer/property/type', $path, [], $withsub);
 	}
 
 
@@ -142,11 +142,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array List of attribute items implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/property/type/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, [], $withsub);
 	}
 
 
@@ -157,9 +157,9 @@ 
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
 	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g types, lists etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', 'property/type/' . $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', 'property/type/'.$manager, ($name === null ? 'Laravel' : $name));
 	}
 
 

lib/custom/src/MShop/Customer/Manager/Property/Laravel.php

Spacing +14 added lines, -14 removed lines

@@ -25,7 +25,7 @@ 
 		'customer.property.id' => array(
 			'code' => 'customer.property.id',
 			'internalcode' => 'lvupr."id"',
-			'internaldeps'=>array( 'LEFT JOIN "users_property" AS lvupr ON ( lvupr."parentid" = fos."id" )' ),
+			'internaldeps'=>array('LEFT JOIN "users_property" AS lvupr ON ( lvupr."parentid" = fos."id" )'),
 			'label' => 'Property ID',
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -101,10 +101,10 @@ 
 	 *
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context object
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context)
 	{
-		parent::__construct( $context );
-		$this->setResourceName( 'db-customer' );
+		parent::__construct($context);
+		$this->setResourceName('db-customer');
 	}
 
 
@@ -113,14 +113,14 @@ 
 	 *
 	 * @param integer[] $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/property/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/property/laravel/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/laravel/delete');
 	}
 
 
@@ -130,11 +130,11 @@ 
 	 * @param boolean $withsub Return also the resource type of sub-managers if true
 	 * @return array Type of the manager and submanagers, subtypes are separated by slashes
 	 */
-	public function getResourceType( $withsub = true )
+	public function getResourceType($withsub = true)
 	{
 		$path = 'mshop/customer/manager/property/submanagers';
 
-		return $this->getResourceTypeBase( 'customer/property', $path, array( 'type' ), $withsub );
+		return $this->getResourceTypeBase('customer/property', $path, array('type'), $withsub);
 	}
 
 
@@ -144,11 +144,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/property/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, array( 'type' ), $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, array('type'), $withsub);
 	}
 
 
@@ -160,9 +160,9 @@ 
 	 * configuration (or Default) if null
 	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g property types, property lists etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', 'property/' . $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', 'property/'.$manager, ($name === null ? 'Laravel' : $name));
 	}
 
 

lib/custom/tests/MShop/Customer/Manager/Property/LaravelTest.php

Spacing +92 added lines, -92 removed lines

@@ -20,91 +20,91 @@ 
 		$context = \TestHelper::getContext();
 		$this->editor = $context->getEditor();
 
-		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( $context, 'Laravel' );
-		$this->object = $manager->getSubManager( 'property', 'Laravel' );
+		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager($context, 'Laravel');
+		$this->object = $manager->getSubManager('property', 'Laravel');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testCreateItem()
 	{
 		$item = $this->object->createItem();
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Property\\Iface', $item );
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Property\\Iface', $item);
 	}
 
 
 	public function testSaveInvalid()
 	{
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$this->object->saveItem( new \Aimeos\MShop\Locale\Item\Standard() );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$this->object->saveItem(new \Aimeos\MShop\Locale\Item\Standard());
 	}
 
 
 	public function testSaveUpdateDeleteItem()
 	{
 		$search = $this->object->createSearch();
-		$search->setConditions( $search->compare( '==', 'customer.property.editor', $this->editor ) );
-		$results = $this->object->searchItems( $search );
+		$search->setConditions($search->compare('==', 'customer.property.editor', $this->editor));
+		$results = $this->object->searchItems($search);
 
-		if( ( $item = reset($results) ) === false ) {
-			throw new \RuntimeException( 'No property item found' );
+		if (($item = reset($results)) === false) {
+			throw new \RuntimeException('No property item found');
 		}
 
 		$item->setId(null);
-		$item->setLanguageId( 'en' );
-		$resultSaved = $this->object->saveItem( $item );
-		$itemSaved = $this->object->getItem( $item->getId() );
+		$item->setLanguageId('en');
+		$resultSaved = $this->object->saveItem($item);
+		$itemSaved = $this->object->getItem($item->getId());
 
 		$itemExp = clone $itemSaved;
-		$itemExp->setValue( 'unittest' );
-		$resultUpd = $this->object->saveItem( $itemExp );
-		$itemUpd = $this->object->getItem( $itemExp->getId() );
+		$itemExp->setValue('unittest');
+		$resultUpd = $this->object->saveItem($itemExp);
+		$itemUpd = $this->object->getItem($itemExp->getId());
 
-		$this->object->deleteItem( $itemSaved->getId() );
+		$this->object->deleteItem($itemSaved->getId());
 
 		$context = \TestHelper::getContext();
 
-		$this->assertTrue( $item->getId() !== null );
-		$this->assertTrue( $itemSaved->getType() !== null );
-		$this->assertEquals( $item->getId(), $itemSaved->getId() );
-		$this->assertEquals( $item->getParentId(), $itemSaved->getParentId() );
-		$this->assertEquals( $item->getSiteId(), $itemSaved->getSiteId() );
-		$this->assertEquals( $item->getTypeId(), $itemSaved->getTypeId() );
-		$this->assertEquals( $item->getLanguageId(), $itemSaved->getLanguageId() );
-		$this->assertEquals( $item->getValue(), $itemSaved->getValue() );
-
-		$this->assertEquals( $context->getEditor(), $itemSaved->getEditor() );
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated() );
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified() );
-
-		$this->assertTrue( $itemUpd->getType() !== null );
-		$this->assertEquals( $itemExp->getId(), $itemUpd->getId() );
-		$this->assertEquals( $itemExp->getParentId(), $itemUpd->getParentId() );
-		$this->assertEquals( $itemExp->getSiteId(), $itemUpd->getSiteId() );
-		$this->assertEquals( $itemExp->getTypeId(), $itemUpd->getTypeId() );
-		$this->assertEquals( $itemExp->getLanguageId(), $itemUpd->getLanguageId() );
-		$this->assertEquals( $itemExp->getValue(), $itemUpd->getValue() );
-
-		$this->assertEquals( $context->getEditor(), $itemUpd->getEditor() );
-		$this->assertEquals( $itemExp->getTimeCreated(), $itemUpd->getTimeCreated() );
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemUpd->getTimeModified() );
-
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Item\Iface', $resultSaved );
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Item\Iface', $resultUpd );
-
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$this->object->getItem( $itemSaved->getId() );
+		$this->assertTrue($item->getId() !== null);
+		$this->assertTrue($itemSaved->getType() !== null);
+		$this->assertEquals($item->getId(), $itemSaved->getId());
+		$this->assertEquals($item->getParentId(), $itemSaved->getParentId());
+		$this->assertEquals($item->getSiteId(), $itemSaved->getSiteId());
+		$this->assertEquals($item->getTypeId(), $itemSaved->getTypeId());
+		$this->assertEquals($item->getLanguageId(), $itemSaved->getLanguageId());
+		$this->assertEquals($item->getValue(), $itemSaved->getValue());
+
+		$this->assertEquals($context->getEditor(), $itemSaved->getEditor());
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated());
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified());
+
+		$this->assertTrue($itemUpd->getType() !== null);
+		$this->assertEquals($itemExp->getId(), $itemUpd->getId());
+		$this->assertEquals($itemExp->getParentId(), $itemUpd->getParentId());
+		$this->assertEquals($itemExp->getSiteId(), $itemUpd->getSiteId());
+		$this->assertEquals($itemExp->getTypeId(), $itemUpd->getTypeId());
+		$this->assertEquals($itemExp->getLanguageId(), $itemUpd->getLanguageId());
+		$this->assertEquals($itemExp->getValue(), $itemUpd->getValue());
+
+		$this->assertEquals($context->getEditor(), $itemUpd->getEditor());
+		$this->assertEquals($itemExp->getTimeCreated(), $itemUpd->getTimeCreated());
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemUpd->getTimeModified());
+
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Item\Iface', $resultSaved);
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Item\Iface', $resultUpd);
+
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getItem($itemSaved->getId());
 	}
 
 
@@ -112,19 +112,19 @@ 
 	{
 		$search = $this->object->createSearch();
 		$conditions = array(
-			$search->compare( '~=', 'customer.property.value', '1'),
-			$search->compare( '==', 'customer.property.editor', $this->editor )
+			$search->compare('~=', 'customer.property.value', '1'),
+			$search->compare('==', 'customer.property.editor', $this->editor)
 		);
-		$search->setConditions( $search->combine( '&&', $conditions ) );
-		$results = $this->object->searchItems( $search );
+		$search->setConditions($search->combine('&&', $conditions));
+		$results = $this->object->searchItems($search);
 
-		if( ($expected = reset($results)) === false ) {
-			throw new \RuntimeException( sprintf( 'No customer property item found for value "%1$s".', '1' ) );
+		if (($expected = reset($results)) === false) {
+			throw new \RuntimeException(sprintf('No customer property item found for value "%1$s".', '1'));
 		}
 
-		$actual = $this->object->getItem( $expected->getId() );
-		$this->assertNotEquals( '', $actual->getTypeName() );
-		$this->assertEquals( $expected, $actual );
+		$actual = $this->object->getItem($expected->getId());
+		$this->assertNotEquals('', $actual->getTypeName());
+		$this->assertEquals($expected, $actual);
 	}
 
 
@@ -132,15 +132,15 @@ 
 	{
 		$result = $this->object->getResourceType();
 
-		$this->assertContains( 'customer/property', $result );
-		$this->assertContains( 'customer/property/type', $result );
+		$this->assertContains('customer/property', $result);
+		$this->assertContains('customer/property/type', $result);
 	}
 
 
 	public function testGetSearchAttributes()
 	{
-		foreach( $this->object->getSearchAttributes() as $attribute ) {
-			$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute );
+		foreach ($this->object->getSearchAttributes() as $attribute) {
+			$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute);
 		}
 	}
 
@@ -151,51 +151,51 @@ 
 		$search = $this->object->createSearch();
 
 		$expr = [];
-		$expr[] = $search->compare( '!=', 'customer.property.id', null );
-		$expr[] = $search->compare( '!=', 'customer.property.parentid', null );
-		$expr[] = $search->compare( '!=', 'customer.property.siteid', null );
-		$expr[] = $search->compare( '!=', 'customer.property.typeid', null );
-		$expr[] = $search->compare( '==', 'customer.property.languageid', null );
-		$expr[] = $search->compare( '==', 'customer.property.value', '1' );
-		$expr[] = $search->compare( '==', 'customer.property.editor', $this->editor );
-
-		$expr[] = $search->compare( '!=', 'customer.property.type.id', null );
-		$expr[] = $search->compare( '!=', 'customer.property.type.siteid', null );
-		$expr[] = $search->compare( '==', 'customer.property.type.domain', 'customer' );
-		$expr[] = $search->compare( '==', 'customer.property.type.code', 'newsletter' );
-		$expr[] = $search->compare( '>', 'customer.property.type.label', '' );
-		$expr[] = $search->compare( '==', 'customer.property.type.status', 1 );
-		$expr[] = $search->compare( '>=', 'customer.property.type.mtime', '1970-01-01 00:00:00' );
-		$expr[] = $search->compare( '>=', 'customer.property.type.ctime', '1970-01-01 00:00:00' );
-		$expr[] = $search->compare( '==', 'customer.property.type.editor', $this->editor );
-
-		$search->setConditions( $search->combine('&&', $expr) );
-		$results = $this->object->searchItems( $search, [], $total );
-		$this->assertEquals( 1, count( $results ) );
+		$expr[] = $search->compare('!=', 'customer.property.id', null);
+		$expr[] = $search->compare('!=', 'customer.property.parentid', null);
+		$expr[] = $search->compare('!=', 'customer.property.siteid', null);
+		$expr[] = $search->compare('!=', 'customer.property.typeid', null);
+		$expr[] = $search->compare('==', 'customer.property.languageid', null);
+		$expr[] = $search->compare('==', 'customer.property.value', '1');
+		$expr[] = $search->compare('==', 'customer.property.editor', $this->editor);
+
+		$expr[] = $search->compare('!=', 'customer.property.type.id', null);
+		$expr[] = $search->compare('!=', 'customer.property.type.siteid', null);
+		$expr[] = $search->compare('==', 'customer.property.type.domain', 'customer');
+		$expr[] = $search->compare('==', 'customer.property.type.code', 'newsletter');
+		$expr[] = $search->compare('>', 'customer.property.type.label', '');
+		$expr[] = $search->compare('==', 'customer.property.type.status', 1);
+		$expr[] = $search->compare('>=', 'customer.property.type.mtime', '1970-01-01 00:00:00');
+		$expr[] = $search->compare('>=', 'customer.property.type.ctime', '1970-01-01 00:00:00');
+		$expr[] = $search->compare('==', 'customer.property.type.editor', $this->editor);
+
+		$search->setConditions($search->combine('&&', $expr));
+		$results = $this->object->searchItems($search, [], $total);
+		$this->assertEquals(1, count($results));
 
 
 		$search = $this->object->createSearch();
 		$conditions = array(
-			$search->compare( '=~', 'customer.property.type.code', 'newsletter' ),
-			$search->compare( '==', 'customer.property.editor', $this->editor )
+			$search->compare('=~', 'customer.property.type.code', 'newsletter'),
+			$search->compare('==', 'customer.property.editor', $this->editor)
 		);
-		$search->setConditions( $search->combine( '&&', $conditions ) );
+		$search->setConditions($search->combine('&&', $conditions));
 		$search->setSlice(0, 1);
-		$items = $this->object->searchItems( $search, [], $total );
+		$items = $this->object->searchItems($search, [], $total);
 
-		$this->assertEquals( 1, count( $items ) );
-		$this->assertEquals( 1, $total );
+		$this->assertEquals(1, count($items));
+		$this->assertEquals(1, $total);
 
-		foreach($items as $itemId => $item) {
-			$this->assertEquals( $itemId, $item->getId() );
+		foreach ($items as $itemId => $item) {
+			$this->assertEquals($itemId, $item->getId());
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type') );
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type'));
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard'));
 
 		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
 		$this->object->getSubManager('unknown');