aimeos / aimeos-core

lib/mshoplib/src/MShop/Index/Manager/Supplier/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -521,7 +521,7 @@
 
 					try {
 						$stmt->execute()->finish();
-					} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+					} catch( \Aimeos\MW\DB\Exception $e ) {; } // Ignore duplicates
 				}
 			}
 

Braces +6 added lines, -2 removed lines

@@ -472,9 +472,13 @@
 					$stmt->bind( 5, $date ); //mtime
 					$stmt->bind( 6, $siteid, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 
-					try {
+					try
+					{
 						$stmt->execute()->finish();
-					} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+					}
+					catch( \Aimeos\MW\DB\Exception $e )
+					{
+; } // Ignore duplicates
 				}
 			}
 

lib/mwlib/src/MW/Cache/DB.php

Spacing +1 added lines, -1 removed lines

@@ -254,7 +254,7 @@
 			$result = $stmt->execute();
 
 			while( ( $row = $result->fetch() ) !== false ) {
-				$list[ $row['id'] ] = (string) $row['value'];
+				$list[$row['id']] = (string) $row['value'];
 			}
 
 			$this->dbm->release( $conn, $this->dbname );

lib/mwlib/tests/MW/Criteria/Expression/Compare/SQLTest.php

Spacing +5 added lines, -5 removed lines

@@ -85,10 +85,10 @@ 
 			'bool' => 't.bool',
 		);
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list NOT IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '~=', 'string', 'value' );
@@ -97,13 +97,13 @@ 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '<', 'float', 0.1 );
 		$this->assertEquals( "t.float < 0.1", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '>', 'int', 10 );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '>', 'int', 10 );
 		$this->assertEquals( "t.int > 10", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'undefined', null );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'undefined', null );
 		$this->assertEquals( "t.undefined IS NOT NULL", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'bool', true );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'bool', true );
 		$this->assertEquals( "t.bool = 1", $expr->toSource( $types, $translations ) );
 	}
 

lib/mwlib/tests/MW/Criteria/Expression/Compare/PgSQLTest.php

Spacing +6 added lines, -6 removed lines

@@ -52,10 +52,10 @@ 
 			'bool' => 't.bool',
 		);
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list NOT IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '~=', 'string', 'value' );
@@ -64,16 +64,16 @@ 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '<', 'float', 0.1 );
 		$this->assertEquals( "t.float < 0.1", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '>', 'int', 10 );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '>', 'int', 10 );
 		$this->assertEquals( "t.int > 10", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'undefined', null );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'undefined', null );
 		$this->assertEquals( "t.undefined IS NOT NULL", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', true );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', true );
 		$this->assertEquals( "t.bool = 't'", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', false );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', false );
 		$this->assertEquals( "t.bool = 'f'", $expr->toSource( $types, $translations ) );
 	}
 }

lib/mwlib/tests/MW/Criteria/PgSQLTest.php

Spacing +2 added lines, -2 removed lines

@@ -43,11 +43,11 @@
 
 	public function testGetConditionSource()
 	{
-		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL);
+		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL );
 		$this->object->setConditions( $this->object->compare( '==', 'column', 0 ) );
 		$this->assertEquals( "column = 'f'", $this->object->getConditionSource( $types ) );
 
-		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL);
+		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL );
 		$this->object->setConditions( $this->object->compare( '==', 'column', 1 ) );
 		$this->assertEquals( "column = 't'", $this->object->getConditionSource( $types ) );
 	}

controller/common/src/Controller/Common/Order/Standard.php

Braces +2 added lines, -1 removed lines

@@ -334,7 +334,8 @@
 				{
 					$stockManager->decrease( [$item->getProductCode() => $how * -1 * $item->getQuantity()], $item->getStockType() );
 
-					switch( $item->getType() ) {
+					switch( $item->getType() )
+					{
 						case 'default':
 							$this->updateStockBundle( $item->getProductId(), $item->getStockType() ); break;
 						case 'select':

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/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/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, [] );