aimeos / aimeos-core

lib/mshoplib/setup/TypesMigrateColumns.php

Braces +3 added lines, -6 removed lines

@@ -167,8 +167,7 @@ 
 
 				$this->executeList( $schemaDiff->toSaveSql( $platform ), $rname );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}
@@ -207,8 +206,7 @@ 
 
 				$this->executeList( $schemaDiff->toSaveSql( $platform ), $rname );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}
@@ -229,8 +227,7 @@ 
 			if( $schema->columnExists( $table, 'typeid' ) ) {
 				$this->execute( $stmt, $rname );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

lib/mshoplib/setup/unitperf/CatalogAddPerfData.php

Spacing +25 added lines, -25 removed lines

@@ -76,8 +76,8 @@ 
 
 			if( $level > 0 )
 			{
-				for( $i = 0; $i < $numCatPerLevel; $i++ ){
-					$treeFcn( $parents, $catItem->getId(), $numCatPerLevel, $level - 1, $catLabel . '-' . ($i+1), $i );
+				for( $i = 0; $i < $numCatPerLevel; $i++ ) {
+					$treeFcn( $parents, $catItem->getId(), $numCatPerLevel, $level - 1, $catLabel . '-' . ( $i + 1 ), $i );
 				}
 			}
 			else
@@ -117,7 +117,7 @@ 
 		}
 
 		for( $i = $begin; $i < $end; $i++ ) {
-			$treeFcn( [$catRootItem], $catRootItem->getId(), $numCatPerLevel, $this->numCatLevels - 1, $i+1, $i );
+			$treeFcn( [$catRootItem], $catRootItem->getId(), $numCatPerLevel, $this->numCatLevels - 1, $i + 1, $i );
 		}
 
 		$this->additional->getProcess()->wait();
@@ -163,7 +163,7 @@ 
 			{
 				if( $item->pos % $fraction === 0 )
 				{
-					$litem = (clone $defListItem)->setRefId( $item->getId() )->setPosition( $start + round( $item->pos / $fraction ) );
+					$litem = ( clone $defListItem )->setRefId( $item->getId() )->setPosition( $start + round( $item->pos / $fraction ) );
 					$catItem->addListItem( 'product', $litem );
 				}
 			}
@@ -199,13 +199,13 @@ 
 		$listItem = $productListManager->createItem()->setType( 'default' );
 
 		foreach( $attrIds as $attrId ) {
-			$prodItem->addListItem( 'attribute', (clone $listItem)->setRefId( $attrId ) );
+			$prodItem->addListItem( 'attribute', ( clone $listItem )->setRefId( $attrId ) );
 		}
 
 		$listItem = $productListManager->createItem()->setType( 'config' );
 
 		foreach( $this->attributes['sticker'] as $attrId => $label ) {
-			$prodItem->addListItem( 'attribute', (clone $listItem)->setRefId( $attrId ) );
+			$prodItem->addListItem( 'attribute', ( clone $listItem )->setRefId( $attrId ) );
 		}
 
 		return $prodItem;
@@ -244,7 +244,7 @@ 
 		{
 			$text = current( $color ) . ' ' . current( $property ) . ' ' . current( $material ) . ' ' . current( $articles );
 
-			$item = (clone $newItem)
+			$item = ( clone $newItem )
 				->setLabel( $text . ' (' . $catLabel . ')' )
 				->setCode( 'p-' . $i . ':' . $catLabel )
 				->setStatus( 1 );
@@ -308,18 +308,18 @@ 
 
 		foreach( array_values( $this->shuffle( range( 0, 3 ) ) ) as $pos => $i )
 		{
-			$num = (($idx + $i) % 4) + 1;
-			$mediaItem = (clone $newItem)
-				->setLabel( ($pos+1) . '. picture for ' . $prodItem->getLabel() )
+			$num = ( ( $idx + $i ) % 4 ) + 1;
+			$mediaItem = ( clone $newItem )
+				->setLabel( ( $pos + 1 ) . '. picture for ' . $prodItem->getLabel() )
 				->setPreview( $prefix . 'unitperf/' . $num . '.jpg' )
 				->setUrl( $prefix . 'unitperf/' . $num . '-big.jpg' )
 				->setMimeType( 'image/jpeg' )
 				->setStatus( 1 );
 
-			$prodItem->addListItem( 'media', (clone $litem)->setPosition( $pos ), $mediaItem );
+			$prodItem->addListItem( 'media', ( clone $litem )->setPosition( $pos ), $mediaItem );
 		}
 
-		$mediaItem = (clone $newItem)
+		$mediaItem = ( clone $newItem )
 			->setPreview( $prefix . 'unitperf/download-preview.jpg' )
 			->setUrl( $prefix . 'unitperf/download.pdf' )
 			->setMimeType( 'application/pdf' )
@@ -328,7 +328,7 @@ 
 
 		$litem = $productListManager->createItem()->setType( 'download' );
 
-		return $prodItem->addListItem( 'media', (clone $litem), $mediaItem );
+		return $prodItem->addListItem( 'media', ( clone $litem ), $mediaItem );
 	}
 
 
@@ -343,15 +343,15 @@ 
 
 		for( $i = 0; $i < 3; $i++ )
 		{
-			$priceItem = (clone $newItem)
+			$priceItem = ( clone $newItem )
 				->setLabel( $prodItem->getLabel() . ': from ' . ( 1 + $i * 5 ) )
-				->setValue( 100 + (( $base + $idx ) % 900) - $i * 10 )
+				->setValue( 100 + ( ( $base + $idx ) % 900 ) - $i * 10 )
 				->setQuantity( 1 + $i * 10 )
 				->setCurrencyId( 'EUR' )
 				->setRebate( $i * 10 )
 				->setStatus( 1 );
 
-			$prodItem->addListItem( 'price', (clone $litem)->setPosition( $i ), $priceItem );
+			$prodItem->addListItem( 'price', ( clone $litem )->setPosition( $i ), $priceItem );
 		}
 
 		return $prodItem;
@@ -372,7 +372,7 @@ 
 			{
 				if( !in_array( $litem->getRefId(), $ids ) )
 				{
-					$prodItem->addListItem( 'product', (clone $listItem)->setRefId( $litem->getRefId() ) );
+					$prodItem->addListItem( 'product', ( clone $listItem )->setRefId( $litem->getRefId() ) );
 					$ids[] = $litem->getRefId();
 					$num--;
 				}
@@ -397,7 +397,7 @@ 
 			->setType( 'url' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem ), $textItem );
 
 		$textItem = $textManager->createItem()
 			->setLanguageId( 'en' )
@@ -406,7 +406,7 @@ 
 			->setType( 'name' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 0 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 0 ), $textItem );
 
 		$textItem = $textManager->createItem()
 			->setContent( 'Short description for ' . $label )
@@ -415,7 +415,7 @@ 
 			->setType( 'short' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 1 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 1 ), $textItem );
 
 		$textItem = $textManager->createItem()
 			->setContent( 'Long description for ' . $label . '. This may include some "lorem ipsum" text' )
@@ -424,7 +424,7 @@ 
 			->setType( 'long' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 2 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 2 ), $textItem );
 
 		return $prodItem;
 	}
@@ -447,14 +447,14 @@ 
 		{
 			$text = current( $length ) . ', ' . current( $width ) . ', ' . $prodItem->getLabel() . ' (' . current( $size ) . ')';
 
-			$item = (clone $newItem)
+			$item = ( clone $newItem )
 				->setCode( 'v-' . $idx . '/' . $i . ':' . $prodItem->getCode() )
 				->setLabel( $text )
 				->setStatus( 1 );
 
-			$item->addListItem( 'attribute', (clone $varListItem)->setRefId( key( $length ) ) );
-			$item->addListItem( 'attribute', (clone $varListItem)->setRefId( key( $width ) ) );
-			$item->addListItem( 'attribute', (clone $varListItem)->setRefId( key( $size ) ) );
+			$item->addListItem( 'attribute', ( clone $varListItem )->setRefId( key( $length ) ) );
+			$item->addListItem( 'attribute', ( clone $varListItem )->setRefId( key( $width ) ) );
+			$item->addListItem( 'attribute', ( clone $varListItem )->setRefId( key( $size ) ) );
 
 			$prodItem->addListItem( 'product', clone $defListItem, $item );
 

Braces +6 added lines, -4 removed lines

@@ -79,8 +79,7 @@ 
 				for( $i = 0; $i < $numCatPerLevel; $i++ ){
 					$treeFcn( $parents, $catItem->getId(), $numCatPerLevel, $level - 1, $catLabel . '-' . ($i+1), $i );
 				}
-			}
-			else
+			} else
 			{
 				$fcn = function( array $parents, $catLabel ) {
 
@@ -139,9 +138,12 @@ 
 
 		while( true )
 		{
-			try {
+			try
+			{
 				return $catalogManager->insertItem( $item, $parentId );
-			} catch( \Aimeos\MW\DB\Exception $e ) {
+			}
+			catch( \Aimeos\MW\DB\Exception $e )
+			{
 				if( $e->getCode() !== 40001 ) { throw $e; } // transaction deadlock
 			}
 		}

lib/mshoplib/src/MShop/Locale/Manager/Currency/Standard.php

Braces +6 added lines, -4 removed lines

@@ -108,9 +108,12 @@ 
 	 */
 	public function createItem( array $values = [] )
 	{
-		try {
+		try
+		{
 			$values['locale.currency.siteid'] = $this->getContext()->getLocale()->getSiteId();
-		} catch( \Exception $e ) {
+		}
+		catch( \Exception $e )
+		{
 			$values['locale.currency.siteid'] = null;
 		}
 
@@ -180,8 +183,7 @@ 
 				 * @see mshop/locale/manager/currency/standard/count/ansi
 				 */
 				$path = 'mshop/locale/manager/currency/standard/insert';
-			}
-			else
+			} else
 			{
 				/** mshop/locale/manager/currency/standard/update/mysql
 				 * Updates an existing currency record in the database

lib/mshoplib/src/MShop/Locale/Manager/Language/Standard.php

Braces +6 added lines, -4 removed lines

@@ -108,9 +108,12 @@ 
 	 */
 	public function createItem( array $values = [] )
 	{
-		try {
+		try
+		{
 			$values['locale.language.siteid'] = $this->getContext()->getLocale()->getSiteId();
-		} catch( \Exception $ex ) {
+		}
+		catch( \Exception $ex )
+		{
 			$values['locale.language.siteid'] = null;
 		}
 
@@ -177,8 +180,7 @@ 
 				 * @see mshop/locale/manager/language/standard/count/ansi
 				 */
 				$path = 'mshop/locale/manager/language/standard/insert';
-			}
-			else
+			} else
 			{
 				/** mshop/locale/manager/language/standard/update/mysql
 				 * Updates an existing language record in the database

lib/mshoplib/src/MAdmin/Log/Manager/Standard.php

Braces +11 added lines, -6 removed lines

@@ -155,9 +155,12 @@ 
 	 */
 	public function createItem( array $values = [] )
 	{
-		try {
+		try
+		{
 			$values['log.siteid'] = $this->getContext()->getLocale()->getSiteId();
-		} catch( \Exception $e ) {
+		}
+		catch( \Exception $e )
+		{
 			$values['log.siteid'] = null;
 		}
 
@@ -182,9 +185,12 @@ 
 
 		$context = $this->getContext();
 
-		try {
+		try
+		{
 			$siteid = $context->getLocale()->getSiteId();
-		} catch( \Exception $e ) {
+		}
+		catch( \Exception $e )
+		{
 			$siteid = null;
 		}
 
@@ -234,8 +240,7 @@ 
 				 * @see madmin/log/manager/standard/count/ansi
 				 */
 				$path = 'madmin/log/manager/standard/insert';
-			}
-			else
+			} else
 			{
 				/** madmin/log/manager/standard/update/mysql
 				 * Updates an existing log record in the database

lib/mwlib/src/MW/Common/Base.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
 	 */
 	public static function checkClass( $name, $object )
 	{
-		if( ($object instanceof $name) === false ) {
+		if( ( $object instanceof $name ) === false ) {
 			throw new \Aimeos\MW\Common\Exception( sprintf( 'Object doesn\'t implement "%1$s"', $name ) );
 		}
 
@@ -50,7 +50,7 @@ 
 	{
 		foreach( $list as $object )
 		{
-			if( ($object instanceof $name) === false ) {
+			if( ( $object instanceof $name ) === false ) {
 				throw new \Aimeos\MW\Common\Exception( sprintf( 'Object doesn\'t implement "%1$s"', $name ) );
 			}
 		}

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

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 			throw new \Aimeos\MW\Cache\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
 		}
 
-		$object =  new $classname( $config, $resource );
+		$object = new $classname( $config, $resource );
 
 		if( !( $object instanceof $iface ) ) {
 			throw new \Aimeos\MW\Cache\Exception( sprintf( 'Class "%1$s" does not implement interface "%2$s"', $classname, $iface ) );

lib/mshoplib/src/MAdmin/Cache/Manager/Standard.php

Braces +10 added lines, -4 removed lines

@@ -217,9 +217,12 @@ 
 			);
 			$dbm = $context->getDatabaseManager();
 
-			try {
+			try
+			{
 				$this->object = \Aimeos\MW\Cache\Factory::create( $name, $config, $dbm );
-			} catch( \Exception $e ) {
+			}
+			catch( \Exception $e )
+			{
 				$this->object = \Aimeos\MW\Cache\Factory::create( 'DB', $config, $dbm );
 			}
 		}
@@ -288,9 +291,12 @@ 
 	 */
 	public function createItem( array $values = [] )
 	{
-		try {
+		try
+		{
 			$values['siteid'] = $this->getContext()->getLocale()->getSiteId();
-		} catch( \Exception $e ) {
+		}
+		catch( \Exception $e )
+		{
 			$values['siteid'] = null;
 		}
 

Indentation +1 added lines, -1 removed lines

@@ -404,7 +404,7 @@
 	 * Removes multiple items specified by ids in the array.
 	 *
 	 * @param string[] $ids List of IDs
-	* @return \Aimeos\MAdmin\Cache\Manager\Iface Manager object for chaining method calls
+	 * @return \Aimeos\MAdmin\Cache\Manager\Iface Manager object for chaining method calls
 	 */
 	public function deleteItems( array $ids )
 	{

lib/mshoplib/setup/MShopAddLocaleLangCurData.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
 		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::create( $this->additional, 'Standard' );
 
 
-		$filename = __DIR__ . $ds . 'default'.  $ds . 'data'. $ds . 'language.php';
+		$filename = __DIR__ . $ds . 'default' . $ds . 'data' . $ds . 'language.php';
 
 		if( ( $data = include( $filename ) ) == false ) {
 			throw new \Aimeos\MW\Setup\Exception( sprintf( 'No data file "%1$s" found', $filename ) );
@@ -62,7 +62,7 @@ 
 		}
 
 
-		$filename = __DIR__ . $ds . 'default'.  $ds . 'data'. $ds . 'currency.php';
+		$filename = __DIR__ . $ds . 'default' . $ds . 'data' . $ds . 'currency.php';
 
 		if( ( $data = include( $filename ) ) == false ) {
 			throw new \Aimeos\MW\Setup\Exception( sprintf( 'No data file "%1$s" found', $filename ) );