Inspection of "Cleanup after stock import" - aimeos/ai-controller-jobs - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (c3c77b)

by Aimeos

created 2019-01-18 12:18 UTC

Status

Braces +1 added lines, -2 removed lines patch added patch discarded remove patch

@@ -87,8 +87,7 @@
 block discarded – undo
 			foreach( $manager->searchItems( $search ) as $item ) {
 				$this->listTypes[$item->getCode()] = $item->getCode();
 			}
-		}
-		else
+		} else
 		{
 			$this->listTypes = array_flip( $this->listTypes );
 		}

Please login to merge, or discard this patch.

src/Controller/Common/Product/Import/Csv/Processor/Property/Standard.php 2 patches

Braces +1 added lines, -2 removed lines patch added patch discarded remove patch

@@ -91,8 +91,7 @@
 block discarded – undo
 			{
 				$item = $propMap[$value][$type];
 				unset( $items[ $item->getId() ] );
-			}
-			else
+			} else
 			{
 				$item = $manager->createItem()->setType( $type );
 			}

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -72,7 +72,7 @@  discard block
 block discarded – undo
 		$map = $this->getMappedChunk( $data, $this->getMapping() );
 
 		foreach( $items as $item ) {
-			$propMap[ $item->getValue() ][ $item->getType() ] = $item;
+			$propMap[$item->getValue()][$item->getType()] = $item;
 		}
 
 		foreach( $map as $list )
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
 			if( isset( $propMap[$value][$type] ) )
 			{
 				$item = $propMap[$value][$type];
-				unset( $items[ $item->getId() ] );
+				unset( $items[$item->getId()] );
 			}
 			else
 			{

Please login to merge, or discard this patch.

src/Controller/Common/Product/Import/Csv/Processor/Media/Standard.php 2 patches

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

@@ -81,8 +81,7 @@  discard block
 block discarded – undo
 			foreach( $manager->searchItems( $search ) as $item ) {
 				$this->listTypes[$item->getCode()] = $item->getCode();
 			}
-		}
-		else
+		} else
 		{
 			$this->listTypes = array_flip( $this->listTypes );
 		}
@@ -143,8 +142,7 @@  discard block
 block discarded – undo
 					$listItem = $listMap[$url][$type][$listtype];
 					$refItem = $listItem->getRefItem();
 					unset( $listItems[ $listItem->getId() ] );
-				}
-				else
+				} else
 				{
 					$listItem = $listManager->createItem()->setType( $listtype );
 					$refItem = $manager->createItem()->setType( $type );

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -120,7 +120,7 @@  discard block
 block discarded – undo
 		foreach( $listItems as $listItem )
 		{
 			if( ( $refItem = $listItem->getRefItem() ) !== null ) {
-				$listMap[ $refItem->getUrl() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem;
+				$listMap[$refItem->getUrl()][$refItem->getType()][$listItem->getType()] = $listItem;
 			}
 		}
 
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
 				{
 					$listItem = $listMap[$url][$type][$listtype];
 					$refItem = $listItem->getRefItem();
-					unset( $listItems[ $listItem->getId() ] );
+					unset( $listItems[$listItem->getId()] );
 				}
 				else
 				{

Please login to merge, or discard this patch.

src/Controller/Common/Product/Import/Csv/Processor/Price/Standard.php 2 patches

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

@@ -81,8 +81,7 @@  discard block
 block discarded – undo
 			foreach( $manager->searchItems( $search ) as $item ) {
 				$this->listTypes[$item->getCode()] = $item->getCode();
 			}
-		}
-		else
+		} else
 		{
 			$this->listTypes = array_flip( $this->listTypes );
 		}
@@ -137,8 +136,7 @@  discard block
 block discarded – undo
 				$listItem = $listMap[$value][$type][$listtype];
 				$refItem = $listItem->getRefItem();
 				unset( $listItems[ $listItem->getId() ] );
-			}
-			else
+			} else
 			{
 				$listItem = $listManager->createItem()->setType( $listtype );
 				$refItem = $manager->createItem()->setType( $type );

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -118,7 +118,7 @@  discard block
 block discarded – undo
 		foreach( $listItems as $listItem )
 		{
 			if( ( $refItem = $listItem->getRefItem() ) !== null ) {
-				$listMap[ $refItem->getValue() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem;
+				$listMap[$refItem->getValue()][$refItem->getType()][$listItem->getType()] = $listItem;
 			}
 		}
 
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
 			{
 				$listItem = $listMap[$value][$type][$listtype];
 				$refItem = $listItem->getRefItem();
-				unset( $listItems[ $listItem->getId() ] );
+				unset( $listItems[$listItem->getId()] );
 			}
 			else
 			{

Please login to merge, or discard this patch.

common/src/Controller/Common/Product/Import/Csv/Processor/Text/Standard.php 2 patches

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

@@ -81,8 +81,7 @@  discard block
 block discarded – undo
 			foreach( $manager->searchItems( $search ) as $item ) {
 				$this->listTypes[$item->getCode()] = $item->getCode();
 			}
-		}
-		else
+		} else
 		{
 			$this->listTypes = array_flip( $this->listTypes );
 		}
@@ -137,8 +136,7 @@  discard block
 block discarded – undo
 				$listItem = $listMap[$content][$type][$listtype];
 				$refItem = $listItem->getRefItem();
 				unset( $listItems[ $listItem->getId() ] );
-			}
-			else
+			} else
 			{
 				$listItem = $listManager->createItem()->setType( $listtype );
 				$refItem = $manager->createItem()->setType( $type );

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -118,7 +118,7 @@  discard block
 block discarded – undo
 		foreach( $listItems as $listItem )
 		{
 			if( ( $refItem = $listItem->getRefItem() ) !== null ) {
-				$listMap[ $refItem->getContent() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem;
+				$listMap[$refItem->getContent()][$refItem->getType()][$listItem->getType()] = $listItem;
 			}
 		}
 
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
 			{
 				$listItem = $listMap[$content][$type][$listtype];
 				$refItem = $listItem->getRefItem();
-				unset( $listItems[ $listItem->getId() ] );
+				unset( $listItems[$listItem->getId()] );
 			}
 			else
 			{

Please login to merge, or discard this patch.

src/Controller/Common/Product/Import/Csv/Processor/Attribute/Standard.php 2 patches

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

@@ -82,8 +82,7 @@  discard block
 block discarded – undo
 			foreach( $manager->searchItems( $search ) as $item ) {
 				$this->listTypes[$item->getCode()] = $item->getCode();
 			}
-		}
-		else
+		} else
 		{
 			$this->listTypes = array_flip( $this->listTypes );
 		}
@@ -144,8 +143,7 @@  discard block
 block discarded – undo
 				{
 					$listItem = $listMap[$code][$listtype];
 					unset( $listItems[ $listItem->getId() ] );
-				}
-				else
+				} else
 				{
 					$listItem = $listManager->createItem()->setType( $listtype );
 				}

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -123,7 +123,7 @@  discard block
 block discarded – undo
 		foreach( $listItems as $listItem )
 		{
 			if( ( $refItem = $listItem->getRefItem() ) !== null ) {
-				$listMap[ $refItem->getCode() ][ $listItem->getType() ] = $listItem;
+				$listMap[$refItem->getCode()][$listItem->getType()] = $listItem;
 			}
 		}
 
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
 				if( isset( $listMap[$code][$listtype] ) )
 				{
 					$listItem = $listMap[$code][$listtype];
-					unset( $listItems[ $listItem->getId() ] );
+					unset( $listItems[$listItem->getId()] );
 				}
 				else
 				{

Please login to merge, or discard this patch.

src/Controller/Common/Catalog/Import/Csv/Processor/Media/Standard.php 2 patches

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

@@ -81,8 +81,7 @@  discard block
 block discarded – undo
 			foreach( $manager->searchItems( $search ) as $item ) {
 				$this->listTypes[$item->getCode()] = $item->getCode();
 			}
-		}
-		else
+		} else
 		{
 			$this->listTypes = array_flip( $this->listTypes );
 		}
@@ -143,8 +142,7 @@  discard block
 block discarded – undo
 					$listItem = $listMap[$url][$type][$listtype];
 					$refItem = $listItem->getRefItem();
 					unset( $listItems[ $listItem->getId() ] );
-				}
-				else
+				} else
 				{
 					$listItem = $listManager->createItem()->setType( $listtype );
 					$refItem = $manager->createItem()->setType( $type );

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -120,7 +120,7 @@  discard block
 block discarded – undo
 		foreach( $listItems as $listItem )
 		{
 			if( ( $refItem = $listItem->getRefItem() ) !== null ) {
-				$listMap[ $refItem->getUrl() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem;
+				$listMap[$refItem->getUrl()][$refItem->getType()][$listItem->getType()] = $listItem;
 			}
 		}
 
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
 				{
 					$listItem = $listMap[$url][$type][$listtype];
 					$refItem = $listItem->getRefItem();
-					unset( $listItems[ $listItem->getId() ] );
+					unset( $listItems[$listItem->getId()] );
 				}
 				else
 				{

Please login to merge, or discard this patch.

common/src/Controller/Common/Catalog/Import/Csv/Processor/Text/Standard.php 2 patches

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

@@ -81,8 +81,7 @@  discard block
 block discarded – undo
 			foreach( $manager->searchItems( $search ) as $item ) {
 				$this->listTypes[$item->getCode()] = $item->getCode();
 			}
-		}
-		else
+		} else
 		{
 			$this->listTypes = array_flip( $this->listTypes );
 		}
@@ -137,8 +136,7 @@  discard block
 block discarded – undo
 				$listItem = $listMap[$content][$type][$listtype];
 				$refItem = $listItem->getRefItem();
 				unset( $listItems[ $listItem->getId() ] );
-			}
-			else
+			} else
 			{
 				$listItem = $listManager->createItem()->setType( $listtype );
 				$refItem = $manager->createItem()->setType( $type );

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -118,7 +118,7 @@  discard block
 block discarded – undo
 		foreach( $listItems as $listItem )
 		{
 			if( ( $refItem = $listItem->getRefItem() ) !== null ) {
-				$listMap[ $refItem->getContent() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem;
+				$listMap[$refItem->getContent()][$refItem->getType()][$listItem->getType()] = $listItem;
 			}
 		}
 
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
 			{
 				$listItem = $listMap[$content][$type][$listtype];
 				$refItem = $listItem->getRefItem();
-				unset( $listItems[ $listItem->getId() ] );
+				unset( $listItems[$listItem->getId()] );
 			}
 			else
 			{

Please login to merge, or discard this patch.

controller/jobs/templates/product/export/items-body-standard.php 1 patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -23,7 +23,7 @@  discard block
 block discarded – undo
             <list>
                 <text>
                     <?php foreach( $item->getListItems( 'text' ) as $listItem ) : ?>
-                        <?php if ($refItem = $listItem->getRefItem()) { ?>
+                        <?php if( $refItem = $listItem->getRefItem() ) { ?>
 
                             <textitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.domain="<?php echo $enc->attr( $listItem->getDomain() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
                                 <type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
                 </text>
                 <media>
                     <?php foreach( $item->getListItems( 'media' ) as $listItem ) : ?>
-                        <?php if ($refItem = $listItem->getRefItem()) { ?>
+                        <?php if( $refItem = $listItem->getRefItem() ) { ?>
 
                             <mediaitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
                                 <type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
                 </media>
                 <price>
                     <?php foreach( $item->getListItems( 'price' ) as $listItem ) : ?>
-                        <?php if ($refItem = $listItem->getRefItem()) { ?>
+                        <?php if( $refItem = $listItem->getRefItem() ) { ?>
 
                             <priceitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
                                 <type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
                 </price>
                 <attribute>
                     <?php foreach( $item->getListItems( 'attribute' ) as $listItem ) : ?>
-                        <?php if ($refItem = $listItem->getRefItem()) { ?>
+                        <?php if( $refItem = $listItem->getRefItem() ) { ?>
 
                             <attributeitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
                                 <type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
                 </attribute>
                 <product>
                     <?php foreach( $item->getListItems( 'product' ) as $listItem ) : ?>
-                        <?php if ($refItem = $listItem->getRefItem()) { ?>
+                        <?php if( $refItem = $listItem->getRefItem() ) { ?>
 
                             <productitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
                                 <type><![CDATA[<?php echo $item->getType(); ?>]]></type>

Please login to merge, or discard this patch.

		@@ -91,8 +91,7 @@
		block discarded – undo
91	91	{
92	92	$item = $propMap[$value][$type];
93	93	unset( $items[ $item->getId() ] );
94		- }
95		- else
	94	+ } else
96	95	{
97	96	$item = $manager->createItem()->setType( $type );
98	97	}

		@@ -123,7 +123,7 @@ discard block
		block discarded – undo
123	123	foreach( $listItems as $listItem )
124	124	{
125	125	if( ( $refItem = $listItem->getRefItem() ) !== null ) {
126		- $listMap[ $refItem->getCode() ][ $listItem->getType() ] = $listItem;
	126	+ $listMap[$refItem->getCode()][$listItem->getType()] = $listItem;
127	127	}
128	128	}
129	129
		@@ -143,7 +143,7 @@ discard block
		block discarded – undo
143	143	if( isset( $listMap[$code][$listtype] ) )
144	144	{
145	145	$listItem = $listMap[$code][$listtype];
146		- unset( $listItems[ $listItem->getId() ] );
	146	+ unset( $listItems[$listItem->getId()] );
147	147	}
148	148	else
149	149	{

		@@ -87,8 +87,7 @@
		block discarded – undo
87	87	foreach( $manager->searchItems( $search ) as $item ) {
88	88	$this->listTypes[$item->getCode()] = $item->getCode();
89	89	}
90		- }
91		- else
	90	+ } else
92	91	{
93	92	$this->listTypes = array_flip( $this->listTypes );
94	93	}

		@@ -72,7 +72,7 @@ discard block
		block discarded – undo
72	72	$map = $this->getMappedChunk( $data, $this->getMapping() );
73	73
74	74	foreach( $items as $item ) {
75		- $propMap[ $item->getValue() ][ $item->getType() ] = $item;
	75	+ $propMap[$item->getValue()][$item->getType()] = $item;
76	76	}
77	77
78	78	foreach( $map as $list )
		@@ -90,7 +90,7 @@ discard block
		block discarded – undo
90	90	if( isset( $propMap[$value][$type] ) )
91	91	{
92	92	$item = $propMap[$value][$type];
93		- unset( $items[ $item->getId() ] );
	93	+ unset( $items[$item->getId()] );
94	94	}
95	95	else
96	96	{

		@@ -81,8 +81,7 @@ discard block
		block discarded – undo
81	81	foreach( $manager->searchItems( $search ) as $item ) {
82	82	$this->listTypes[$item->getCode()] = $item->getCode();
83	83	}
84		- }
85		- else
	84	+ } else
86	85	{
87	86	$this->listTypes = array_flip( $this->listTypes );
88	87	}
		@@ -143,8 +142,7 @@ discard block
		block discarded – undo
143	142	$listItem = $listMap[$url][$type][$listtype];
144	143	$refItem = $listItem->getRefItem();
145	144	unset( $listItems[ $listItem->getId() ] );
146		- }
147		- else
	145	+ } else
148	146	{
149	147	$listItem = $listManager->createItem()->setType( $listtype );
150	148	$refItem = $manager->createItem()->setType( $type );

		@@ -120,7 +120,7 @@ discard block
		block discarded – undo
120	120	foreach( $listItems as $listItem )
121	121	{
122	122	if( ( $refItem = $listItem->getRefItem() ) !== null ) {
123		- $listMap[ $refItem->getUrl() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem;
	123	+ $listMap[$refItem->getUrl()][$refItem->getType()][$listItem->getType()] = $listItem;
124	124	}
125	125	}
126	126
		@@ -142,7 +142,7 @@ discard block
		block discarded – undo
142	142	{
143	143	$listItem = $listMap[$url][$type][$listtype];
144	144	$refItem = $listItem->getRefItem();
145		- unset( $listItems[ $listItem->getId() ] );
	145	+ unset( $listItems[$listItem->getId()] );
146	146	}
147	147	else
148	148	{

		@@ -118,7 +118,7 @@ discard block
		block discarded – undo
118	118	foreach( $listItems as $listItem )
119	119	{
120	120	if( ( $refItem = $listItem->getRefItem() ) !== null ) {
121		- $listMap[ $refItem->getValue() ][ $refItem->getType() ][ $listItem->getType() ] = $listItem;
	121	+ $listMap[$refItem->getValue()][$refItem->getType()][$listItem->getType()] = $listItem;
122	122	}
123	123	}
124	124
		@@ -136,7 +136,7 @@ discard block
		block discarded – undo
136	136	{
137	137	$listItem = $listMap[$value][$type][$listtype];
138	138	$refItem = $listItem->getRefItem();
139		- unset( $listItems[ $listItem->getId() ] );
	139	+ unset( $listItems[$listItem->getId()] );
140	140	}
141	141	else
142	142	{

		@@ -82,8 +82,7 @@ discard block
		block discarded – undo
82	82	foreach( $manager->searchItems( $search ) as $item ) {
83	83	$this->listTypes[$item->getCode()] = $item->getCode();
84	84	}
85		- }
86		- else
	85	+ } else
87	86	{
88	87	$this->listTypes = array_flip( $this->listTypes );
89	88	}
		@@ -144,8 +143,7 @@ discard block
		block discarded – undo
144	143	{
145	144	$listItem = $listMap[$code][$listtype];
146	145	unset( $listItems[ $listItem->getId() ] );
147		- }
148		- else
	146	+ } else
149	147	{
150	148	$listItem = $listManager->createItem()->setType( $listtype );
151	149	}

		@@ -23,7 +23,7 @@ discard block
		block discarded – undo
23	23	<list>
24	24	<text>
25	25	<?php foreach( $item->getListItems( 'text' ) as $listItem ) : ?>
26		- <?php if ($refItem = $listItem->getRefItem()) { ?>
	26	+ <?php if( $refItem = $listItem->getRefItem() ) { ?>
27	27
28	28	<textitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.domain="<?php echo $enc->attr( $listItem->getDomain() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
29	29	<type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
		@@ -38,7 +38,7 @@ discard block
		block discarded – undo
38	38	</text>
39	39	<media>
40	40	<?php foreach( $item->getListItems( 'media' ) as $listItem ) : ?>
41		- <?php if ($refItem = $listItem->getRefItem()) { ?>
	41	+ <?php if( $refItem = $listItem->getRefItem() ) { ?>
42	42
43	43	<mediaitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
44	44	<type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
		@@ -55,7 +55,7 @@ discard block
		block discarded – undo
55	55	</media>
56	56	<price>
57	57	<?php foreach( $item->getListItems( 'price' ) as $listItem ) : ?>
58		- <?php if ($refItem = $listItem->getRefItem()) { ?>
	58	+ <?php if( $refItem = $listItem->getRefItem() ) { ?>
59	59
60	60	<priceitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
61	61	<type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
		@@ -74,7 +74,7 @@ discard block
		block discarded – undo
74	74	</price>
75	75	<attribute>
76	76	<?php foreach( $item->getListItems( 'attribute' ) as $listItem ) : ?>
77		- <?php if ($refItem = $listItem->getRefItem()) { ?>
	77	+ <?php if( $refItem = $listItem->getRefItem() ) { ?>
78	78
79	79	<attributeitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
80	80	<type><![CDATA[<?php echo $refItem->getType(); ?>]]></type>
		@@ -89,7 +89,7 @@ discard block
		block discarded – undo
89	89	</attribute>
90	90	<product>
91	91	<?php foreach( $item->getListItems( 'product' ) as $listItem ) : ?>
92		- <?php if ($refItem = $listItem->getRefItem()) { ?>
	92	+ <?php if( $refItem = $listItem->getRefItem() ) { ?>
93	93
94	94	<productitem id="<?php echo $enc->attr( $refItem->getId() ); ?>" siteid="<?php echo $enc->attr( $refItem->getSiteId() ); ?>" ctime="<?php echo $enc->attr( $refItem->getTimeCreated() ); ?>" mtime="<?php echo $enc->attr( $refItem->getTimeModified() ); ?>" editor="<?php echo $enc->attr( $refItem->getEditor() ); ?>" list.id="<?php echo $enc->attr( $listItem->getId() ); ?>" list.siteid="<?php echo $enc->attr( $listItem->getSiteId() ); ?>" list.parentid="<?php echo $enc->attr( $listItem->getParentId() ); ?>" list.type="<?php echo $enc->attr( $listItem->getType() ); ?>" list.config="<?php echo $enc->attr( json_encode( $listItem->getConfig() ) ); ?>" list.start="<?php echo $enc->attr( $listItem->getDateStart() ); ?>" list.end="<?php echo $enc->attr( $listItem->getDateEnd() ); ?>" list.position="<?php echo $enc->attr( $listItem->getPosition() ); ?>" list.status="<?php echo $enc->attr( $listItem->getStatus() ); ?>" list.ctime="<?php echo $enc->attr( $listItem->getTimeCreated() ); ?>" list.mtime="<?php echo $enc->attr( $listItem->getTimeModified() ); ?>" list.editor="<?php echo $enc->attr( $listItem->getEditor() ); ?>">
95	95	<type><![CDATA[<?php echo $item->getType(); ?>]]></type>

aimeos / ai-controller-jobs

Branch — master (c3c77b)

Status

Category

Braces +1 added lines, -2 removed lines patch added patch discarded remove patch

Braces +1 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +2 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch