Aimeos\Upscheme\Task\MShopAddDataAbstract::addMedia()

Complexity

Conditions	6
Paths	17

Size

Total Lines	54
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	MShopAddDataAbstract::addSuppliers()	0	13	2
A	MShopAddDataAbstract::removeRefItems()	0	15	3

<?php

/**
 * @license LGPLv3, https://opensource.org/licenses/LGPL-3.0
 * @copyright Metaways Infosystems GmbH, 2014
 * @copyright Aimeos (aimeos.org), 2015-2024
 */


namespace Aimeos\Upscheme\Task;


/**
 * Adds records to tables.
 */
class MShopAddDataAbstract extends Base
{
	private $attributes;

The private property $attributes is not used, and could be removed.

	public function up()
	{
	}


	/**
	 * Adds the referenced items from the given entry data.
	 *
	 * @param \Aimeos\MShop\Common\Item\ListsRef\Iface $item Item with list items
	 * @param array $entry Associative list of data with stock, attribute, media, price, text and product sections
	 * @return \Aimeos\MShop\Common\Item\ListsRef\Iface $item Updated item
	 */
	protected function addRefItems( \Aimeos\MShop\Common\Item\ListsRef\Iface $item, array $entry )
	{
		$context = $this->context();

The method context() does not exist on Aimeos\Upscheme\Task\MShopAddDataAbstract. Since you implemented __call, consider adding a @method annotation.

		foreach( ['media', 'price', 'text'] as $refDomain )
		{
			if( isset( $entry[$refDomain] ) )
			{
				$manager = \Aimeos\MShop::create( $context, $refDomain );

				foreach( $entry[$refDomain] as $data )
				{
					$listItem = $manager->createListItem()->fromArray( $data );
					$refItem = $manager->create()->fromArray( $data );

					if( isset( $data['property'] ) )
					{
						foreach( (array) $data['property'] as $property )
						{
							$propItem = $manager->createPropertyItem()->fromArray( $property );
							$refItem->addPropertyItem( $propItem );
						}
					}

					$item->addListItem( $refDomain, $listItem, $refItem );
				}
			}
		}

		$this->addAttributes( $item, $entry['attribute'] ?? [] );
		$this->addCategories( $item, $entry['catalog'] ?? [] );
		$this->addSuppliers( $item, $entry['supplier'] ?? [] );
		$this->addProducts( $item, $entry['product'] ?? [] );

		return $item;
	}


	protected function addAttributes( \Aimeos\MShop\Common\Item\ListsRef\Iface $item, array $entries ) : \Aimeos\MShop\Common\Item\ListsRef\Iface
	{
		$manager = \Aimeos\MShop::create( $this->context(), 'attribute' );

		foreach( $entries as $data )
		{
			$listItem = $manager->createListItem()->fromArray( $data );
			$refItem = $manager->create()->fromArray( $data );

			try {
				$refItem = $manager->find( $refItem->getCode(), [], $item->getResourceType(), $refItem->getType() );
			} catch( \Exception $e ) { ; }

Consider adding a comment why this CATCH block is empty.

			$refItem = $this->addRefItems( $refItem, $data );
			$item->addListItem( 'attribute', $listItem, $refItem );
		}

		return $item;
	}


	protected function addCategories( \Aimeos\MShop\Common\Item\ListsRef\Iface $item, array $entries ) : \Aimeos\MShop\Common\Item\ListsRef\Iface
	{
		$manager = \Aimeos\MShop::create( $this->context(), 'catalog' );

		foreach( $entries as $data )
		{
			$listItem = $manager->createListItem()->fromArray( $data );
			$refItem = $manager->find( $data['catalog.code'] );

			$item->addListItem( 'catalog', $listItem, $refItem );
		}

		return $item;
	}


	protected function addProducts( \Aimeos\MShop\Common\Item\ListsRef\Iface $item, array $entries ) : \Aimeos\MShop\Common\Item\ListsRef\Iface
	{
		$manager = \Aimeos\MShop::create( $this->context(), 'product' );

		foreach( $entries as $data )
		{
			$listItem = $manager->createListItem()->fromArray( $data );
			$refItem = $manager->find( $data['product.code'] );

			$item->addListItem( 'product', $listItem, $refItem );
		}

		return $item;
	}


	protected function addSuppliers( \Aimeos\MShop\Common\Item\ListsRef\Iface $item, array $entries ) : \Aimeos\MShop\Common\Item\ListsRef\Iface
	{
		$manager = \Aimeos\MShop::create( $this->context(), 'supplier' );

		foreach( $entries as $data )
		{
			$listItem = $manager->createListItem()->fromArray( $data );
			$refItem = $manager->find( $data['supplier.code'] );

			$item->addListItem( 'supplier', $listItem, $refItem );
		}

		return $item;
	}


	/**
	 * Removes the referenced items from the given items.
	 *
	 * @param \Aimeos\Map $items List of items
	 * @param array $domains List of domain names
	 */
	public function removeRefItems( \Aimeos\Map $items, array $domains )
	{
		$context = $this->context();

		foreach( $domains as $domain )
		{
			$rmItems = map();

			foreach( $items as $item ) {
				$rmItems->merge( $item->getRefItems( $domain, null, null, false )->filter( function( $item ) {
					return strncmp( $item->getLabel(), 'Demo', 4 ) === 0;
				} ) );
			}

			\Aimeos\MShop::create( $context, $domain )->delete( $rmItems );
		}
	}
}