@@ -29,13 +29,13 @@ discard block |
||
29 | 29 | * @return \Aimeos\MW\Criteria\Iface Criteria object containing the conditions for searching |
30 | 30 | * @since 2017.03 |
31 | 31 | */ |
32 | - public function addFilterCodes( \Aimeos\MW\Criteria\Iface $filter, array $codes ) |
|
32 | + public function addFilterCodes(\Aimeos\MW\Criteria\Iface $filter, array $codes) |
|
33 | 33 | { |
34 | 34 | $expr = [ |
35 | - $filter->compare( '==', 'stock.productcode', $codes ), |
|
35 | + $filter->compare('==', 'stock.productcode', $codes), |
|
36 | 36 | $filter->getConditions(), |
37 | 37 | ]; |
38 | - $filter->setConditions( $filter->combine( '&&', $expr ) ); |
|
38 | + $filter->setConditions($filter->combine('&&', $expr)); |
|
39 | 39 | |
40 | 40 | return $filter; |
41 | 41 | } |
@@ -49,15 +49,15 @@ discard block |
||
49 | 49 | * @return \Aimeos\MW\Criteria\Iface Criteria object containing the conditions for searching |
50 | 50 | * @since 2017.03 |
51 | 51 | */ |
52 | - public function addFilterTypes( \Aimeos\MW\Criteria\Iface $filter, array $codes ) |
|
52 | + public function addFilterTypes(\Aimeos\MW\Criteria\Iface $filter, array $codes) |
|
53 | 53 | { |
54 | - if( !empty( $codes ) ) |
|
54 | + if (!empty($codes)) |
|
55 | 55 | { |
56 | 56 | $expr = [ |
57 | - $filter->compare( '==', 'stock.type.code', $codes ), |
|
57 | + $filter->compare('==', 'stock.type.code', $codes), |
|
58 | 58 | $filter->getConditions(), |
59 | 59 | ]; |
60 | - $filter->setConditions( $filter->combine( '&&', $expr ) ); |
|
60 | + $filter->setConditions($filter->combine('&&', $expr)); |
|
61 | 61 | } |
62 | 62 | |
63 | 63 | return $filter; |
@@ -73,7 +73,7 @@ discard block |
||
73 | 73 | */ |
74 | 74 | public function createFilter() |
75 | 75 | { |
76 | - return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'stock' )->createSearch( true ); |
|
76 | + return \Aimeos\MShop\Factory::createManager($this->getContext(), 'stock')->createSearch(true); |
|
77 | 77 | } |
78 | 78 | |
79 | 79 | |
@@ -85,9 +85,9 @@ discard block |
||
85 | 85 | * @return \Aimeos\MShop\Stock\Item\Iface Stock item including the referenced domains items |
86 | 86 | * @since 2017.03 |
87 | 87 | */ |
88 | - public function getItem( $id ) |
|
88 | + public function getItem($id) |
|
89 | 89 | { |
90 | - return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'stock' )->getItem( $id ); |
|
90 | + return \Aimeos\MShop\Factory::createManager($this->getContext(), 'stock')->getItem($id); |
|
91 | 91 | } |
92 | 92 | |
93 | 93 | |
@@ -99,8 +99,8 @@ discard block |
||
99 | 99 | * @return array Ordered list of stock items implementing \Aimeos\MShop\Stock\Item\Iface |
100 | 100 | * @since 2017.03 |
101 | 101 | */ |
102 | - public function searchItems( \Aimeos\MW\Criteria\Iface $filter, &$total = null ) |
|
102 | + public function searchItems(\Aimeos\MW\Criteria\Iface $filter, &$total = null) |
|
103 | 103 | { |
104 | - return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'stock' )->searchItems( $filter, [], $total ); |
|
104 | + return \Aimeos\MShop\Factory::createManager($this->getContext(), 'stock')->searchItems($filter, [], $total); |
|
105 | 105 | } |
106 | 106 | } |
@@ -15,31 +15,31 @@ |
||
15 | 15 | { |
16 | 16 | $target = '\\Aimeos\\Controller\\Frontend\\Attribute\\Iface'; |
17 | 17 | |
18 | - $controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext() ); |
|
19 | - $this->assertInstanceOf( $target, $controller ); |
|
18 | + $controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext()); |
|
19 | + $this->assertInstanceOf($target, $controller); |
|
20 | 20 | |
21 | - $controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' ); |
|
22 | - $this->assertInstanceOf( $target, $controller ); |
|
21 | + $controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'Standard'); |
|
22 | + $this->assertInstanceOf($target, $controller); |
|
23 | 23 | } |
24 | 24 | |
25 | 25 | |
26 | 26 | public function testCreateControllerInvalidImplementation() |
27 | 27 | { |
28 | - $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' ); |
|
29 | - \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' ); |
|
28 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
29 | + \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid'); |
|
30 | 30 | } |
31 | 31 | |
32 | 32 | |
33 | 33 | public function testCreateControllerInvalidName() |
34 | 34 | { |
35 | - $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' ); |
|
36 | - \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), '%^' ); |
|
35 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
36 | + \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), '%^'); |
|
37 | 37 | } |
38 | 38 | |
39 | 39 | |
40 | 40 | public function testCreateControllerNotExisting() |
41 | 41 | { |
42 | - $this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' ); |
|
43 | - \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' ); |
|
42 | + $this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception'); |
|
43 | + \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'notexist'); |
|
44 | 44 | } |
45 | 45 | } |
@@ -16,13 +16,13 @@ discard block |
||
16 | 16 | |
17 | 17 | protected function setUp() |
18 | 18 | { |
19 | - $this->object = new \Aimeos\Controller\Frontend\Attribute\Standard( \TestHelperFrontend::getContext() ); |
|
19 | + $this->object = new \Aimeos\Controller\Frontend\Attribute\Standard(\TestHelperFrontend::getContext()); |
|
20 | 20 | } |
21 | 21 | |
22 | 22 | |
23 | 23 | protected function tearDown() |
24 | 24 | { |
25 | - unset( $this->object ); |
|
25 | + unset($this->object); |
|
26 | 26 | } |
27 | 27 | |
28 | 28 | |
@@ -30,51 +30,51 @@ discard block |
||
30 | 30 | { |
31 | 31 | $filter = $this->object->createFilter(); |
32 | 32 | |
33 | - $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter ); |
|
34 | - $this->assertEquals( 1, count( $filter->getSortations() ) ); |
|
35 | - $this->assertEquals( 0, $filter->getSliceStart() ); |
|
36 | - $this->assertEquals( 100, $filter->getSliceSize() ); |
|
33 | + $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter); |
|
34 | + $this->assertEquals(1, count($filter->getSortations())); |
|
35 | + $this->assertEquals(0, $filter->getSliceStart()); |
|
36 | + $this->assertEquals(100, $filter->getSliceSize()); |
|
37 | 37 | } |
38 | 38 | |
39 | 39 | |
40 | 40 | public function testAddFilterTypes() |
41 | 41 | { |
42 | 42 | $filter = $this->object->createFilter(); |
43 | - $filter = $this->object->addFilterTypes( $filter, ['size'] ); |
|
43 | + $filter = $this->object->addFilterTypes($filter, ['size']); |
|
44 | 44 | |
45 | 45 | $list = $filter->getConditions()->getExpressions(); |
46 | 46 | |
47 | - if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) { |
|
48 | - throw new \RuntimeException( 'Wrong expression' ); |
|
47 | + if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) { |
|
48 | + throw new \RuntimeException('Wrong expression'); |
|
49 | 49 | } |
50 | 50 | |
51 | - $this->assertEquals( 'attribute.type.code', $list[0]->getName() ); |
|
52 | - $this->assertEquals( 1, count( $list[0]->getValue() ) ); |
|
51 | + $this->assertEquals('attribute.type.code', $list[0]->getName()); |
|
52 | + $this->assertEquals(1, count($list[0]->getValue())); |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | |
56 | 56 | public function testGetItem() |
57 | 57 | { |
58 | 58 | $context = \TestHelperFrontend::getContext(); |
59 | - $manager = \Aimeos\MShop\Factory::createManager( $context, 'attribute' ); |
|
60 | - $id = $manager->findItem( 'xs', [], 'product', 'size' )->getId(); |
|
59 | + $manager = \Aimeos\MShop\Factory::createManager($context, 'attribute'); |
|
60 | + $id = $manager->findItem('xs', [], 'product', 'size')->getId(); |
|
61 | 61 | |
62 | - $result = $this->object->getItem( $id, ['text'] ); |
|
62 | + $result = $this->object->getItem($id, ['text']); |
|
63 | 63 | |
64 | - $this->assertInstanceOf( '\Aimeos\MShop\Attribute\Item\Iface', $result ); |
|
65 | - $this->assertEquals( 3, count( $result->getRefItems( 'text' ) ) ); |
|
64 | + $this->assertInstanceOf('\Aimeos\MShop\Attribute\Item\Iface', $result); |
|
65 | + $this->assertEquals(3, count($result->getRefItems('text'))); |
|
66 | 66 | } |
67 | 67 | |
68 | 68 | |
69 | 69 | public function testSearchItems() |
70 | 70 | { |
71 | 71 | $filter = $this->object->createFilter(); |
72 | - $filter = $this->object->addFilterTypes( $filter, ['size'] ); |
|
72 | + $filter = $this->object->addFilterTypes($filter, ['size']); |
|
73 | 73 | |
74 | 74 | $total = 0; |
75 | - $results = $this->object->searchItems( $filter, ['text'], $total ); |
|
75 | + $results = $this->object->searchItems($filter, ['text'], $total); |
|
76 | 76 | |
77 | - $this->assertEquals( 6, $total ); |
|
78 | - $this->assertEquals( 6, count( $results ) ); |
|
77 | + $this->assertEquals(6, $total); |
|
78 | + $this->assertEquals(6, count($results)); |
|
79 | 79 | } |
80 | 80 | } |
@@ -29,15 +29,15 @@ discard block |
||
29 | 29 | * @return \Aimeos\MW\Criteria\Iface Criteria object containing the conditions for searching |
30 | 30 | * @since 2017.03 |
31 | 31 | */ |
32 | - public function addFilterTypes( \Aimeos\MW\Criteria\Iface $filter, array $codes ) |
|
32 | + public function addFilterTypes(\Aimeos\MW\Criteria\Iface $filter, array $codes) |
|
33 | 33 | { |
34 | - if( !empty( $codes ) ) |
|
34 | + if (!empty($codes)) |
|
35 | 35 | { |
36 | 36 | $expr = [ |
37 | - $filter->compare( '==', 'attribute.type.code', $codes ), |
|
37 | + $filter->compare('==', 'attribute.type.code', $codes), |
|
38 | 38 | $filter->getConditions(), |
39 | 39 | ]; |
40 | - $filter->setConditions( $filter->combine( '&&', $expr ) ); |
|
40 | + $filter->setConditions($filter->combine('&&', $expr)); |
|
41 | 41 | } |
42 | 42 | |
43 | 43 | return $filter; |
@@ -53,15 +53,15 @@ discard block |
||
53 | 53 | */ |
54 | 54 | public function createFilter() |
55 | 55 | { |
56 | - $filter = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' )->createSearch( true ); |
|
56 | + $filter = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute')->createSearch(true); |
|
57 | 57 | |
58 | 58 | $expr = array( |
59 | - $filter->compare( '==', 'attribute.domain', 'product' ), |
|
59 | + $filter->compare('==', 'attribute.domain', 'product'), |
|
60 | 60 | $filter->getConditions(), |
61 | 61 | ); |
62 | 62 | |
63 | - $filter->setConditions( $filter->combine( '&&', $expr ) ); |
|
64 | - $filter->setSortations( array( $filter->sort( '+', 'attribute.position' ) ) ); |
|
63 | + $filter->setConditions($filter->combine('&&', $expr)); |
|
64 | + $filter->setSortations(array($filter->sort('+', 'attribute.position'))); |
|
65 | 65 | |
66 | 66 | return $filter; |
67 | 67 | } |
@@ -75,9 +75,9 @@ discard block |
||
75 | 75 | * @return \Aimeos\MShop\Attribute\Item\Iface Attribute item including the referenced domains items |
76 | 76 | * @since 2017.03 |
77 | 77 | */ |
78 | - public function getItem( $id, array $domains = array( 'media', 'price', 'text' ) ) |
|
78 | + public function getItem($id, array $domains = array('media', 'price', 'text')) |
|
79 | 79 | { |
80 | - return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' )->getItem( $id, $domains ); |
|
80 | + return \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute')->getItem($id, $domains); |
|
81 | 81 | } |
82 | 82 | |
83 | 83 | |
@@ -90,8 +90,8 @@ discard block |
||
90 | 90 | * @return array Ordered list of attribute items implementing \Aimeos\MShop\Attribute\Item\Iface |
91 | 91 | * @since 2017.03 |
92 | 92 | */ |
93 | - public function searchItems( \Aimeos\MW\Criteria\Iface $filter, array $domains = array( 'media', 'price', 'text' ), &$total = null ) |
|
93 | + public function searchItems(\Aimeos\MW\Criteria\Iface $filter, array $domains = array('media', 'price', 'text'), &$total = null) |
|
94 | 94 | { |
95 | - return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' )->searchItems( $filter, $domains, $total ); |
|
95 | + return \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute')->searchItems($filter, $domains, $total); |
|
96 | 96 | } |
97 | 97 | } |
@@ -27,7 +27,7 @@ discard block |
||
27 | 27 | * @return \Aimeos\MW\Criteria\Iface Criteria object containing the conditions for searching |
28 | 28 | * @since 2017.03 |
29 | 29 | */ |
30 | - public function addFilterTypes( \Aimeos\MW\Criteria\Iface $filter, array $codes ); |
|
30 | + public function addFilterTypes(\Aimeos\MW\Criteria\Iface $filter, array $codes); |
|
31 | 31 | |
32 | 32 | |
33 | 33 | /** |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | * @return \Aimeos\MShop\Attribute\Item\Iface Attribute item including the referenced domains items |
48 | 48 | * @since 2017.03 |
49 | 49 | */ |
50 | - public function getItem( $id, array $domains = array( 'media', 'price', 'text' ) ); |
|
50 | + public function getItem($id, array $domains = array('media', 'price', 'text')); |
|
51 | 51 | |
52 | 52 | |
53 | 53 | /** |
@@ -59,5 +59,5 @@ discard block |
||
59 | 59 | * @return array Ordered list of attribute items implementing \Aimeos\MShop\Attribute\Item\Iface |
60 | 60 | * @since 2017.03 |
61 | 61 | */ |
62 | - public function searchItems( \Aimeos\MW\Criteria\Iface $filter, array $domains = array( 'media', 'price', 'text' ), &$total = null ); |
|
62 | + public function searchItems(\Aimeos\MW\Criteria\Iface $filter, array $domains = array('media', 'price', 'text'), &$total = null); |
|
63 | 63 | } |
@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | * @param string|null $name Name of the controller implementaton (default: "Standard") |
29 | 29 | * @return \Aimeos\Controller\Frontend\Attribute\Iface Controller object |
30 | 30 | */ |
31 | - public static function createController( \Aimeos\MShop\Context\Item\Iface $context, $name = null ) |
|
31 | + public static function createController(\Aimeos\MShop\Context\Item\Iface $context, $name = null) |
|
32 | 32 | { |
33 | 33 | /** controller/frontend/attribute/name |
34 | 34 | * Class name of the used attribute frontend controller implementation |
@@ -63,20 +63,20 @@ discard block |
||
63 | 63 | * @since 2014.03 |
64 | 64 | * @category Developer |
65 | 65 | */ |
66 | - if( $name === null ) { |
|
67 | - $name = $context->getConfig()->get( 'controller/frontend/attribute/name', 'Standard' ); |
|
66 | + if ($name === null) { |
|
67 | + $name = $context->getConfig()->get('controller/frontend/attribute/name', 'Standard'); |
|
68 | 68 | } |
69 | 69 | |
70 | - if( ctype_alnum( $name ) === false ) |
|
70 | + if (ctype_alnum($name) === false) |
|
71 | 71 | { |
72 | - $classname = is_string( $name ) ? '\\Aimeos\\Controller\\Frontend\\Attribute\\' . $name : '<not a string>'; |
|
73 | - throw new \Aimeos\Controller\Frontend\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) ); |
|
72 | + $classname = is_string($name) ? '\\Aimeos\\Controller\\Frontend\\Attribute\\' . $name : '<not a string>'; |
|
73 | + throw new \Aimeos\Controller\Frontend\Exception(sprintf('Invalid characters in class name "%1$s"', $classname)); |
|
74 | 74 | } |
75 | 75 | |
76 | 76 | $iface = '\\Aimeos\\Controller\\Frontend\\Attribute\\Iface'; |
77 | 77 | $classname = '\\Aimeos\\Controller\\Frontend\\Attribute\\' . $name; |
78 | 78 | |
79 | - $manager = self::createControllerBase( $context, $classname, $iface ); |
|
79 | + $manager = self::createControllerBase($context, $classname, $iface); |
|
80 | 80 | |
81 | 81 | /** controller/frontend/attribute/decorators/excludes |
82 | 82 | * Excludes decorators added by the "common" option from the attribute frontend controllers |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | * @see controller/frontend/attribute/decorators/excludes |
153 | 153 | * @see controller/frontend/attribute/decorators/global |
154 | 154 | */ |
155 | - return self::addControllerDecorators( $context, $manager, 'attribute' ); |
|
155 | + return self::addControllerDecorators($context, $manager, 'attribute'); |
|
156 | 156 | } |
157 | 157 | |
158 | 158 | } |
@@ -20,106 +20,106 @@ |
||
20 | 20 | { |
21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
22 | 22 | |
23 | - $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Order\Standard' ) |
|
23 | + $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Order\Standard') |
|
24 | 24 | ->disableOriginalConstructor() |
25 | 25 | ->getMock(); |
26 | 26 | |
27 | - $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Order\Decorator\Base' ) |
|
28 | - ->setConstructorArgs( [$this->stub, $this->context] ) |
|
27 | + $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Order\Decorator\Base') |
|
28 | + ->setConstructorArgs([$this->stub, $this->context]) |
|
29 | 29 | ->getMockForAbstractClass(); |
30 | 30 | } |
31 | 31 | |
32 | 32 | |
33 | 33 | protected function tearDown() |
34 | 34 | { |
35 | - unset( $this->context, $this->object, $this->stub ); |
|
35 | + unset($this->context, $this->object, $this->stub); |
|
36 | 36 | } |
37 | 37 | |
38 | 38 | |
39 | 39 | public function testConstructException() |
40 | 40 | { |
41 | - $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock(); |
|
41 | + $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock(); |
|
42 | 42 | |
43 | - $this->setExpectedException( '\Aimeos\Controller\Frontend\Exception' ); |
|
43 | + $this->setExpectedException('\Aimeos\Controller\Frontend\Exception'); |
|
44 | 44 | |
45 | - $this->getMockBuilder( '\Aimeos\Controller\Frontend\Order\Decorator\Base' ) |
|
46 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
45 | + $this->getMockBuilder('\Aimeos\Controller\Frontend\Order\Decorator\Base') |
|
46 | + ->setConstructorArgs([$stub, $this->context]) |
|
47 | 47 | ->getMockForAbstractClass(); |
48 | 48 | } |
49 | 49 | |
50 | 50 | |
51 | 51 | public function testCall() |
52 | 52 | { |
53 | - $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Order\Standard' ) |
|
53 | + $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Order\Standard') |
|
54 | 54 | ->disableOriginalConstructor() |
55 | - ->setMethods( ['invalid'] ) |
|
55 | + ->setMethods(['invalid']) |
|
56 | 56 | ->getMock(); |
57 | 57 | |
58 | - $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Order\Decorator\Base' ) |
|
59 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
58 | + $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Order\Decorator\Base') |
|
59 | + ->setConstructorArgs([$stub, $this->context]) |
|
60 | 60 | ->getMockForAbstractClass(); |
61 | 61 | |
62 | - $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) ); |
|
62 | + $stub->expects($this->once())->method('invalid')->will($this->returnValue(true)); |
|
63 | 63 | |
64 | - $this->assertTrue( $object->invalid() ); |
|
64 | + $this->assertTrue($object->invalid()); |
|
65 | 65 | } |
66 | 66 | |
67 | 67 | |
68 | 68 | public function testStore() |
69 | 69 | { |
70 | - $orderItem = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem(); |
|
71 | - $basket = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base' )->createItem(); |
|
70 | + $orderItem = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem(); |
|
71 | + $basket = \Aimeos\MShop\Factory::createManager($this->context, 'order/base')->createItem(); |
|
72 | 72 | |
73 | - $this->stub->expects( $this->once() )->method( 'store' ) |
|
74 | - ->will( $this->returnValue( $orderItem ) ); |
|
73 | + $this->stub->expects($this->once())->method('store') |
|
74 | + ->will($this->returnValue($orderItem)); |
|
75 | 75 | |
76 | - $this->assertInstanceOf( '\Aimeos\MShop\Order\Item\Iface', $this->object->store( $basket ) ); |
|
76 | + $this->assertInstanceOf('\Aimeos\MShop\Order\Item\Iface', $this->object->store($basket)); |
|
77 | 77 | } |
78 | 78 | |
79 | 79 | |
80 | 80 | public function testBlock() |
81 | 81 | { |
82 | - $orderItem = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem(); |
|
82 | + $orderItem = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem(); |
|
83 | 83 | |
84 | - $this->stub->expects( $this->once() )->method( 'block' ); |
|
84 | + $this->stub->expects($this->once())->method('block'); |
|
85 | 85 | |
86 | - $this->object->block( $orderItem ); |
|
86 | + $this->object->block($orderItem); |
|
87 | 87 | } |
88 | 88 | |
89 | 89 | |
90 | 90 | public function testUnblock() |
91 | 91 | { |
92 | - $orderItem = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem(); |
|
92 | + $orderItem = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem(); |
|
93 | 93 | |
94 | - $this->stub->expects( $this->once() )->method( 'unblock' ); |
|
94 | + $this->stub->expects($this->once())->method('unblock'); |
|
95 | 95 | |
96 | - $this->object->unblock( $orderItem ); |
|
96 | + $this->object->unblock($orderItem); |
|
97 | 97 | } |
98 | 98 | |
99 | 99 | |
100 | 100 | public function testUpdate() |
101 | 101 | { |
102 | - $orderItem = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem(); |
|
102 | + $orderItem = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem(); |
|
103 | 103 | |
104 | - $this->stub->expects( $this->once() )->method( 'update' ); |
|
104 | + $this->stub->expects($this->once())->method('update'); |
|
105 | 105 | |
106 | - $this->object->update( $orderItem ); |
|
106 | + $this->object->update($orderItem); |
|
107 | 107 | } |
108 | 108 | |
109 | 109 | |
110 | 110 | public function testGetController() |
111 | 111 | { |
112 | - $result = $this->access( 'getController' )->invokeArgs( $this->object, [] ); |
|
112 | + $result = $this->access('getController')->invokeArgs($this->object, []); |
|
113 | 113 | |
114 | - $this->assertSame( $this->stub, $result ); |
|
114 | + $this->assertSame($this->stub, $result); |
|
115 | 115 | } |
116 | 116 | |
117 | 117 | |
118 | - protected function access( $name ) |
|
118 | + protected function access($name) |
|
119 | 119 | { |
120 | - $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Order\Decorator\Base' ); |
|
121 | - $method = $class->getMethod( $name ); |
|
122 | - $method->setAccessible( true ); |
|
120 | + $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Order\Decorator\Base'); |
|
121 | + $method = $class->getMethod($name); |
|
122 | + $method->setAccessible(true); |
|
123 | 123 | |
124 | 124 | return $method; |
125 | 125 | } |
@@ -18,31 +18,31 @@ |
||
18 | 18 | { |
19 | 19 | $context = \TestHelperFrontend::getContext(); |
20 | 20 | |
21 | - $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Base' ) |
|
22 | - ->setConstructorArgs( [$context] ) |
|
21 | + $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Base') |
|
22 | + ->setConstructorArgs([$context]) |
|
23 | 23 | ->getMockForAbstractClass(); |
24 | 24 | } |
25 | 25 | |
26 | 26 | |
27 | 27 | protected function tearDown() |
28 | 28 | { |
29 | - unset( $this->object ); |
|
29 | + unset($this->object); |
|
30 | 30 | } |
31 | 31 | |
32 | 32 | |
33 | 33 | public function testGetContext() |
34 | 34 | { |
35 | - $result = $this->access( 'getContext' )->invokeArgs( $this->object, [] ); |
|
35 | + $result = $this->access('getContext')->invokeArgs($this->object, []); |
|
36 | 36 | |
37 | - $this->assertInstanceOf( '\Aimeos\MShop\Context\Item\Iface', $result ); |
|
37 | + $this->assertInstanceOf('\Aimeos\MShop\Context\Item\Iface', $result); |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | |
41 | - protected function access( $name ) |
|
41 | + protected function access($name) |
|
42 | 42 | { |
43 | - $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Base' ); |
|
44 | - $method = $class->getMethod( $name ); |
|
45 | - $method->setAccessible( true ); |
|
43 | + $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Base'); |
|
44 | + $method = $class->getMethod($name); |
|
45 | + $method->setAccessible(true); |
|
46 | 46 | |
47 | 47 | return $method; |
48 | 48 | } |
@@ -20,108 +20,108 @@ |
||
20 | 20 | { |
21 | 21 | $this->context = \TestHelperFrontend::getContext(); |
22 | 22 | |
23 | - $this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Standard' ) |
|
23 | + $this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard') |
|
24 | 24 | ->disableOriginalConstructor() |
25 | 25 | ->getMock(); |
26 | 26 | |
27 | - $this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' ) |
|
28 | - ->setConstructorArgs( [$this->stub, $this->context] ) |
|
27 | + $this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base') |
|
28 | + ->setConstructorArgs([$this->stub, $this->context]) |
|
29 | 29 | ->getMockForAbstractClass(); |
30 | 30 | } |
31 | 31 | |
32 | 32 | |
33 | 33 | protected function tearDown() |
34 | 34 | { |
35 | - unset( $this->context, $this->object, $this->stub ); |
|
35 | + unset($this->context, $this->object, $this->stub); |
|
36 | 36 | } |
37 | 37 | |
38 | 38 | |
39 | 39 | public function testConstructException() |
40 | 40 | { |
41 | - $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Iface' )->getMock(); |
|
41 | + $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock(); |
|
42 | 42 | |
43 | - $this->setExpectedException( '\Aimeos\Controller\Frontend\Exception' ); |
|
43 | + $this->setExpectedException('\Aimeos\Controller\Frontend\Exception'); |
|
44 | 44 | |
45 | - $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' ) |
|
46 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
45 | + $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base') |
|
46 | + ->setConstructorArgs([$stub, $this->context]) |
|
47 | 47 | ->getMockForAbstractClass(); |
48 | 48 | } |
49 | 49 | |
50 | 50 | |
51 | 51 | public function testCall() |
52 | 52 | { |
53 | - $stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Standard' ) |
|
53 | + $stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard') |
|
54 | 54 | ->disableOriginalConstructor() |
55 | - ->setMethods( ['invalid'] ) |
|
55 | + ->setMethods(['invalid']) |
|
56 | 56 | ->getMock(); |
57 | 57 | |
58 | - $object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' ) |
|
59 | - ->setConstructorArgs( [$stub, $this->context] ) |
|
58 | + $object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base') |
|
59 | + ->setConstructorArgs([$stub, $this->context]) |
|
60 | 60 | ->getMockForAbstractClass(); |
61 | 61 | |
62 | - $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) ); |
|
62 | + $stub->expects($this->once())->method('invalid')->will($this->returnValue(true)); |
|
63 | 63 | |
64 | - $this->assertTrue( $object->invalid() ); |
|
64 | + $this->assertTrue($object->invalid()); |
|
65 | 65 | } |
66 | 66 | |
67 | 67 | |
68 | 68 | public function testAddFilterTypes() |
69 | 69 | { |
70 | - $search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch(); |
|
70 | + $search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch(); |
|
71 | 71 | |
72 | - $this->stub->expects( $this->once() )->method( 'addFilterTypes' ) |
|
73 | - ->will( $this->returnArgument( 0 ) ); |
|
72 | + $this->stub->expects($this->once())->method('addFilterTypes') |
|
73 | + ->will($this->returnArgument(0)); |
|
74 | 74 | |
75 | - $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes( $search, [] ) ); |
|
75 | + $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes($search, [])); |
|
76 | 76 | } |
77 | 77 | |
78 | 78 | |
79 | 79 | public function testCreateFilter() |
80 | 80 | { |
81 | - $search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch(); |
|
81 | + $search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch(); |
|
82 | 82 | |
83 | - $this->stub->expects( $this->once() )->method( 'createFilter' ) |
|
84 | - ->will( $this->returnValue( $search ) ); |
|
83 | + $this->stub->expects($this->once())->method('createFilter') |
|
84 | + ->will($this->returnValue($search)); |
|
85 | 85 | |
86 | - $this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() ); |
|
86 | + $this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter()); |
|
87 | 87 | } |
88 | 88 | |
89 | 89 | |
90 | 90 | public function testGetItem() |
91 | 91 | { |
92 | - $item = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createItem(); |
|
92 | + $item = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createItem(); |
|
93 | 93 | |
94 | - $this->stub->expects( $this->once() )->method( 'getItem' ) |
|
95 | - ->will( $this->returnValue( $item ) ); |
|
94 | + $this->stub->expects($this->once())->method('getItem') |
|
95 | + ->will($this->returnValue($item)); |
|
96 | 96 | |
97 | - $this->assertInstanceOf( '\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 ) ); |
|
97 | + $this->assertInstanceOf('\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 )); |
|
98 | 98 | } |
99 | 99 | |
100 | 100 | |
101 | 101 | public function testSearchItems() |
102 | 102 | { |
103 | - $filter = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch(); |
|
103 | + $filter = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch(); |
|
104 | 104 | |
105 | - $this->stub->expects( $this->once() )->method( 'searchItems' ) |
|
106 | - ->will( $this->returnValue( [] ) ); |
|
105 | + $this->stub->expects($this->once())->method('searchItems') |
|
106 | + ->will($this->returnValue([])); |
|
107 | 107 | |
108 | - $this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) ); |
|
108 | + $this->assertEquals([], $this->object->searchItems($filter, ['media'])); |
|
109 | 109 | } |
110 | 110 | |
111 | 111 | |
112 | 112 | public function testGetController() |
113 | 113 | { |
114 | - $result = $this->access( 'getController' )->invokeArgs( $this->object, [] ); |
|
114 | + $result = $this->access('getController')->invokeArgs($this->object, []); |
|
115 | 115 | |
116 | - $this->assertSame( $this->stub, $result ); |
|
116 | + $this->assertSame($this->stub, $result); |
|
117 | 117 | } |
118 | 118 | |
119 | 119 | |
120 | - protected function access( $name ) |
|
120 | + protected function access($name) |
|
121 | 121 | { |
122 | - $class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' ); |
|
123 | - $method = $class->getMethod( $name ); |
|
124 | - $method->setAccessible( true ); |
|
122 | + $class = new \ReflectionClass('\Aimeos\Controller\Frontend\Attribute\Decorator\Base'); |
|
123 | + $method = $class->getMethod($name); |
|
124 | + $method->setAccessible(true); |
|
125 | 125 | |
126 | 126 | return $method; |
127 | 127 | } |