aimeos / ai-ezpublish

lib/custom/src/MShop/Customer/Manager/Group/Ezpublish.php

Spacing +18 added lines, -18 removed lines

@@ -72,7 +72,7 @@ 
 	 *
 	 * @param integer[] $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 	}
 
@@ -82,9 +82,9 @@ 
 	 *
 	 * @param array $ids List of IDs
 	 */
-	public function deleteItems( array $ids )
+	public function deleteItems(array $ids)
 	{
-		throw new \Aimeos\MShop\Customer\Exception( sprintf( 'Deleting groups is not supported, please use the eZ Publish backend' ) );
+		throw new \Aimeos\MShop\Customer\Exception(sprintf('Deleting groups is not supported, please use the eZ Publish backend'));
 	}
 
 
@@ -94,11 +94,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/group/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, [], $withsub);
 	}
 
 
@@ -109,9 +109,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
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer/group', $manager, ( $name === null ? 'Ezpublish' : $name ) );
+		return $this->getSubManagerBase('customer/group', $manager, ($name === null ? 'Ezpublish' : $name));
 	}
 
 
@@ -121,9 +121,9 @@ 
 	 * @param \Aimeos\MShop\Customer\Item\Group\Iface $item Customer group item
 	 * @param boolean $fetch True if the new ID should be returned in the item
 	 */
-	public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true )
+	public function saveItem(\Aimeos\MShop\Common\Item\Iface $item, $fetch = true)
 	{
-		throw new \Aimeos\MShop\Customer\Exception( sprintf( 'Saving groups is not supported, please use the eZ Publish backend' ) );
+		throw new \Aimeos\MShop\Customer\Exception(sprintf('Saving groups is not supported, please use the eZ Publish backend'));
 	}
 
 
@@ -136,33 +136,33 @@ 
 	 * @return array List of items implementing \Aimeos\MShop\Customer\Item\Group\Iface
 	 * @throws \Aimeos\MShop\Exception If retrieving 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)
 	{
 		$map = [];
 		$context = $this->getContext();
 
 		$dbm = $context->getDatabaseManager();
 		$dbname = $this->getResourceName();
-		$conn = $dbm->acquire( $dbname );
+		$conn = $dbm->acquire($dbname);
 
 		try
 		{
-			$required = array( 'customer.group' );
+			$required = array('customer.group');
 			$level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL;
 			$cfgPathSearch = 'mshop/customer/manager/group/ezpublish/search';
 			$cfgPathCount = 'mshop/customer/manager/group/ezpublish/count';
 
-			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
+			$results = $this->searchItemsBase($conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level);
 
-			while( ( $row = $results->fetch() ) !== false ) {
-				$map[$row['customer.group.id']] = $this->createItemBase( $row );
+			while (($row = $results->fetch()) !== false) {
+				$map[$row['customer.group.id']] = $this->createItemBase($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;
 		}
 

lib/custom/src/MShop/Customer/Manager/Lists/Type/Ezpublish.php

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
 		'customer.lists.type.id' => array(
 			'code'=>'customer.lists.type.id',
 			'internalcode'=>'ezulity."id"',
-			'internaldeps'=>array( 'LEFT JOIN "ezuser_list_type" AS ezulity ON ( ezuli."typeid" = ezulity."id" )' ),
+			'internaldeps'=>array('LEFT JOIN "ezuser_list_type" AS ezulity ON ( ezuli."typeid" = ezulity."id" )'),
 			'label'=>'Customer list type ID',
 			'type'=> 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -96,14 +96,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/lists/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, $this->getConfigPath() . 'delete' );
+		$this->cleanupBase($siteids, $this->getConfigPath().'delete');
 	}
 
 
@@ -113,11 +113,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/lists/type/submanagers';
 
-		return $this->getSearchAttributesBase( $this->getSearchConfig(), $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->getSearchConfig(), $path, [], $withsub);
 	}
 
 
@@ -128,9 +128,9 @@ 
 	 * @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', 'lists/type/' . $manager, ( $name === null ? 'Ezpublish' : $name ) );
+		return $this->getSubManagerBase('customer', 'lists/type/'.$manager, ($name === null ? 'Ezpublish' : $name));
 	}
 
 

lib/custom/src/MShop/Customer/Manager/Address/Ezpublish.php

Spacing +9 added lines, -9 removed lines

@@ -26,7 +26,7 @@ 
 			'label' => 'Customer address ID',
 			'code' => 'customer.address.id',
 			'internalcode' => 'ezuad."id"',
-			'internaldeps' => array( 'LEFT JOIN "ezuser_address" AS ezuad ON ( ezu."contentobject_id" = ezuad."parentid" )' ),
+			'internaldeps' => array('LEFT JOIN "ezuser_address" AS ezuad ON ( ezu."contentobject_id" = ezuad."parentid" )'),
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
 			'public' => false,
@@ -209,14 +209,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/address/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/lists/ezpublish/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/lists/ezpublish/delete');
 	}
 
 
@@ -226,11 +226,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/address/submanagers';
 
-		return $this->getSearchAttributesBase( $this->getSearchConfig(), $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->getSearchConfig(), $path, [], $withsub);
 	}
 
 
@@ -241,9 +241,9 @@ 
 	 * @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', 'address/' . $manager, ( $name === null ? 'Ezpublish' : $name ) );
+		return $this->getSubManagerBase('customer', 'address/'.$manager, ($name === null ? 'Ezpublish' : $name));
 	}
 
 

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

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
 		'customer.property.id' => array(
 			'code' => 'customer.property.id',
 			'internalcode' => 'ezppr."id"',
-			'internaldeps'=>array( 'LEFT JOIN "ezp_user_property" AS ezppr ON ( ezppr."parentid" = ezp."id" )' ),
+			'internaldeps'=>array('LEFT JOIN "ezp_user_property" AS ezppr ON ( ezppr."parentid" = ezp."id" )'),
 			'label' => 'Property ID',
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -100,14 +100,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/ezpublish/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/ezpublish/delete');
 	}
 
 
@@ -117,11 +117,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, ['type'], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, ['type'], $withsub);
 	}
 
 
@@ -133,9 +133,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 ? 'Ezpublish' : $name ) );
+		return $this->getSubManagerBase('customer', 'property/'.$manager, ($name === null ? 'Ezpublish' : $name));
 	}
 
 

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

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
 		'customer.property.type.id' => array(
 			'code' => 'customer.property.type.id',
 			'internalcode' => 'ezpprty."id"',
-			'internaldeps' => array( 'LEFT JOIN "ezp_user_property_type" AS ezpprty ON ( ezppr."typeid" = ezpprty."id" )' ),
+			'internaldeps' => array('LEFT JOIN "ezp_user_property_type" AS ezpprty ON ( ezppr."typeid" = ezpprty."id" )'),
 			'label' => 'Property type ID',
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -105,14 +105,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/ezpublish/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/type/ezpublish/delete');
 	}
 
 
@@ -122,11 +122,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/type/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, [], $withsub);
 	}
 
 
@@ -137,9 +137,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 ? 'Ezpublish' : $name ) );
+		return $this->getSubManagerBase('customer', 'property/type/'.$manager, ($name === null ? 'Ezpublish' : $name));
 	}
 
 

lib/custom/tests/MShop/Customer/Manager/Lists/EzpublishTest.php

Spacing +16 added lines, -16 removed lines

@@ -16,57 +16,57 @@
 
 	protected function setUp()
 	{
-		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( \TestHelper::getContext(), 'Ezpublish' );
-		$this->object = $manager->getSubManager( 'lists', 'Ezpublish' );
+		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager(\TestHelper::getContext(), 'Ezpublish');
+		$this->object = $manager->getSubManager('lists', 'Ezpublish');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\Aimeos\MW\Criteria\Attribute\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\Aimeos\MW\Criteria\Attribute\Iface', $attribute);
 		}
 	}
 
 
 	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' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getSubManager('unknown');
 	}
 
 
 	public function testSaveItemInvalidItem()
 	{
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
-		$this->object->saveItem( new \Aimeos\MShop\Common\Item\Type\Standard( 'common.lists.type.' ) );
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
+		$this->object->saveItem(new \Aimeos\MShop\Common\Item\Type\Standard('common.lists.type.'));
 	}
 
 
 	public function testSaveItemInvalidDomain()
 	{
 		$item = $this->object->createItem();
-		$item->setDomain( 'customer/group' );
+		$item->setDomain('customer/group');
 
-		$this->setExpectedException( '\Aimeos\MShop\Customer\Exception' );
-		$this->object->saveItem( $item );
+		$this->setExpectedException('\Aimeos\MShop\Customer\Exception');
+		$this->object->saveItem($item);
 	}
 }

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

Spacing +13 added lines, -13 removed lines

@@ -16,47 +16,47 @@
 
 	protected function setUp()
 	{
-		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( \TestHelper::getContext(), 'Ezpublish' );
-		$this->object = $manager->getSubManager( 'property', 'Ezpublish' );
+		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager(\TestHelper::getContext(), 'Ezpublish');
+		$this->object = $manager->getSubManager('property', 'Ezpublish');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\Aimeos\MW\Criteria\Attribute\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\Aimeos\MW\Criteria\Attribute\Iface', $attribute);
 		}
 	}
 
 
 	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' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getSubManager('unknown');
 	}
 
 
 	public function testSaveItemInvalidItem()
 	{
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
-		$this->object->saveItem( new \Aimeos\MShop\Common\Item\Type\Standard( 'common.property.type.' ) );
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
+		$this->object->saveItem(new \Aimeos\MShop\Common\Item\Type\Standard('common.property.type.'));
 	}
 }

lib/custom/src/MShop/Customer/Manager/Lists/Ezpublish.php

Spacing +14 added lines, -14 removed lines

@@ -25,7 +25,7 @@ 
 		'customer.lists.id'=> array(
 			'code'=>'customer.lists.id',
 			'internalcode'=>'ezuli."id"',
-			'internaldeps' => array( 'LEFT JOIN "ezuser_list" AS ezuli ON ( ezu."id" = ezuli."parentid" )' ),
+			'internaldeps' => array('LEFT JOIN "ezuser_list" AS ezuli ON ( ezu."id" = ezuli."parentid" )'),
 			'label'=>'Customer list ID',
 			'type'=> 'integer',
 			'internaltype'=> \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -133,14 +133,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/lists/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, array( 'type' ) ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, array('type')) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, $this->getConfigPath() . 'delete' );
+		$this->cleanupBase($siteids, $this->getConfigPath().'delete');
 	}
 
 
@@ -150,11 +150,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/lists/submanagers';
 
-		return $this->getSearchAttributesBase( $this->getSearchConfig(), $path, ['type'], $withsub );
+		return $this->getSearchAttributesBase($this->getSearchConfig(), $path, ['type'], $withsub);
 	}
 
 
@@ -165,9 +165,9 @@ 
 	 * @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', 'lists/' . $manager, ( $name === null ? 'Ezpublish' : $name ) );
+		return $this->getSubManagerBase('customer', 'lists/'.$manager, ($name === null ? 'Ezpublish' : $name));
 	}
 
 
@@ -177,15 +177,15 @@ 
 	 * @param \Aimeos\MShop\Common\Item\Lists\Iface $item List item object which should be saved
 	 * @param boolean $fetch True if the new ID should be returned in the item
 	 */
-	public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true )
+	public function saveItem(\Aimeos\MShop\Common\Item\Iface $item, $fetch = true)
 	{
-		self::checkClass( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $item );
+		self::checkClass('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $item);
 
-		if( $item->getDomain() === 'customer/group' ) {
-			throw new \Aimeos\MShop\Customer\Exception( sprintf( 'Adding groups to customers is not supported, please use the eZ Publish backend' ) );
+		if ($item->getDomain() === 'customer/group') {
+			throw new \Aimeos\MShop\Customer\Exception(sprintf('Adding groups to customers is not supported, please use the eZ Publish backend'));
 		}
 
-		parent::saveItem( $item, $fetch );
+		parent::saveItem($item, $fetch);
 	}
 
 

lib/custom/tests/MShop/Customer/Manager/EzpublishTest.php

Spacing +99 added lines, -99 removed lines

@@ -18,206 +18,206 @@
 
 	protected function setUp()
 	{
-		if( !interface_exists( 'eZ\Publish\API\Repository\UserService' ) ) {
-			$this->markTestSkipped( 'Install ezsystems/ezpublish-api first' );
+		if (!interface_exists('eZ\Publish\API\Repository\UserService')) {
+			$this->markTestSkipped('Install ezsystems/ezpublish-api first');
 		}
 
 		$this->context = \TestHelper::getContext();
 
-		$this->object = new \Aimeos\MShop\Customer\Manager\Ezpublish( $this->context );
-		$this->address = new \Aimeos\MShop\Common\Item\Address\Standard( 'common.address.' );
+		$this->object = new \Aimeos\MShop\Customer\Manager\Ezpublish($this->context);
+		$this->address = new \Aimeos\MShop\Common\Item\Address\Standard('common.address.');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->context, $this->address );
+		unset($this->object, $this->context, $this->address);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testCreateItem()
 	{
-		$this->assertInstanceOf( 'Aimeos\MShop\Customer\Item\Iface', $this->object->createItem() );
+		$this->assertInstanceOf('Aimeos\MShop\Customer\Item\Iface', $this->object->createItem());
 	}
 
 
 	public function testDeleteItems()
 	{
-		$mock = $this->getMockBuilder( 'eZ\Publish\API\Repository\UserService' )->getMock();
-		$user = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\User' )->getMock();
+		$mock = $this->getMockBuilder('eZ\Publish\API\Repository\UserService')->getMock();
+		$user = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\User')->getMock();
 
-		$mock->expects( $this->once() )->method( 'loadUser' )->will( $this->returnValue( $user ) );
-		$mock->expects( $this->once() )->method( 'deleteUser' );
+		$mock->expects($this->once())->method('loadUser')->will($this->returnValue($user));
+		$mock->expects($this->once())->method('deleteUser');
 
-		$this->context->setEzUserService( $mock );
-		$this->object->deleteItems( array( -1 ) );
+		$this->context->setEzUserService($mock);
+		$this->object->deleteItems(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\Aimeos\MW\Criteria\Attribute\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\Aimeos\MW\Criteria\Attribute\Iface', $attribute);
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager( 'address' ) );
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager( 'address', 'Standard' ) );
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager('address'));
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager('address', 'Standard'));
 
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$this->object->getSubManager( 'unknown' );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$this->object->getSubManager('unknown');
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$this->object->getSubManager( 'address', 'unknown' );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$this->object->getSubManager('address', 'unknown');
 	}
 
 
 	public function testSaveItemInsert()
 	{
-		$service = $this->getMockBuilder( 'eZ\Publish\API\Repository\UserService' )->getMock();
-		$struct = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\UserCreateStruct' )->getMock();
-		$user = $this->getMockBuilder( '\eZ\Publish\API\Repository\Values\User\User' )
-			->setMethods( array( 'getUserId', 'getVersionInfo', 'getFieldValue', 'getFields', 'getFieldsByLanguage' ) )
+		$service = $this->getMockBuilder('eZ\Publish\API\Repository\UserService')->getMock();
+		$struct = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\UserCreateStruct')->getMock();
+		$user = $this->getMockBuilder('\eZ\Publish\API\Repository\Values\User\User')
+			->setMethods(array('getUserId', 'getVersionInfo', 'getFieldValue', 'getFields', 'getFieldsByLanguage'))
 			->getMock();
 
-		$dbm = $this->getMockBuilder( 'Aimeos\MW\DB\Manager\Iface' )->getMock();
-		$conn = $this->getMockBuilder( 'Aimeos\MW\DB\Connection\Iface' )->getMock();
-		$stmt = $this->getMockBuilder( 'Aimeos\MW\DB\Statement\Iface' )->getMock();
-		$result = $this->getMockBuilder( 'Aimeos\MW\DB\Result\Iface' )->getMock();
+		$dbm = $this->getMockBuilder('Aimeos\MW\DB\Manager\Iface')->getMock();
+		$conn = $this->getMockBuilder('Aimeos\MW\DB\Connection\Iface')->getMock();
+		$stmt = $this->getMockBuilder('Aimeos\MW\DB\Statement\Iface')->getMock();
+		$result = $this->getMockBuilder('Aimeos\MW\DB\Result\Iface')->getMock();
 
-		$service->expects( $this->once() )->method( 'newUserCreateStruct' )->will( $this->returnValue( $struct ) );
-		$service->expects( $this->once() )->method( 'createUser' )->will( $this->returnValue( $user ) );
-		$user->expects( $this->once() )->method( 'getUserId' )->will( $this->returnValue( 1 ) );
-		$dbm->expects( $this->once() )->method( 'acquire' )->will( $this->returnValue( $conn ) );
-		$conn->expects( $this->once() )->method( 'create' )->will( $this->returnValue( $stmt ) );
-		$stmt->expects( $this->once() )->method( 'execute' )->will( $this->returnValue( $result ) );
+		$service->expects($this->once())->method('newUserCreateStruct')->will($this->returnValue($struct));
+		$service->expects($this->once())->method('createUser')->will($this->returnValue($user));
+		$user->expects($this->once())->method('getUserId')->will($this->returnValue(1));
+		$dbm->expects($this->once())->method('acquire')->will($this->returnValue($conn));
+		$conn->expects($this->once())->method('create')->will($this->returnValue($stmt));
+		$stmt->expects($this->once())->method('execute')->will($this->returnValue($result));
 
 
-		$this->context->setDatabaseManager( $dbm );
-		$this->context->setEzUserService( $service );
+		$this->context->setDatabaseManager($dbm);
+		$this->context->setEzUserService($service);
 
-		$this->object->saveItem( new \Aimeos\MShop\Customer\Item\Standard( $this->address ) );
+		$this->object->saveItem(new \Aimeos\MShop\Customer\Item\Standard($this->address));
 	}
 
 
 	public function testSaveItemUpdate()
 	{
-		$service = $this->getMockBuilder( 'eZ\Publish\API\Repository\UserService' )->getMock();
-		$user = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\User' )->getMock();
-		$struct = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\UserUpdateStruct' )->getMock();
-		$dbm = $this->getMockBuilder( 'Aimeos\MW\DB\Manager\Iface' )->getMock();
-		$conn = $this->getMockBuilder( 'Aimeos\MW\DB\Connection\Iface' )->getMock();
-		$stmt = $this->getMockBuilder( 'Aimeos\MW\DB\Statement\Iface' )->getMock();
-		$result = $this->getMockBuilder( 'Aimeos\MW\DB\Result\Iface' )->getMock();
-
-		$service->expects( $this->once() )->method( 'updateUser' );
-		$service->expects( $this->once() )->method( 'loadUser' )->will( $this->returnValue( $user ) );
-		$service->expects( $this->once() )->method( 'newUserUpdateStruct' )->will( $this->returnValue( $struct ) );
-		$dbm->expects( $this->once() )->method( 'acquire' )->will( $this->returnValue( $conn ) );
-		$conn->expects( $this->once() )->method( 'create' )->will( $this->returnValue( $stmt ) );
-		$stmt->expects( $this->once() )->method( 'execute' )->will( $this->returnValue( $result ) );
-		$result->expects( $this->once() )->method( 'finish' );
-		$dbm->expects( $this->once() )->method( 'release' );
-
-
-		$item = new \Aimeos\MShop\Customer\Item\Standard( $this->address, array( 'customer.id' => 1 ) );
-		$item->setStatus( 1 );
-
-		$this->context->setDatabaseManager( $dbm );
-		$this->context->setEzUserService( $service );
-		$this->object->saveItem( $item );
+		$service = $this->getMockBuilder('eZ\Publish\API\Repository\UserService')->getMock();
+		$user = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\User')->getMock();
+		$struct = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\UserUpdateStruct')->getMock();
+		$dbm = $this->getMockBuilder('Aimeos\MW\DB\Manager\Iface')->getMock();
+		$conn = $this->getMockBuilder('Aimeos\MW\DB\Connection\Iface')->getMock();
+		$stmt = $this->getMockBuilder('Aimeos\MW\DB\Statement\Iface')->getMock();
+		$result = $this->getMockBuilder('Aimeos\MW\DB\Result\Iface')->getMock();
+
+		$service->expects($this->once())->method('updateUser');
+		$service->expects($this->once())->method('loadUser')->will($this->returnValue($user));
+		$service->expects($this->once())->method('newUserUpdateStruct')->will($this->returnValue($struct));
+		$dbm->expects($this->once())->method('acquire')->will($this->returnValue($conn));
+		$conn->expects($this->once())->method('create')->will($this->returnValue($stmt));
+		$stmt->expects($this->once())->method('execute')->will($this->returnValue($result));
+		$result->expects($this->once())->method('finish');
+		$dbm->expects($this->once())->method('release');
+
+
+		$item = new \Aimeos\MShop\Customer\Item\Standard($this->address, array('customer.id' => 1));
+		$item->setStatus(1);
+
+		$this->context->setDatabaseManager($dbm);
+		$this->context->setEzUserService($service);
+		$this->object->saveItem($item);
 	}
 
 
 	public function testSaveItemInvalidItem()
 	{
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
-		$this->object->saveItem( new \Aimeos\MShop\Common\Item\Lists\Standard( 'common.lists.' ) );
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
+		$this->object->saveItem(new \Aimeos\MShop\Common\Item\Lists\Standard('common.lists.'));
 	}
 
 
 	public function testSaveItemNotModified()
 	{
-		$item = new \Aimeos\MShop\Customer\Item\Standard( $this->address, array( 'customer.id' => 1 ) );
-		$this->object->saveItem( $item );
+		$item = new \Aimeos\MShop\Customer\Item\Standard($this->address, array('customer.id' => 1));
+		$this->object->saveItem($item);
 	}
 
 
 	public function testSaveItemInvalidContext()
 	{
 		$context = new \Aimeos\MShop\Context\Item\Standard();
-		$context->setConfig( $this->context->getConfig() );
+		$context->setConfig($this->context->getConfig());
 
-		$object = new \Aimeos\MShop\Customer\Manager\Ezpublish( $context );
+		$object = new \Aimeos\MShop\Customer\Manager\Ezpublish($context);
 
-		$this->setExpectedException( '\Aimeos\MShop\Customer\Exception' );
-		$object->saveItem( new \Aimeos\MShop\Customer\Item\Standard( $this->address ) );
+		$this->setExpectedException('\Aimeos\MShop\Customer\Exception');
+		$object->saveItem(new \Aimeos\MShop\Customer\Item\Standard($this->address));
 	}
 
 
 	public function testSearchItems()
 	{
-		$mock = $this->getMockBuilder( '\Aimeos\MShop\Customer\Manager\Ezpublish' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'searchItemsBase' ) )
+		$mock = $this->getMockBuilder('\Aimeos\MShop\Customer\Manager\Ezpublish')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('searchItemsBase'))
 			->getMock();
 
-		$result1 = $this->getMockBuilder( '\Aimeos\MW\DB\Result\Iface' )->getMock();
+		$result1 = $this->getMockBuilder('\Aimeos\MW\DB\Result\Iface')->getMock();
 
-		$mock->expects( $this->once() )->method( 'searchItemsBase' )->will( $this->returnValue( $result1 ) );
-		$result1->expects( $this->exactly( 2 ) )->method( 'fetch' )
-			->will( $this->onConsecutiveCalls( array( 'customer.id' => -1 ), false ) );
+		$mock->expects($this->once())->method('searchItemsBase')->will($this->returnValue($result1));
+		$result1->expects($this->exactly(2))->method('fetch')
+			->will($this->onConsecutiveCalls(array('customer.id' => -1), false));
 
 
-		$dbm = $this->getMockBuilder( 'Aimeos\MW\DB\Manager\Iface' )->getMock();
-		$conn = $this->getMockBuilder( 'Aimeos\MW\DB\Connection\Iface' )->getMock();
-		$stmt = $this->getMockBuilder( 'Aimeos\MW\DB\Statement\Iface' )->getMock();
-		$result2 = $this->getMockBuilder( '\Aimeos\MW\DB\Result\Iface' )->getMock();
+		$dbm = $this->getMockBuilder('Aimeos\MW\DB\Manager\Iface')->getMock();
+		$conn = $this->getMockBuilder('Aimeos\MW\DB\Connection\Iface')->getMock();
+		$stmt = $this->getMockBuilder('Aimeos\MW\DB\Statement\Iface')->getMock();
+		$result2 = $this->getMockBuilder('\Aimeos\MW\DB\Result\Iface')->getMock();
 
-		$dbm->expects( $this->any() )->method( 'acquire' )->will( $this->returnValue( $conn ) );
-		$conn->expects( $this->once() )->method( 'create' )->will( $this->returnValue( $stmt ) );
-		$stmt->expects( $this->once() )->method( 'execute' )->will( $this->returnValue( $result2 ) );
-		$result2->expects( $this->exactly( 2 ) )->method( 'fetch' )
-			->will( $this->onConsecutiveCalls( array( 'contentobject_id' => -1, 'role_id' => -2 ), false ) );
+		$dbm->expects($this->any())->method('acquire')->will($this->returnValue($conn));
+		$conn->expects($this->once())->method('create')->will($this->returnValue($stmt));
+		$stmt->expects($this->once())->method('execute')->will($this->returnValue($result2));
+		$result2->expects($this->exactly(2))->method('fetch')
+			->will($this->onConsecutiveCalls(array('contentobject_id' => -1, 'role_id' => -2), false));
 
-		$dbm->expects( $this->any() )->method( 'release' );
+		$dbm->expects($this->any())->method('release');
 
 
-		$this->context->setDatabaseManager( $dbm );
-		$list = $mock->searchItems( $mock->createSearch() );
+		$this->context->setDatabaseManager($dbm);
+		$list = $mock->searchItems($mock->createSearch());
 
-		$this->assertEquals( 1, count( $list ) );
-		$this->assertInstanceOf( '\Aimeos\MShop\Customer\Item\Iface', $list[-1] );
+		$this->assertEquals(1, count($list));
+		$this->assertInstanceOf('\Aimeos\MShop\Customer\Item\Iface', $list[-1]);
 	}
 
 
 	public function testSearchItemsException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\MShop\Customer\Manager\Ezpublish' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'searchItemsBase' ) )
+		$object = $this->getMockBuilder('\Aimeos\MShop\Customer\Manager\Ezpublish')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('searchItemsBase'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'searchItemsBase' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('searchItemsBase')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$object->searchItems( $object->createSearch() );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$object->searchItems($object->createSearch());
 	}
 }