aimeos / ai-laravel

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;

Doc Comments +3 added lines, -3 removed lines

@@ -239,7 +239,7 @@ 
 	/**
 	 * Removes old entries from the storage.
 	 *
-	 * @param array $siteids List of IDs for sites whose entries should be deleted
+	 * @param integer[] $siteids List of IDs for sites whose entries should be deleted
 	 */
 	public function cleanup( array $siteids )
 	{
@@ -292,7 +292,7 @@ 
 	 *
 	 * @param \Aimeos\MShop\Customer\Item\Iface $item Customer item object
 	 * @param boolean $fetch True if the new ID should be returned in the item
-	 * @return \Aimeos\MShop\Common\Item\Iface $item Updated item including the generated ID
+	 * @return \Aimeos\MShop\Common\Item\ListRef\Iface $item Updated item including the generated ID
 	 */
 	public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true )
 	{
@@ -520,7 +520,7 @@ 
 	 *
 	 * @param string $manager Name of the sub manager type in lower case
 	 * @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.
+	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
 	public function getSubManager( $manager, $name = null )
 	{

Spacing +77 added lines, -77 removed lines

@@ -241,13 +241,13 @@ 
 	 *
 	 * @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';
 		$default = ['address', 'group', 'lists', 'property'];
 
-		foreach( $this->getContext()->getConfig()->get( $path, $default ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, $default) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 	}
 
@@ -257,10 +257,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);
 	}
 
 
@@ -270,12 +270,12 @@ 
 	 * @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';
 		$default = ['address', 'group', 'lists', 'property'];
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, $default, $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, $default, $withsub);
 	}
 
 
@@ -286,29 +286,29 @@ 
 	 * @param boolean $fetch True if the new ID should be returned in the item
 	 * @return \Aimeos\MShop\Common\Item\Iface $item Updated item including the generated ID
 	 */
-	public function saveItem( \Aimeos\MShop\Common\Item\Iface $item, $fetch = true )
+	public function saveItem(\Aimeos\MShop\Common\Item\Iface $item, $fetch = true)
 	{
-		self::checkClass( '\\Aimeos\\MShop\\Customer\\Item\\Iface', $item );
+		self::checkClass('\\Aimeos\\MShop\\Customer\\Item\\Iface', $item);
 
-		if( !$item->isModified() )
+		if (!$item->isModified())
 		{
-			$item = $this->savePropertyItems( $item, 'customer' );
-			$item = $this->saveAddressItems( $item, 'customer' );
-			return $this->saveListItems( $item, 'customer' );
+			$item = $this->savePropertyItems($item, 'customer');
+			$item = $this->saveAddressItems($item, 'customer');
+			return $this->saveListItems($item, 'customer');
 		}
 
 		$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
@@ -372,48 +372,48 @@ 
 				$path = 'mshop/customer/manager/laravel/update';
 			}
 
-			$stmt = $this->getCachedStatement( $conn, $path );
-
-			$stmt->bind( 1, $context->getLocale()->getSiteId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-			$stmt->bind( 2, $item->getCode() );
-			$stmt->bind( 3, $billingAddress->getCompany() );
-			$stmt->bind( 4, $billingAddress->getVatID() );
-			$stmt->bind( 5, $billingAddress->getSalutation() );
-			$stmt->bind( 6, $billingAddress->getTitle() );
-			$stmt->bind( 7, $billingAddress->getFirstname() );
-			$stmt->bind( 8, $billingAddress->getLastname() );
-			$stmt->bind( 9, $billingAddress->getAddress1() );
-			$stmt->bind( 10, $billingAddress->getAddress2() );
-			$stmt->bind( 11, $billingAddress->getAddress3() );
-			$stmt->bind( 12, $billingAddress->getPostal() );
-			$stmt->bind( 13, $billingAddress->getCity() );
-			$stmt->bind( 14, $billingAddress->getState() );
-			$stmt->bind( 15, $billingAddress->getCountryId() );
-			$stmt->bind( 16, $billingAddress->getLanguageId() );
-			$stmt->bind( 17, $billingAddress->getTelephone() );
-			$stmt->bind( 18, $billingAddress->getTelefax() );
-			$stmt->bind( 19, $billingAddress->getWebsite() );
-			$stmt->bind( 20, $billingAddress->getEmail() );
-			$stmt->bind( 21, $billingAddress->getLongitude(), \Aimeos\MW\DB\Statement\Base::PARAM_FLOAT );
-			$stmt->bind( 22, $billingAddress->getLatitude(), \Aimeos\MW\DB\Statement\Base::PARAM_FLOAT );
-			$stmt->bind( 23, $item->getLabel() );
-			$stmt->bind( 24, $item->getBirthday() );
-			$stmt->bind( 25, $item->getStatus(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-			$stmt->bind( 26, $item->getDateVerified() );
-			$stmt->bind( 27, $item->getPassword() );
-			$stmt->bind( 28, $date ); // Modification time
-			$stmt->bind( 29, $context->getEditor() );
-
-			if( $id !== null ) {
-				$stmt->bind( 30, $id, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-				$item->setId( $id );
+			$stmt = $this->getCachedStatement($conn, $path);
+
+			$stmt->bind(1, $context->getLocale()->getSiteId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+			$stmt->bind(2, $item->getCode());
+			$stmt->bind(3, $billingAddress->getCompany());
+			$stmt->bind(4, $billingAddress->getVatID());
+			$stmt->bind(5, $billingAddress->getSalutation());
+			$stmt->bind(6, $billingAddress->getTitle());
+			$stmt->bind(7, $billingAddress->getFirstname());
+			$stmt->bind(8, $billingAddress->getLastname());
+			$stmt->bind(9, $billingAddress->getAddress1());
+			$stmt->bind(10, $billingAddress->getAddress2());
+			$stmt->bind(11, $billingAddress->getAddress3());
+			$stmt->bind(12, $billingAddress->getPostal());
+			$stmt->bind(13, $billingAddress->getCity());
+			$stmt->bind(14, $billingAddress->getState());
+			$stmt->bind(15, $billingAddress->getCountryId());
+			$stmt->bind(16, $billingAddress->getLanguageId());
+			$stmt->bind(17, $billingAddress->getTelephone());
+			$stmt->bind(18, $billingAddress->getTelefax());
+			$stmt->bind(19, $billingAddress->getWebsite());
+			$stmt->bind(20, $billingAddress->getEmail());
+			$stmt->bind(21, $billingAddress->getLongitude(), \Aimeos\MW\DB\Statement\Base::PARAM_FLOAT);
+			$stmt->bind(22, $billingAddress->getLatitude(), \Aimeos\MW\DB\Statement\Base::PARAM_FLOAT);
+			$stmt->bind(23, $item->getLabel());
+			$stmt->bind(24, $item->getBirthday());
+			$stmt->bind(25, $item->getStatus(), \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+			$stmt->bind(26, $item->getDateVerified());
+			$stmt->bind(27, $item->getPassword());
+			$stmt->bind(28, $date); // Modification time
+			$stmt->bind(29, $context->getEditor());
+
+			if ($id !== null) {
+				$stmt->bind(30, $id, \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+				$item->setId($id);
 			} else {
-				$stmt->bind( 30, $date ); // Creation time
+				$stmt->bind(30, $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
@@ -446,22 +446,22 @@ 
 				 * @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;
 		}
 
-		$this->addGroups( $item );
+		$this->addGroups($item);
 
-		$item = $this->savePropertyItems( $item, 'customer' );
-		$item = $this->saveAddressItems( $item, 'customer' );
-		return $this->saveListItems( $item, 'customer' );
+		$item = $this->savePropertyItems($item, 'customer');
+		$item = $this->saveAddressItems($item, 'customer');
+		return $this->saveListItems($item, 'customer');
 	}
 
 
@@ -473,11 +473,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
@@ -485,32 +485,32 @@ 
 			$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( 'customer/address', $ref, true ) ) {
-			$addrItems = $this->getAddressItems( array_keys( $map ), 'customer' );
+		if (in_array('customer/address', $ref, true)) {
+			$addrItems = $this->getAddressItems(array_keys($map), 'customer');
 		}
 
 		$propItems = [];
-		if( in_array( 'customer/property', $ref, true ) ) {
-			$propItems = $this->getPropertyItems( array_keys( $map ), 'customer' );
+		if (in_array('customer/property', $ref, true)) {
+			$propItems = $this->getPropertyItems(array_keys($map), 'customer');
 		}
 
-		return $this->buildItems( $map, $ref, 'customer', $addrItems, $propItems );
+		return $this->buildItems($map, $ref, 'customer', $addrItems, $propItems);
 	}
 
 
@@ -521,8 +521,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/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/Laravel.php

Indentation +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
 	 * @param string $path Path to the directory
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
-	*/
+	 */
 	public function mkdir( $path )
 	{
 		try {
@@ -69,7 +69,7 @@ 
 	 * @param string $path Path to the directory
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
-	*/
+	 */
 	public function rmdir( $path )
 	{
 		try {

Spacing +70 added lines, -70 removed lines

@@ -29,7 +29,7 @@ 
 	 * @param \Illuminate\Contracts\Filesystem\Filesystem $fs Laravel file system object
 	 * @param string $tempdir Directory for storing temporary files
 	 */
-	public function __construct( \Illuminate\Contracts\Filesystem\Filesystem $fs, $tempdir )
+	public function __construct(\Illuminate\Contracts\Filesystem\Filesystem $fs, $tempdir)
 	{
 		$this->fs = $fs;
 		$this->tempdir = $tempdir;
@@ -43,9 +43,9 @@ 
 	 * @return boolean True if directory, false if not
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function isdir( $path )
+	public function isdir($path)
 	{
-		return in_array( basename( $path ), $this->fs->directories( dirname( $path ) ) );
+		return in_array(basename($path), $this->fs->directories(dirname($path)));
 	}
 
 
@@ -56,12 +56,12 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	*/
-	public function mkdir( $path )
+	public function mkdir($path)
 	{
 		try {
-			$this->fs->makeDirectory( $path );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$this->fs->makeDirectory($path);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -73,12 +73,12 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	*/
-	public function rmdir( $path )
+	public function rmdir($path)
 	{
 		try {
-			$this->fs->deleteDirectory( $path );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$this->fs->deleteDirectory($path);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -92,12 +92,12 @@ 
 	 * @return \Iterator|array Iterator over the entries or array with entries
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function scan( $path = null )
+	public function scan($path = null)
 	{
 		try {
-			return array_merge( $this->fs->directories( $path ), $this->fs->files( $path ) );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			return array_merge($this->fs->directories($path), $this->fs->files($path));
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -109,12 +109,12 @@ 
 	 * @return integer Size in bytes
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function size( $path )
+	public function size($path)
 	{
 		try {
-			return $this->fs->size( $path );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			return $this->fs->size($path);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -126,12 +126,12 @@ 
 	 * @return integer Unix time stamp in seconds
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function time( $path )
+	public function time($path)
 	{
 		try {
-			return $this->fs->lastModified( $path );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			return $this->fs->lastModified($path);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -143,12 +143,12 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function rm( $path )
+	public function rm($path)
 	{
 		try {
-			$this->fs->delete( $path );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$this->fs->delete($path);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -159,9 +159,9 @@ 
 	 * @param string $path Path to the file
 	 * @return boolean True if it exists, false if not
 	 */
-	public function has( $path )
+	public function has($path)
 	{
-		return $this->fs->exists( $path );
+		return $this->fs->exists($path);
 	}
 
 
@@ -174,12 +174,12 @@ 
 	 * @return string File content
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function read( $path )
+	public function read($path)
 	{
 		try {
-			return $this->fs->get( $path );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			return $this->fs->get($path);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -191,14 +191,14 @@ 
 	 * @return string Path of the local file
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function readf( $path )
+	public function readf($path)
 	{
-		if( ( $filename = tempnam( $this->tempdir, 'ai-' ) ) === false ) {
-			throw new Exception( sprintf( 'Unable to create file in "%1$s"', $this->tempdir ) );
+		if (($filename = tempnam($this->tempdir, 'ai-')) === false) {
+			throw new Exception(sprintf('Unable to create file in "%1$s"', $this->tempdir));
 		}
 
-		if( @file_put_contents( $filename, $this->fs->get( $path ) ) === false ) {
-			throw new Exception( sprintf( 'Couldn\'t write file "%1$s"', $filename ) );
+		if (@file_put_contents($filename, $this->fs->get($path)) === false) {
+			throw new Exception(sprintf('Couldn\'t write file "%1$s"', $filename));
 		}
 
 		return $filename;
@@ -214,24 +214,24 @@ 
 	 * @return resource File stream descriptor
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function reads( $path )
+	public function reads($path)
 	{
 		try {
-			$content = $this->fs->get( $path );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$content = $this->fs->get($path);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 
-		if( ( $stream = tmpfile() ) === false ) {
-			throw new Exception( 'Couldn\'t create temporary file' );
+		if (($stream = tmpfile()) === false) {
+			throw new Exception('Couldn\'t create temporary file');
 		}
 
-		if( fwrite( $stream, $content ) === false ) {
-			throw new Exception( 'Couldn\'t write to temporary file' );
+		if (fwrite($stream, $content) === false) {
+			throw new Exception('Couldn\'t write to temporary file');
 		}
 
-		if( rewind( $stream ) === false ) {
-			throw new Exception( 'Couldn\'t rewind temporary file' );
+		if (rewind($stream) === false) {
+			throw new Exception('Couldn\'t rewind temporary file');
 		}
 
 		return $stream;
@@ -248,12 +248,12 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function write( $path, $content )
+	public function write($path, $content)
 	{
 		try {
-			$this->fs->put( $path, $content );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$this->fs->put($path, $content);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -268,13 +268,13 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function writef( $path, $local )
+	public function writef($path, $local)
 	{
-		if( ( $content = @file_get_contents( $local ) ) === false ) {
-			throw new Exception( sprintf( 'Couldn\'t read file "%1$s"', $local ) );
+		if (($content = @file_get_contents($local)) === false) {
+			throw new Exception(sprintf('Couldn\'t read file "%1$s"', $local));
 		}
 
-		$this->write( $path, $content );
+		$this->write($path, $content);
 	}
 
 
@@ -288,17 +288,17 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function writes( $path, $stream )
+	public function writes($path, $stream)
 	{
-		if( ( $content = @fread( $stream, 0x7ffffffd ) ) === false ) {
+		if (($content = @fread($stream, 0x7ffffffd)) === false) {
 			$error = error_get_last();
-			throw new Exception( $error['message'] );
+			throw new Exception($error['message']);
 		}
 
 		try {
-			$this->fs->put( $path, $content );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$this->fs->put($path, $content);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -311,12 +311,12 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function move( $from, $to )
+	public function move($from, $to)
 	{
 		try {
-			$this->fs->move( $from, $to );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$this->fs->move($from, $to);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 
@@ -329,12 +329,12 @@ 
 	 * @return void
 	 * @throws \Aimeos\MW\Filesystem\Exception If an error occurs
 	 */
-	public function copy( $from, $to )
+	public function copy($from, $to)
 	{
 		try {
-			$this->fs->copy( $from, $to );
-		} catch( \Exception $e ) {
-			throw new Exception( $e->getMessage(), 0, $e );
+			$this->fs->copy($from, $to);
+		} catch (\Exception $e) {
+			throw new Exception($e->getMessage(), 0, $e);
 		}
 	}
 }

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/src/MShop/Customer/Manager/Address/Laravel.php

Doc Comments +1 added lines, -1 removed lines

@@ -249,7 +249,7 @@
 	 *
 	 * @param string $manager Name of the sub manager type in lower case
 	 * @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.
+	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
 	public function getSubManager( $manager, $name = null )
 	{

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
 			'label' => 'Customer address ID',
 			'code' => 'customer.address.id',
 			'internalcode' => 'lvuad."id"',
-			'internaldeps' => array( 'LEFT JOIN "users_address" AS lvuad ON ( lvu."id" = lvuad."parentid" )' ),
+			'internaldeps' => array('LEFT JOIN "users_address" AS lvuad ON ( lvu."id" = lvuad."parentid" )'),
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
 			'public' => false,
@@ -222,14 +222,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/address/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/address/laravel/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/address/laravel/delete');
 	}
 
 
@@ -239,11 +239,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/address/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, [], $withsub);
 	}
 
 
@@ -254,9 +254,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', 'address/' . $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', 'address/'.$manager, ($name === null ? 'Laravel' : $name));
 	}
 
 

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

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	/**
 	 * Initializes the request view helper.
 	 *
-	 * @param \\Aimeos\MW\View\Iface $view View instance with registered view helpers
+	 * @param \Aimeos\MW\View\Iface $view View instance with registered view helpers
 	 * @param \Illuminate\Http\Request $request Laravel request object
 	 */
 	public function __construct( \Aimeos\MW\View\Iface $view, \Illuminate\Http\Request $request )

Braces +1 added lines, -2 removed lines

@@ -117,8 +117,7 @@
 					$value->getClientOriginalName(),
 					$value->getClientMimeType()
 				);
-			}
-			elseif( is_array( $value ) )
+			} elseif( is_array( $value ) )
 			{
 				$list[$key] = $this->getFiles( $value );
 			}

Spacing +16 added lines, -16 removed lines

@@ -34,11 +34,11 @@ 
 	 * @param \\Aimeos\MW\View\Iface $view View instance with registered view helpers
 	 * @param \Illuminate\Http\Request $request Laravel request object
 	 */
-	public function __construct( \Aimeos\MW\View\Iface $view, \Illuminate\Http\Request $request )
+	public function __construct(\Aimeos\MW\View\Iface $view, \Illuminate\Http\Request $request)
 	{
 		$this->request = $request;
 
-		parent::__construct( $view, $this->createRequest( $request ) );
+		parent::__construct($view, $this->createRequest($request));
 	}
 
 
@@ -60,7 +60,7 @@ 
 	 */
 	public function getTarget()
 	{
-		if( ( $route = $this->request->route() ) !== null ) {
+		if (($route = $this->request->route()) !== null) {
 			return $route->getName();
 		}
 	}
@@ -72,10 +72,10 @@ 
 	 * @param \Illuminate\Http\Request $nativeRequest Laravel request object
 	 * @return \Psr\Http\Message\ServerRequestInterface PSR-7 request object
 	 */
-	protected function createRequest( \Illuminate\Http\Request $nativeRequest )
+	protected function createRequest(\Illuminate\Http\Request $nativeRequest)
 	{
-		$files = ServerRequestFactory::normalizeFiles( $this->getFiles( $nativeRequest->files->all() ) );
-		$server = ServerRequestFactory::normalizeServer( $nativeRequest->server->all() );
+		$files = ServerRequestFactory::normalizeFiles($this->getFiles($nativeRequest->files->all()));
+		$server = ServerRequestFactory::normalizeServer($nativeRequest->server->all());
 		$headers = $nativeRequest->headers->all();
 		$cookies = $nativeRequest->cookies->all();
 		$post = $nativeRequest->request->all();
@@ -83,13 +83,13 @@ 
 		$method = $nativeRequest->getMethod();
 		$uri = $nativeRequest->getUri();
 
-		$body = new Stream( 'php://temp', 'wb+' );
-		$body->write( $nativeRequest->getContent() );
+		$body = new Stream('php://temp', 'wb+');
+		$body->write($nativeRequest->getContent());
 
-		$request = new ServerRequest( $server, $files, $uri, $method, $body, $headers, $cookies, $query, $post );
+		$request = new ServerRequest($server, $files, $uri, $method, $body, $headers, $cookies, $query, $post);
 
-		foreach( $nativeRequest->attributes->all() as $key => $value ) {
-			$request = $request->withAttribute( $key, $value );
+		foreach ($nativeRequest->attributes->all() as $key => $value) {
+			$request = $request->withAttribute($key, $value);
 		}
 
 		return $request;
@@ -102,13 +102,13 @@ 
 	 * @param array $files Multi-dimensional list of uploaded files from Symfony request
 	 * @return array Multi-dimensional list of uploaded files as PSR-7 objects
 	 */
-	protected function getFiles( array $files )
+	protected function getFiles(array $files)
 	{
 		$list = [];
 
-		foreach( $files as $key => $value )
+		foreach ($files as $key => $value)
 		{
-			if( $value instanceof \Symfony\Component\HttpFoundation\File\UploadedFile )
+			if ($value instanceof \Symfony\Component\HttpFoundation\File\UploadedFile)
 			{
 				$list[$key] = new \Zend\Diactoros\UploadedFile(
 					$value->getRealPath(),
@@ -118,9 +118,9 @@ 
 					$value->getClientMimeType()
 				);
 			}
-			elseif( is_array( $value ) )
+			elseif (is_array($value))
 			{
-				$list[$key] = $this->getFiles( $value );
+				$list[$key] = $this->getFiles($value);
 			}
 		}
 

lib/custom/src/MW/View/Engine/Blade.php

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
 	 *
 	 * @param \Illuminate\View\Factory $factory Laravel view factory
 	 */
-	public function __construct( \Illuminate\View\Factory $factory )
+	public function __construct(\Illuminate\View\Factory $factory)
 	{
 		$this->factory = $factory;
 	}
@@ -42,19 +42,19 @@ 
 	 * @return string Output generated by the template
 	 * @throws \Aimeos\MW\View\Exception If the template isn't found
 	 */
-	public function render( \Aimeos\MW\View\Iface $view, $filename, array $values )
+	public function render(\Aimeos\MW\View\Iface $view, $filename, array $values)
 	{
 		$factory = $this->factory;
-		$lv = $factory->file( $filename, $values );
+		$lv = $factory->file($filename, $values);
 
-		$fcn = function() use ( $factory, $view )
+		$fcn = function() use ($factory, $view)
 		{
-			foreach( $factory->getSections() as $name => $section ) {
-				$view->block()->set( $name, $section );
+			foreach ($factory->getSections() as $name => $section) {
+				$view->block()->set($name, $section);
 			}
 		};
 
-		$contents = $lv->render( $fcn );
+		$contents = $lv->render($fcn);
 		$this->factory->flushSections();
 
 		return $contents;

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

Spacing +5 added lines, -5 removed lines

@@ -28,7 +28,7 @@ 
 	 *
 	 * @param \Illuminate\Session\Store $object Laravel session object
 	 */
-	public function __construct( \Illuminate\Session\Store $object )
+	public function __construct(\Illuminate\Session\Store $object)
 	{
 		$this->object = $object;
 	}
@@ -43,9 +43,9 @@ 
 	 * @param mixed $default Value returned if requested key isn't found
 	 * @return mixed Value associated to the requested key
 	 */
-	public function get( $name, $default = null )
+	public function get($name, $default = null)
 	{
-		return $this->object->get( $name, $default );
+		return $this->object->get($name, $default);
 	}
 
 
@@ -58,8 +58,8 @@ 
 	 * @param mixed $value Value that should be associated with the given key
 	 * @return void
 	 */
-	public function set( $name, $value )
+	public function set($name, $value)
 	{
-		$this->object->put( $name, $value );
+		$this->object->put($name, $value);
 	}
 }