Completed
Push — master ( 500452...c6784a )
by Aimeos
01:42
created
lib/custom/setup/CustomerChangeAddressRefidParentidFosUser.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	public function getPostDependencies()
42 42
 	{
43
-		return array( 'TablesCreateFosUser' );
43
+		return array('TablesCreateFosUser');
44 44
 	}
45 45
 
46 46
 
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 	 */
50 50
 	public function migrate()
51 51
 	{
52
-		$this->process( $this->mysql );
52
+		$this->process($this->mysql);
53 53
 	}
54 54
 
55 55
 
@@ -58,22 +58,22 @@  discard block
 block discarded – undo
58 58
 	 *
59 59
 	 * array string $stmts List of SQL statements for changing the columns
60 60
 	 */
61
-	protected function process( array $stmts )
61
+	protected function process(array $stmts)
62 62
 	{
63 63
 		$table = 'fos_user_address';
64
-		$this->msg( sprintf( 'Rename "refid" to "parentid" in table "%1$s"', $table ), 0 ); $this->status( '' );
64
+		$this->msg(sprintf('Rename "refid" to "parentid" in table "%1$s"', $table), 0); $this->status('');
65 65
 
66
-		foreach( $stmts as $column => $stmts )
66
+		foreach ($stmts as $column => $stmts)
67 67
 		{
68
-			$this->msg( sprintf( 'Checking column "%1$s"', $column ), 1 );
68
+			$this->msg(sprintf('Checking column "%1$s"', $column), 1);
69 69
 
70
-			if( $this->schema->tableExists( $table )
71
-				&& $this->schema->columnExists( $table, $column ) === true
70
+			if ($this->schema->tableExists($table)
71
+				&& $this->schema->columnExists($table, $column) === true
72 72
 			) {
73
-				$this->executeList( $stmts );
74
-				$this->status( 'done' );
73
+				$this->executeList($stmts);
74
+				$this->status('done');
75 75
 			} else {
76
-				$this->status( 'OK' );
76
+				$this->status('OK');
77 77
 			}
78 78
 		}
79 79
 	}
Please login to merge, or discard this patch.
lib/custom/setup/unittest/data/customer-property.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -5,12 +5,12 @@
 block discarded – undo
5 5
  * @copyright Aimeos (aimeos.org), 2018
6 6
  */
7 7
 
8
-return array (
8
+return array(
9 9
 	'customer/property/type' => array(
10
-		'customer/property/type/newsletter' => array( 'domain' => 'customer', 'code' => 'newsletter', 'label' => 'Newsletter', 'status' => 1 ),
10
+		'customer/property/type/newsletter' => array('domain' => 'customer', 'code' => 'newsletter', 'label' => 'Newsletter', 'status' => 1),
11 11
 	),
12 12
 
13 13
 	'customer/property' => array(
14
-		'customer/property/UTC001/newsletter' => array( 'parentid' => 'customer/UTC001', 'typeid' => 'customer/property/type/newsletter', 'langid' => null, 'value' => '1' ),
14
+		'customer/property/UTC001/newsletter' => array('parentid' => 'customer/UTC001', 'typeid' => 'customer/property/type/newsletter', 'langid' => null, 'value' => '1'),
15 15
 	),
16 16
 );
17 17
\ No newline at end of file
Please login to merge, or discard this patch.
lib/custom/setup/CustomerRemoveLostUserDataFosuser.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 	 */
39 39
 	public function getPostDependencies()
40 40
 	{
41
-		return array( 'TablesCreateMShop' );
41
+		return array('TablesCreateMShop');
42 42
 	}
43 43
 
44 44
 
@@ -47,29 +47,29 @@  discard block
 block discarded – undo
47 47
 	 */
48 48
 	public function migrate()
49 49
 	{
50
-		$this->msg( 'Remove left over FOS user address records', 0 );
50
+		$this->msg('Remove left over FOS user address records', 0);
51 51
 
52
-		if( $this->schema->tableExists( 'fos_user' ) && $this->schema->tableExists( 'fos_user_address' ) )
52
+		if ($this->schema->tableExists('fos_user') && $this->schema->tableExists('fos_user_address'))
53 53
 		{
54
-			$this->execute( $this->sql['address'] );
55
-			$this->status( 'done' );
54
+			$this->execute($this->sql['address']);
55
+			$this->status('done');
56 56
 		}
57 57
 		else
58 58
 		{
59
-			$this->status( 'OK' );
59
+			$this->status('OK');
60 60
 		}
61 61
 
62 62
 
63
-		$this->msg( 'Remove left over FOS user list records', 0 );
63
+		$this->msg('Remove left over FOS user list records', 0);
64 64
 
65
-		if( $this->schema->tableExists( 'fos_user' ) && $this->schema->tableExists( 'fos_user_list' ) )
65
+		if ($this->schema->tableExists('fos_user') && $this->schema->tableExists('fos_user_list'))
66 66
 		{
67
-			$this->execute( $this->sql['list'] );
68
-			$this->status( 'done' );
67
+			$this->execute($this->sql['list']);
68
+			$this->status('done');
69 69
 		}
70 70
 		else
71 71
 		{
72
-			$this->status( 'OK' );
72
+			$this->status('OK');
73 73
 		}
74 74
 	}
75 75
 }
Please login to merge, or discard this patch.
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -53,8 +53,7 @@  discard block
 block discarded – undo
53 53
 		{
54 54
 			$this->execute( $this->sql['address'] );
55 55
 			$this->status( 'done' );
56
-		}
57
-		else
56
+		} else
58 57
 		{
59 58
 			$this->status( 'OK' );
60 59
 		}
@@ -66,8 +65,7 @@  discard block
 block discarded – undo
66 65
 		{
67 66
 			$this->execute( $this->sql['list'] );
68 67
 			$this->status( 'done' );
69
-		}
70
-		else
68
+		} else
71 69
 		{
72 70
 			$this->status( 'OK' );
73 71
 		}
Please login to merge, or discard this patch.
lib/custom/src/MShop/Customer/Manager/Property/Type/FosUser.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 		'customer.property.type.id' => array(
25 25
 			'code' => 'customer.property.type.id',
26 26
 			'internalcode' => 'fosprty."id"',
27
-			'internaldeps' => array( 'LEFT JOIN "fos_user_property_type" AS fosprty ON ( fospr."typeid" = fosprty."id" )' ),
27
+			'internaldeps' => array('LEFT JOIN "fos_user_property_type" AS fosprty ON ( fospr."typeid" = fosprty."id" )'),
28 28
 			'label' => 'Property type ID',
29 29
 			'type' => 'integer',
30 30
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -105,14 +105,14 @@  discard block
 block discarded – undo
105 105
 	 *
106 106
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
107 107
 	 */
108
-	public function cleanup( array $siteids )
108
+	public function cleanup(array $siteids)
109 109
 	{
110 110
 		$path = 'mshop/customer/manager/property/type/submanagers';
111
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
112
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
111
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
112
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
113 113
 		}
114 114
 
115
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/property/type/fosuser/delete' );
115
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/type/fosuser/delete');
116 116
 	}
117 117
 
118 118
 
@@ -122,11 +122,11 @@  discard block
 block discarded – undo
122 122
 	 * @param boolean $withsub Return also attributes of sub-managers if true
123 123
 	 * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface
124 124
 	 */
125
-	public function getSearchAttributes( $withsub = true )
125
+	public function getSearchAttributes($withsub = true)
126 126
 	{
127 127
 		$path = 'mshop/customer/manager/property/type/submanagers';
128 128
 
129
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, [], $withsub );
129
+		return $this->getSearchAttributesBase($this->searchConfig, $path, [], $withsub);
130 130
 	}
131 131
 
132 132
 
@@ -137,9 +137,9 @@  discard block
 block discarded – undo
137 137
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
138 138
 	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g types, lists etc.
139 139
 	 */
140
-	public function getSubManager( $manager, $name = null )
140
+	public function getSubManager($manager, $name = null)
141 141
 	{
142
-		return $this->getSubManagerBase( 'customer', 'property/type/' . $manager, ( $name === null ? 'FosUser' : $name ) );
142
+		return $this->getSubManagerBase('customer', 'property/type/' . $manager, ($name === null ? 'FosUser' : $name));
143 143
 	}
144 144
 
145 145
 
Please login to merge, or discard this patch.
lib/custom/src/MShop/Customer/Manager/Property/FosUser.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 		'customer.property.id' => array(
25 25
 			'code' => 'customer.property.id',
26 26
 			'internalcode' => 'fospr."id"',
27
-			'internaldeps'=>array( 'LEFT JOIN "fos_user_property" AS fospr ON ( fospr."parentid" = fos."id" )' ),
27
+			'internaldeps'=>array('LEFT JOIN "fos_user_property" AS fospr ON ( fospr."parentid" = fos."id" )'),
28 28
 			'label' => 'Property ID',
29 29
 			'type' => 'integer',
30 30
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -100,14 +100,14 @@  discard block
 block discarded – undo
100 100
 	 *
101 101
 	 * @param integer[] $siteids List of IDs for sites whose entries should be deleted
102 102
 	 */
103
-	public function cleanup( array $siteids )
103
+	public function cleanup(array $siteids)
104 104
 	{
105 105
 		$path = 'mshop/customer/manager/property/submanagers';
106
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
107
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
106
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
107
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
108 108
 		}
109 109
 
110
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/property/fosuser/delete' );
110
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/fosuser/delete');
111 111
 	}
112 112
 
113 113
 
@@ -117,11 +117,11 @@  discard block
 block discarded – undo
117 117
 	 * @param boolean $withsub Return also attributes of sub-managers if true
118 118
 	 * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface
119 119
 	 */
120
-	public function getSearchAttributes( $withsub = true )
120
+	public function getSearchAttributes($withsub = true)
121 121
 	{
122 122
 		$path = 'mshop/customer/manager/property/submanagers';
123 123
 
124
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, ['type'], $withsub );
124
+		return $this->getSearchAttributesBase($this->searchConfig, $path, ['type'], $withsub);
125 125
 	}
126 126
 
127 127
 
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
 	 * configuration (or Default) if null
134 134
 	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g property types, property lists etc.
135 135
 	 */
136
-	public function getSubManager( $manager, $name = null )
136
+	public function getSubManager($manager, $name = null)
137 137
 	{
138
-		return $this->getSubManagerBase( 'customer', 'property/' . $manager, ( $name === null ? 'FosUser' : $name ) );
138
+		return $this->getSubManagerBase('customer', 'property/' . $manager, ($name === null ? 'FosUser' : $name));
139 139
 	}
140 140
 
141 141
 
Please login to merge, or discard this patch.
lib/custom/setup/unittest/CustomerAddFosUserTestData.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 	 */
22 22
 	public function getPreDependencies()
23 23
 	{
24
-		return array( 'TablesCreateFosUser' );
24
+		return array('TablesCreateFosUser');
25 25
 	}
26 26
 
27 27
 
@@ -30,36 +30,36 @@  discard block
 block discarded – undo
30 30
 	 */
31 31
 	public function migrate()
32 32
 	{
33
-		\Aimeos\MW\Common\Base::checkClass( '\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional );
33
+		\Aimeos\MW\Common\Base::checkClass('\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional);
34 34
 
35
-		$this->msg( 'Adding Fos user bundle customer test data', 0 );
35
+		$this->msg('Adding Fos user bundle customer test data', 0);
36 36
 
37 37
 		$parentIds = [];
38 38
 		$ds = DIRECTORY_SEPARATOR;
39
-		$this->additional->setEditor( 'ai-fosuser:unittest' );
39
+		$this->additional->setEditor('ai-fosuser:unittest');
40 40
 		$path = __DIR__ . $ds . 'data' . $ds . 'customer.php';
41 41
 
42
-		if( ( $testdata = include( $path ) ) === false ){
43
-			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for customer domain', $path ) );
42
+		if (($testdata = include($path)) === false) {
43
+			throw new \Aimeos\MShop\Exception(sprintf('No file "%1$s" found for customer domain', $path));
44 44
 		}
45 45
 
46 46
 
47
-		$customerManager = \Aimeos\MShop\Customer\Manager\Factory::createManager( $this->additional, 'FosUser' );
48
-		$customerAddressManager = $customerManager->getSubManager( 'address', 'FosUser' );
47
+		$customerManager = \Aimeos\MShop\Customer\Manager\Factory::createManager($this->additional, 'FosUser');
48
+		$customerAddressManager = $customerManager->getSubManager('address', 'FosUser');
49 49
 
50 50
 		$search = $customerManager->createSearch();
51
-		$search->setConditions( $search->compare( '==', 'customer.code', array( 'UTC001', 'UTC002', 'UTC003' ) ) );
52
-		$items = $customerManager->searchItems( $search );
51
+		$search->setConditions($search->compare('==', 'customer.code', array('UTC001', 'UTC002', 'UTC003')));
52
+		$items = $customerManager->searchItems($search);
53 53
 
54 54
 		$this->conn->begin();
55 55
 
56
-		$customerManager->deleteItems( array_keys( $items ) );
57
-		$parentIds = $this->addCustomerData( $testdata, $customerManager, $customerAddressManager->createItem() );
58
-		$this->addCustomerAddressData( $testdata, $customerAddressManager, $parentIds );
56
+		$customerManager->deleteItems(array_keys($items));
57
+		$parentIds = $this->addCustomerData($testdata, $customerManager, $customerAddressManager->createItem());
58
+		$this->addCustomerAddressData($testdata, $customerAddressManager, $parentIds);
59 59
 
60 60
 		$this->conn->commit();
61 61
 
62 62
 
63
-		$this->status( 'done' );
63
+		$this->status('done');
64 64
 	}
65 65
 }
Please login to merge, or discard this patch.
lib/custom/setup/unittest/CustomerListAddFosUserTestData.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 	 */
23 23
 	public function getPreDependencies()
24 24
 	{
25
-		return array( 'TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData', 'TextAddTestData' );
25
+		return array('TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData', 'TextAddTestData');
26 26
 	}
27 27
 
28 28
 
@@ -31,27 +31,27 @@  discard block
 block discarded – undo
31 31
 	 */
32 32
 	public function migrate()
33 33
 	{
34
-		\Aimeos\MW\Common\Base::checkClass( '\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional );
34
+		\Aimeos\MW\Common\Base::checkClass('\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional);
35 35
 
36
-		$this->msg( 'Adding customer-list Fos user bundle test data', 0 );
37
-		$this->additional->setEditor( 'ai-fosuser:unittest' );
36
+		$this->msg('Adding customer-list Fos user bundle test data', 0);
37
+		$this->additional->setEditor('ai-fosuser:unittest');
38 38
 
39 39
 		$ds = DIRECTORY_SEPARATOR;
40
-		$path = dirname( __FILE__ ) . $ds . 'data' . $ds . 'customer-list.php';
40
+		$path = dirname(__FILE__) . $ds . 'data' . $ds . 'customer-list.php';
41 41
 
42
-		if( ( $testdata = include( $path ) ) === false ){
43
-			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for customer list domain', $path ) );
42
+		if (($testdata = include($path)) === false) {
43
+			throw new \Aimeos\MShop\Exception(sprintf('No file "%1$s" found for customer list domain', $path));
44 44
 		}
45 45
 
46 46
 		$refKeys = [];
47
-		foreach( $testdata['customer/lists'] as $dataset ) {
48
-			$refKeys[ $dataset['domain'] ][] = $dataset['refid'];
47
+		foreach ($testdata['customer/lists'] as $dataset) {
48
+			$refKeys[$dataset['domain']][] = $dataset['refid'];
49 49
 		}
50 50
 
51 51
 		$refIds = [];
52
-		$refIds['text'] = $this->getTextData( $refKeys['text'] );
53
-		$this->addCustomerListData( $testdata, $refIds, 'FosUser' );
52
+		$refIds['text'] = $this->getTextData($refKeys['text']);
53
+		$this->addCustomerListData($testdata, $refIds, 'FosUser');
54 54
 
55
-		$this->status( 'done' );
55
+		$this->status('done');
56 56
 	}
57 57
 }
Please login to merge, or discard this patch.
lib/custom/setup/unittest/CustomerPropertyAddFosUserTestData.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public function getPreDependencies()
25 25
 	{
26
-		return array( 'TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData' );
26
+		return array('TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData');
27 27
 	}
28 28
 
29 29
 
@@ -32,20 +32,20 @@  discard block
 block discarded – undo
32 32
 	 */
33 33
 	public function migrate()
34 34
 	{
35
-		\Aimeos\MW\Common\Base::checkClass( '\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional );
35
+		\Aimeos\MW\Common\Base::checkClass('\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional);
36 36
 
37
-		$this->msg( 'Adding customer property Fos user bundle test data', 0 );
38
-		$this->additional->setEditor( 'ai-fosuser:unittest' );
37
+		$this->msg('Adding customer property Fos user bundle test data', 0);
38
+		$this->additional->setEditor('ai-fosuser:unittest');
39 39
 
40 40
 		$ds = DIRECTORY_SEPARATOR;
41 41
 		$path = __DIR__ . $ds . 'data' . $ds . 'customer-property.php';
42 42
 
43
-		if( ( $testdata = include( $path ) ) == false ) {
44
-			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for customer domain', $path ) );
43
+		if (($testdata = include($path)) == false) {
44
+			throw new \Aimeos\MShop\Exception(sprintf('No file "%1$s" found for customer domain', $path));
45 45
 		}
46 46
 
47
-		$this->addCustomerPropertyData( $testdata, 'FosUser' );
47
+		$this->addCustomerPropertyData($testdata, 'FosUser');
48 48
 
49
-		$this->status( 'done' );
49
+		$this->status('done');
50 50
 	}
51 51
 }
52 52
\ No newline at end of file
Please login to merge, or discard this patch.
lib/custom/tests/MShop/Customer/Manager/Property/FosUserTest.php 1 patch
Spacing   +92 added lines, -92 removed lines patch added patch discarded remove patch
@@ -20,91 +20,91 @@  discard block
 block discarded – undo
20 20
 		$context = \TestHelper::getContext();
21 21
 		$this->editor = $context->getEditor();
22 22
 
23
-		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( $context, 'FosUser' );
24
-		$this->object = $manager->getSubManager( 'property', 'FosUser' );
23
+		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager($context, 'FosUser');
24
+		$this->object = $manager->getSubManager('property', 'FosUser');
25 25
 	}
26 26
 
27 27
 
28 28
 	protected function tearDown()
29 29
 	{
30
-		unset( $this->object );
30
+		unset($this->object);
31 31
 	}
32 32
 
33 33
 
34 34
 	public function testCleanup()
35 35
 	{
36
-		$this->object->cleanup( array( -1 ) );
36
+		$this->object->cleanup(array( -1 ));
37 37
 	}
38 38
 
39 39
 
40 40
 	public function testCreateItem()
41 41
 	{
42 42
 		$item = $this->object->createItem();
43
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Property\\Iface', $item );
43
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Property\\Iface', $item);
44 44
 	}
45 45
 
46 46
 
47 47
 	public function testSaveInvalid()
48 48
 	{
49
-		$this->setExpectedException( '\Aimeos\MW\Common\Exception' );
50
-		$this->object->saveItem( new \Aimeos\MShop\Locale\Item\Standard() );
49
+		$this->setExpectedException('\Aimeos\MW\Common\Exception');
50
+		$this->object->saveItem(new \Aimeos\MShop\Locale\Item\Standard());
51 51
 	}
52 52
 
53 53
 
54 54
 	public function testSaveUpdateDeleteItem()
55 55
 	{
56 56
 		$search = $this->object->createSearch();
57
-		$search->setConditions( $search->compare( '==', 'customer.property.editor', $this->editor ) );
58
-		$results = $this->object->searchItems( $search );
57
+		$search->setConditions($search->compare('==', 'customer.property.editor', $this->editor));
58
+		$results = $this->object->searchItems($search);
59 59
 
60
-		if( ( $item = reset($results) ) === false ) {
61
-			throw new \RuntimeException( 'No property item found' );
60
+		if (($item = reset($results)) === false) {
61
+			throw new \RuntimeException('No property item found');
62 62
 		}
63 63
 
64 64
 		$item->setId(null);
65
-		$item->setLanguageId( 'en' );
66
-		$resultSaved = $this->object->saveItem( $item );
67
-		$itemSaved = $this->object->getItem( $item->getId() );
65
+		$item->setLanguageId('en');
66
+		$resultSaved = $this->object->saveItem($item);
67
+		$itemSaved = $this->object->getItem($item->getId());
68 68
 
69 69
 		$itemExp = clone $itemSaved;
70
-		$itemExp->setValue( 'unittest' );
71
-		$resultUpd = $this->object->saveItem( $itemExp );
72
-		$itemUpd = $this->object->getItem( $itemExp->getId() );
70
+		$itemExp->setValue('unittest');
71
+		$resultUpd = $this->object->saveItem($itemExp);
72
+		$itemUpd = $this->object->getItem($itemExp->getId());
73 73
 
74
-		$this->object->deleteItem( $itemSaved->getId() );
74
+		$this->object->deleteItem($itemSaved->getId());
75 75
 
76 76
 		$context = \TestHelper::getContext();
77 77
 
78
-		$this->assertTrue( $item->getId() !== null );
79
-		$this->assertTrue( $itemSaved->getType() !== null );
80
-		$this->assertEquals( $item->getId(), $itemSaved->getId() );
81
-		$this->assertEquals( $item->getParentId(), $itemSaved->getParentId() );
82
-		$this->assertEquals( $item->getSiteId(), $itemSaved->getSiteId() );
83
-		$this->assertEquals( $item->getTypeId(), $itemSaved->getTypeId() );
84
-		$this->assertEquals( $item->getLanguageId(), $itemSaved->getLanguageId() );
85
-		$this->assertEquals( $item->getValue(), $itemSaved->getValue() );
86
-
87
-		$this->assertEquals( $context->getEditor(), $itemSaved->getEditor() );
88
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated() );
89
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified() );
90
-
91
-		$this->assertTrue( $itemUpd->getType() !== null );
92
-		$this->assertEquals( $itemExp->getId(), $itemUpd->getId() );
93
-		$this->assertEquals( $itemExp->getParentId(), $itemUpd->getParentId() );
94
-		$this->assertEquals( $itemExp->getSiteId(), $itemUpd->getSiteId() );
95
-		$this->assertEquals( $itemExp->getTypeId(), $itemUpd->getTypeId() );
96
-		$this->assertEquals( $itemExp->getLanguageId(), $itemUpd->getLanguageId() );
97
-		$this->assertEquals( $itemExp->getValue(), $itemUpd->getValue() );
98
-
99
-		$this->assertEquals( $context->getEditor(), $itemUpd->getEditor() );
100
-		$this->assertEquals( $itemExp->getTimeCreated(), $itemUpd->getTimeCreated() );
101
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemUpd->getTimeModified() );
102
-
103
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Item\Iface', $resultSaved );
104
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Item\Iface', $resultUpd );
105
-
106
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
107
-		$this->object->getItem( $itemSaved->getId() );
78
+		$this->assertTrue($item->getId() !== null);
79
+		$this->assertTrue($itemSaved->getType() !== null);
80
+		$this->assertEquals($item->getId(), $itemSaved->getId());
81
+		$this->assertEquals($item->getParentId(), $itemSaved->getParentId());
82
+		$this->assertEquals($item->getSiteId(), $itemSaved->getSiteId());
83
+		$this->assertEquals($item->getTypeId(), $itemSaved->getTypeId());
84
+		$this->assertEquals($item->getLanguageId(), $itemSaved->getLanguageId());
85
+		$this->assertEquals($item->getValue(), $itemSaved->getValue());
86
+
87
+		$this->assertEquals($context->getEditor(), $itemSaved->getEditor());
88
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated());
89
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified());
90
+
91
+		$this->assertTrue($itemUpd->getType() !== null);
92
+		$this->assertEquals($itemExp->getId(), $itemUpd->getId());
93
+		$this->assertEquals($itemExp->getParentId(), $itemUpd->getParentId());
94
+		$this->assertEquals($itemExp->getSiteId(), $itemUpd->getSiteId());
95
+		$this->assertEquals($itemExp->getTypeId(), $itemUpd->getTypeId());
96
+		$this->assertEquals($itemExp->getLanguageId(), $itemUpd->getLanguageId());
97
+		$this->assertEquals($itemExp->getValue(), $itemUpd->getValue());
98
+
99
+		$this->assertEquals($context->getEditor(), $itemUpd->getEditor());
100
+		$this->assertEquals($itemExp->getTimeCreated(), $itemUpd->getTimeCreated());
101
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemUpd->getTimeModified());
102
+
103
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Item\Iface', $resultSaved);
104
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Item\Iface', $resultUpd);
105
+
106
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
107
+		$this->object->getItem($itemSaved->getId());
108 108
 	}
109 109
 
110 110
 
@@ -112,19 +112,19 @@  discard block
 block discarded – undo
112 112
 	{
113 113
 		$search = $this->object->createSearch();
114 114
 		$conditions = array(
115
-			$search->compare( '~=', 'customer.property.value', '1'),
116
-			$search->compare( '==', 'customer.property.editor', $this->editor )
115
+			$search->compare('~=', 'customer.property.value', '1'),
116
+			$search->compare('==', 'customer.property.editor', $this->editor)
117 117
 		);
118
-		$search->setConditions( $search->combine( '&&', $conditions ) );
119
-		$results = $this->object->searchItems( $search );
118
+		$search->setConditions($search->combine('&&', $conditions));
119
+		$results = $this->object->searchItems($search);
120 120
 
121
-		if( ($expected = reset($results)) === false ) {
122
-			throw new \RuntimeException( sprintf( 'No customer property item found for value "%1$s".', '1' ) );
121
+		if (($expected = reset($results)) === false) {
122
+			throw new \RuntimeException(sprintf('No customer property item found for value "%1$s".', '1'));
123 123
 		}
124 124
 
125
-		$actual = $this->object->getItem( $expected->getId() );
126
-		$this->assertNotEquals( '', $actual->getTypeName() );
127
-		$this->assertEquals( $expected, $actual );
125
+		$actual = $this->object->getItem($expected->getId());
126
+		$this->assertNotEquals('', $actual->getTypeName());
127
+		$this->assertEquals($expected, $actual);
128 128
 	}
129 129
 
130 130
 
@@ -132,15 +132,15 @@  discard block
 block discarded – undo
132 132
 	{
133 133
 		$result = $this->object->getResourceType();
134 134
 
135
-		$this->assertContains( 'customer/property', $result );
136
-		$this->assertContains( 'customer/property/type', $result );
135
+		$this->assertContains('customer/property', $result);
136
+		$this->assertContains('customer/property/type', $result);
137 137
 	}
138 138
 
139 139
 
140 140
 	public function testGetSearchAttributes()
141 141
 	{
142
-		foreach( $this->object->getSearchAttributes() as $attribute ) {
143
-			$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute );
142
+		foreach ($this->object->getSearchAttributes() as $attribute) {
143
+			$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute);
144 144
 		}
145 145
 	}
146 146
 
@@ -151,51 +151,51 @@  discard block
 block discarded – undo
151 151
 		$search = $this->object->createSearch();
152 152
 
153 153
 		$expr = [];
154
-		$expr[] = $search->compare( '!=', 'customer.property.id', null );
155
-		$expr[] = $search->compare( '!=', 'customer.property.parentid', null );
156
-		$expr[] = $search->compare( '!=', 'customer.property.siteid', null );
157
-		$expr[] = $search->compare( '!=', 'customer.property.typeid', null );
158
-		$expr[] = $search->compare( '==', 'customer.property.languageid', null );
159
-		$expr[] = $search->compare( '==', 'customer.property.value', '1' );
160
-		$expr[] = $search->compare( '==', 'customer.property.editor', $this->editor );
161
-
162
-		$expr[] = $search->compare( '!=', 'customer.property.type.id', null );
163
-		$expr[] = $search->compare( '!=', 'customer.property.type.siteid', null );
164
-		$expr[] = $search->compare( '==', 'customer.property.type.domain', 'customer' );
165
-		$expr[] = $search->compare( '==', 'customer.property.type.code', 'newsletter' );
166
-		$expr[] = $search->compare( '>', 'customer.property.type.label', '' );
167
-		$expr[] = $search->compare( '==', 'customer.property.type.status', 1 );
168
-		$expr[] = $search->compare( '>=', 'customer.property.type.mtime', '1970-01-01 00:00:00' );
169
-		$expr[] = $search->compare( '>=', 'customer.property.type.ctime', '1970-01-01 00:00:00' );
170
-		$expr[] = $search->compare( '==', 'customer.property.type.editor', $this->editor );
171
-
172
-		$search->setConditions( $search->combine('&&', $expr) );
173
-		$results = $this->object->searchItems( $search, [], $total );
174
-		$this->assertEquals( 1, count( $results ) );
154
+		$expr[] = $search->compare('!=', 'customer.property.id', null);
155
+		$expr[] = $search->compare('!=', 'customer.property.parentid', null);
156
+		$expr[] = $search->compare('!=', 'customer.property.siteid', null);
157
+		$expr[] = $search->compare('!=', 'customer.property.typeid', null);
158
+		$expr[] = $search->compare('==', 'customer.property.languageid', null);
159
+		$expr[] = $search->compare('==', 'customer.property.value', '1');
160
+		$expr[] = $search->compare('==', 'customer.property.editor', $this->editor);
161
+
162
+		$expr[] = $search->compare('!=', 'customer.property.type.id', null);
163
+		$expr[] = $search->compare('!=', 'customer.property.type.siteid', null);
164
+		$expr[] = $search->compare('==', 'customer.property.type.domain', 'customer');
165
+		$expr[] = $search->compare('==', 'customer.property.type.code', 'newsletter');
166
+		$expr[] = $search->compare('>', 'customer.property.type.label', '');
167
+		$expr[] = $search->compare('==', 'customer.property.type.status', 1);
168
+		$expr[] = $search->compare('>=', 'customer.property.type.mtime', '1970-01-01 00:00:00');
169
+		$expr[] = $search->compare('>=', 'customer.property.type.ctime', '1970-01-01 00:00:00');
170
+		$expr[] = $search->compare('==', 'customer.property.type.editor', $this->editor);
171
+
172
+		$search->setConditions($search->combine('&&', $expr));
173
+		$results = $this->object->searchItems($search, [], $total);
174
+		$this->assertEquals(1, count($results));
175 175
 
176 176
 
177 177
 		$search = $this->object->createSearch();
178 178
 		$conditions = array(
179
-			$search->compare( '=~', 'customer.property.type.code', 'newsletter' ),
180
-			$search->compare( '==', 'customer.property.editor', $this->editor )
179
+			$search->compare('=~', 'customer.property.type.code', 'newsletter'),
180
+			$search->compare('==', 'customer.property.editor', $this->editor)
181 181
 		);
182
-		$search->setConditions( $search->combine( '&&', $conditions ) );
182
+		$search->setConditions($search->combine('&&', $conditions));
183 183
 		$search->setSlice(0, 1);
184
-		$items = $this->object->searchItems( $search, [], $total );
184
+		$items = $this->object->searchItems($search, [], $total);
185 185
 
186
-		$this->assertEquals( 1, count( $items ) );
187
-		$this->assertEquals( 1, $total );
186
+		$this->assertEquals(1, count($items));
187
+		$this->assertEquals(1, $total);
188 188
 
189
-		foreach($items as $itemId => $item) {
190
-			$this->assertEquals( $itemId, $item->getId() );
189
+		foreach ($items as $itemId => $item) {
190
+			$this->assertEquals($itemId, $item->getId());
191 191
 		}
192 192
 	}
193 193
 
194 194
 
195 195
 	public function testGetSubManager()
196 196
 	{
197
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type') );
198
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard') );
197
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type'));
198
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard'));
199 199
 
200 200
 		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
201 201
 		$this->object->getSubManager('unknown');
Please login to merge, or discard this patch.