| @@ -4,17 +4,17 @@ | ||
| 4 | 4 | * Set error reporting to maximum | 
| 5 | 5 | */ | 
| 6 | 6 | error_reporting( -1 ); | 
| 7 | -ini_set( 'display_errors', true ); | |
| 7 | +ini_set('display_errors', true); | |
| 8 | 8 | |
| 9 | -date_default_timezone_set( 'UTC' ); | |
| 9 | +date_default_timezone_set('UTC'); | |
| 10 | 10 | |
| 11 | 11 | /* | 
| 12 | 12 | * Set locale settings to reasonable defaults | 
| 13 | 13 | */ | 
| 14 | -setlocale( LC_ALL, 'en_US.UTF-8' ); | |
| 15 | -setlocale( LC_CTYPE, 'en_US.UTF-8' ); | |
| 16 | -setlocale( LC_NUMERIC, 'POSIX' ); | |
| 17 | -setlocale( LC_TIME, 'POSIX' ); | |
| 14 | +setlocale(LC_ALL, 'en_US.UTF-8'); | |
| 15 | +setlocale(LC_CTYPE, 'en_US.UTF-8'); | |
| 16 | +setlocale(LC_NUMERIC, 'POSIX'); | |
| 17 | +setlocale(LC_TIME, 'POSIX'); | |
| 18 | 18 | |
| 19 | 19 | |
| 20 | 20 | require_once 'TestHelper.php'; | 
| @@ -17,7 +17,7 @@ | ||
| 17 | 17 | /** | 
| 18 | 18 | * Returns the list of task names which depends on this task. | 
| 19 | 19 | * | 
| 20 | - * @return array List of task names | |
| 20 | + * @return string[] List of task names | |
| 21 | 21 | */ | 
| 22 | 22 | public function getPreDependencies() | 
| 23 | 23 |  	{ | 
| @@ -21,7 +21,7 @@ discard block | ||
| 21 | 21 | */ | 
| 22 | 22 | public function getPreDependencies() | 
| 23 | 23 |  	{ | 
| 24 | - return array( 'TablesCreateEzpublish' ); | |
| 24 | +		return array('TablesCreateEzpublish'); | |
| 25 | 25 | } | 
| 26 | 26 | |
| 27 | 27 | |
| @@ -41,29 +41,29 @@ discard block | ||
| 41 | 41 | */ | 
| 42 | 42 | public function migrate() | 
| 43 | 43 |  	{ | 
| 44 | - $this->msg( 'Adding address fields to ezuser table', 0 ); | |
| 44 | +		$this->msg('Adding address fields to ezuser table', 0); | |
| 45 | 45 | |
| 46 | - $dbal = $this->getConnection( 'db-customer' )->getRawObject(); | |
| 46 | +		$dbal = $this->getConnection('db-customer')->getRawObject(); | |
| 47 | 47 | |
| 48 | -		if( !( $dbal instanceof \Doctrine\DBAL\Connection ) ) { | |
| 49 | - throw new \Aimeos\MW\Setup\Exception( 'Not a DBAL connection' ); | |
| 48 | +		if (!($dbal instanceof \Doctrine\DBAL\Connection)) { | |
| 49 | +			throw new \Aimeos\MW\Setup\Exception('Not a DBAL connection'); | |
| 50 | 50 | } | 
| 51 | 51 | |
| 52 | 52 | |
| 53 | 53 | $fromSchema = $dbal->getSchemaManager()->createSchema(); | 
| 54 | 54 | $toSchema = clone $fromSchema; | 
| 55 | 55 | |
| 56 | - $this->addIndexes( $this->addColumns( $toSchema->getTable( 'ezuser' ) ) ); | |
| 57 | - $sql = $fromSchema->getMigrateToSql( $toSchema, $dbal->getDatabasePlatform() ); | |
| 56 | +		$this->addIndexes($this->addColumns($toSchema->getTable('ezuser'))); | |
| 57 | + $sql = $fromSchema->getMigrateToSql($toSchema, $dbal->getDatabasePlatform()); | |
| 58 | 58 | |
| 59 | - if( $sql !== [] ) | |
| 59 | + if ($sql !== []) | |
| 60 | 60 |  		{ | 
| 61 | - $this->executeList( $sql, 'db-customer' ); | |
| 62 | - $this->status( 'done' ); | |
| 61 | + $this->executeList($sql, 'db-customer'); | |
| 62 | +			$this->status('done'); | |
| 63 | 63 | } | 
| 64 | 64 | else | 
| 65 | 65 |  		{ | 
| 66 | - $this->status( 'OK' ); | |
| 66 | +			$this->status('OK'); | |
| 67 | 67 | } | 
| 68 | 68 | } | 
| 69 | 69 | |
| @@ -74,38 +74,38 @@ discard block | ||
| 74 | 74 | * @param \Doctrine\DBAL\Schema\Table $table Table object | 
| 75 | 75 | * @return \Doctrine\DBAL\Schema\Table Updated table object | 
| 76 | 76 | */ | 
| 77 | - protected function addColumns( \Doctrine\DBAL\Schema\Table $table ) | |
| 77 | + protected function addColumns(\Doctrine\DBAL\Schema\Table $table) | |
| 78 | 78 |  	{ | 
| 79 | 79 | $columns = array( | 
| 80 | - 'company' => array( 'string', array( 'length' => 100 ) ), | |
| 81 | - 'vatid' => array( 'string', array( 'length' => 32 ) ), | |
| 82 | - 'salutation' => array( 'string', array( 'length' => 8 ) ), | |
| 83 | - 'title' => array( 'string', array( 'length' => 64 ) ), | |
| 84 | - 'firstname' => array( 'string', array( 'length' => 64 ) ), | |
| 85 | - 'lastname' => array( 'string', array( 'length' => 64 ) ), | |
| 86 | - 'address1' => array( 'string', array( 'length' => 255 ) ), | |
| 87 | - 'address2' => array( 'string', array( 'length' => 255 ) ), | |
| 88 | - 'address3' => array( 'string', array( 'length' => 255 ) ), | |
| 89 | - 'postal' => array( 'string', array( 'length' => 16 ) ), | |
| 90 | - 'city' => array( 'string', array( 'length' => 255 ) ), | |
| 91 | - 'state' => array( 'string', array( 'length' => 255 ) ), | |
| 92 | - 'langid' => array( 'string', array( 'length' => 5, 'notnull' => false ) ), | |
| 93 | - 'countryid' => array( 'string', array( 'length' => 2, 'notnull' => false, 'fixed' => true ) ), | |
| 94 | - 'telephone' => array( 'string', array( 'length' => 32 ) ), | |
| 95 | - 'telefax' => array( 'string', array( 'length' => 32 ) ), | |
| 96 | - 'website' => array( 'string', array( 'length' => 255 ) ), | |
| 97 | - 'birthday' => array( 'date', array( 'notnull' => false ) ), | |
| 98 | - 'vdate' => array( 'date', array( 'notnull' => false ) ), | |
| 99 | - 'status' => array( 'smallint', [] ), | |
| 100 | - 'mtime' => array( 'datetime', [] ), | |
| 101 | - 'ctime' => array( 'datetime', [] ), | |
| 102 | - 'editor' => array( 'string', array( 'length' => 255 ) ), | |
| 80 | +			'company' => array('string', array('length' => 100)), | |
| 81 | +			'vatid' => array('string', array('length' => 32)), | |
| 82 | +			'salutation' => array('string', array('length' => 8)), | |
| 83 | +			'title' => array('string', array('length' => 64)), | |
| 84 | +			'firstname' => array('string', array('length' => 64)), | |
| 85 | +			'lastname' => array('string', array('length' => 64)), | |
| 86 | +			'address1' => array('string', array('length' => 255)), | |
| 87 | +			'address2' => array('string', array('length' => 255)), | |
| 88 | +			'address3' => array('string', array('length' => 255)), | |
| 89 | +			'postal' => array('string', array('length' => 16)), | |
| 90 | +			'city' => array('string', array('length' => 255)), | |
| 91 | +			'state' => array('string', array('length' => 255)), | |
| 92 | +			'langid' => array('string', array('length' => 5, 'notnull' => false)), | |
| 93 | +			'countryid' => array('string', array('length' => 2, 'notnull' => false, 'fixed' => true)), | |
| 94 | +			'telephone' => array('string', array('length' => 32)), | |
| 95 | +			'telefax' => array('string', array('length' => 32)), | |
| 96 | +			'website' => array('string', array('length' => 255)), | |
| 97 | +			'birthday' => array('date', array('notnull' => false)), | |
| 98 | +			'vdate' => array('date', array('notnull' => false)), | |
| 99 | +			'status' => array('smallint', []), | |
| 100 | +			'mtime' => array('datetime', []), | |
| 101 | +			'ctime' => array('datetime', []), | |
| 102 | +			'editor' => array('string', array('length' => 255)), | |
| 103 | 103 | ); | 
| 104 | 104 | |
| 105 | - foreach( $columns as $name => $def ) | |
| 105 | + foreach ($columns as $name => $def) | |
| 106 | 106 |  		{ | 
| 107 | -			if( $table->hasColumn( $name ) === false ) { | |
| 108 | - $table->addColumn( $name, $def[0], $def[1] ); | |
| 107 | +			if ($table->hasColumn($name) === false) { | |
| 108 | + $table->addColumn($name, $def[0], $def[1]); | |
| 109 | 109 | } | 
| 110 | 110 | } | 
| 111 | 111 | |
| @@ -119,23 +119,23 @@ discard block | ||
| 119 | 119 | * @param \Doctrine\DBAL\Schema\Table $table Table object | 
| 120 | 120 | * @return \Doctrine\DBAL\Schema\Table Updated table object | 
| 121 | 121 | */ | 
| 122 | - protected function addIndexes( \Doctrine\DBAL\Schema\Table $table ) | |
| 122 | + protected function addIndexes(\Doctrine\DBAL\Schema\Table $table) | |
| 123 | 123 |  	{ | 
| 124 | 124 | $indexes = array( | 
| 125 | - 'idx_ezpus_langid' => array( 'langid' ), | |
| 126 | - 'idx_ezpus_status_ln_fn' => array( 'status', 'lastname', 'firstname' ), | |
| 127 | - 'idx_ezpus_status_ad1_ad2' => array( 'status', 'address1', 'address2' ), | |
| 128 | - 'idx_ezpus_status_postal_city' => array( 'status', 'postal', 'city' ), | |
| 129 | - 'idx_ezpus_lastname' => array( 'lastname' ), | |
| 130 | - 'idx_ezpus_address1' => array( 'address1' ), | |
| 131 | - 'idx_ezpus_postal' => array( 'postal' ), | |
| 132 | - 'idx_ezpus_city' => array( 'city' ), | |
| 125 | +			'idx_ezpus_langid' => array('langid'), | |
| 126 | +			'idx_ezpus_status_ln_fn' => array('status', 'lastname', 'firstname'), | |
| 127 | +			'idx_ezpus_status_ad1_ad2' => array('status', 'address1', 'address2'), | |
| 128 | +			'idx_ezpus_status_postal_city' => array('status', 'postal', 'city'), | |
| 129 | +			'idx_ezpus_lastname' => array('lastname'), | |
| 130 | +			'idx_ezpus_address1' => array('address1'), | |
| 131 | +			'idx_ezpus_postal' => array('postal'), | |
| 132 | +			'idx_ezpus_city' => array('city'), | |
| 133 | 133 | ); | 
| 134 | 134 | |
| 135 | - foreach( $indexes as $name => $def ) | |
| 135 | + foreach ($indexes as $name => $def) | |
| 136 | 136 |  		{ | 
| 137 | -			if( $table->hasIndex( $name ) === false ) { | |
| 138 | - $table->addIndex( $def, $name ); | |
| 137 | +			if ($table->hasIndex($name) === false) { | |
| 138 | + $table->addIndex($def, $name); | |
| 139 | 139 | } | 
| 140 | 140 | } | 
| 141 | 141 | |
| @@ -16,44 +16,44 @@ | ||
| 16 | 16 | |
| 17 | 17 | protected function setUp() | 
| 18 | 18 |  	{ | 
| 19 | - $customer = new \Aimeos\MShop\Customer\Manager\Ezpublish( \TestHelper::getContext() ); | |
| 19 | + $customer = new \Aimeos\MShop\Customer\Manager\Ezpublish(\TestHelper::getContext()); | |
| 20 | 20 | |
| 21 | - $this->object = $customer->getSubManager( 'address', 'Ezpublish' ); | |
| 21 | +		$this->object = $customer->getSubManager('address', 'Ezpublish'); | |
| 22 | 22 | } | 
| 23 | 23 | |
| 24 | 24 | |
| 25 | 25 | protected function tearDown() | 
| 26 | 26 |  	{ | 
| 27 | - unset( $this->object ); | |
| 27 | + unset($this->object); | |
| 28 | 28 | } | 
| 29 | 29 | |
| 30 | 30 | |
| 31 | 31 | public function testCleanup() | 
| 32 | 32 |  	{ | 
| 33 | - $this->object->cleanup( array( -1 ) ); | |
| 33 | + $this->object->cleanup(array( -1 )); | |
| 34 | 34 | } | 
| 35 | 35 | |
| 36 | 36 | |
| 37 | 37 | public function testDeleteItems() | 
| 38 | 38 |  	{ | 
| 39 | - $this->object->deleteItems( array( -1 ) ); | |
| 39 | + $this->object->deleteItems(array( -1 )); | |
| 40 | 40 | } | 
| 41 | 41 | |
| 42 | 42 | |
| 43 | 43 | public function testGetSearchAttributes() | 
| 44 | 44 |  	{ | 
| 45 | 45 | $attributes = $this->object->getSearchAttributes(); | 
| 46 | - $this->assertGreaterThan( 0, count( $attributes ) ); | |
| 46 | + $this->assertGreaterThan(0, count($attributes)); | |
| 47 | 47 | |
| 48 | -		foreach( $attributes as $attribute ) { | |
| 49 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute ); | |
| 48 | +		foreach ($attributes as $attribute) { | |
| 49 | +			$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute); | |
| 50 | 50 | } | 
| 51 | 51 | } | 
| 52 | 52 | |
| 53 | 53 | |
| 54 | 54 | public function testGetSubManager() | 
| 55 | 55 |  	{ | 
| 56 | - $this->setExpectedException( '\\Aimeos\\MShop\\Exception' ); | |
| 57 | - $this->object->getSubManager( 'unknown' ); | |
| 56 | +		$this->setExpectedException('\\Aimeos\\MShop\\Exception'); | |
| 57 | +		$this->object->getSubManager('unknown'); | |
| 58 | 58 | } | 
| 59 | 59 | } | 
| @@ -39,9 +39,9 @@ | ||
| 39 | 39 | * @param string $value password of the customer item | 
| 40 | 40 | * @return \Aimeos\MShop\Customer\Item\Iface Customer item for chaining method calls | 
| 41 | 41 | */ | 
| 42 | - public function setPassword( $value ) | |
| 42 | + public function setPassword($value) | |
| 43 | 43 |  	{ | 
| 44 | - if( $value !== '' && $value != $this->getPassword() ) | |
| 44 | + if ($value !== '' && $value != $this->getPassword()) | |
| 45 | 45 |  		{ | 
| 46 | 46 | $this->password = (string) $value; | 
| 47 | 47 | $this->setModified(); | 
| @@ -28,7 +28,7 @@ discard block | ||
| 28 | 28 | * @param \eZ\Publish\API\Repository\UserService $service eZ user service object | 
| 29 | 29 | * @return \Aimeos\MShop\Context\Item\Iface Context item for chaining method calls | 
| 30 | 30 | */ | 
| 31 | - public function setEzUserService( \eZ\Publish\API\Repository\UserService $service ) | |
| 31 | + public function setEzUserService(\eZ\Publish\API\Repository\UserService $service) | |
| 32 | 32 |  	{ | 
| 33 | 33 | $this->ezUserService = $service; | 
| 34 | 34 | return $this; | 
| @@ -42,8 +42,8 @@ discard block | ||
| 42 | 42 | */ | 
| 43 | 43 | public function getEzUserService() | 
| 44 | 44 |  	{ | 
| 45 | -		if( !isset( $this->ezUserService ) ) { | |
| 46 | - throw new \Aimeos\MShop\Exception( sprintf( 'eZ user service not available' ) ); | |
| 45 | +		if (!isset($this->ezUserService)) { | |
| 46 | +			throw new \Aimeos\MShop\Exception(sprintf('eZ user service not available')); | |
| 47 | 47 | } | 
| 48 | 48 | |
| 49 | 49 | return $this->ezUserService; | 
| @@ -248,7 +248,7 @@ | ||
| 248 | 248 | * | 
| 249 | 249 | * @param string $manager Name of the sub manager type in lower case | 
| 250 | 250 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 251 | - * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | |
| 251 | + * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc. | |
| 252 | 252 | */ | 
| 253 | 253 | public function getSubManager( $manager, $name = null ) | 
| 254 | 254 |  	{ | 
| @@ -26,7 +26,7 @@ discard block | ||
| 26 | 26 | 'label' => 'Customer address ID', | 
| 27 | 27 | 'code' => 'customer.address.id', | 
| 28 | 28 | 'internalcode' => 'ezuad."id"', | 
| 29 | - 'internaldeps' => array( 'LEFT JOIN "ezuser_address" AS ezuad ON ( ezu."contentobject_id" = ezuad."parentid" )' ), | |
| 29 | +			'internaldeps' => array('LEFT JOIN "ezuser_address" AS ezuad ON ( ezu."contentobject_id" = ezuad."parentid" )'), | |
| 30 | 30 | 'type' => 'integer', | 
| 31 | 31 | 'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT, | 
| 32 | 32 | 'public' => false, | 
| @@ -209,14 +209,14 @@ discard block | ||
| 209 | 209 | * | 
| 210 | 210 | * @param array $siteids List of IDs for sites whose entries should be deleted | 
| 211 | 211 | */ | 
| 212 | - public function cleanup( array $siteids ) | |
| 212 | + public function cleanup(array $siteids) | |
| 213 | 213 |  	{ | 
| 214 | 214 | $path = 'mshop/customer/manager/address/submanagers'; | 
| 215 | -		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) { | |
| 216 | - $this->getObject()->getSubManager( $domain )->cleanup( $siteids ); | |
| 215 | +		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) { | |
| 216 | + $this->getObject()->getSubManager($domain)->cleanup($siteids); | |
| 217 | 217 | } | 
| 218 | 218 | |
| 219 | - $this->cleanupBase( $siteids, 'mshop/customer/manager/lists/ezpublish/delete' ); | |
| 219 | + $this->cleanupBase($siteids, 'mshop/customer/manager/lists/ezpublish/delete'); | |
| 220 | 220 | } | 
| 221 | 221 | |
| 222 | 222 | |
| @@ -226,11 +226,11 @@ discard block | ||
| 226 | 226 | * @param boolean $withsub Return also attributes of sub-managers if true | 
| 227 | 227 | * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface | 
| 228 | 228 | */ | 
| 229 | - public function getSearchAttributes( $withsub = true ) | |
| 229 | + public function getSearchAttributes($withsub = true) | |
| 230 | 230 |  	{ | 
| 231 | 231 | $path = 'mshop/customer/manager/address/submanagers'; | 
| 232 | 232 | |
| 233 | - return $this->getSearchAttributesBase( $this->getSearchConfig(), $path, [], $withsub ); | |
| 233 | + return $this->getSearchAttributesBase($this->getSearchConfig(), $path, [], $withsub); | |
| 234 | 234 | } | 
| 235 | 235 | |
| 236 | 236 | |
| @@ -241,9 +241,9 @@ discard block | ||
| 241 | 241 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 242 | 242 | * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | 
| 243 | 243 | */ | 
| 244 | - public function getSubManager( $manager, $name = null ) | |
| 244 | + public function getSubManager($manager, $name = null) | |
| 245 | 245 |  	{ | 
| 246 | - return $this->getSubManagerBase( 'customer', 'address/' . $manager, ( $name === null ? 'Ezpublish' : $name ) ); | |
| 246 | +		return $this->getSubManagerBase('customer', 'address/'.$manager, ($name === null ? 'Ezpublish' : $name)); | |
| 247 | 247 | } | 
| 248 | 248 | |
| 249 | 249 | |
| @@ -248,7 +248,7 @@ | ||
| 248 | 248 | * | 
| 249 | 249 | * @param string $manager Name of the sub manager type in lower case | 
| 250 | 250 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 251 | - * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | |
| 251 | + * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc. | |
| 252 | 252 | */ | 
| 253 | 253 | public function getSubManager( $manager, $name = null ) | 
| 254 | 254 |  	{ | 
| @@ -25,7 +25,7 @@ discard block | ||
| 25 | 25 | 'customer.lists.id'=> array( | 
| 26 | 26 | 'code'=>'customer.lists.id', | 
| 27 | 27 | 'internalcode'=>'ezuli."id"', | 
| 28 | - 'internaldeps' => array( 'LEFT JOIN "ezuser_list" AS ezuli ON ( ezu."id" = ezuli."parentid" )' ), | |
| 28 | +			'internaldeps' => array('LEFT JOIN "ezuser_list" AS ezuli ON ( ezu."id" = ezuli."parentid" )'), | |
| 29 | 29 | 'label'=>'Customer list ID', | 
| 30 | 30 | 'type'=> 'integer', | 
| 31 | 31 | 'internaltype'=> \Aimeos\MW\DB\Statement\Base::PARAM_INT, | 
| @@ -132,14 +132,14 @@ discard block | ||
| 132 | 132 | * | 
| 133 | 133 | * @param array $siteids List of IDs for sites whose entries should be deleted | 
| 134 | 134 | */ | 
| 135 | - public function cleanup( array $siteids ) | |
| 135 | + public function cleanup(array $siteids) | |
| 136 | 136 |  	{ | 
| 137 | 137 | $path = 'mshop/customer/manager/lists/submanagers'; | 
| 138 | -		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) { | |
| 139 | - $this->getObject()->getSubManager( $domain )->cleanup( $siteids ); | |
| 138 | +		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) { | |
| 139 | + $this->getObject()->getSubManager($domain)->cleanup($siteids); | |
| 140 | 140 | } | 
| 141 | 141 | |
| 142 | - $this->cleanupBase( $siteids, $this->getConfigPath() . 'delete' ); | |
| 142 | + $this->cleanupBase($siteids, $this->getConfigPath().'delete'); | |
| 143 | 143 | } | 
| 144 | 144 | |
| 145 | 145 | |
| @@ -149,11 +149,11 @@ discard block | ||
| 149 | 149 | * @param boolean $withsub Return also attributes of sub-managers if true | 
| 150 | 150 | * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface | 
| 151 | 151 | */ | 
| 152 | - public function getSearchAttributes( $withsub = true ) | |
| 152 | + public function getSearchAttributes($withsub = true) | |
| 153 | 153 |  	{ | 
| 154 | 154 | $path = 'mshop/customer/manager/lists/submanagers'; | 
| 155 | 155 | |
| 156 | - return $this->getSearchAttributesBase( $this->getSearchConfig(), $path, [], $withsub ); | |
| 156 | + return $this->getSearchAttributesBase($this->getSearchConfig(), $path, [], $withsub); | |
| 157 | 157 | } | 
| 158 | 158 | |
| 159 | 159 | |
| @@ -164,9 +164,9 @@ discard block | ||
| 164 | 164 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 165 | 165 | * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | 
| 166 | 166 | */ | 
| 167 | - public function getSubManager( $manager, $name = null ) | |
| 167 | + public function getSubManager($manager, $name = null) | |
| 168 | 168 |  	{ | 
| 169 | - return $this->getSubManagerBase( 'customer', 'lists/' . $manager, ( $name === null ? 'Ezpublish' : $name ) ); | |
| 169 | +		return $this->getSubManagerBase('customer', 'lists/'.$manager, ($name === null ? 'Ezpublish' : $name)); | |
| 170 | 170 | } | 
| 171 | 171 | |
| 172 | 172 | |
| @@ -176,15 +176,15 @@ discard block | ||
| 176 | 176 | * @param \Aimeos\MShop\Common\Item\Lists\Iface $item List item object which should be saved | 
| 177 | 177 | * @param boolean $fetch True if the new ID should be returned in the item | 
| 178 | 178 | */ | 
| 179 | - public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true ) | |
| 179 | + public function saveItem(\Aimeos\MShop\Common\Item\Iface $item, $fetch = true) | |
| 180 | 180 |  	{ | 
| 181 | - self::checkClass( '\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $item ); | |
| 181 | +		self::checkClass('\\Aimeos\\MShop\\Common\\Item\\Lists\\Iface', $item); | |
| 182 | 182 | |
| 183 | -		if( $item->getDomain() === 'customer/group' ) { | |
| 184 | - throw new \Aimeos\MShop\Customer\Exception( sprintf( 'Adding groups to customers is not supported, please use the eZ Publish backend' ) ); | |
| 183 | +		if ($item->getDomain() === 'customer/group') { | |
| 184 | +			throw new \Aimeos\MShop\Customer\Exception(sprintf('Adding groups to customers is not supported, please use the eZ Publish backend')); | |
| 185 | 185 | } | 
| 186 | 186 | |
| 187 | - parent::saveItem( $item, $fetch ); | |
| 187 | + parent::saveItem($item, $fetch); | |
| 188 | 188 | } | 
| 189 | 189 | |
| 190 | 190 | |
| @@ -248,7 +248,7 @@ | ||
| 248 | 248 | * | 
| 249 | 249 | * @param string $manager Name of the sub manager type in lower case | 
| 250 | 250 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 251 | - * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | |
| 251 | + * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc. | |
| 252 | 252 | */ | 
| 253 | 253 | public function getSubManager( $manager, $name = null ) | 
| 254 | 254 |  	{ | 
| @@ -95,14 +95,14 @@ discard block | ||
| 95 | 95 | * | 
| 96 | 96 | * @param array $siteids List of IDs for sites whose entries should be deleted | 
| 97 | 97 | */ | 
| 98 | - public function cleanup( array $siteids ) | |
| 98 | + public function cleanup(array $siteids) | |
| 99 | 99 |  	{ | 
| 100 | 100 | $path = 'mshop/customer/manager/lists/type/submanagers'; | 
| 101 | -		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) { | |
| 102 | - $this->getObject()->getSubManager( $domain )->cleanup( $siteids ); | |
| 101 | +		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) { | |
| 102 | + $this->getObject()->getSubManager($domain)->cleanup($siteids); | |
| 103 | 103 | } | 
| 104 | 104 | |
| 105 | - $this->cleanupBase( $siteids, $this->getConfigPath() . 'delete' ); | |
| 105 | + $this->cleanupBase($siteids, $this->getConfigPath().'delete'); | |
| 106 | 106 | } | 
| 107 | 107 | |
| 108 | 108 | |
| @@ -112,11 +112,11 @@ discard block | ||
| 112 | 112 | * @param boolean $withsub Return also attributes of sub-managers if true | 
| 113 | 113 | * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface | 
| 114 | 114 | */ | 
| 115 | - public function getSearchAttributes( $withsub = true ) | |
| 115 | + public function getSearchAttributes($withsub = true) | |
| 116 | 116 |  	{ | 
| 117 | 117 | $path = 'mshop/customer/manager/lists/type/submanagers'; | 
| 118 | 118 | |
| 119 | - return $this->getSearchAttributesBase( $this->getSearchConfig(), $path, [], $withsub ); | |
| 119 | + return $this->getSearchAttributesBase($this->getSearchConfig(), $path, [], $withsub); | |
| 120 | 120 | } | 
| 121 | 121 | |
| 122 | 122 | |
| @@ -127,9 +127,9 @@ discard block | ||
| 127 | 127 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 128 | 128 | * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | 
| 129 | 129 | */ | 
| 130 | - public function getSubManager( $manager, $name = null ) | |
| 130 | + public function getSubManager($manager, $name = null) | |
| 131 | 131 |  	{ | 
| 132 | - return $this->getSubManagerBase( 'customer', 'lists/type/' . $manager, ( $name === null ? 'Ezpublish' : $name ) ); | |
| 132 | +		return $this->getSubManagerBase('customer', 'lists/type/'.$manager, ($name === null ? 'Ezpublish' : $name)); | |
| 133 | 133 | } | 
| 134 | 134 | |
| 135 | 135 | |
| @@ -226,7 +226,7 @@ discard block | ||
| 226 | 226 | /** | 
| 227 | 227 | * Removes old entries from the storage. | 
| 228 | 228 | * | 
| 229 | - * @param array $siteids List of IDs for sites whose entries should be deleted | |
| 229 | + * @param integer[] $siteids List of IDs for sites whose entries should be deleted | |
| 230 | 230 | */ | 
| 231 | 231 | public function cleanup( array $siteids ) | 
| 232 | 232 |  	{ | 
| @@ -251,7 +251,7 @@ discard block | ||
| 251 | 251 | /** | 
| 252 | 252 | * Removes multiple items specified by ids in the array. | 
| 253 | 253 | * | 
| 254 | - * @param array $ids List of IDs | |
| 254 | + * @param integer[] $ids List of IDs | |
| 255 | 255 | */ | 
| 256 | 256 | public function deleteItems( array $ids ) | 
| 257 | 257 |  	{ | 
| @@ -282,7 +282,7 @@ discard block | ||
| 282 | 282 | * | 
| 283 | 283 | * @param string $manager Name of the sub manager type in lower case | 
| 284 | 284 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 285 | - * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | |
| 285 | + * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc. | |
| 286 | 286 | */ | 
| 287 | 287 | public function getSubManager( $manager, $name = null ) | 
| 288 | 288 |  	{ | 
| @@ -295,7 +295,7 @@ discard block | ||
| 295 | 295 | * | 
| 296 | 296 | * @param \Aimeos\MShop\Customer\Item\Iface $item Customer item object | 
| 297 | 297 | * @param boolean $fetch True if the new ID should be returned in the item | 
| 298 | - * @return \Aimeos\MShop\Common\Item\Iface $item Updated item including the generated ID | |
| 298 | + * @return \Aimeos\MShop\Common\Item\ListRef\Iface $item Updated item including the generated ID | |
| 299 | 299 | */ | 
| 300 | 300 | public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true ) | 
| 301 | 301 |  	{ | 
| @@ -226,11 +226,11 @@ discard block | ||
| 226 | 226 | * | 
| 227 | 227 | * @param array $siteids List of IDs for sites whose entries should be deleted | 
| 228 | 228 | */ | 
| 229 | - public function cleanup( array $siteids ) | |
| 229 | + public function cleanup(array $siteids) | |
| 230 | 230 |  	{ | 
| 231 | 231 | $path = 'mshop/customer/manager/submanagers'; | 
| 232 | -		foreach( $this->getContext()->getConfig()->get( $path, ['address', 'group', 'lists', 'property'] ) as $domain ) { | |
| 233 | - $this->getObject()->getSubManager( $domain )->cleanup( $siteids ); | |
| 232 | +		foreach ($this->getContext()->getConfig()->get($path, ['address', 'group', 'lists', 'property']) as $domain) { | |
| 233 | + $this->getObject()->getSubManager($domain)->cleanup($siteids); | |
| 234 | 234 | } | 
| 235 | 235 | } | 
| 236 | 236 | |
| @@ -240,12 +240,12 @@ discard block | ||
| 240 | 240 | * | 
| 241 | 241 | * @param array $ids List of IDs | 
| 242 | 242 | */ | 
| 243 | - public function deleteItems( array $ids ) | |
| 243 | + public function deleteItems(array $ids) | |
| 244 | 244 |  	{ | 
| 245 | 245 | $service = $this->getContext()->getEzUserService(); | 
| 246 | 246 | |
| 247 | -		foreach( $ids as $id ) { | |
| 248 | - $service->deleteUser( $service->loadUser( $id ) ); | |
| 247 | +		foreach ($ids as $id) { | |
| 248 | + $service->deleteUser($service->loadUser($id)); | |
| 249 | 249 | } | 
| 250 | 250 | } | 
| 251 | 251 | |
| @@ -256,11 +256,11 @@ discard block | ||
| 256 | 256 | * @param boolean $withsub Return also attributes of sub-managers if true | 
| 257 | 257 | * @return array List of attribute items implementing \Aimeos\MW\Criteria\Attribute\Iface | 
| 258 | 258 | */ | 
| 259 | - public function getSearchAttributes( $withsub = true ) | |
| 259 | + public function getSearchAttributes($withsub = true) | |
| 260 | 260 |  	{ | 
| 261 | 261 | $path = 'mshop/customer/manager/submanagers'; | 
| 262 | 262 | |
| 263 | - return $this->getSearchAttributesBase( $this->searchConfig, $path, ['address', 'group', 'lists', 'property'], $withsub ); | |
| 263 | + return $this->getSearchAttributesBase($this->searchConfig, $path, ['address', 'group', 'lists', 'property'], $withsub); | |
| 264 | 264 | } | 
| 265 | 265 | |
| 266 | 266 | |
| @@ -271,9 +271,9 @@ discard block | ||
| 271 | 271 | * @param string|null $name Name of the implementation, will be from configuration (or Default) if null | 
| 272 | 272 | * @return mixed Manager for different extensions, e.g stock, tags, locations, etc. | 
| 273 | 273 | */ | 
| 274 | - public function getSubManager( $manager, $name = null ) | |
| 274 | + public function getSubManager($manager, $name = null) | |
| 275 | 275 |  	{ | 
| 276 | - return $this->getSubManagerBase( 'customer', $manager, ( $name === null ? 'Ezpublish' : $name ) ); | |
| 276 | +		return $this->getSubManagerBase('customer', $manager, ($name === null ? 'Ezpublish' : $name)); | |
| 277 | 277 | } | 
| 278 | 278 | |
| 279 | 279 | |
| @@ -284,96 +284,96 @@ discard block | ||
| 284 | 284 | * @param boolean $fetch True if the new ID should be returned in the item | 
| 285 | 285 | * @return \Aimeos\MShop\Common\Item\Iface $item Updated item including the generated ID | 
| 286 | 286 | */ | 
| 287 | - public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true ) | |
| 287 | + public function saveItem(\Aimeos\MShop\Common\Item\Iface $item, $fetch = true) | |
| 288 | 288 |  	{ | 
| 289 | - self::checkClass( '\\Aimeos\\MShop\\Customer\\Item\\Iface', $item ); | |
| 289 | +		self::checkClass('\\Aimeos\\MShop\\Customer\\Item\\Iface', $item); | |
| 290 | 290 | |
| 291 | - if( !$item->isModified() ) | |
| 291 | + if (!$item->isModified()) | |
| 292 | 292 |  		{ | 
| 293 | - $item = $this->savePropertyItems( $item, 'customer' ); | |
| 294 | - $item = $this->saveAddressItems( $item, 'customer' ); | |
| 295 | - return $this->saveListItems( $item, 'customer' ); | |
| 293 | + $item = $this->savePropertyItems($item, 'customer'); | |
| 294 | + $item = $this->saveAddressItems($item, 'customer'); | |
| 295 | + return $this->saveListItems($item, 'customer'); | |
| 296 | 296 | } | 
| 297 | 297 | |
| 298 | 298 | $context = $this->getContext(); | 
| 299 | 299 | |
| 300 | 300 | $class = '\Aimeos\MShop\Context\Item\Ezpublish'; | 
| 301 | -		if( !is_a( $context, $class ) ) { | |
| 302 | - throw new \Aimeos\MShop\Customer\Exception( sprintf( 'Object is not of required type "%1$s"', $class ) ); | |
| 301 | +		if (!is_a($context, $class)) { | |
| 302 | +			throw new \Aimeos\MShop\Customer\Exception(sprintf('Object is not of required type "%1$s"', $class)); | |
| 303 | 303 | } | 
| 304 | 304 | |
| 305 | 305 | $service = $context->getEzUserService(); | 
| 306 | 306 | $email = $item->getPaymentAddress()->getEmail(); | 
| 307 | 307 | |
| 308 | - if( $item->getId() !== null ) | |
| 308 | + if ($item->getId() !== null) | |
| 309 | 309 |  		{ | 
| 310 | 310 | $struct = $service->newUserUpdateStruct(); | 
| 311 | 311 | $struct->password = $item->getPassword(); | 
| 312 | 312 | $struct->enabled = $item->getStatus(); | 
| 313 | 313 | $struct->email = $email; | 
| 314 | 314 | |
| 315 | - $user = $service->loadUser( $item->getId() ); | |
| 316 | - $service->updateUser( $user, $struct ); | |
| 315 | + $user = $service->loadUser($item->getId()); | |
| 316 | + $service->updateUser($user, $struct); | |
| 317 | 317 | } | 
| 318 | 318 | else | 
| 319 | 319 |  		{ | 
| 320 | - $struct = $service->newUserCreateStruct( $item->getCode(), $email, $item->getPassword(), 'eng-GB' ); | |
| 320 | + $struct = $service->newUserCreateStruct($item->getCode(), $email, $item->getPassword(), 'eng-GB'); | |
| 321 | 321 | $struct->enabled = $item->getStatus(); | 
| 322 | 322 | |
| 323 | - $user = $service->createUser( $struct, [] ); | |
| 324 | - $item->setId( $user->getUserId() ); | |
| 323 | + $user = $service->createUser($struct, []); | |
| 324 | + $item->setId($user->getUserId()); | |
| 325 | 325 | } | 
| 326 | 326 | |
| 327 | 327 | $dbm = $context->getDatabaseManager(); | 
| 328 | 328 | $dbname = $this->getResourceName(); | 
| 329 | - $conn = $dbm->acquire( $dbname ); | |
| 329 | + $conn = $dbm->acquire($dbname); | |
| 330 | 330 | |
| 331 | 331 | try | 
| 332 | 332 |  		{ | 
| 333 | - $date = date( 'Y-m-d H:i:s' ); | |
| 334 | - $ctime = ( $item->getTimeCreated() ? $item->getTimeCreated() : $date ); | |
| 333 | +			$date = date('Y-m-d H:i:s'); | |
| 334 | + $ctime = ($item->getTimeCreated() ? $item->getTimeCreated() : $date); | |
| 335 | 335 | $billingAddress = $item->getPaymentAddress(); | 
| 336 | 336 | |
| 337 | 337 | $path = 'mshop/customer/manager/ezpublish/update'; | 
| 338 | - $stmt = $this->getCachedStatement( $conn, $path ); | |
| 339 | - | |
| 340 | - $stmt->bind( 1, $billingAddress->getCompany() ); | |
| 341 | - $stmt->bind( 2, $billingAddress->getVatID() ); | |
| 342 | - $stmt->bind( 3, $billingAddress->getSalutation() ); | |
| 343 | - $stmt->bind( 4, $billingAddress->getTitle() ); | |
| 344 | - $stmt->bind( 5, $billingAddress->getFirstname() ); | |
| 345 | - $stmt->bind( 6, $billingAddress->getLastname() ); | |
| 346 | - $stmt->bind( 7, $billingAddress->getAddress1() ); | |
| 347 | - $stmt->bind( 8, $billingAddress->getAddress2() ); | |
| 348 | - $stmt->bind( 9, $billingAddress->getAddress3() ); | |
| 349 | - $stmt->bind( 10, $billingAddress->getPostal() ); | |
| 350 | - $stmt->bind( 11, $billingAddress->getCity() ); | |
| 351 | - $stmt->bind( 12, $billingAddress->getState() ); | |
| 352 | - $stmt->bind( 13, $billingAddress->getCountryId() ); | |
| 353 | - $stmt->bind( 14, $billingAddress->getLanguageId() ); | |
| 354 | - $stmt->bind( 15, $billingAddress->getTelephone() ); | |
| 355 | - $stmt->bind( 16, $billingAddress->getTelefax() ); | |
| 356 | - $stmt->bind( 17, $billingAddress->getWebsite() ); | |
| 357 | - $stmt->bind( 18, $item->getBirthday() ); | |
| 358 | - $stmt->bind( 19, $item->getDateVerified() ); | |
| 359 | - $stmt->bind( 20, $date ); // Modification time | |
| 360 | - $stmt->bind( 21, $context->getEditor() ); | |
| 361 | - $stmt->bind( 22, $ctime ); // Creation time | |
| 362 | - $stmt->bind( 23, $item->getId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT ); | |
| 338 | + $stmt = $this->getCachedStatement($conn, $path); | |
| 339 | + | |
| 340 | + $stmt->bind(1, $billingAddress->getCompany()); | |
| 341 | + $stmt->bind(2, $billingAddress->getVatID()); | |
| 342 | + $stmt->bind(3, $billingAddress->getSalutation()); | |
| 343 | + $stmt->bind(4, $billingAddress->getTitle()); | |
| 344 | + $stmt->bind(5, $billingAddress->getFirstname()); | |
| 345 | + $stmt->bind(6, $billingAddress->getLastname()); | |
| 346 | + $stmt->bind(7, $billingAddress->getAddress1()); | |
| 347 | + $stmt->bind(8, $billingAddress->getAddress2()); | |
| 348 | + $stmt->bind(9, $billingAddress->getAddress3()); | |
| 349 | + $stmt->bind(10, $billingAddress->getPostal()); | |
| 350 | + $stmt->bind(11, $billingAddress->getCity()); | |
| 351 | + $stmt->bind(12, $billingAddress->getState()); | |
| 352 | + $stmt->bind(13, $billingAddress->getCountryId()); | |
| 353 | + $stmt->bind(14, $billingAddress->getLanguageId()); | |
| 354 | + $stmt->bind(15, $billingAddress->getTelephone()); | |
| 355 | + $stmt->bind(16, $billingAddress->getTelefax()); | |
| 356 | + $stmt->bind(17, $billingAddress->getWebsite()); | |
| 357 | + $stmt->bind(18, $item->getBirthday()); | |
| 358 | + $stmt->bind(19, $item->getDateVerified()); | |
| 359 | + $stmt->bind(20, $date); // Modification time | |
| 360 | + $stmt->bind(21, $context->getEditor()); | |
| 361 | + $stmt->bind(22, $ctime); // Creation time | |
| 362 | + $stmt->bind(23, $item->getId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT); | |
| 363 | 363 | |
| 364 | 364 | $stmt->execute()->finish(); | 
| 365 | 365 | |
| 366 | - $dbm->release( $conn, $dbname ); | |
| 366 | + $dbm->release($conn, $dbname); | |
| 367 | 367 | } | 
| 368 | - catch( \Exception $e ) | |
| 368 | + catch (\Exception $e) | |
| 369 | 369 |  		{ | 
| 370 | - $dbm->release( $conn, $dbname ); | |
| 370 | + $dbm->release($conn, $dbname); | |
| 371 | 371 | throw $e; | 
| 372 | 372 | } | 
| 373 | 373 | |
| 374 | - $item = $this->savePropertyItems( $item, 'customer' ); | |
| 375 | - $item = $this->saveAddressItems( $item, 'customer' ); | |
| 376 | - return $this->saveListItems( $item, 'customer' ); | |
| 374 | + $item = $this->savePropertyItems($item, 'customer'); | |
| 375 | + $item = $this->saveAddressItems($item, 'customer'); | |
| 376 | + return $this->saveListItems($item, 'customer'); | |
| 377 | 377 | } | 
| 378 | 378 | |
| 379 | 379 | |
| @@ -385,11 +385,11 @@ discard block | ||
| 385 | 385 | * @return array List of items implementing \Aimeos\MShop\Customer\Item\Iface | 
| 386 | 386 | * @throws \Aimeos\MShop\Customer\Exception If creating items failed | 
| 387 | 387 | */ | 
| 388 | - public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null ) | |
| 388 | + public function searchItems(\Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null) | |
| 389 | 389 |  	{ | 
| 390 | 390 | $dbm = $this->getContext()->getDatabaseManager(); | 
| 391 | 391 | $dbname = $this->getResourceName(); | 
| 392 | - $conn = $dbm->acquire( $dbname ); | |
| 392 | + $conn = $dbm->acquire($dbname); | |
| 393 | 393 | $map = []; | 
| 394 | 394 | |
| 395 | 395 | try | 
| @@ -397,34 +397,34 @@ discard block | ||
| 397 | 397 | $level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL; | 
| 398 | 398 | $cfgPathSearch = 'mshop/customer/manager/ezpublish/search'; | 
| 399 | 399 | $cfgPathCount = 'mshop/customer/manager/ezpublish/count'; | 
| 400 | - $required = array( 'customer' ); | |
| 400 | +			$required = array('customer'); | |
| 401 | 401 | |
| 402 | - $results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level ); | |
| 402 | + $results = $this->searchItemsBase($conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level); | |
| 403 | 403 | |
| 404 | - while( ( $row = $results->fetch() ) !== false ) | |
| 404 | + while (($row = $results->fetch()) !== false) | |
| 405 | 405 |  			{ | 
| 406 | - $map[ $row['customer.id'] ] = $row; | |
| 407 | - $map[ $row['customer.id'] ]['groups'] = []; | |
| 406 | + $map[$row['customer.id']] = $row; | |
| 407 | + $map[$row['customer.id']]['groups'] = []; | |
| 408 | 408 | } | 
| 409 | 409 | |
| 410 | 410 | |
| 411 | 411 | $path = 'mshop/customer/manager/ezpublish/groups'; | 
| 412 | - $stmt = $conn->create( $this->getGroupSql( array_keys( $map ), $path ) ); | |
| 412 | + $stmt = $conn->create($this->getGroupSql(array_keys($map), $path)); | |
| 413 | 413 | $results = $stmt->execute(); | 
| 414 | 414 | |
| 415 | -			while( ( $row = $results->fetch() ) !== false ) { | |
| 416 | - $map[ $row['contentobject_id'] ]['groups'][] = $row['role_id']; | |
| 415 | +			while (($row = $results->fetch()) !== false) { | |
| 416 | + $map[$row['contentobject_id']]['groups'][] = $row['role_id']; | |
| 417 | 417 | } | 
| 418 | 418 | |
| 419 | - $dbm->release( $conn, $dbname ); | |
| 419 | + $dbm->release($conn, $dbname); | |
| 420 | 420 | } | 
| 421 | - catch( \Exception $e ) | |
| 421 | + catch (\Exception $e) | |
| 422 | 422 |  		{ | 
| 423 | - $dbm->release( $conn, $dbname ); | |
| 423 | + $dbm->release($conn, $dbname); | |
| 424 | 424 | throw $e; | 
| 425 | 425 | } | 
| 426 | 426 | |
| 427 | - return $this->buildItems( $map, $ref, 'customer' ); | |
| 427 | + return $this->buildItems($map, $ref, 'customer'); | |
| 428 | 428 | } | 
| 429 | 429 | |
| 430 | 430 | |
| @@ -438,10 +438,10 @@ discard block | ||
| 438 | 438 | * @param \Aimeos\MShop\Common\Item\Property\Iface[] $propItems List of property items | 
| 439 | 439 | * @return \Aimeos\MShop\Customer\Item\Iface New customer item | 
| 440 | 440 | */ | 
| 441 | - protected function createItemBase( array $values = [], array $listItems = [], array $refItems = [], | |
| 442 | - array $addrItems = [], array $propItems = [] ) | |
| 441 | + protected function createItemBase(array $values = [], array $listItems = [], array $refItems = [], | |
| 442 | + array $addrItems = [], array $propItems = []) | |
| 443 | 443 |  	{ | 
| 444 | - $address = new \Aimeos\MShop\Common\Item\Address\Simple( 'customer.', $values ); | |
| 444 | +		$address = new \Aimeos\MShop\Common\Item\Address\Simple('customer.', $values); | |
| 445 | 445 | |
| 446 | 446 | return new \Aimeos\MShop\Customer\Item\Ezpublish( | 
| 447 | 447 | $address, $values, $listItems, $refItems, $addrItems, $propItems | 
| @@ -456,18 +456,18 @@ discard block | ||
| 456 | 456 | * @param string $cfgpath Configuration path to the SQL statement | 
| 457 | 457 | * @return string SQL statement ready for execution | 
| 458 | 458 | */ | 
| 459 | - protected function getGroupSql( array $ids, $cfgpath ) | |
| 459 | + protected function getGroupSql(array $ids, $cfgpath) | |
| 460 | 460 |  	{ | 
| 461 | -		if( empty( $ids ) ) { return '1=1'; } | |
| 461 | +		if (empty($ids)) { return '1=1'; } | |
| 462 | 462 | |
| 463 | 463 | $search = $this->getObject()->createSearch(); | 
| 464 | - $search->setConditions( $search->compare( '==', 'id', $ids ) ); | |
| 464 | +		$search->setConditions($search->compare('==', 'id', $ids)); | |
| 465 | 465 | |
| 466 | - $types = array( 'id' => \Aimeos\MW\DB\Statement\Base::PARAM_INT ); | |
| 467 | - $translations = array( 'id' => '"contentobject_id"' ); | |
| 466 | +		$types = array('id' => \Aimeos\MW\DB\Statement\Base::PARAM_INT); | |
| 467 | +		$translations = array('id' => '"contentobject_id"'); | |
| 468 | 468 | |
| 469 | - $cond = $search->getConditionSource( $types, $translations ); | |
| 469 | + $cond = $search->getConditionSource($types, $translations); | |
| 470 | 470 | |
| 471 | - return str_replace( ':cond', $cond, $this->getSqlConfig( $cfgpath ) ); | |
| 471 | +		return str_replace(':cond', $cond, $this->getSqlConfig($cfgpath)); | |
| 472 | 472 | } | 
| 473 | 473 | } |