aimeos / ai-laravel

lib/custom/setup/CustomerChangeAddressRefidParentidLaravel.php

Spacing +11 added lines, -11 removed lines

@@ -40,7 +40,7 @@ 
 	 */
 	public function getPostDependencies()
 	{
-		return array( 'TablesCreateLaravel' );
+		return array('TablesCreateLaravel');
 	}
 
 
@@ -49,7 +49,7 @@ 
 	 */
 	protected function mysql()
 	{
-		$this->process( $this->mysql );
+		$this->process($this->mysql);
 	}
 
 
@@ -58,22 +58,22 @@ 
 	 *
 	 * array string $stmts List of SQL statements for changing the columns
 	 */
-	protected function process( array $stmts )
+	protected function process(array $stmts)
 	{
 		$table = 'users_address';
-		$this->msg( sprintf( 'Rename "refid" to "parentid" in table "%1$s"', $table ), 0 ); $this->status( '' );
+		$this->msg(sprintf('Rename "refid" to "parentid" in table "%1$s"', $table), 0); $this->status('');
 
-		foreach( $stmts as $column => $stmts )
+		foreach ($stmts as $column => $stmts)
 		{
-			$this->msg( sprintf( 'Checking column "%1$s"', $column ), 1 );
+			$this->msg(sprintf('Checking column "%1$s"', $column), 1);
 
-			if( $this->schema->tableExists( $table )
-				&& $this->schema->columnExists( $table, $column ) === true
+			if ($this->schema->tableExists($table)
+				&& $this->schema->columnExists($table, $column) === true
 			) {
-				$this->executeList( $stmts );
-				$this->status( 'done' );
+				$this->executeList($stmts);
+				$this->status('done');
 			} else {
-				$this->status( 'OK' );
+				$this->status('OK');
 			}
 		}
 	}

lib/custom/src/MShop/Customer/Manager/Laravel.php

Braces +3 added lines, -6 removed lines

@@ -330,8 +330,7 @@ 
 				 * @see mshop/customer/manager/laravel/count
 				 */
 				$path = 'mshop/customer/manager/laravel/insert';
-			}
-			else
+			} else
 			{
 				/** mshop/customer/manager/laravel/update
 				 * Updates an existing customer record in the database
@@ -437,8 +436,7 @@ 
 			}
 
 			$dbm->release( $conn, $dbname );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$dbm->release( $conn, $dbname );
 			throw $e;
@@ -474,8 +472,7 @@ 
 			}
 
 			$dbm->release( $conn, $dbname );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$dbm->release( $conn, $dbname  );
 			throw $e;

Spacing +68 added lines, -68 removed lines

@@ -240,11 +240,11 @@ 
 	 *
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, array( 'address', 'lists' ) ) as $domain ) {
-			$this->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, array('address', 'lists')) as $domain) {
+			$this->getSubManager($domain)->cleanup($siteids);
 		}
 	}
 
@@ -265,10 +265,10 @@ 
 	 *
 	 * @param array $ids List of IDs
 	 */
-	public function deleteItems( array $ids )
+	public function deleteItems(array $ids)
 	{
 		$path = 'mshop/customer/manager/laravel/delete';
-		$this->deleteItemsBase( $ids, $path, false );
+		$this->deleteItemsBase($ids, $path, false);
 	}
 
 
@@ -278,11 +278,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array List of attribute items implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, array( 'address', 'lists' ), $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, array('address', 'lists'), $withsub);
 	}
 
 
@@ -292,27 +292,27 @@ 
 	 * @param \Aimeos\MShop\Customer\Item\Iface $item Customer item object
 	 * @param boolean $fetch True if the new ID should be returned in the item
 	 */
-	public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true )
+	public function saveItem(\Aimeos\MShop\Common\Item\Iface $item, $fetch = true)
 	{
 		$iface = '\\Aimeos\\MShop\\Customer\\Item\\Iface';
-		if( !( $item instanceof $iface ) ) {
-			throw new \Aimeos\MShop\Customer\Exception( sprintf( 'Object is not of required type "%1$s"', $iface ) );
+		if (!($item instanceof $iface)) {
+			throw new \Aimeos\MShop\Customer\Exception(sprintf('Object is not of required type "%1$s"', $iface));
 		}
 
-		if( !$item->isModified() ) { return; }
+		if (!$item->isModified()) { return; }
 
 		$context = $this->getContext();
 		$dbm = $context->getDatabaseManager();
 		$dbname = $this->getResourceName();
-		$conn = $dbm->acquire( $dbname );
+		$conn = $dbm->acquire($dbname);
 
 		try
 		{
 			$id = $item->getId();
-			$date = date( 'Y-m-d H:i:s' );
+			$date = date('Y-m-d H:i:s');
 			$billingAddress = $item->getPaymentAddress();
 
-			if( $id === null )
+			if ($id === null)
 			{
 				/** mshop/customer/manager/laravel/insert
 				 * Inserts a new customer record into the database table
@@ -376,47 +376,47 @@ 
 				$path = 'mshop/customer/manager/laravel/update';
 			}
 
-			$stmt = $this->getCachedStatement( $conn, $path );
-
-			$stmt->bind( 1, $item->getCode() );
-			$stmt->bind( 2, $billingAddress->getCompany() );
-			$stmt->bind( 3, $billingAddress->getVatID() );
-			$stmt->bind( 4, $billingAddress->getSalutation() );
-			$stmt->bind( 5, $billingAddress->getTitle() );
-			$stmt->bind( 6, $billingAddress->getFirstname() );
-			$stmt->bind( 7, $billingAddress->getLastname() );
-			$stmt->bind( 8, $billingAddress->getAddress1() );
-			$stmt->bind( 9, $billingAddress->getAddress2() );
-			$stmt->bind( 10, $billingAddress->getAddress3() );
-			$stmt->bind( 11, $billingAddress->getPostal() );
-			$stmt->bind( 12, $billingAddress->getCity() );
-			$stmt->bind( 13, $billingAddress->getState() );
-			$stmt->bind( 14, $billingAddress->getCountryId() );
-			$stmt->bind( 15, $billingAddress->getLanguageId() );
-			$stmt->bind( 16, $billingAddress->getTelephone() );
-			$stmt->bind( 17, $billingAddress->getTelefax() );
-			$stmt->bind( 18, $billingAddress->getWebsite() );
-			$stmt->bind( 19, $billingAddress->getEmail() );
-			$stmt->bind( 20, $billingAddress->getLongitude() );
-			$stmt->bind( 21, $billingAddress->getLatitude() );
-			$stmt->bind( 22, $item->getLabel() );
-			$stmt->bind( 23, $item->getBirthday() );
-			$stmt->bind( 24, $item->getStatus(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-			$stmt->bind( 25, $item->getDateVerified() );
-			$stmt->bind( 26, $item->getPassword() );
-			$stmt->bind( 27, $date ); // Modification time
-			$stmt->bind( 28, $context->getEditor() );
-
-			if( $id !== null ) {
-				$stmt->bind( 29, $id, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-				$item->setId( $id );
+			$stmt = $this->getCachedStatement($conn, $path);
+
+			$stmt->bind(1, $item->getCode());
+			$stmt->bind(2, $billingAddress->getCompany());
+			$stmt->bind(3, $billingAddress->getVatID());
+			$stmt->bind(4, $billingAddress->getSalutation());
+			$stmt->bind(5, $billingAddress->getTitle());
+			$stmt->bind(6, $billingAddress->getFirstname());
+			$stmt->bind(7, $billingAddress->getLastname());
+			$stmt->bind(8, $billingAddress->getAddress1());
+			$stmt->bind(9, $billingAddress->getAddress2());
+			$stmt->bind(10, $billingAddress->getAddress3());
+			$stmt->bind(11, $billingAddress->getPostal());
+			$stmt->bind(12, $billingAddress->getCity());
+			$stmt->bind(13, $billingAddress->getState());
+			$stmt->bind(14, $billingAddress->getCountryId());
+			$stmt->bind(15, $billingAddress->getLanguageId());
+			$stmt->bind(16, $billingAddress->getTelephone());
+			$stmt->bind(17, $billingAddress->getTelefax());
+			$stmt->bind(18, $billingAddress->getWebsite());
+			$stmt->bind(19, $billingAddress->getEmail());
+			$stmt->bind(20, $billingAddress->getLongitude());
+			$stmt->bind(21, $billingAddress->getLatitude());
+			$stmt->bind(22, $item->getLabel());
+			$stmt->bind(23, $item->getBirthday());
+			$stmt->bind(24, $item->getStatus(), \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+			$stmt->bind(25, $item->getDateVerified());
+			$stmt->bind(26, $item->getPassword());
+			$stmt->bind(27, $date); // Modification time
+			$stmt->bind(28, $context->getEditor());
+
+			if ($id !== null) {
+				$stmt->bind(29, $id, \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+				$item->setId($id);
 			} else {
-				$stmt->bind( 29, $date ); // Creation time
+				$stmt->bind(29, $date); // Creation time
 			}
 
 			$stmt->execute()->finish();
 
-			if( $id === null && $fetch === true )
+			if ($id === null && $fetch === true)
 			{
 				/** mshop/customer/manager/laravel/newid
 				 * Retrieves the ID generated by the database when inserting a new record
@@ -449,14 +449,14 @@ 
 				 * @see mshop/customer/manager/laravel/count
 				 */
 				$path = 'mshop/customer/manager/laravel/newid';
-				$item->setId( $this->newId( $conn, $path ) );
+				$item->setId($this->newId($conn, $path));
 			}
 
-			$dbm->release( $conn, $dbname );
+			$dbm->release($conn, $dbname);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$dbm->release( $conn, $dbname );
+			$dbm->release($conn, $dbname);
 			throw $e;
 		}
 	}
@@ -470,11 +470,11 @@ 
 	 * @return array List of items implementing \Aimeos\MShop\Customer\Item\Iface
 	 * @throws \Aimeos\MShop\Customer\Exception If creating items failed
 	 */
-	public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null )
+	public function searchItems(\Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null)
 	{
 		$dbm = $this->getContext()->getDatabaseManager();
 		$dbname = $this->getResourceName();
-		$conn = $dbm->acquire( $dbname );
+		$conn = $dbm->acquire($dbname);
 		$map = [];
 
 		try
@@ -482,27 +482,27 @@ 
 			$level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL;
 			$cfgPathSearch = 'mshop/customer/manager/laravel/search';
 			$cfgPathCount = 'mshop/customer/manager/laravel/count';
-			$required = array( 'customer' );
+			$required = array('customer');
 
-			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
-			while( ( $row = $results->fetch() ) !== false ) {
-				$map[ $row['customer.id'] ] = $row;
+			$results = $this->searchItemsBase($conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level);
+			while (($row = $results->fetch()) !== false) {
+				$map[$row['customer.id']] = $row;
 			}
 
-			$dbm->release( $conn, $dbname );
+			$dbm->release($conn, $dbname);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$dbm->release( $conn, $dbname  );
+			$dbm->release($conn, $dbname);
 			throw $e;
 		}
 
 		$addrItems = [];
-		if( in_array( 'address', $ref, true ) ) {
-			$addrItems = $this->getAddressItems( array_keys( $map ) );
+		if (in_array('address', $ref, true)) {
+			$addrItems = $this->getAddressItems(array_keys($map));
 		}
 
-		return $this->buildItems( $map, $ref, 'customer', $addrItems );
+		return $this->buildItems($map, $ref, 'customer', $addrItems);
 	}
 
 
@@ -513,8 +513,8 @@ 
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
 	 * @return mixed Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', $manager, ($name === null ? 'Laravel' : $name));
 	}
 }

lib/custom/src/MShop/Customer/Manager/Lists/Laravel.php

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
 		'customer.lists.id'=> array(
 			'code'=>'customer.lists.id',
 			'internalcode'=>'lvuli."id"',
-			'internaldeps' => array( 'LEFT JOIN "users_list" AS lvuli ON ( lvu."id" = lvuli."parentid" )' ),
+			'internaldeps' => array('LEFT JOIN "users_list" AS lvuli ON ( lvu."id" = lvuli."parentid" )'),
 			'label'=>'Customer list ID',
 			'type'=> 'integer',
 			'internaltype'=> \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -133,14 +133,14 @@ 
 	 *
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/lists/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, array( 'type' ) ) as $domain ) {
-			$this->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, array('type')) as $domain) {
+			$this->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/lists/laravel/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/lists/laravel/delete');
 	}
 
 
@@ -150,11 +150,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array List of attribute items implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/lists/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, array( 'type' ), $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, array('type'), $withsub);
 	}
 
 
@@ -165,9 +165,9 @@ 
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
 	 * @return mixed Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', 'lists/' . $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', 'lists/'.$manager, ($name === null ? 'Laravel' : $name));
 	}
 
 

lib/custom/src/MW/Logger/Laravel5.php

Spacing +10 added lines, -10 removed lines

@@ -29,7 +29,7 @@ 
 	 *
 	 * @param \Illuminate\Contracts\Logging\Log $logger Laravel logger object
 	 */
-	public function __construct( \Illuminate\Contracts\Logging\Log $logger )
+	public function __construct(\Illuminate\Contracts\Logging\Log $logger)
 	{
 		$this->logger = $logger;
 	}
@@ -44,18 +44,18 @@ 
 	 * @throws \Aimeos\MW\Logger\Exception If an error occurs in Zend_Log
 	 * @see \Aimeos\MW\Logger\Base for available log level constants
 	 */
-	public function log( $message, $priority = \Aimeos\MW\Logger\Base::ERR, $facility = 'message' )
+	public function log($message, $priority = \Aimeos\MW\Logger\Base::ERR, $facility = 'message')
 	{
 		try
 		{
-			if( !is_scalar( $message ) ) {
-				$message = json_encode( $message );
+			if (!is_scalar($message)) {
+				$message = json_encode($message);
 			}
 
-			$this->logger->log( $message, $this->translatePriority( $priority ) );
+			$this->logger->log($message, $this->translatePriority($priority));
 		}
-		catch( \Exception $e )	{
-			throw new \Aimeos\MW\Logger\Exception( $e->getMessage(), $e->getCode(), $e );
+		catch (\Exception $e) {
+			throw new \Aimeos\MW\Logger\Exception($e->getMessage(), $e->getCode(), $e);
 		}
 	}
 
@@ -67,9 +67,9 @@ 
 	 * @return integer Log level from Monolog\Logger
 	 * @throws \Aimeos\MW\Logger\Exception If log level is unknown
 	 */
-	protected function translatePriority( $priority )
+	protected function translatePriority($priority)
 	{
-		switch( $priority )
+		switch ($priority)
 		{
 			case \Aimeos\MW\Logger\Base::EMERG:
 				return 'emergency';
@@ -88,7 +88,7 @@ 
 			case \Aimeos\MW\Logger\Base::DEBUG:
 				return 'debug';
 			default:
-				throw new \Aimeos\MW\Logger\Exception( 'Invalid log level' );
+				throw new \Aimeos\MW\Logger\Exception('Invalid log level');
 		}
 	}
 }
\ No newline at end of file

Braces +1 added lines, -2 removed lines

@@ -53,8 +53,7 @@
 			}
 
 			$this->logger->log( $message, $this->translatePriority( $priority ) );
-		}
-		catch( \Exception $e )	{
+		} catch( \Exception $e )	{
 			throw new \Aimeos\MW\Logger\Exception( $e->getMessage(), $e->getCode(), $e );
 		}
 	}

lib/custom/src/MW/Filesystem/Manager/Laravel.php

Spacing +8 added lines, -8 removed lines

@@ -31,9 +31,9 @@ 
 	 * @param \Aimeos\MW\Config\Iface $config Configuration object
 	 * @param string $tempdir Directory for storing temporary files
 	 */
-	public function __construct( \Illuminate\Filesystem\FilesystemManager $fsm, \Aimeos\MW\Config\Iface $config, $tempdir )
+	public function __construct(\Illuminate\Filesystem\FilesystemManager $fsm, \Aimeos\MW\Config\Iface $config, $tempdir)
 	{
-		parent::__construct( $config );
+		parent::__construct($config);
 
 		$this->fsm = $fsm;
 		$this->tempdir = $tempdir;
@@ -47,19 +47,19 @@ 
 	 * @return \Aimeos\MW\Filesystem\Iface File system object
 	 * @throws \Aimeos\MW\Filesystem\Exception If an no configuration for that name is found
 	 */
-	public function get( $name )
+	public function get($name)
 	{
-		$key = $this->getConfig( $name );
+		$key = $this->getConfig($name);
 
-		if( is_string( $key ) )
+		if (is_string($key))
 		{
-			if( !isset( $this->objects[$key] ) ) {
-				$this->objects[$key] = new \Aimeos\MW\Filesystem\Laravel( $this->fsm->disk( $key ), $this->tempdir );
+			if (!isset($this->objects[$key])) {
+				$this->objects[$key] = new \Aimeos\MW\Filesystem\Laravel($this->fsm->disk($key), $this->tempdir);
 			}
 
 			return $this->objects[$key];
 		}
 
-		return parent::get( $name );
+		return parent::get($name);
 	}
 }

lib/custom/src/MW/View/Helper/Response/Laravel5.php

Spacing +2 added lines, -2 removed lines

@@ -26,8 +26,8 @@
 	 *
 	 * @param \Aimeos\MW\View\Iface $view View instance with registered view helpers
 	 */
-	public function __construct( \Aimeos\MW\View\Iface $view )
+	public function __construct(\Aimeos\MW\View\Iface $view)
 	{
-		parent::__construct( $view, new \Zend\Diactoros\Response() );
+		parent::__construct($view, new \Zend\Diactoros\Response());
 	}
 }

lib/custom/tests/MW/View/Helper/Response/Laravel5Test.php

Spacing +7 added lines, -7 removed lines

@@ -17,27 +17,27 @@
 
 	protected function setUp()
 	{
-		if( !class_exists( '\Illuminate\Http\Response' ) ) {
-			$this->markTestSkipped( '\Illuminate\Http\Response is not available' );
+		if (!class_exists('\Illuminate\Http\Response')) {
+			$this->markTestSkipped('\Illuminate\Http\Response is not available');
 		}
 
-		if( !class_exists( '\Zend\Diactoros\Response' ) ) {
-			$this->markTestSkipped( '\Zend\Diactoros\Response is not available' );
+		if (!class_exists('\Zend\Diactoros\Response')) {
+			$this->markTestSkipped('\Zend\Diactoros\Response is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
-		$this->object = new \Aimeos\MW\View\Helper\Response\Laravel5( $view );
+		$this->object = new \Aimeos\MW\View\Helper\Response\Laravel5($view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testTransform()
 	{
-		$this->assertInstanceOf( '\Aimeos\MW\View\Helper\Response\Laravel5', $this->object->transform() );
+		$this->assertInstanceOf('\Aimeos\MW\View\Helper\Response\Laravel5', $this->object->transform());
 	}
 }

lib/custom/tests/MW/Logger/Laravel5Test.php

Spacing +21 added lines, -21 removed lines

@@ -20,12 +20,12 @@ 
 	 */
 	protected function setUp()
 	{
-		if( interface_exists( '\\Illuminate\\Contracts\\Logging\\Log' ) === false ) {
-			$this->markTestSkipped( 'Class \\Illuminate\\Contracts\\Logging\\Log not found' );
+		if (interface_exists('\\Illuminate\\Contracts\\Logging\\Log') === false) {
+			$this->markTestSkipped('Class \\Illuminate\\Contracts\\Logging\\Log not found');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Logging\\Log' )->getMock();
-		$this->object = new \Aimeos\MW\Logger\Laravel5( $this->mock );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Logging\\Log')->getMock();
+		$this->object = new \Aimeos\MW\Logger\Laravel5($this->mock);
 	}
 
 
@@ -37,42 +37,42 @@ 
 	 */
 	protected function tearDown()
 	{
-		unset( $this->mock, $this->object );
+		unset($this->mock, $this->object);
 	}
 
 
 	public function testLog()
 	{
-		$this->mock->expects( $this->once() )->method( 'log' )
-			->with( $this->equalTo( 'msg' ), $this->equalTo( 'error' ) );
-		$this->object->log( 'msg' );
+		$this->mock->expects($this->once())->method('log')
+			->with($this->equalTo('msg'), $this->equalTo('error'));
+		$this->object->log('msg');
 	}
 
 
 	public function testNonScalarLog()
 	{
-		$this->mock->expects( $this->once() )->method( 'log' )
-			->with( $this->equalTo( '["error","error2",2]' ), $this->equalTo( 'error' ) );
-		$this->object->log( array( 'error', 'error2', 2 ) );
+		$this->mock->expects($this->once())->method('log')
+			->with($this->equalTo('["error","error2",2]'), $this->equalTo('error'));
+		$this->object->log(array('error', 'error2', 2));
 	}
 
 
 	public function testLogDebug()
 	{
-		$this->object->log( 'emergency', \Aimeos\MW\Logger\Base::EMERG );
-		$this->object->log( 'alert', \Aimeos\MW\Logger\Base::ALERT );
-		$this->object->log( 'critical', \Aimeos\MW\Logger\Base::CRIT );
-		$this->object->log( 'error', \Aimeos\MW\Logger\Base::ERR );
-		$this->object->log( 'warning', \Aimeos\MW\Logger\Base::WARN );
-		$this->object->log( 'notice', \Aimeos\MW\Logger\Base::NOTICE );
-		$this->object->log( 'info', \Aimeos\MW\Logger\Base::INFO );
-		$this->object->log( 'debug', \Aimeos\MW\Logger\Base::DEBUG );
+		$this->object->log('emergency', \Aimeos\MW\Logger\Base::EMERG);
+		$this->object->log('alert', \Aimeos\MW\Logger\Base::ALERT);
+		$this->object->log('critical', \Aimeos\MW\Logger\Base::CRIT);
+		$this->object->log('error', \Aimeos\MW\Logger\Base::ERR);
+		$this->object->log('warning', \Aimeos\MW\Logger\Base::WARN);
+		$this->object->log('notice', \Aimeos\MW\Logger\Base::NOTICE);
+		$this->object->log('info', \Aimeos\MW\Logger\Base::INFO);
+		$this->object->log('debug', \Aimeos\MW\Logger\Base::DEBUG);
 	}
 
 
 	public function testBadPriority()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Logger\\Exception' );
-		$this->object->log( 'error', -1 );
+		$this->setExpectedException('\\Aimeos\\MW\\Logger\\Exception');
+		$this->object->log('error', -1);
 	}
 }

lib/custom/tests/MW/Filesystem/LaravelTest.php

Spacing +124 added lines, -124 removed lines

@@ -11,348 +11,348 @@
 
 	protected function setUp()
 	{
-		if( !interface_exists( '\\Illuminate\\Contracts\\Filesystem\\Filesystem' ) ) {
-			$this->markTestSkipped( 'Install Laravel framework first' );
+		if (!interface_exists('\\Illuminate\\Contracts\\Filesystem\\Filesystem')) {
+			$this->markTestSkipped('Install Laravel framework first');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Filesystem\\Filesystem' )
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Filesystem\\Filesystem')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = new \Aimeos\MW\Filesystem\Laravel( $this->mock, sys_get_temp_dir() );
+		$this->object = new \Aimeos\MW\Filesystem\Laravel($this->mock, sys_get_temp_dir());
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testIsdir()
 	{
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->returnValue( array( 't', 'test', 'es' ) ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->returnValue(array('t', 'test', 'es')));
 
-		$this->assertTrue( $this->object->isdir( 'test' ) );
+		$this->assertTrue($this->object->isdir('test'));
 	}
 
 
 	public function testIsdirFalse()
 	{
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->returnValue( array( 't', 'es' ) ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->returnValue(array('t', 'es')));
 
-		$this->assertFalse( $this->object->isdir( 'test' ) );
+		$this->assertFalse($this->object->isdir('test'));
 	}
 
 
 	public function testMkdir()
 	{
-		$this->mock->expects( $this->once() )->method( 'makeDirectory' );
+		$this->mock->expects($this->once())->method('makeDirectory');
 
-		$this->object->mkdir( 'test' );
+		$this->object->mkdir('test');
 	}
 
 
 	public function testMkdirException()
 	{
-		$this->mock->expects( $this->once() )->method( 'makeDirectory' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('makeDirectory')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->mkdir( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->mkdir('test');
 	}
 
 
 	public function testRmdir()
 	{
-		$this->mock->expects( $this->once() )->method( 'deleteDirectory' );
+		$this->mock->expects($this->once())->method('deleteDirectory');
 
-		$this->object->rmdir( 'test' );
+		$this->object->rmdir('test');
 	}
 
 
 	public function testRmdirException()
 	{
-		$this->mock->expects( $this->once() )->method( 'deleteDirectory' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('deleteDirectory')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->rmdir( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->rmdir('test');
 	}
 
 
 	public function testScan()
 	{
-		$list = array( 't', 'es', 'st' );
+		$list = array('t', 'es', 'st');
 
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->returnValue( array( 't', 'es' ) ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->returnValue(array('t', 'es')));
 
-		$this->mock->expects( $this->once() )->method( 'files' )
-			->will( $this->returnValue( array( 'st' ) ) );
+		$this->mock->expects($this->once())->method('files')
+			->will($this->returnValue(array('st')));
 
 		$result = $this->object->scan();
 
-		$this->assertInternalType( 'array', $result );
+		$this->assertInternalType('array', $result);
 
-		foreach( $result as $entry ) {
-			$this->assertTrue( in_array( $entry, $list ) );
+		foreach ($result as $entry) {
+			$this->assertTrue(in_array($entry, $list));
 		}
 	}
 
 
 	public function testScanException()
 	{
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->scan( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->scan('test');
 	}
 
 
 	public function testSize()
 	{
-		$this->mock->expects( $this->once() )->method( 'size' )
-			->will( $this->returnValue( 4 ) );
+		$this->mock->expects($this->once())->method('size')
+			->will($this->returnValue(4));
 
-		$result = $this->object->size( 'test' );
+		$result = $this->object->size('test');
 
-		$this->assertEquals( 4, $result );
+		$this->assertEquals(4, $result);
 	}
 
 
 	public function testSizeException()
 	{
-		$this->mock->expects( $this->once() )->method( 'size' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('size')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->size( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->size('test');
 	}
 
 
 	public function testTime()
 	{
-		$this->mock->expects( $this->once() )->method( 'lastModified' )
-			->will( $this->returnValue( 1 ) );
+		$this->mock->expects($this->once())->method('lastModified')
+			->will($this->returnValue(1));
 
-		$result = $this->object->time( 'test' );
+		$result = $this->object->time('test');
 
-		$this->assertGreaterThan( 0, $result );
+		$this->assertGreaterThan(0, $result);
 	}
 
 
 	public function testTimeException()
 	{
-		$this->mock->expects( $this->once() )->method( 'lastModified' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('lastModified')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->time( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->time('test');
 	}
 
 
 	public function testRm()
 	{
-		$this->mock->expects( $this->once() )->method( 'delete' );
+		$this->mock->expects($this->once())->method('delete');
 
-		$this->object->rm( 'test' );
+		$this->object->rm('test');
 	}
 
 
 	public function testRmException()
 	{
-		$this->mock->expects( $this->once() )->method( 'delete' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('delete')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->rm( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->rm('test');
 	}
 
 
 	public function testHas()
 	{
-		$this->mock->expects( $this->once() )->method( 'exists' )
-			->will( $this->returnValue( true ) );
+		$this->mock->expects($this->once())->method('exists')
+			->will($this->returnValue(true));
 
-		$result = $this->object->has( 'test' );
+		$result = $this->object->has('test');
 
-		$this->assertTrue( $result );
+		$this->assertTrue($result);
 	}
 
 
 	public function testHasFalse()
 	{
-		$this->mock->expects( $this->once() )->method( 'exists' )
-			->will( $this->returnValue( false ) );
+		$this->mock->expects($this->once())->method('exists')
+			->will($this->returnValue(false));
 
-		$result = $this->object->has( 'test' );
+		$result = $this->object->has('test');
 
-		$this->assertFalse( $result );
+		$this->assertFalse($result);
 	}
 
 
 	public function testRead()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->returnValue('test'));
 
-		$result = $this->object->read( 'file' );
+		$result = $this->object->read('file');
 
-		$this->assertEquals( 'test', $result );
+		$this->assertEquals('test', $result);
 	}
 
 
 	public function testReadException()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->read( 'readinvalid' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->read('readinvalid');
 	}
 
 
 	public function testReadf()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->returnValue('test'));
 
-		$result = $this->object->readf( 'file' );
+		$result = $this->object->readf('file');
 
-		$this->assertEquals( 'test', file_get_contents( $result ) );
-		unlink( $result );
+		$this->assertEquals('test', file_get_contents($result));
+		unlink($result);
 	}
 
 
 	public function testReads()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->returnValue('test'));
 
-		$handle = $this->object->reads( 'file' );
+		$handle = $this->object->reads('file');
 
-		$this->assertInternalType( 'resource', $handle );
-		$this->assertEquals( 'test', fgets( $handle ) );
+		$this->assertInternalType('resource', $handle);
+		$this->assertEquals('test', fgets($handle));
 
-		fclose( $handle );
+		fclose($handle);
 	}
 
 
 	public function testReadsException()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->reads( 'readinvalid' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->reads('readinvalid');
 	}
 
 
 	public function testWrite()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' );
+		$this->mock->expects($this->once())->method('put');
 
-		$this->object->write( 'file', 'test' );
+		$this->object->write('file', 'test');
 	}
 
 
 	public function testWriteException()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('put')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->write( '', 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->write('', 'test');
 	}
 
 
 	public function testWritef()
 	{
-		$file = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'file99';
-		file_put_contents( $file, 'test' );
+		$file = sys_get_temp_dir().DIRECTORY_SEPARATOR.'file99';
+		file_put_contents($file, 'test');
 
-		$this->mock->expects( $this->once() )->method( 'put' );
+		$this->mock->expects($this->once())->method('put');
 
-		$this->object->writef( 'file', $file );
+		$this->object->writef('file', $file);
 
-		unlink( $file );
+		unlink($file);
 	}
 
 
 	public function testWritefException()
 	{
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->writef( '', 'invalid' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->writef('', 'invalid');
 	}
 
 
 	public function testWrites()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' );
+		$this->mock->expects($this->once())->method('put');
 
-		$handle = fopen( __FILE__, 'r' );
+		$handle = fopen(__FILE__, 'r');
 
-		$this->object->writes( 'file', $handle );
+		$this->object->writes('file', $handle);
 
-		fclose( $handle );
+		fclose($handle);
 	}
 
 
 	public function testWritesException()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('put')
+			->will($this->throwException(new \RuntimeException()));
 
-		$handle = fopen( __FILE__, 'r' );
+		$handle = fopen(__FILE__, 'r');
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->writes( 'file', $handle );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->writes('file', $handle);
 	}
 
 
 	public function testWritesException2()
 	{
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->writes( 'file', null );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->writes('file', null);
 	}
 
 
 	public function testMove()
 	{
-		$this->mock->expects( $this->once() )->method( 'move' );
+		$this->mock->expects($this->once())->method('move');
 
-		$this->object->move( 'file1', 'file2' );
+		$this->object->move('file1', 'file2');
 	}
 
 
 	public function testMoveException()
 	{
-		$this->mock->expects( $this->once() )->method( 'move' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('move')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->move( 'file1', 'file2' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->move('file1', 'file2');
 	}
 
 
 	public function testCopy()
 	{
-		$this->mock->expects( $this->once() )->method( 'copy' );
+		$this->mock->expects($this->once())->method('copy');
 
-		$this->object->copy( 'file1', 'file2' );
+		$this->object->copy('file1', 'file2');
 	}
 
 
 	public function testCopyException()
 	{
-		$this->mock->expects( $this->once() )->method( 'copy' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('copy')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->copy( 'file1', 'file2' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->copy('file1', 'file2');
 	}
 }