Passed
Push — master ( 1a8053...1690e8 )
by Aimeos
03:06
created
lib/custom/tests/MShop/Customer/Manager/Property/Type/LaravelTest.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 
28 28
 	protected function tearDown()
29 29
 	{
30
-		unset($this->object);
30
+		unset( $this->object );
31 31
 	}
32 32
 
33 33
 
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 
71 71
 		$results = $this->object->searchItems( $search );
72 72
 
73
-		if( ($expected = reset($results) ) === false )
73
+		if( ( $expected = reset( $results ) ) === false )
74 74
 		{
75 75
 			throw new \RuntimeException( 'No property type item found.' );
76 76
 		}
@@ -92,13 +92,13 @@  discard block
 block discarded – undo
92 92
 	{
93 93
 		$search = $this->object->createSearch();
94 94
 		$search->setConditions( $search->compare( '==', 'customer.property.type.editor', $this->editor ) );
95
-		$results = $this->object->searchItems($search);
95
+		$results = $this->object->searchItems( $search );
96 96
 
97
-		if( ( $item = reset($results) ) === false ) {
97
+		if( ( $item = reset( $results ) ) === false ) {
98 98
 			throw new \RuntimeException( 'No type item found' );
99 99
 		}
100 100
 
101
-		$item->setId(null);
101
+		$item->setId( null );
102 102
 		$item->setCode( 'unitTestSave' );
103 103
 		$resultSaved = $this->object->saveItem( $item );
104 104
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -159,24 +159,24 @@  discard block
 block discarded – undo
159 159
 		$expr[] = $search->compare( '>=', 'customer.property.type.ctime', '1970-01-01 00:00:00' );
160 160
 		$expr[] = $search->compare( '==', 'customer.property.type.editor', $this->editor );
161 161
 
162
-		$search->setConditions( $search->combine('&&', $expr) );
162
+		$search->setConditions( $search->combine( '&&', $expr ) );
163 163
 		$results = $this->object->searchItems( $search, [], $total );
164 164
 		$this->assertEquals( 1, count( $results ) );
165 165
 
166 166
 
167 167
 		$search = $this->object->createSearch();
168 168
 		$conditions = array(
169
-			$search->compare( '=~', 'customer.property.type.code', 'newsletter'),
169
+			$search->compare( '=~', 'customer.property.type.code', 'newsletter' ),
170 170
 			$search->compare( '==', 'customer.property.type.editor', $this->editor )
171 171
 		);
172
-		$search->setConditions( $search->combine('&&', $conditions ) );
173
-		$search->setSlice(0, 1);
174
-		$items = $this->object->searchItems( $search, [], $total);
172
+		$search->setConditions( $search->combine( '&&', $conditions ) );
173
+		$search->setSlice( 0, 1 );
174
+		$items = $this->object->searchItems( $search, [], $total );
175 175
 
176 176
 		$this->assertEquals( 1, count( $items ) );
177 177
 		$this->assertEquals( 1, $total );
178 178
 
179
-		foreach($items as $itemId => $item) {
179
+		foreach( $items as $itemId => $item ) {
180 180
 			$this->assertEquals( $itemId, $item->getId() );
181 181
 		}
182 182
 	}
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 
185 185
 	public function testGetSubManager()
186 186
 	{
187
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
188
-		$this->object->getSubManager('unknown');
187
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
188
+		$this->object->getSubManager( 'unknown' );
189 189
 	}
190 190
 }
Please login to merge, or discard this patch.
lib/custom/tests/MW/View/Helper/Url/Laravel5Test.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 	public function testTransform()
37 37
 	{
38 38
 		$this->mock->expects( $this->once() )->method( 'route' )
39
-			->with( $this->equalTo( 'route'), $this->equalTo( array( 'key' => 'value' ) ), $this->equalTo( false ) );
39
+			->with( $this->equalTo( 'route' ), $this->equalTo( array( 'key' => 'value' ) ), $this->equalTo( false ) );
40 40
 
41 41
 		$this->object->transform( 'route', 'catalog', 'lists', array( 'key' => 'value' ) );
42 42
 	}
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 	public function testTransformAbsolute()
46 46
 	{
47 47
 		$this->mock->expects( $this->once() )->method( 'route' )
48
-			->with( $this->equalTo( 'route'), $this->equalTo( [] ), $this->equalTo( true ) );
48
+			->with( $this->equalTo( 'route' ), $this->equalTo( [] ), $this->equalTo( true ) );
49 49
 
50 50
 		$options = array( 'absoluteUri' => true );
51 51
 		$this->object->transform( 'route', 'catalog', 'lists', [], [], $options );
Please login to merge, or discard this patch.
lib/custom/tests/MW/View/Engine/BladeTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
 			->will( $this->returnValue( 'test' ) );
49 49
 
50 50
 		$this->mock->expects( $this->once() )->method( 'file' )
51
-			->will( $this->returnValue( $view) );
51
+			->will( $this->returnValue( $view ) );
52 52
 
53 53
 		$result = $this->object->render( $v, 'filepath', array( 'key' => 'value' ) );
54 54
 		$this->assertEquals( 'test', $result );
Please login to merge, or discard this patch.
lib/custom/tests/bootstrap.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -10,16 +10,16 @@
 block discarded – undo
10 10
  * Set error reporting to maximum
11 11
  */
12 12
 error_reporting( -1 );
13
-ini_set('display_errors', '1');
13
+ini_set( 'display_errors', '1' );
14 14
 
15 15
 
16 16
 /*
17 17
  * Set locale settings to reasonable defaults
18 18
  */
19
-setlocale(LC_ALL, 'en_US.UTF-8');
20
-setlocale(LC_NUMERIC, 'POSIX');
21
-setlocale(LC_CTYPE, 'en_US.UTF-8');
22
-setlocale(LC_TIME, 'POSIX');
19
+setlocale( LC_ALL, 'en_US.UTF-8' );
20
+setlocale( LC_NUMERIC, 'POSIX' );
21
+setlocale( LC_CTYPE, 'en_US.UTF-8' );
22
+setlocale( LC_TIME, 'POSIX' );
23 23
 
24 24
 
25 25
 require_once 'TestHelper.php';
Please login to merge, or discard this patch.
lib/custom/src/MShop/Customer/Manager/Laravel.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -489,14 +489,14 @@
 block discarded – undo
489 489
 
490 490
 			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
491 491
 			while( ( $row = $results->fetch() ) !== false ) {
492
-				$map[ $row['customer.id'] ] = $row;
492
+				$map[$row['customer.id']] = $row;
493 493
 			}
494 494
 
495 495
 			$dbm->release( $conn, $dbname );
496 496
 		}
497 497
 		catch( \Exception $e )
498 498
 		{
499
-			$dbm->release( $conn, $dbname  );
499
+			$dbm->release( $conn, $dbname );
500 500
 			throw $e;
501 501
 		}
502 502
 
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -340,8 +340,7 @@
 block discarded – undo
340 340
 				 * @see mshop/customer/manager/laravel/count
341 341
 				 */
342 342
 				$path = 'mshop/customer/manager/laravel/insert';
343
-			}
344
-			else
343
+			} else
345 344
 			{
346 345
 				/** mshop/customer/manager/laravel/update
347 346
 				 * Updates an existing customer record in the database
Please login to merge, or discard this patch.
lib/custom/src/MW/Logger/Laravel5.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
 
55 55
 			$this->logger->log( $message, $this->translatePriority( $priority ) );
56 56
 		}
57
-		catch( \Exception $e )	{
57
+		catch( \Exception $e ) {
58 58
 			throw new \Aimeos\MW\Logger\Exception( $e->getMessage(), $e->getCode(), $e );
59 59
 		}
60 60
 	}
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,8 @@
 block discarded – undo
54 54
 
55 55
 			$this->logger->log( $message, $this->translatePriority( $priority ) );
56 56
 		}
57
-		catch( \Exception $e )	{
57
+		catch( \Exception $e )
58
+		{
58 59
 			throw new \Aimeos\MW\Logger\Exception( $e->getMessage(), $e->getCode(), $e );
59 60
 		}
60 61
 	}
Please login to merge, or discard this patch.
lib/custom/src/MW/View/Engine/Blade.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -47,8 +47,7 @@
 block discarded – undo
47 47
 		$factory = $this->factory;
48 48
 		$lv = $factory->file( $filename, $values );
49 49
 
50
-		$fcn = function() use ( $factory, $view )
51
-		{
50
+		$fcn = function() use ( $factory, $view ) {
52 51
 			foreach( $factory->getSections() as $name => $section ) {
53 52
 				$view->block()->set( $name, $section );
54 53
 			}
Please login to merge, or discard this patch.
lib/custom/setup/CustomerRemoveSignedConstraints.php 1 patch
Braces   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -44,9 +44,12 @@  discard block
 block discarded – undo
44 44
 		$schema = $this->getSchema( 'db-customer' );
45 45
 		$sql = 'SELECT COLUMN_TYPE FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_NAME = \'users\' AND COLUMN_NAME = \'id\'';
46 46
 
47
-		try {
47
+		try
48
+		{
48 49
 			$type = $this->getValue( $sql, 'COLUMN_TYPE', 'db-customer' );
49
-		} catch( \Aimeos\MW\Setup\Exception $e ) {
50
+		}
51
+		catch( \Aimeos\MW\Setup\Exception $e )
52
+		{
50 53
 			$type = null;
51 54
 		}
52 55
 
@@ -62,8 +65,7 @@  discard block
 block discarded – undo
62 65
 			{
63 66
 				$this->execute( 'ALTER TABLE "users_address" DROP FOREIGN KEY "fk_lvuad_pid"', 'db-customer' );
64 67
 				$this->status( 'done' );
65
-			}
66
-			else
68
+			} else
67 69
 			{
68 70
 				$this->status( 'OK' );
69 71
 			}
@@ -75,8 +77,7 @@  discard block
 block discarded – undo
75 77
 			{
76 78
 				$this->execute( 'ALTER TABLE "users_list" DROP FOREIGN KEY "fk_lvuli_pid"', 'db-customer' );
77 79
 				$this->status( 'done' );
78
-			}
79
-			else
80
+			} else
80 81
 			{
81 82
 				$this->status( 'OK' );
82 83
 			}
@@ -88,8 +89,7 @@  discard block
 block discarded – undo
88 89
 			{
89 90
 				$this->execute( 'ALTER TABLE "users_property" DROP FOREIGN KEY "fk_lvupr_pid"', 'db-customer' );
90 91
 				$this->status( 'done' );
91
-			}
92
-			else
92
+			} else
93 93
 			{
94 94
 				$this->status( 'OK' );
95 95
 			}
Please login to merge, or discard this patch.
lib/custom/tests/MShop/Customer/Manager/Lists/LaravelTest.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 	public function testGetItem()
64 64
 	{
65 65
 		$search = $this->object->createSearch();
66
-		$search->setSlice(0, 1);
66
+		$search->setSlice( 0, 1 );
67 67
 		$results = $this->object->searchItems( $search );
68 68
 
69 69
 		if( ( $item = reset( $results ) ) === false ) {
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 	public function testSaveUpdateDeleteItem()
78 78
 	{
79 79
 		$search = $this->object->createSearch();
80
-		$search->setSlice(0, 1);
80
+		$search->setSlice( 0, 1 );
81 81
 		$items = $this->object->searchItems( $search );
82 82
 
83 83
 		if( ( $item = reset( $items ) ) === false ) {
@@ -108,12 +108,12 @@  discard block
 block discarded – undo
108 108
 		$this->assertEquals( $item->getDateEnd(), $itemSaved->getDateEnd() );
109 109
 		$this->assertEquals( $item->getPosition(), $itemSaved->getPosition() );
110 110
 		$this->assertEquals( $this->editor, $itemSaved->getEditor() );
111
-		$this->assertStringStartsWith(date('Y-m-d', time()), $itemSaved->getTimeCreated());
112
-		$this->assertStringStartsWith(date('Y-m-d', time()), $itemSaved->getTimeModified());
111
+		$this->assertStringStartsWith( date( 'Y-m-d', time() ), $itemSaved->getTimeCreated() );
112
+		$this->assertStringStartsWith( date( 'Y-m-d', time() ), $itemSaved->getTimeModified() );
113 113
 
114 114
 		$this->assertEquals( $this->editor, $itemSaved->getEditor() );
115 115
 		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated() );
116
-		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified() );
116
+		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified() );
117 117
 
118 118
 		$this->assertEquals( $itemExp->getId(), $itemUpd->getId() );
119 119
 		$this->assertEquals( $itemExp->getSiteId(), $itemUpd->getSiteId() );
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultSaved );
133 133
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultUpd );
134 134
 
135
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
135
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
136 136
 		$this->object->getItem( $itemSaved->getId() );
137 137
 	}
138 138
 
@@ -278,8 +278,8 @@  discard block
 block discarded – undo
278 278
 
279 279
 	public function testGetSubManager()
280 280
 	{
281
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type') );
282
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard') );
281
+		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager( 'type' ) );
282
+		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager( 'type', 'Standard' ) );
283 283
 
284 284
 		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
285 285
 		$this->object->getSubManager( 'unknown' );
Please login to merge, or discard this patch.