aimeos / aimeos-core

lib/mshoplib/setup/unittest/SubscriptionAddTestData.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 
 		foreach( $testdata['subscription'] as $key => $dataset )
 		{
-			$ordProdItem = $this->getOrderProductItem( $dataset['ordprodid'] ) ;
+			$ordProdItem = $this->getOrderProductItem( $dataset['ordprodid'] );
 
 			$item = $subscriptionManager->createItem();
 			$item->setOrderBaseId( $ordProdItem->getBaseId() );

lib/mshoplib/setup/unittest/AttributeAddPropertyTestData.php

Spacing +5 added lines, -5 removed lines

@@ -87,19 +87,19 @@ 
 			$type->setStatus( $dataset['status'] );
 
 			$attributePropertyTypeManager->saveItem( $type );
-			$typeIds[ $key ] = $type->getId();
+			$typeIds[$key] = $type->getId();
 		}
 
 		$prodProperty = $attributePropertyManager->createItem();
 		foreach( $testdata['attribute/property'] as $key => $dataset )
 		{
-			if( !isset( $typeIds[ $dataset['typeid'] ] ) ) {
+			if( !isset( $typeIds[$dataset['typeid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No attribute property type ID found for "%1$s"', $dataset['typeid'] ) );
 			}
 
 			$prodProperty->setId( null );
-			$prodProperty->setParentId( $prodIds[ $dataset['parentid'] ] );
-			$prodProperty->setTypeId( $typeIds[ $dataset['typeid'] ] );
+			$prodProperty->setParentId( $prodIds[$dataset['parentid']] );
+			$prodProperty->setTypeId( $typeIds[$dataset['typeid']] );
 			$prodProperty->setLanguageId( $dataset['langid'] );
 			$prodProperty->setValue( $dataset['value'] );
 
@@ -122,7 +122,7 @@ 
 		$search = $attributeManager->createSearch();
 
 		foreach( $attributeManager->searchItems( $search ) as $id => $item ) {
-			$entry[ 'attribute/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getCode() ] = $id;
+			$entry['attribute/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getCode()] = $id;
 		}
 
 		return $entry;

lib/mshoplib/setup/unittest/MediaAddPropertyTestData.php

Spacing +5 added lines, -5 removed lines

@@ -86,19 +86,19 @@ 
 			$type->setStatus( $dataset['status'] );
 
 			$mediaPropertyTypeManager->saveItem( $type );
-			$typeIds[ $key ] = $type->getId();
+			$typeIds[$key] = $type->getId();
 		}
 
 		$prodProperty = $mediaPropertyManager->createItem();
 		foreach( $testdata['media/property'] as $key => $dataset )
 		{
-			if( !isset( $typeIds[ $dataset['typeid'] ] ) ) {
+			if( !isset( $typeIds[$dataset['typeid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No media property type ID found for "%1$s"', $dataset['typeid'] ) );
 			}
 
 			$prodProperty->setId( null );
-			$prodProperty->setParentId( $prodIds[ $dataset['parentid'] ] );
-			$prodProperty->setTypeId( $typeIds[ $dataset['typeid'] ] );
+			$prodProperty->setParentId( $prodIds[$dataset['parentid']] );
+			$prodProperty->setTypeId( $typeIds[$dataset['typeid']] );
 			$prodProperty->setLanguageId( $dataset['langid'] );
 			$prodProperty->setValue( $dataset['value'] );
 
@@ -121,7 +121,7 @@ 
 		$search = $mediaManager->createSearch();
 
 		foreach( $mediaManager->searchItems( $search ) as $id => $item ) {
-			$entry[ 'media/' . $item->getUrl() ] = $id;
+			$entry['media/' . $item->getUrl()] = $id;
 		}
 
 		return $entry;

lib/mshoplib/setup/unittest/CustomerAddPropertyTestData.php

Spacing +5 added lines, -5 removed lines

@@ -87,19 +87,19 @@ 
 			$type->setStatus( $dataset['status'] );
 
 			$customerPropertyTypeManager->saveItem( $type );
-			$typeIds[ $key ] = $type->getId();
+			$typeIds[$key] = $type->getId();
 		}
 
 		$custProperty = $customerPropertyManager->createItem();
 		foreach( $testdata['customer/property'] as $key => $dataset )
 		{
-			if( !isset( $typeIds[ $dataset['typeid'] ] ) ) {
+			if( !isset( $typeIds[$dataset['typeid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No customer property type ID found for "%1$s"', $dataset['typeid'] ) );
 			}
 
 			$custProperty->setId( null );
-			$custProperty->setParentId( $custIds[ $dataset['parentid'] ] );
-			$custProperty->setTypeId( $typeIds[ $dataset['typeid'] ] );
+			$custProperty->setParentId( $custIds[$dataset['parentid']] );
+			$custProperty->setTypeId( $typeIds[$dataset['typeid']] );
 			$custProperty->setLanguageId( $dataset['langid'] );
 			$custProperty->setValue( $dataset['value'] );
 
@@ -122,7 +122,7 @@ 
 		$search = $customerManager->createSearch();
 
 		foreach( $customerManager->searchItems( $search ) as $id => $item ) {
-			$entry[ 'customer/' . $item->getCode() ] = $id;
+			$entry['customer/' . $item->getCode()] = $id;
 		}
 
 		return $entry;

lib/mshoplib/setup/unittest/SupplierListAddTestData.php

Spacing +18 added lines, -18 removed lines

@@ -49,13 +49,13 @@ 
 		$ds = DIRECTORY_SEPARATOR;
 		$path = __DIR__ . $ds . 'data' . $ds . 'supplier-list.php';
 
-		if( ( $testdata = include( $path ) ) == false ){
+		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for supplier list domain', $path ) );
 		}
 
 		$refKeys = [];
 		foreach( $testdata['supplier/lists'] as $dataset ) {
-			$refKeys[ $dataset['domain'] ][] = $dataset['refid'];
+			$refKeys[$dataset['domain']][] = $dataset['refid'];
 		}
 
 		$refIds = [];
@@ -118,7 +118,7 @@ 
 		$codes = [];
 		foreach( $keys as $dataset )
 		{
-			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos+1 ) ) === false ) {
+			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos + 1 ) ) === false ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'Some keys for ref products are set wrong "%1$s"', $dataset ) );
 			}
 
@@ -130,7 +130,7 @@ 
 
 		$refIds = [];
 		foreach( $manager->searchItems( $search ) as $item ) {
-			$refIds[ 'product/' . $item->getCode() ] = $item->getId();
+			$refIds['product/' . $item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -150,7 +150,7 @@ 
 		$labels = [];
 		foreach( $keys as $dataset )
 		{
-			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos+1 ) ) === false ) {
+			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos + 1 ) ) === false ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'Some keys for ref text are set wrong "%1$s"', $dataset ) );
 			}
 
@@ -161,8 +161,8 @@ 
 		$search->setConditions( $search->compare( '==', 'text.label', $labels ) );
 
 		$refIds = [];
-		foreach( $textManager->searchItems( $search ) as $item )	{
-			$refIds[ 'text/'.$item->getLabel() ] = $item->getId();
+		foreach( $textManager->searchItems( $search ) as $item ) {
+			$refIds['text/' . $item->getLabel()] = $item->getId();
 		}
 
 		return $refIds;
@@ -186,7 +186,7 @@ 
 		$itemCode = [];
 		foreach( $testdata['supplier/lists'] as $dataset )
 		{
-			if( ( $pos = strpos( $dataset['parentid'], '/' ) ) === false || ( $str = substr( $dataset['parentid'], $pos+1 ) ) === false ) {
+			if( ( $pos = strpos( $dataset['parentid'], '/' ) ) === false || ( $str = substr( $dataset['parentid'], $pos + 1 ) ) === false ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'Some keys for parentid are set wrong "%1$s"', $dataset['parentid'] ) );
 			}
 
@@ -194,11 +194,11 @@ 
 		}
 
 		$search = $supplierManager->createSearch();
-		$search->setConditions( $search->compare( '==', 'supplier.code', $itemCode) );
+		$search->setConditions( $search->compare( '==', 'supplier.code', $itemCode ) );
 
 		$parentIds = [];
-		foreach( $supplierManager->searchItems( $search ) as $item )	{
-			$parentIds[ 'supplier/'.$item->getCode() ] = $item->getId();
+		foreach( $supplierManager->searchItems( $search ) as $item ) {
+			$parentIds['supplier/' . $item->getCode()] = $item->getId();
 		}
 
 		$listItemTypeIds = [];
@@ -215,28 +215,28 @@ 
 			$listItemType->setStatus( $dataset['status'] );
 
 			$supplierListTypeManager->saveItem( $listItemType );
-			$listItemTypeIds[ $key ] = $listItemType->getId();
+			$listItemTypeIds[$key] = $listItemType->getId();
 		}
 
 		$listItem = $supplierListManager->createItem();
 		foreach( $testdata['supplier/lists'] as $dataset )
 		{
-			if( !isset( $parentIds[ $dataset['parentid'] ] ) ) {
+			if( !isset( $parentIds[$dataset['parentid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No supplier ID found for "%1$s"', $dataset['parentid'] ) );
 			}
 
-			if( !isset( $refIds[ $dataset['domain'] ][ $dataset['refid'] ] ) ) {
+			if( !isset( $refIds[$dataset['domain']][$dataset['refid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No "%2$s" ref ID found for "%1$s"', $dataset['refid'], $dataset['domain'] ) );
 			}
 
-			if( !isset( $listItemTypeIds[ $dataset['typeid'] ] ) ) {
+			if( !isset( $listItemTypeIds[$dataset['typeid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No supplier list type ID found for "%1$s"', $dataset['typeid'] ) );
 			}
 
 			$listItem->setId( null );
-			$listItem->setParentId( $parentIds[ $dataset['parentid'] ] );
-			$listItem->setTypeId( $listItemTypeIds[ $dataset['typeid'] ] );
-			$listItem->setRefId( $refIds[ $dataset['domain'] ] [ $dataset['refid'] ] );
+			$listItem->setParentId( $parentIds[$dataset['parentid']] );
+			$listItem->setTypeId( $listItemTypeIds[$dataset['typeid']] );
+			$listItem->setRefId( $refIds[$dataset['domain']] [$dataset['refid']] );
 			$listItem->setDomain( $dataset['domain'] );
 			$listItem->setDateStart( $dataset['start'] );
 			$listItem->setDateEnd( $dataset['end'] );

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

Spacing +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
 			$stmt->bind( 6, $item->getFirstname() );
 			$stmt->bind( 7, $item->getLastname() );
 			$stmt->bind( 8, $item->getAddress1() );
-			$stmt->bind( 9, $item->getAddress2());
+			$stmt->bind( 9, $item->getAddress2() );
 			$stmt->bind( 10, $item->getAddress3() );
 			$stmt->bind( 11, $item->getPostal() );
 			$stmt->bind( 12, $item->getCity() );

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

Spacing +5 added lines, -5 removed lines

@@ -229,8 +229,8 @@ 
 			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
 			while( ( $row = $results->fetch() ) !== false )
 			{
-				$map[ $row[$this->prefix . 'id'] ] = $row;
-				$typeIds[ $row[$this->prefix . 'typeid'] ] = null;
+				$map[$row[$this->prefix . 'id']] = $row;
+				$typeIds[$row[$this->prefix . 'typeid']] = null;
 			}
 
 			$dbm->release( $conn, $dbname );
@@ -251,10 +251,10 @@ 
 
 			foreach( $map as $id => $row )
 			{
-				if( isset( $typeItems[ $row[$this->prefix . 'typeid'] ] ) )
+				if( isset( $typeItems[$row[$this->prefix . 'typeid']] ) )
 				{
-					$row[$this->prefix . 'type'] = $typeItems[ $row[$this->prefix . 'typeid'] ]->getCode();
-					$row[$this->prefix . 'typename'] = $typeItems[ $row[$this->prefix . 'typeid'] ]->getName();
+					$row[$this->prefix . 'type'] = $typeItems[$row[$this->prefix . 'typeid']]->getCode();
+					$row[$this->prefix . 'typename'] = $typeItems[$row[$this->prefix . 'typeid']]->getName();
 				}
 
 				$items[$id] = $this->createItemBase( $row );

lib/mshoplib/src/MShop/Common/Item/Helper/Password/Hash.php

Spacing +3 added lines, -3 removed lines

@@ -55,11 +55,11 @@
 		$iterations = ( isset( $this->options['iterations'] ) ? (int) $this->options['iterations'] : 1 );
 
 		$salted = sprintf( $format, $password, $salt );
-		$digest = hash( $this->options['algorithm'], $salted, true);
+		$digest = hash( $this->options['algorithm'], $salted, true );
 
 		// "stretch" hash
-		for ($i = 1; $i < $iterations; $i++ ) {
-			$digest = hash( $this->options['algorithm'], $digest . $salted, true);
+		for( $i = 1; $i < $iterations; $i++ ) {
+			$digest = hash( $this->options['algorithm'], $digest . $salted, true );
 		}
 
 		return ( $encode ? base64_encode( $digest ) : bin2hex( $digest ) );

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

Spacing +1 added lines, -1 removed lines

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

Braces +8 added lines, -3 removed lines

@@ -73,7 +73,8 @@ 
 			$siteIds = array_merge( $siteIds, $locale->getSiteSubTree() );
 		}
 
-		$this->searchConfig['index.attribute:all']['function'] = function( $source, array $params ) {
+		$this->searchConfig['index.attribute:all']['function'] = function( $source, array $params )
+		{
 			return [$params[0], count( explode( ',', $params[0] ) )];
 		};
 
@@ -673,9 +674,13 @@ 
 				$stmt->bind( 6, $date ); // mtime
 				$stmt->bind( 7, $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
 			}
 		}
 	}