aimeos / ai-admin-jqadm

admin/jqadm/src/Admin/JQAdm/Product/Category/Decorator/Cache.php

Spacing +3 added lines, -3 removed lines

@@ -27,13 +27,13 @@
 	public function save()
 	{
 		$result = $this->getClient()->save();
-		$ids = array( 'catalog' );
+		$ids = array('catalog');
 
-		foreach( $this->getView()->param( 'category/catalog.id', array() ) as $id ) {
+		foreach ($this->getView()->param('category/catalog.id', array()) as $id) {
 			$ids[] = 'catalog-' . $id;
 		}
 
-		$this->getContext()->getCache()->deleteByTags( $ids );
+		$this->getContext()->getCache()->deleteByTags($ids);
 
 		return $result;
 	}

admin/jqadm/src/Admin/JQAdm/Product/Category/Standard.php

Spacing +61 added lines, -61 removed lines

@@ -67,17 +67,17 @@ 
 	{
 		$view = $this->getView();
 
-		$this->setData( $view );
+		$this->setData($view);
 		$view->categoryBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->categoryBody .= $client->copy();
 		}
 
 		$tplconf = 'admin/jqadm/product/category/template-item';
 		$default = 'product/item-category-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -90,17 +90,17 @@ 
 	{
 		$view = $this->getView();
 
-		$this->setData( $view );
+		$this->setData($view);
 		$view->categoryBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->categoryBody .= $client->create();
 		}
 
 		$tplconf = 'admin/jqadm/product/category/template-item';
 		$default = 'product/item-category-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -113,17 +113,17 @@ 
 	{
 		$view = $this->getView();
 
-		$this->setData( $view );
+		$this->setData($view);
 		$view->categoryBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->categoryBody .= $client->get();
 		}
 
 		$tplconf = 'admin/jqadm/product/category/template-item';
 		$default = 'product/item-category-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -137,32 +137,32 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'product/lists' );
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'product/lists');
 		$manager->begin();
 
 		try
 		{
-			$this->updateItems( $view );
+			$this->updateItems($view);
 			$view->categoryBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->categoryBody .= $client->save();
 			}
 
 			$manager->commit();
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item-category' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item-category' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 			$manager->rollback();
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
-			$error = array( 'product-item-category' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$context->getLogger()->log($e->getMessage() . ' - ' . $e->getTraceAsString());
+			$error = array('product-item-category' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 			$manager->rollback();
 		}
 
@@ -177,7 +177,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/product/category/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -252,7 +252,7 @@ 
 		 * @see admin/jqadm/product/category/decorators/excludes
 		 * @see admin/jqadm/product/category/decorators/global
 		 */
-		return $this->createSubClient( 'product/category/' . $type, $name );
+		return $this->createSubClient('product/category/' . $type, $name);
 	}
 
 
@@ -263,7 +263,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -273,20 +273,20 @@ 
 	 * @param array $listItems List of items implementing \Aimeos\Common\Item\Lists\Iface
 	 * @return array Associative list of catalog IDs as keys and items implementing \Aimeos\Catalog\Item\Iface as values
 	 */
-	protected function getCatalogItems( array $listItems )
+	protected function getCatalogItems(array $listItems)
 	{
 		$ids = array();
 
-		foreach( $listItems as $listItem ) {
+		foreach ($listItems as $listItem) {
 			$ids[] = $listItem->getParentId();
 		}
 
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'catalog' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'catalog');
 
 		$search = $manager->createSearch();
-		$search->setConditions( $search->compare( '==', 'catalog.id', $ids ) );
+		$search->setConditions($search->compare('==', 'catalog.id', $ids));
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 
 
@@ -296,19 +296,19 @@ 
 	 * @param string $prodid Unique product ID
 	 * @return array Associative list of category list IDs as keys and list items as values
 	 */
-	protected function getListItems( $prodid )
+	protected function getListItems($prodid)
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'catalog/lists' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'catalog/lists');
 
 		$search = $manager->createSearch();
 		$expr = array(
-			$search->compare( '==', 'catalog.lists.refid', $prodid ),
-			$search->compare( '==', 'catalog.lists.domain', 'product' ),
-			$search->compare( '==', 'catalog.lists.type.domain', 'product' ),
+			$search->compare('==', 'catalog.lists.refid', $prodid),
+			$search->compare('==', 'catalog.lists.domain', 'product'),
+			$search->compare('==', 'catalog.lists.type.domain', 'product'),
 		);
-		$search->setConditions( $search->combine( '&&', $expr ) );
+		$search->setConditions($search->combine('&&', $expr));
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 
 
@@ -320,13 +320,13 @@ 
 	protected function getListTypes()
 	{
 		$list = array();
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'catalog/lists/type' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'catalog/lists/type');
 
 		$search = $manager->createSearch();
-		$search->setConditions( $search->compare( '==', 'catalog.lists.type.domain', 'product' ) );
-		$search->setSlice( 0, 0x7fffffff );
+		$search->setConditions($search->compare('==', 'catalog.lists.type.domain', 'product'));
+		$search->setSlice(0, 0x7fffffff);
 
-		foreach( $manager->searchItems( $search ) as $item ) {
+		foreach ($manager->searchItems($search) as $item) {
 			$list[$item->getCode()] = $item->getId();
 		}
 
@@ -339,29 +339,29 @@ 
 	 *
 	 * @param \Aimeos\MW\View\Iface $view View object with helpers and assigned parameters
 	 */
-	protected function setData( \Aimeos\MW\View\Iface $view )
+	protected function setData(\Aimeos\MW\View\Iface $view)
 	{
-		$view->categoryData = (array) $view->param( 'category', array() );
+		$view->categoryData = (array) $view->param('category', array());
 
-		if( !empty( $view->categoryData ) ) {
+		if (!empty($view->categoryData)) {
 			return;
 		}
 
-		$listItems = $this->getListItems( $view->item->getId() );
-		$catItems = $this->getCatalogItems( $listItems );
+		$listItems = $this->getListItems($view->item->getId());
+		$catItems = $this->getCatalogItems($listItems);
 		$data = array();
 
-		foreach( $listItems as $listItem )
+		foreach ($listItems as $listItem)
 		{
 			$catId = $listItem->getParentId();
 
-			if( isset( $catItems[$catId] ) )
+			if (isset($catItems[$catId]))
 			{
 				$data['catalog.id'][] = $catId;
 				$data['catalog.label'][] = $catItems[$catId]->getLabel();
 			}
 
-			foreach( $listItem->toArray() as $key => $value ) {
+			foreach ($listItem->toArray() as $key => $value) {
 				$data[$key][] = $value;
 			}
 		}
@@ -376,36 +376,36 @@ 
 	 *
 	 * @param \Aimeos\MW\View\Iface $view View object with helpers and assigned parameters
 	 */
-	protected function updateItems( \Aimeos\MW\View\Iface $view )
+	protected function updateItems(\Aimeos\MW\View\Iface $view)
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'catalog/lists' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'catalog/lists');
 
 		$id = $view->item->getId();
-		$map = $this->getListItems( $id );
-		$listIds = (array) $view->param( 'category/catalog.lists.id', array() );
+		$map = $this->getListItems($id);
+		$listIds = (array) $view->param('category/catalog.lists.id', array());
 
 
-		foreach( $listIds as $pos => $listid )
+		foreach ($listIds as $pos => $listid)
 		{
-			if( isset( $map[$listid] ) ) {
-				unset( $map[$listid], $listIds[$pos] );
+			if (isset($map[$listid])) {
+				unset($map[$listid], $listIds[$pos]);
 			}
 		}
 
-		$manager->deleteItems( array_keys( $map ) );
+		$manager->deleteItems(array_keys($map));
 
 
 		$item = $manager->createItem();
-		$item->setDomain( 'product' );
-		$item->setRefId( $id );
+		$item->setDomain('product');
+		$item->setRefId($id);
 
-		foreach( $listIds as $pos => $listid )
+		foreach ($listIds as $pos => $listid)
 		{
-			$item->setId( null );
-			$item->setParentId( $view->param( 'category/catalog.id/' . $pos ) );
-			$item->setTypeId( $view->param( 'category/catalog.lists.typeid/' . $pos ) );
+			$item->setId(null);
+			$item->setParentId($view->param('category/catalog.id/' . $pos));
+			$item->setTypeId($view->param('category/catalog.lists.typeid/' . $pos));
 
-			$manager->saveItem( $item, false );
+			$manager->saveItem($item, false);
 		}
 	}
 }
\ No newline at end of file

Braces +2 added lines, -4 removed lines

@@ -151,14 +151,12 @@
 
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-category' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
 			$error = array( 'product-item-category' => $e->getMessage() );

admin/jqadm/src/Admin/JQAdm/Product/Standard.php

Spacing +82 added lines, -82 removed lines

@@ -55,7 +55,7 @@ 
 	 * @category Developer
 	 */
 	private $subPartPath = 'admin/jqadm/product/standard/subparts';
-	private $subPartNames = array( 'selection', 'bundle', 'image', 'text', 'price', 'stock', 'category', 'characteristic', 'physical', 'download' );
+	private $subPartNames = array('selection', 'bundle', 'image', 'text', 'price', 'stock', 'category', 'characteristic', 'physical', 'download');
 
 
 	/**
@@ -70,39 +70,39 @@ 
 
 		try
 		{
-			$this->setData( $view );
+			$this->setData($view);
 
-			if( isset( $view->itemData['product.code'] ) )
+			if (isset($view->itemData['product.code']))
 			{
 				$data = $view->itemData;
 				$data['product.id'] = '';
 				$data['product.code'] = $data['product.code'] . '_copy';
 
-				$view->item->setCode( $data['product.code'] );
+				$view->item->setCode($data['product.code']);
 				$view->itemData = $data;
 			}
 
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->copy();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 		}
 
 		$tplconf = 'admin/jqadm/product/template-item';
 		$default = 'product/item-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -118,28 +118,28 @@ 
 
 		try
 		{
-			$this->setData( $view );
+			$this->setData($view);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 		}
 
 		$tplconf = 'admin/jqadm/product/template-item';
 		$default = 'product/item-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -153,30 +153,30 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'product');
 		$manager->begin();
 
 		try
 		{
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->deleteItems( (array) $view->param( 'id' ) );
+			$manager->deleteItems((array) $view->param('id'));
 			$manager->commit();
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 			$manager->rollback();
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 			$manager->rollback();
 		}
 	}
@@ -194,28 +194,28 @@ 
 
 		try
 		{
-			$this->setData( $view );
+			$this->setData($view);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->get();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 		}
 
 		$tplconf = 'admin/jqadm/product/template-item';
 		$default = 'product/item-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -229,41 +229,41 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'product');
 		$manager->begin();
 
 		$item = $manager->createItem();
 
 		try
 		{
-			$item->fromArray( $view->param( 'item', array() ) );
-			$item->setConfig( $this->getItemConfig( $view ) );
-			$manager->saveItem( $item );
+			$item->fromArray($view->param('item', array()));
+			$item->setConfig($this->getItemConfig($view));
+			$manager->saveItem($item);
 
-			$view->item = $manager->getItem( $item->getId() ); // product.type must be available
+			$view->item = $manager->getItem($item->getId()); // product.type must be available
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->save();
 			}
 
 			$manager->commit();
 			return;
 		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
 			// fall through to create()
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 			$manager->rollback();
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 			$manager->rollback();
 		}
 
@@ -284,34 +284,34 @@ 
 		try
 		{
 			$total = 0;
-			$manager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
-			$search = $this->initCriteria( $manager->createSearch(), $view->param() );
+			$manager = \Aimeos\MShop\Factory::createManager($context, 'product');
+			$search = $this->initCriteria($manager->createSearch(), $view->param());
 
 			$view->pageParams = $this->getClientParams();
-			$view->items = $manager->searchItems( $search, array(), $total );
+			$view->items = $manager->searchItems($search, array(), $total);
 			$view->filterOperators = $search->getOperators();
 			$view->total = $total;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('product-item' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 		}
 
 		$tplconf = 'admin/jqadm/product/template-list';
 		$default = 'product/list-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -322,7 +322,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/product/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -397,7 +397,7 @@ 
 		 * @see admin/jqadm/product/decorators/excludes
 		 * @see admin/jqadm/product/decorators/global
 		 */
-		return $this->createSubClient( 'product/' . $type, $name );
+		return $this->createSubClient('product/' . $type, $name);
 	}
 
 
@@ -407,17 +407,17 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with helpers and assigned parameters
 	 * @return array Multi-dimensional associative array
 	 */
-	protected function setData( \Aimeos\MW\View\Iface $view )
+	protected function setData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'product');
 
 		$view->pageParams = $this->getClientParams();
-		$view->itemData = (array) $view->param( 'item', array() );
+		$view->itemData = (array) $view->param('item', array());
 		$view->itemTypes = $this->getTypeItems();
 		$view->item = $manager->createItem();
 
-		if( !empty( $view->itemData ) || ( $id = $view->param( 'id' ) ) === null ) {
+		if (!empty($view->itemData) || ($id = $view->param('id')) === null) {
 			return;
 		}
 
@@ -432,14 +432,14 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		$domains = array( 'attribute', 'media', 'price', 'product', 'text' );
-		$domains = $context->getConfig()->get( 'admin/jqadm/product/domains', $domains );
-		$item = $manager->getItem( $id, $domains );
+		$domains = array('attribute', 'media', 'price', 'product', 'text');
+		$domains = $context->getConfig()->get('admin/jqadm/product/domains', $domains);
+		$item = $manager->getItem($id, $domains);
 
 		$data = $item->toArray();
-		$data['config'] = array( 'key' => array(), 'val' => array() );
+		$data['config'] = array('key' => array(), 'val' => array());
 
-		foreach( $item->getConfig() as $key => $value )
+		foreach ($item->getConfig() as $key => $value)
 		{
 			$data['config']['key'][] = $key;
 			$data['config']['val'][] = $value;
@@ -456,14 +456,14 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with helpers and assigned parameters
 	 * @return array Associative list of key/value pairs
 	 */
-	protected function getItemConfig( \Aimeos\MW\View\Iface $view )
+	protected function getItemConfig(\Aimeos\MW\View\Iface $view)
 	{
 		$config = array();
 
-		foreach( (array) $view->param( 'item/config/key' ) as $idx => $key )
+		foreach ((array) $view->param('item/config/key') as $idx => $key)
 		{
-			if( trim( $key ) !== '' ) {
-				$config[$key] = $view->param( 'item/config/val/' . $idx );
+			if (trim($key) !== '') {
+				$config[$key] = $view->param('item/config/val/' . $idx);
 			}
 		}
 
@@ -478,7 +478,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		$result = $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		$result = $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 		return $result;
 	}
 
@@ -490,11 +490,11 @@ 
 	 */
 	protected function getTypeItems()
 	{
-		$typeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product/type' );
+		$typeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product/type');
 
 		$search = $typeManager->createSearch();
-		$search->setSortations( array( $search->sort( '+', 'product.type.label' ) ) );
+		$search->setSortations(array($search->sort('+', 'product.type.label')));
 
-		return $typeManager->searchItems( $search );
+		return $typeManager->searchItems($search);
 	}
 }

Braces +13 added lines, -26 removed lines

@@ -87,13 +87,11 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->copy();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'product-item' => $e->getMessage() );
 			$view->errors = $view->get( 'errors', array() ) + $error;
@@ -124,13 +122,11 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->create();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'product-item' => $e->getMessage() );
 			$view->errors = $view->get( 'errors', array() ) + $error;
@@ -166,14 +162,12 @@ 
 			$manager->commit();
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'product-item' => $e->getMessage() );
 			$view->errors = $view->get( 'errors', array() ) + $error;
@@ -200,13 +194,11 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->get();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'product-item' => $e->getMessage() );
 			$view->errors = $view->get( 'errors', array() ) + $error;
@@ -249,18 +241,15 @@ 
 
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		} catch( \Aimeos\Admin\JQAdm\Exception $e )
 		{
 			// fall through to create()
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'product-item' => $e->getMessage() );
 			$view->errors = $view->get( 'errors', array() ) + $error;
@@ -296,13 +285,11 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'product-item' => $e->getMessage() );
 			$view->errors = $view->get( 'errors', array() ) + $error;

Doc Comments +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
 	/**
 	 * Copies a resource
 	 *
-	 * @return string|null admin output to display or null for redirecting to the list
+	 * @return string admin output to display or null for redirecting to the list
 	 */
 	public function copy()
 	{
@@ -86,7 +86,7 @@ 
 	/**
 	 * Creates a new resource
 	 *
-	 * @return string|null admin output to display or null for redirecting to the list
+	 * @return string admin output to display or null for redirecting to the list
 	 */
 	public function create()
 	{
@@ -109,7 +109,7 @@ 
 	/**
 	 * Returns a single resource
 	 *
-	 * @return string|null admin output to display or null for redirecting to the list
+	 * @return string admin output to display or null for redirecting to the list
 	 */
 	public function get()
 	{

admin/jqadm/src/Admin/JQAdm/Product/Factory.php

Spacing +8 added lines, -8 removed lines

@@ -30,7 +30,7 @@ 
 	 * @return \Aimeos\Admin\JQAdm\Iface Filter part implementing \Aimeos\Admin\JQAdm\Iface
 	 * @throws \Aimeos\Admin\JQAdm\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function createClient( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null )
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null)
 	{
 		/** admin/jqadm/product/name
 		 * Class name of the used account favorite client implementation
@@ -65,22 +65,22 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'admin/jqadm/product/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('admin/jqadm/product/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Admin\\JQAdm\\Product\\' . $name : '<not a string>';
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Admin\\JQAdm\\Product\\' . $name : '<not a string>';
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Admin\\JQAdm\\Iface';
 		$classname = '\\Aimeos\\Admin\\JQAdm\\Product\\' . $name;
 
-		$client = self::createClientBase( $context, $classname, $iface, $templatePaths );
+		$client = self::createClientBase($context, $classname, $iface, $templatePaths);
 
-		return self::addClientDecorators( $context, $client, $templatePaths, 'product' );
+		return self::addClientDecorators($context, $client, $templatePaths, 'product');
 	}
 
 }
\ No newline at end of file

admin/jqadm/src/Admin/JQAdm/Common/Admin/Factory/Base.php

Spacing +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
 	 * 	and a list of relative paths inside the core or the extension as values
 	 * @param \Aimeos\Admin\JQAdm\Iface $client Admin object
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths)
 	{
-		parent::__construct( $context, $templatePaths );
+		parent::__construct($context, $templatePaths);
 	}
 }
\ No newline at end of file

admin/jqadm/src/Admin/JQAdm/Common/Admin/Factory/Iface.php

Spacing +1 added lines, -1 removed lines

@@ -28,5 +28,5 @@
 	 * 	and a list of relative paths inside the core or the extension as values
 	 * @return void
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths );
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths);
 }

admin/jqadm/src/Admin/JQAdm/Common/Factory/Base.php

Spacing +34 added lines, -34 removed lines

@@ -30,7 +30,7 @@ 
 	 * @param string $classname Full name of the class for which the object should be returned
 	 * @param \Aimeos\Admin\JQAdm\Iface|null $client ExtJS client object
 	 */
-	public static function injectClient( $classname, \Aimeos\Admin\JQAdm\Iface $client = null )
+	public static function injectClient($classname, \Aimeos\Admin\JQAdm\Iface $client = null)
 	{
 		self::$objects[$classname] = $client;
 	}
@@ -46,29 +46,29 @@ 
 	 * @param string $classprefix Decorator class prefix, e.g. "\Aimeos\Admin\JQAdm\Catalog\Decorator\"
 	 * @return \Aimeos\Admin\JQAdm\Iface Admin object
 	 */
-	protected static function addDecorators( \Aimeos\MShop\Context\Item\Iface $context,
-		\Aimeos\Admin\JQAdm\Iface $client, array $templatePaths, array $decorators, $classprefix )
+	protected static function addDecorators(\Aimeos\MShop\Context\Item\Iface $context,
+		\Aimeos\Admin\JQAdm\Iface $client, array $templatePaths, array $decorators, $classprefix)
 	{
 		$iface = '\\Aimeos\\Admin\\JQAdm\\Common\\Decorator\\Iface';
 
-		foreach( $decorators as $name )
+		foreach ($decorators as $name)
 		{
-			if( ctype_alnum( $name ) === false )
+			if (ctype_alnum($name) === false)
 			{
-				$classname = is_string( $name ) ? $classprefix . $name : '<not a string>';
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid class name "%1$s"', $classname ) );
+				$classname = is_string($name) ? $classprefix . $name : '<not a string>';
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid class name "%1$s"', $classname));
 			}
 
 			$classname = $classprefix . $name;
 
-			if( class_exists( $classname ) === false ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Class "%1$s" not found', $classname ) );
+			if (class_exists($classname) === false) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Class "%1$s" not found', $classname));
 			}
 
-			$client = new $classname( $client, $context, $templatePaths );
+			$client = new $classname($client, $context, $templatePaths);
 
-			if( !( $client instanceof $iface ) ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Class "%1$s" does not implement "%2$s"', $classname, $iface ) );
+			if (!($client instanceof $iface)) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Class "%1$s" does not implement "%2$s"', $classname, $iface));
 			}
 		}
 
@@ -85,14 +85,14 @@ 
 	 * @param string $path Path of the client in lower case, e.g. "catalog/detail"
 	 * @return \Aimeos\Admin\JQAdm\Iface Admin object
 	 */
-	protected static function addClientDecorators( \Aimeos\MShop\Context\Item\Iface $context,
-		\Aimeos\Admin\JQAdm\Iface $client, array $templatePaths, $path )
+	protected static function addClientDecorators(\Aimeos\MShop\Context\Item\Iface $context,
+		\Aimeos\Admin\JQAdm\Iface $client, array $templatePaths, $path)
 	{
-		if( !is_string( $path ) || $path === '' ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid domain "%1$s"', $path ) );
+		if (!is_string($path) || $path === '') {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid domain "%1$s"', $path));
 		}
 
-		$localClass = str_replace( ' ', '\\', ucwords( str_replace( '/', ' ', $path ) ) );
+		$localClass = str_replace(' ', '\\', ucwords(str_replace('/', ' ', $path)));
 		$config = $context->getConfig();
 
 		/** admin/jqadm/common/decorators/default
@@ -117,26 +117,26 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		$decorators = $config->get( 'admin/jqadm/common/decorators/default', array() );
-		$excludes = $config->get( 'admin/jqadm/' . $path . '/decorators/excludes', array() );
+		$decorators = $config->get('admin/jqadm/common/decorators/default', array());
+		$excludes = $config->get('admin/jqadm/' . $path . '/decorators/excludes', array());
 
-		foreach( $decorators as $key => $name )
+		foreach ($decorators as $key => $name)
 		{
-			if( in_array( $name, $excludes ) ) {
-				unset( $decorators[$key] );
+			if (in_array($name, $excludes)) {
+				unset($decorators[$key]);
 			}
 		}
 
 		$classprefix = '\\Aimeos\\Admin\\JQAdm\\Common\\Decorator\\';
-		$client = self::addDecorators( $context, $client, $templatePaths, $decorators, $classprefix );
+		$client = self::addDecorators($context, $client, $templatePaths, $decorators, $classprefix);
 
 		$classprefix = '\\Aimeos\\Admin\\JQAdm\\Common\\Decorator\\';
-		$decorators = $config->get( 'admin/jqadm/' . $path . '/decorators/global', array() );
-		$client = self::addDecorators( $context, $client, $templatePaths, $decorators, $classprefix );
+		$decorators = $config->get('admin/jqadm/' . $path . '/decorators/global', array());
+		$client = self::addDecorators($context, $client, $templatePaths, $decorators, $classprefix);
 
 		$classprefix = '\\Aimeos\\Admin\\JQAdm\\' . $localClass . '\\Decorator\\';
-		$decorators = $config->get( 'admin/jqadm/' . $path . '/decorators/local', array() );
-		$client = self::addDecorators( $context, $client, $templatePaths, $decorators, $classprefix );
+		$decorators = $config->get('admin/jqadm/' . $path . '/decorators/local', array());
+		$client = self::addDecorators($context, $client, $templatePaths, $decorators, $classprefix);
 
 		return $client;
 	}
@@ -152,20 +152,20 @@ 
 	 * @return \Aimeos\Admin\JQAdm\\Iface Admin object
 	 * @throws \Aimeos\Admin\JQAdm\Exception If client couldn't be found or doesn't implement the interface
 	 */
-	protected static function createClientBase( \Aimeos\MShop\Context\Item\Iface $context, $classname, $interface, $templatePaths )
+	protected static function createClientBase(\Aimeos\MShop\Context\Item\Iface $context, $classname, $interface, $templatePaths)
 	{
-		if( isset( self::$objects[$classname] ) ) {
+		if (isset(self::$objects[$classname])) {
 			return self::$objects[$classname];
 		}
 
-		if( class_exists( $classname ) === false ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
+		if (class_exists($classname) === false) {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Class "%1$s" not available', $classname));
 		}
 
-		$client = new $classname( $context, $templatePaths );
+		$client = new $classname($context, $templatePaths);
 
-		if( !( $client instanceof $interface ) ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Class "%1$s" does not implement interface "%2$s"', $classname, $interface ) );
+		if (!($client instanceof $interface)) {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Class "%1$s" does not implement interface "%2$s"', $classname, $interface));
 		}
 
 		return $client;

admin/jqadm/src/Admin/JQAdm/Common/Factory/Iface.php

Spacing +1 added lines, -1 removed lines

@@ -27,5 +27,5 @@
 	 * @param string $name Admin name (from configuration or "Standard" if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface New client object
 	 */
-	public static function createClient( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null );
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null);
 }

admin/jqadm/src/Admin/JQAdm/Common/Decorator/Base.php

Indentation +5 added lines, -5 removed lines

@@ -74,7 +74,7 @@ 
 	 * Creates a new resource
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function create()
 	{
 		return $this->client->create();
@@ -85,7 +85,7 @@ 
 	 * Deletes a resource
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function delete()
 	{
 		return $this->client->delete();
@@ -96,7 +96,7 @@ 
 	 * Returns a single resource
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function get()
 	{
 		return $this->client->get();
@@ -107,7 +107,7 @@ 
 	 * Saves the data
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function save()
 	{
 		return $this->client->save();
@@ -118,7 +118,7 @@ 
 	 * Returns a list of resource according to the conditions
 	 *
 	 * @return string admin output to display
-	*/
+	 */
 	public function search()
 	{
 		return $this->client->search();

Spacing +10 added lines, -10 removed lines

@@ -32,10 +32,10 @@ 
 	 * @param array $templatePaths Associative list of the file system paths to the core or the extensions as key
 	 * 	and a list of relative paths inside the core or the extension as values
 	 */
-	public function __construct( \Aimeos\Admin\JQAdm\Iface $client,
-		\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths )
+	public function __construct(\Aimeos\Admin\JQAdm\Iface $client,
+		\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths)
 	{
-		parent::__construct( $context, $templatePaths );
+		parent::__construct($context, $templatePaths);
 
 		$this->client = $client;
 	}
@@ -49,10 +49,10 @@ 
 	 * @return mixed Returns the value of the called method
 	 * @throws \Aimeos\Admin\JQAdm\Exception If method call failed
 	 */
-	public function __call( $name, array $param )
+	public function __call($name, array $param)
 	{
-		if( ( $result = call_user_func_array( array( $this->client, $name ), $param ) ) === false ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Unable to call method "%1$s"', $name ) );
+		if (($result = call_user_func_array(array($this->client, $name), $param)) === false) {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Unable to call method "%1$s"', $name));
 		}
 
 		return $result;
@@ -132,9 +132,9 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
-		return $this->client->getSubClient( $type, $name );
+		return $this->client->getSubClient($type, $name);
 	}
 
 
@@ -155,9 +155,9 @@ 
 	 * @param \Aimeos\MW\View\Iface $view The view object which generates the admin output
 	 * @return \Aimeos\Admin\JQAdm\Iface Reference to this object for fluent calls
 	 */
-	public function setView( \Aimeos\MW\View\Iface $view )
+	public function setView(\Aimeos\MW\View\Iface $view)
 	{
-		$this->client->setView( $view );
+		$this->client->setView($view);
 		return $this;
 	}