aimeos / aimeos-core

lib/mshoplib/tests/MShop/Coupon/Provider/VoucherTest.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 	public function testCheckVoucher()
 	{
 		$this->setExpectedException( \Aimeos\MShop\Coupon\Exception::class );
-		$this->access( 'checkVoucher' )->invokeArgs( $this->object, [-1, [5,6]] );
+		$this->access( 'checkVoucher' )->invokeArgs( $this->object, [-1, [5, 6]] );
 	}
 
 

lib/mshoplib/tests/MShop/Customer/Manager/Property/StandardTest.php

Spacing +11 added lines, -11 removed lines

@@ -54,11 +54,11 @@ 
 		$search->setConditions( $search->compare( '==', 'customer.property.editor', $this->editor ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No property item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setLanguageId( 'en' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -109,13 +109,13 @@ 
 	{
 		$search = $this->object->createSearch();
 		$conditions = array(
-			$search->compare( '~=', 'customer.property.value', '1'),
+			$search->compare( '~=', 'customer.property.value', '1' ),
 			$search->compare( '==', 'customer.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results)) === false ) {
+		if( ( $expected = reset( $results ) ) === false ) {
 			throw new \RuntimeException( sprintf( 'No customer property item found for value "%1$s".', '1' ) );
 		}
 
@@ -166,7 +166,7 @@ 
 		$expr[] = $search->compare( '>=', 'customer.property.type.ctime', '1970-01-01 00:00:00' );
 		$expr[] = $search->compare( '==', 'customer.property.type.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 
@@ -177,13 +177,13 @@ 
 			$search->compare( '==', 'customer.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
-		$search->setSlice(0, 1);
+		$search->setSlice( 0, 1 );
 		$items = $this->object->searchItems( $search, [], $total );
 
 		$this->assertEquals( 1, count( $items ) );
 		$this->assertEquals( 1, $total );
 
-		foreach($items as $itemId => $item) {
+		foreach( $items as $itemId => $item ) {
 			$this->assertEquals( $itemId, $item->getId() );
 		}
 	}
@@ -191,17 +191,17 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type') );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type' ) );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type', 'Standard' ) );
 
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('unknown');
+		$this->object->getSubManager( 'unknown' );
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('type', 'unknown');
+		$this->object->getSubManager( 'type', 'unknown' );
 	}
 }

lib/mshoplib/tests/MShop/Customer/Manager/Property/Type/StandardTest.php

Spacing +13 added lines, -13 removed lines

@@ -19,13 +19,13 @@ 
 	{
 		$this->editor = \TestHelperMShop::getContext()->getEditor();
 		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( \TestHelperMShop::getContext() );
-		$this->object = $manager->getSubManager( 'property' )->getSubManager('type');
+		$this->object = $manager->getSubManager( 'property' )->getSubManager( 'type' );
 	}
 
 
 	protected function tearDown()
 	{
-		unset($this->object);
+		unset( $this->object );
 	}
 
 
@@ -68,7 +68,7 @@ 
 
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results) ) === false )
+		if( ( $expected = reset( $results ) ) === false )
 		{
 			throw new \RuntimeException( 'No property type item found.' );
 		}
@@ -90,13 +90,13 @@ 
 	{
 		$search = $this->object->createSearch();
 		$search->setConditions( $search->compare( '==', 'customer.property.type.editor', $this->editor ) );
-		$results = $this->object->searchItems($search);
+		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No type item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setCode( 'unitTestSave' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -157,25 +157,25 @@ 
 		$expr[] = $search->compare( '>=', 'customer.property.type.ctime', '1970-01-01 00:00:00' );
 		$expr[] = $search->compare( '==', 'customer.property.type.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 
 
 		$search = $this->object->createSearch();
 		$conditions = array(
-			$search->compare( '=~', 'customer.property.type.code', 'newsletter'),
+			$search->compare( '=~', 'customer.property.type.code', 'newsletter' ),
 			$search->compare( '==', 'customer.property.type.editor', $this->editor )
 		);
-		$search->setConditions( $search->combine('&&', $conditions ) );
+		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$search->setSortations( [$search->sort( '-', 'customer.property.type.position' )] );
-		$search->setSlice(0, 1);
-		$items = $this->object->searchItems( $search, [], $total);
+		$search->setSlice( 0, 1 );
+		$items = $this->object->searchItems( $search, [], $total );
 
 		$this->assertEquals( 1, count( $items ) );
 		$this->assertEquals( 1, $total );
 
-		foreach($items as $itemId => $item) {
+		foreach( $items as $itemId => $item ) {
 			$this->assertEquals( $itemId, $item->getId() );
 		}
 	}
@@ -184,6 +184,6 @@ 
 	public function testGetSubManager()
 	{
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('unknown');
+		$this->object->getSubManager( 'unknown' );
 	}
 }

lib/mshoplib/tests/MShop/Common/Item/Property/StandardTest.php

Spacing +8 added lines, -8 removed lines

@@ -68,7 +68,7 @@ 
 
 	public function testSetLanguageId()
 	{
-		$return = $this->object->setLanguageId('fr');
+		$return = $this->object->setLanguageId( 'fr' );
 
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Property\Iface::class, $return );
 		$this->assertEquals( 'fr', $this->object->getLanguageId() );
@@ -96,7 +96,7 @@ 
 
 	public function testSetTypeId()
 	{
-		$return = $this->object->setTypeId(33);
+		$return = $this->object->setTypeId( 33 );
 
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Property\Iface::class, $return );
 		$this->assertEquals( 33, $this->object->getTypeId() );
@@ -162,14 +162,14 @@ 
 			'common.property.value' => 'value',
 		);
 
-		$unknown = $item->fromArray($list);
+		$unknown = $item->fromArray( $list );
 
-		$this->assertEquals([], $unknown);
+		$this->assertEquals( [], $unknown );
 
-		$this->assertEquals($list['common.property.parentid'], $item->getParentId());
-		$this->assertEquals($list['common.property.typeid'], $item->getTypeId());
-		$this->assertEquals($list['common.property.languageid'], $item->getLanguageId());
-		$this->assertEquals($list['common.property.value'], $item->getValue());
+		$this->assertEquals( $list['common.property.parentid'], $item->getParentId() );
+		$this->assertEquals( $list['common.property.typeid'], $item->getTypeId() );
+		$this->assertEquals( $list['common.property.languageid'], $item->getLanguageId() );
+		$this->assertEquals( $list['common.property.value'], $item->getValue() );
 		$this->assertNull( $item->getSiteId() );
 		$this->assertNull( $item->getTypeName() );
 		$this->assertNull( $item->getType() );

lib/mshoplib/src/MShop/Supplier/Manager/Factory.php

Spacing +5 added lines, -5 removed lines

@@ -66,14 +66,14 @@
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if ( $name === null ) {
-			$name = $context->getConfig()->get('mshop/supplier/manager/name', 'Standard');
+		if( $name === null ) {
+			$name = $context->getConfig()->get( 'mshop/supplier/manager/name', 'Standard' );
 		}
 
-		if ( ctype_alnum($name) === false )
+		if( ctype_alnum( $name ) === false )
 		{
-			$classname = is_string($name) ? '\Aimeos\MShop\Supplier\Manager\\' . $name : '<not a string>';
-			throw new \Aimeos\MShop\Supplier\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
+			$classname = is_string( $name ) ? '\Aimeos\MShop\Supplier\Manager\\' . $name : '<not a string>';
+			throw new \Aimeos\MShop\Supplier\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 		}
 
 		$iface = \Aimeos\MShop\Supplier\Manager\Iface::class;

lib/mshoplib/src/MShop/Common/Item/ListRef/Traits.php

Spacing +1 added lines, -1 removed lines

@@ -209,7 +209,7 @@
 				$refItem = $item->getRefItem();
 				$iface = \Aimeos\MShop\Common\Item\Typeid\Iface::class;
 
-				if( $type && ( !$refItem || !($refItem instanceof $iface) || !in_array( $refItem->getType(), $types ) ) ) {
+				if( $type && ( !$refItem || !( $refItem instanceof $iface ) || !in_array( $refItem->getType(), $types ) ) ) {
 					continue;
 				}
 

lib/mwlib/tests/MW/Tree/Manager/DBNestedSetTest.php

Spacing +16 added lines, -16 removed lines

@@ -179,14 +179,14 @@ 
 	public function testConstructorNoDatabaseManager()
 	{
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, null);
+		new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, null );
 	}
 
 
 	public function testConstructorNoConfig()
 	{
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		new \Aimeos\MW\Tree\Manager\DBNestedSet([], self::$dbm);
+		new \Aimeos\MW\Tree\Manager\DBNestedSet( [], self::$dbm );
 	}
 
 
@@ -195,7 +195,7 @@ 
 		unset( $this->config['sql'] );
 
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 	}
 
 
@@ -204,7 +204,7 @@ 
 		unset( $this->config['sql']['newid'] );
 
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 	}
 
 
@@ -213,7 +213,7 @@ 
 		unset( $this->config['search']['id'] );
 
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 	}
 
 
@@ -260,7 +260,7 @@ 
 
 
 		if( ( $node = reset( $nodes ) ) === false ) {
-			throw new \RuntimeException('No node found');
+			throw new \RuntimeException( 'No node found' );
 		}
 
 		$search->setConditions( $search->compare( '==', 'tree.level', 3 ) );
@@ -284,7 +284,7 @@ 
 
 		$this->setExpectedException( \Aimeos\MW\DB\Exception::class );
 
-		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 		$manager->searchNodes( $manager->createSearch() );
 	}
 
@@ -321,8 +321,8 @@ 
 		$this->setExpectedException( \Aimeos\MW\DB\Exception::class );
 
 		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
-		$root = $manager->getNode( null, \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE  );
-		$manager->deleteNode($root->getId());
+		$root = $manager->getNode( null, \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE );
+		$manager->deleteNode( $root->getId() );
 	}
 
 
@@ -392,10 +392,10 @@ 
 
 	public function testGetLevelFromConstantException()
 	{
-		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		$manager->getNode( null, 0);
+		$manager->getNode( null, 0 );
 	}
 
 
@@ -432,7 +432,7 @@ 
 
 	public function testInsertNodeException()
 	{
-		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 		$newNode = $manager->createNode();
 
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
@@ -1023,7 +1023,7 @@ 
 		';
 
 
-		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 
 		$root = $manager->getNode( null, \Aimeos\MW\Tree\Manager\Base::LEVEL_TREE );
 
@@ -1055,7 +1055,7 @@ 
 		$node = $manager->createNode();
 
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		$manager->saveNode($node);
+		$manager->saveNode( $node );
 	}
 
 
@@ -1065,7 +1065,7 @@ 
 			UPDATE "mw_tree_test" SET label123 = ?, status = ? WHERE id = ?
 		';
 
-		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 		$root = $manager->getNode( null, \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE );
 
 		$root->setLabel( 'rooot' );
@@ -1081,7 +1081,7 @@ 
 		$method = $class->getMethod( 'setReadOnly' );
 		$method->setAccessible( true );
 
-		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet($this->config, self::$dbm);
+		$manager = new \Aimeos\MW\Tree\Manager\DBNestedSet( $this->config, self::$dbm );
 
 		$method->invokeArgs( $manager, [] );
 

lib/mwlib/tests/MW/Tree/Node/DBNestedSetTest.php

Spacing +2 added lines, -2 removed lines

@@ -52,14 +52,14 @@
 
 	public function testSetId()
 	{
-		$this->object->setId(null);
+		$this->object->setId( null );
 		$this->assertEquals( true, $this->object->isModified() );
 	}
 
 	public function testGetChild()
 	{
 		$this->setExpectedException( \Aimeos\MW\Tree\Exception::class );
-		$this->object->getChild(null);
+		$this->object->getChild( null );
 	}
 
 	public function testMagicGet()

lib/mwlib/tests/MW/Media/FactoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 	public function testGetImage()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		$object = \Aimeos\MW\Media\Factory::get( __DIR__ . $ds .'_testfiles' . $ds . 'image.png' );
+		$object = \Aimeos\MW\Media\Factory::get( __DIR__ . $ds . '_testfiles' . $ds . 'image.png' );
 
 		$this->assertInstanceOf( \Aimeos\MW\Media\Iface::class, $object );
 		$this->assertInstanceOf( \Aimeos\MW\Media\Image\Iface::class, $object );
@@ -25,7 +25,7 @@ 
 	public function testGetImageAsResource()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		$resource = fopen( __DIR__ . $ds .'_testfiles' . $ds . 'image.png', 'rw' );
+		$resource = fopen( __DIR__ . $ds . '_testfiles' . $ds . 'image.png', 'rw' );
 		$object = \Aimeos\MW\Media\Factory::get( $resource );
 
 		$this->assertInstanceOf( \Aimeos\MW\Media\Iface::class, $object );
@@ -36,7 +36,7 @@ 
 	public function testGetImageAsString()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		$content = file_get_contents( __DIR__ . $ds .'_testfiles' . $ds . 'image.png' );
+		$content = file_get_contents( __DIR__ . $ds . '_testfiles' . $ds . 'image.png' );
 		$object = \Aimeos\MW\Media\Factory::get( $content );
 
 		$this->assertInstanceOf( \Aimeos\MW\Media\Iface::class, $object );