@@ -27,7 +27,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 | } |
@@ -36,7 +36,7 @@ discard block |
||
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 |
||
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 ); |
@@ -48,7 +48,7 @@ |
||
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 ); |
@@ -10,16 +10,16 @@ |
||
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'; |
@@ -489,14 +489,14 @@ |
||
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 |
@@ -340,8 +340,7 @@ |
||
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 |
@@ -54,7 +54,7 @@ |
||
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 | } |
@@ -54,7 +54,8 @@ |
||
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 | } |
@@ -47,8 +47,7 @@ |
||
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 | } |
@@ -44,9 +44,12 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 | } |
@@ -63,7 +63,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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' ); |