aimeos / aimeos-core

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

Spacing +10 added lines, -10 removed lines

@@ -77,10 +77,10 @@ 
 			'undefined' => '$undefined',
 		);
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'listitem', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'listitem', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "( \$listitem == 'a' || \$listitem == 'b' || \$listitem == 'c' )", $expr->toString( $types, $translations ) );
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '!=', 'listitem', array('a', 'b') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '!=', 'listitem', array( 'a', 'b' ) );
 		$this->assertEquals( "( \$listitem != 'a' && \$listitem != 'b' )", $expr->toString( $types, $translations ) );
 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'stringvar', 'value' );
@@ -89,16 +89,16 @@ 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '<', 'floatvar', 0.1 );
 		$this->assertEquals( '$floatvar < 0.1', $expr->toString( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PHP( '>', 'intvar', 10 );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '>', 'intvar', 10 );
 		$this->assertEquals( '$intvar > 10', $expr->toString( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'boolvar', true );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'boolvar', true );
 		$this->assertEquals( '$boolvar == 1', $expr->toString( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PHP( '!=', 'undefined', null );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '!=', 'undefined', null );
 		$this->assertEquals( '$undefined !== null', $expr->toString( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'undefined', null );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'undefined', null );
 		$this->assertEquals( '$undefined === null', $expr->toString( $types, $translations ) );
 
 
@@ -124,15 +124,15 @@ 
 			'undefined' => '$undefined',
 		);
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '>=', 'listitem', array('a', 'b') );
-		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '>=', 'listitem', array( 'a', 'b' ) );
+		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
 		$expr->toString( $types, $translations );
 	}
 
 	public function testToStringExcept2()
 	{
-		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
-		new \Aimeos\MW\Criteria\Expression\Compare\PHP('=', 'undefined', null);
+		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
+		new \Aimeos\MW\Criteria\Expression\Compare\PHP( '=', 'undefined', null );
 	}
 
 	public function testToStringFunction()

lib/mshoplib/setup/OrderMigrateFlag.php

Braces +2 added lines, -4 removed lines

@@ -78,8 +78,7 @@ 
 			{
 				$this->execute( $stmts['index'] );
 				$this->status( 'dropped' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}
@@ -125,8 +124,7 @@ 
 				} else {
 					$this->status( 'OK' );
 				}
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

lib/mshoplib/tests/MShop/Service/Provider/Decorator/BaseTest.php

Spacing +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
 
 		$servManager = \Aimeos\MShop\Service\Manager\Factory::createManager( $this->context );
 		$search = $servManager->createSearch();
-		$search->setConditions($search->compare('==', 'service.provider', 'Standard'));
-		$result = $servManager->searchItems($search, array('price'));
+		$search->setConditions( $search->compare( '==', 'service.provider', 'Standard' ) );
+		$result = $servManager->searchItems( $search, array( 'price' ) );
 
 		if( ( $item = reset( $result ) ) === false ) {
 			throw new \RuntimeException( 'No order base item found' );

lib/mshoplib/src/MShop/Catalog/Manager/Base.php

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 	 * Creates the catalog item objects.
 	 *
 	 * @param array $itemMap Associative list of catalog ID / tree node pairs
-	 * @param array $domains List of domains (e.g. text, media) whose items should be attached to the catalog items
+	 * @param string[] $domains List of domains (e.g. text, media) whose items should be attached to the catalog items
 	 * @param string $prefix Domain prefix
 	 * @param array $local Associative list of IDs as keys and the associative array of items as values
 	 * @return array List of items implementing \Aimeos\MShop\Catalog\Item\Iface

lib/mwlib/src/MW/DB/Statement/Base.php

Spacing +2 added lines, -2 removed lines

@@ -132,9 +132,9 @@
 					&& ( $count += substr_count( $pr, '\'' ) ) % 2 !== 0 )
 				{
 					if( ( $part = next( $parts ) ) === false ) {
-						throw new \Aimeos\MW\DB\Exception( 'Number of apostrophes don\'t match: ' . $sql );
+						throw new \Aimeos\MW\DB\Exception( 'Number of apostrophes don\'t match: '.$sql );
 					}
-					$temp .= '?' . $part;
+					$temp .= '?'.$part;
 				}
 				$result[] = $temp;
 			}

Doc Comments +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
 	 * Reconnects to the database server if not in a transaction
 	 *
 	 * @param \Exception $e Exception to throw if a transaction is running
-	 * @return \Aimeos\MW\DB\Statement\Iface Object for method chaining
+	 * @return Base Object for method chaining
 	 * @throws \Exception If a transaction is running
 	 */
 	protected function reconnect( \Exception $e )

lib/mshoplib/tests/MShop/Order/Manager/Base/StandardTest.php

Spacing +1 added lines, -1 removed lines

@@ -335,7 +335,7 @@
 
 	public function testSearchItemsDefault()
 	{
-		$search = $this->object->createSearch(  true );
+		$search = $this->object->createSearch( true );
 		$items = $this->object->searchItems( $search );
 
 		$this->assertEquals( 0, count( $items ) );

lib/mshoplib/src/MShop/Common/Manager/Type/Base.php

Doc Comments +1 added lines

@@ -200,6 +200,7 @@
 	 * @param \Aimeos\MW\Criteria\Iface $search Search criteria object
 	 * @param string[] $ref List of domains to fetch list items and referenced items for
 	 * @param integer|null &$total Number of items that are available in total
+	 * @param integer $total
 	 * @return array List of type items implementing \Aimeos\MShop\Common\Item\Type\Iface
 	 */
 	public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null )

Spacing +11 added lines, -11 removed lines

@@ -61,7 +61,7 @@ 
 	public function createItem()
 	{
 		$values = array(
-			$this->prefix . 'siteid' => $this->getContext()->getLocale()->getSiteId(),
+			$this->prefix.'siteid' => $this->getContext()->getLocale()->getSiteId(),
 		);
 		return $this->createItemBase( $values );
 	}
@@ -114,7 +114,7 @@ 
 			}
 
 			$time = date( 'Y-m-d H:i:s', time() );
-			$stmt = $conn->create( $this->getSqlConfig( $this->getConfigPath() . $type ) );
+			$stmt = $conn->create( $this->getSqlConfig( $this->getConfigPath().$type ) );
 
 			$stmt->bind( 1, $item->getCode(), \Aimeos\MW\DB\Statement\Base::PARAM_STR );
 			$stmt->bind( 2, $item->getDomain(), \Aimeos\MW\DB\Statement\Base::PARAM_STR );
@@ -135,7 +135,7 @@ 
 			if( $fetch === true )
 			{
 				if( $id === null ) {
-					$item->setId( $this->newId( $conn, $this->getConfigPath() . 'newid' ) );
+					$item->setId( $this->newId( $conn, $this->getConfigPath().'newid' ) );
 				} else {
 					$item->setId( $id ); // modified false
 				}
@@ -158,7 +158,7 @@ 
 	 */
 	public function deleteItems( array $ids )
 	{
-		$this->deleteItemsBase( $ids, $this->getConfigPath() . 'delete' );
+		$this->deleteItemsBase( $ids, $this->getConfigPath().'delete' );
 	}
 
 
@@ -174,8 +174,8 @@ 
 	public function findItem( $code, array $ref = [], $domain = 'product', $type = null )
 	{
 		$find = array(
-			$this->prefix . 'code' => $code,
-			$this->prefix . 'domain' => $domain,
+			$this->prefix.'code' => $code,
+			$this->prefix.'domain' => $domain,
 		);
 		return $this->findItemBase( $find, $ref );
 	}
@@ -191,7 +191,7 @@ 
 	 */
 	public function getItem( $id, array $ref = [], $default = false )
 	{
-		return $this->getItemBase( $this->prefix . 'id', $id, $ref, $default );
+		return $this->getItemBase( $this->prefix.'id', $id, $ref, $default );
 	}
 
 
@@ -220,13 +220,13 @@ 
 			}
 
 			$level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL;
-			$cfgPathSearch = $this->getConfigPath() . 'search';
-			$cfgPathCount = $this->getConfigPath() . 'count';
+			$cfgPathSearch = $this->getConfigPath().'search';
+			$cfgPathCount = $this->getConfigPath().'count';
 			$required = array( trim( $this->prefix, '.' ) );
 
 			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
 			while( ( $row = $results->fetch() ) !== false ) {
-				$items[$row[$this->prefix . 'id']] = $this->createItemBase( $row );
+				$items[$row[$this->prefix.'id']] = $this->createItemBase( $row );
 			}
 
 			$dbm->release( $conn, $dbname );
@@ -250,7 +250,7 @@ 
 	 */
 	public function getSubManager( $manager, $name = null )
 	{
-		return $this->getSubManagerBase( 'common', 'type/' . $manager, $name );
+		return $this->getSubManagerBase( 'common', 'type/'.$manager, $name );
 	}
 
 

lib/mshoplib/src/MShop/Service/Provider/Decorator/Base.php

Doc Comments +1 added lines, -1 removed lines

@@ -180,7 +180,7 @@
 	 * Processes the order
 	 *
 	 * @param \Aimeos\MShop\Order\Item\Iface $order Order invoice object to process
-	 * @param array $params Request parameter if available
+	 * @param string[] $params Request parameter if available
 	 * @return \Aimeos\MShop\Common\Item\Helper\Form\Standard|null Form object or null
 	 */
 	public function process( \Aimeos\MShop\Order\Item\Iface $order, array $params = [] )

controller/common/tests/TestHelperCntl.php

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
 	{
 		if( !isset( self::$aimeos ) )
 		{
-			require_once dirname( dirname( dirname( __DIR__ ) ) ) . DIRECTORY_SEPARATOR . 'Bootstrap.php';
+			require_once dirname( dirname( dirname( __DIR__ ) ) ).DIRECTORY_SEPARATOR.'Bootstrap.php';
 
 			self::$aimeos = new \Aimeos\Bootstrap( [], false );
 		}
@@ -70,8 +70,8 @@ 
 
 
 		$paths = $aimeos->getConfigPaths();
-		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
-		$file = __DIR__ . DIRECTORY_SEPARATOR . 'confdoc.ser';
+		$paths[] = __DIR__.DIRECTORY_SEPARATOR.'config';
+		$file = __DIR__.DIRECTORY_SEPARATOR.'confdoc.ser';
 
 		$conf = new \Aimeos\MW\Config\PHPArray( [], $paths );
 		$conf = new \Aimeos\MW\Config\Decorator\Memory( $conf );