aimeos / aimeos-core

lib/mshoplib/src/MShop/Common/Item/Property/Standard.php

Spacing +27 added lines, -27 removed lines

@@ -47,8 +47,8 @@ 
 	 */
 	public function getLanguageId()
 	{
-		if( isset( $this->values[$this->prefix . 'languageid'] ) ) {
-			return (string) $this->values[$this->prefix . 'languageid'];
+		if( isset( $this->values[$this->prefix.'languageid'] ) ) {
+			return (string) $this->values[$this->prefix.'languageid'];
 		}
 
 		return null;
@@ -65,7 +65,7 @@ 
 	{
 		if( $id !== $this->getLanguageId() )
 		{
-			$this->values[$this->prefix . 'languageid'] = $this->checkLanguageId( $id );
+			$this->values[$this->prefix.'languageid'] = $this->checkLanguageId( $id );
 			$this->setModified();
 		}
 
@@ -80,8 +80,8 @@ 
 	 */
 	public function getParentId()
 	{
-		if( isset( $this->values[$this->prefix . 'parentid'] ) ) {
-			return (string) $this->values[$this->prefix . 'parentid'];
+		if( isset( $this->values[$this->prefix.'parentid'] ) ) {
+			return (string) $this->values[$this->prefix.'parentid'];
 		}
 	}
 
@@ -96,7 +96,7 @@ 
 	{
 		if( (string) $id !== $this->getParentId() )
 		{
-			$this->values[$this->prefix . 'parentid'] = (string) $id;
+			$this->values[$this->prefix.'parentid'] = (string) $id;
 			$this->setModified();
 		}
 
@@ -111,8 +111,8 @@ 
 	 */
 	public function getType()
 	{
-		if( isset( $this->values[$this->prefix . 'type'] ) ) {
-			return (string) $this->values[$this->prefix . 'type'];
+		if( isset( $this->values[$this->prefix.'type'] ) ) {
+			return (string) $this->values[$this->prefix.'type'];
 		}
 	}
 
@@ -124,8 +124,8 @@ 
 	 */
 	public function getTypeName()
 	{
-		if( isset( $this->values[$this->prefix . 'typename'] ) ) {
-			return (string) $this->values[$this->prefix . 'typename'];
+		if( isset( $this->values[$this->prefix.'typename'] ) ) {
+			return (string) $this->values[$this->prefix.'typename'];
 		}
 	}
 
@@ -137,8 +137,8 @@ 
 	 */
 	public function getTypeId()
 	{
-		if( isset( $this->values[$this->prefix . 'typeid'] ) ) {
-			return (string) $this->values[$this->prefix . 'typeid'];
+		if( isset( $this->values[$this->prefix.'typeid'] ) ) {
+			return (string) $this->values[$this->prefix.'typeid'];
 		}
 	}
 
@@ -153,7 +153,7 @@ 
 	{
 		if( (string) $id !== $this->getTypeId() )
 		{
-			$this->values[$this->prefix . 'typeid'] = (string) $id;
+			$this->values[$this->prefix.'typeid'] = (string) $id;
 			$this->setModified();
 		}
 
@@ -168,8 +168,8 @@ 
 	 */
 	public function getValue()
 	{
-		if( isset( $this->values[$this->prefix . 'value'] ) ) {
-			return (string) $this->values[$this->prefix . 'value'];
+		if( isset( $this->values[$this->prefix.'value'] ) ) {
+			return (string) $this->values[$this->prefix.'value'];
 		}
 
 		return '';
@@ -186,7 +186,7 @@ 
 	{
 		if( (string) $value !== $this->getValue() )
 		{
-			$this->values[$this->prefix . 'value'] = (string) $value;
+			$this->values[$this->prefix.'value'] = (string) $value;
 			$this->setModified();
 		}
 
@@ -215,16 +215,16 @@ 
 	{
 		$unknown = [];
 		$list = parent::fromArray( $list );
-		unset( $list[$this->prefix . 'type'], $list[$this->prefix . 'typename'] );
+		unset( $list[$this->prefix.'type'], $list[$this->prefix.'typename'] );
 
 		foreach( $list as $key => $value )
 		{
 			switch( $key )
 			{
-				case $this->prefix . 'parentid': $this->setParentId( $value ); break;
-				case $this->prefix . 'typeid': $this->setTypeId( $value ); break;
-				case $this->prefix . 'languageid': $this->setLanguageId( $value ); break;
-				case $this->prefix . 'value': $this->setValue( $value ); break;
+				case $this->prefix.'parentid': $this->setParentId( $value ); break;
+				case $this->prefix.'typeid': $this->setTypeId( $value ); break;
+				case $this->prefix.'languageid': $this->setLanguageId( $value ); break;
+				case $this->prefix.'value': $this->setValue( $value ); break;
 				default: $unknown[$key] = $value;
 			}
 		}
@@ -243,15 +243,15 @@ 
 	{
 		$list = parent::toArray( $private );
 
-		$list[$this->prefix . 'typename'] = $this->getTypeName();
-		$list[$this->prefix . 'languageid'] = $this->getLanguageId();
-		$list[$this->prefix . 'value'] = $this->getValue();
-		$list[$this->prefix . 'type'] = $this->getType();
+		$list[$this->prefix.'typename'] = $this->getTypeName();
+		$list[$this->prefix.'languageid'] = $this->getLanguageId();
+		$list[$this->prefix.'value'] = $this->getValue();
+		$list[$this->prefix.'type'] = $this->getType();
 
 		if( $private === true )
 		{
-			$list[$this->prefix . 'parentid'] = $this->getParentId();
-			$list[$this->prefix . 'typeid'] = $this->getTypeId();
+			$list[$this->prefix.'parentid'] = $this->getParentId();
+			$list[$this->prefix.'typeid'] = $this->getTypeId();
 		}
 
 		return $list;

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

Spacing +16 added lines, -16 removed lines

@@ -57,7 +57,7 @@ 
 	 */
 	public function createItem()
 	{
-		$values = array( $this->prefix . 'siteid' => $this->getContext()->getLocale()->getSiteId() );
+		$values = array( $this->prefix.'siteid' => $this->getContext()->getLocale()->getSiteId() );
 		return $this->createItemBase( $values );
 	}
 
@@ -77,8 +77,8 @@ 
 			$langid = $this->getContext()->getLocale()->getLanguageId();
 
 			$expr = array(
-				$object->compare( '==', $this->prefix . 'languageid', null ),
-				$object->compare( '==', $this->prefix . 'languageid', $langid ),
+				$object->compare( '==', $this->prefix.'languageid', null ),
+				$object->compare( '==', $this->prefix.'languageid', $langid ),
 			);
 
 			$object->setConditions( $object->combine( '||', $expr ) );
@@ -122,7 +122,7 @@ 
 				$type = 'update';
 			}
 
-			$stmt = $conn->create( $this->getSqlConfig( $this->getConfigPath() . $type ) );
+			$stmt = $conn->create( $this->getSqlConfig( $this->getConfigPath().$type ) );
 
 			$stmt->bind( 1, $item->getParentId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 			$stmt->bind( 2, $item->getTypeId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
@@ -144,7 +144,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
 				}
@@ -169,7 +169,7 @@ 
 	 */
 	public function deleteItems( array $ids )
 	{
-		$this->deleteItemsBase( $ids, $this->getConfigPath() . 'delete' );
+		$this->deleteItemsBase( $ids, $this->getConfigPath().'delete' );
 	}
 
 
@@ -184,7 +184,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 );
 	}
 
 
@@ -214,15 +214,15 @@ 
 			}
 
 			$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 )
 			{
-				$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 );
@@ -237,16 +237,16 @@ 
 		{
 			$typeManager = $this->getObject()->getSubManager( 'type' );
 			$typeSearch = $typeManager->createSearch();
-			$typeSearch->setConditions( $typeSearch->compare( '==', $this->prefix . 'type.id', array_keys( $typeIds ) ) );
+			$typeSearch->setConditions( $typeSearch->compare( '==', $this->prefix.'type.id', array_keys( $typeIds ) ) );
 			$typeSearch->setSlice( 0, $search->getSliceSize() );
 			$typeItems = $typeManager->searchItems( $typeSearch );
 
 			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 );
@@ -267,7 +267,7 @@ 
 	 */
 	public function getSubManager( $manager, $name = null )
 	{
-		return $this->getSubManagerBase( 'common', 'property/' . $manager, $name );
+		return $this->getSubManagerBase( 'common', 'property/'.$manager, $name );
 	}
 
 

lib/mshoplib/src/MShop/Product/Manager/Property/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -288,7 +288,7 @@
 		 * @see mshop/product/manager/property/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'product', 'property/' . $manager, $name );
+		return $this->getSubManagerBase( 'product', 'property/'.$manager, $name );
 	}
 
 

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

Spacing +1 added lines, -1 removed lines

@@ -288,7 +288,7 @@
 		 * @see mshop/attribute/manager/property/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'attribute', 'property/' . $manager, $name );
+		return $this->getSubManagerBase( 'attribute', 'property/'.$manager, $name );
 	}
 
 

lib/mshoplib/src/MShop/Attribute/Manager/Property/Type/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -284,7 +284,7 @@
 		 * @see mshop/attribute/manager/property/type/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'attribute', 'property/type/' . $manager, $name );
+		return $this->getSubManagerBase( 'attribute', 'property/type/'.$manager, $name );
 	}
 
 

lib/mshoplib/setup/default/schema/attribute.php

Spacing +7 added lines, -7 removed lines

@@ -8,7 +8,7 @@ 
 
 return array(
 	'table' => array(
-		'mshop_attribute_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_attribute_type' );
 
@@ -31,7 +31,7 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_attribute' );
 
@@ -45,7 +45,7 @@ 
 			$table->addColumn( 'status', 'smallint', [] );
 			$table->addColumn( 'mtime', 'datetime', [] );
 			$table->addColumn( 'ctime', 'datetime', [] );
-			$table->addColumn( 'editor', 'string', array('length' => 255 ) );
+			$table->addColumn( 'editor', 'string', array( 'length' => 255 ) );
 
 			$table->setPrimaryKey( array( 'id' ), 'pk_msatt_id' );
 			$table->addUniqueIndex( array( 'siteid', 'domain', 'code', 'typeid' ), 'unq_msattr_sid_dom_cod_tid' );
@@ -59,7 +59,7 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_list_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_list_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_attribute_list_type' );
 
@@ -82,7 +82,7 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_list' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_list' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_attribute_list' );
 
@@ -120,7 +120,7 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_property_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_property_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_attribute_property_type' );
 
@@ -143,7 +143,7 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_property' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_property' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_attribute_property' );
 

Braces +12 added lines, -6 removed lines

@@ -8,7 +8,8 @@ 
 
 return array(
 	'table' => array(
-		'mshop_attribute_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_type' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_attribute_type' );
 
@@ -31,7 +32,8 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_attribute' );
 
@@ -59,7 +61,8 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_list_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_list_type' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_attribute_list_type' );
 
@@ -82,7 +85,8 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_list' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_list' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_attribute_list' );
 
@@ -120,7 +124,8 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_property_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_property_type' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_attribute_property_type' );
 
@@ -143,7 +148,8 @@ 
 			return $schema;
 		},
 
-		'mshop_attribute_property' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_attribute_property' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_attribute_property' );
 

lib/mshoplib/setup/unittest/AttributeAddPropertyTestData.php

Spacing +6 added lines, -6 removed lines

@@ -51,7 +51,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'attribute-property.php';
+		$path = __DIR__.$ds.'data'.$ds.'attribute-property.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for attribute domain', $path ) );
@@ -89,19 +89,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'] );
 
@@ -124,7 +124,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/ProductAddPropertyTestData.php

Spacing +6 added lines, -6 removed lines

@@ -51,7 +51,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'product-property.php';
+		$path = __DIR__.$ds.'data'.$ds.'product-property.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for product domain', $path ) );
@@ -89,19 +89,19 @@ 
 			$type->setStatus( $dataset['status'] );
 
 			$productPropertyTypeManager->saveItem( $type );
-			$typeIds[ $key ] = $type->getId();
+			$typeIds[$key] = $type->getId();
 		}
 
 		$prodProperty = $productPropertyManager->createItem();
 		foreach( $testdata['product/property'] as $key => $dataset )
 		{
-			if( !isset( $typeIds[ $dataset['typeid'] ] ) ) {
+			if( !isset( $typeIds[$dataset['typeid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No product 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'] );
 
@@ -124,7 +124,7 @@ 
 		$search = $productManager->createSearch();
 
 		foreach( $productManager->searchItems( $search ) as $id => $item ) {
-			$entry[ 'product/' . $item->getCode() ] = $id;
+			$entry['product/'.$item->getCode()] = $id;
 		}
 
 		return $entry;

lib/mshoplib/setup/unittest/data/attribute-property.php

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
  * @copyright Aimeos (aimeos.org), 2017
  */
 
-return array (
+return array(
 	'attribute/property/type' => array(
 		'attribute/property/type/size' => array( 'domain' => 'attribute', 'code' => 'size', 'label' => 'Size', 'status' => 1 ),
 		'attribute/property/type/mtime' => array( 'domain' => 'attribute', 'code' => 'mtime', 'label' => 'Modification time', 'status' => 1 ),