aimeos / ai-admin-jqadm

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

Doc Comments -1 removed lines

@@ -27,7 +27,6 @@
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context object with required objects
 	 * @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
-	 * @param \Aimeos\Admin\JQAdm\Iface $client Admin object
 	 */
 	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths )
 	{

admin/jqadm/templates/common/partials/filter-default.php

Braces +5 added lines, -2 removed lines

@@ -36,8 +36,11 @@
 		<td>
 <?php	if( $pos < $cnt - 1 ) : ?>
 			<div class="fa fa-minus" aria-label="<?php echo $enc->attr( $this->translate( 'admin', 'Remove filter' ) ); ?>"></div>
-<?php	else : ?>
-			<div class="fa fa-plus" aria-label="<?php echo $enc->attr( $this->translate( 'admin', 'Add filter' ) ); ?>"></div>
+<?php	else {
+	: ?>
+			<div class="fa fa-plus" aria-label="<?php echo $enc->attr( $this->translate( 'admin', 'Add filter' ) );
+}
+?>"></div>
 <?php	endif; ?>
 		</td>
 		<td>

admin/jqadm/templates/product/item-stock-default.php

Braces +5 added lines, -2 removed lines

@@ -33,8 +33,11 @@
 <?php	foreach( $this->get( 'stockWarehouses', array() ) as $whid => $warehouse ) : ?>
 <?php		if( $whid == $this->get( 'stockData/product.stock.warehouseid/' . $idx ) ) : ?>
 							<option value="<?php echo $enc->attr( $whid ); ?>" selected="selected"><?php echo $enc->html( $warehouse->getLabel() ) ?></option>
-<?php		else : ?>
-							<option value="<?php echo $enc->attr( $whid ); ?>"><?php echo $enc->html( $warehouse->getLabel() ) ?></option>
+<?php		else {
+	: ?>
+							<option value="<?php echo $enc->attr( $whid );
+}
+?>"><?php echo $enc->html( $warehouse->getLabel() ) ?></option>
 <?php		endif; ?>
 <?php	endforeach; ?>
 						</select>

admin/jqadm/templates/product/item-price-default.php

Braces +5 added lines, -2 removed lines

@@ -42,8 +42,11 @@
 <?php	foreach( (array) $this->get( 'priceTypes', array() ) as $typeId => $typeItem ) : ?>
 <?php		if( $typeId == $this->get( 'priceData/price.typeid/' . $idx ) ) : ?>
 									<option value="<?php echo $enc->attr( $typeId ); ?>" selected="selected"><?php echo $enc->html( $typeItem->getLabel() ); ?></option>
-<?php		else : ?>
-									<option value="<?php echo $enc->attr( $typeId ); ?>"><?php echo $enc->html( $typeItem->getLabel() ); ?></option>
+<?php		else {
+	: ?>
+									<option value="<?php echo $enc->attr( $typeId );
+}
+?>"><?php echo $enc->html( $typeItem->getLabel() ); ?></option>
 <?php		endif; ?>
 <?php	endforeach; ?>
 								</select>

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

Braces +3 added lines, -6 removed lines

@@ -155,16 +155,14 @@ 
 			$textManager->commit();
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-price' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 
 			$textManager->rollback();
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
 			$error = array( 'product-item-price' => $e->getMessage() );
@@ -365,8 +363,7 @@ 
 
 				$item = $priceItem;
 				$item->setId( null );
-			}
-			else
+			} else
 			{
 				$litem = $listItems[$listid];
 				$item = $litem->getRefItem();

Doc Comments +3 added lines, -3 removed lines

@@ -61,7 +61,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()
 	{
@@ -84,7 +84,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()
 	{
@@ -107,7 +107,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/Download/Standard.php

Braces +3 added lines, -6 removed lines

@@ -196,16 +196,14 @@ 
 			$attrManager->commit();
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-download' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 
 			$attrManager->rollback();
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
 			$error = array( 'product-item-download' => $e->getMessage() );
@@ -482,8 +480,7 @@ 
 
 				$data['time'] = $fs->time( $data['path'] );
 				$data['size'] = $fs->size( $data['path'] );
-			}
-			catch( \Exception $e ) { ; }
+			} catch( \Exception $e ) { ; }
 		}
 
 		$view->downloadData = $data;

Doc Comments +3 added lines, -3 removed lines

@@ -61,7 +61,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()
 	{
@@ -84,7 +84,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()
 	{
@@ -148,7 +148,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/Text/Standard.php

Braces +3 added lines, -6 removed lines

@@ -157,16 +157,14 @@ 
 			$textManager->commit();
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-text' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 
 			$textManager->rollback();
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
 			$error = array( 'product-item-text' => $e->getMessage() );
@@ -401,8 +399,7 @@ 
 
 					$item = $textItem;
 					$item->setId( null );
-				}
-				else
+				} else
 				{
 					$litem = $listItems[$listid];
 					$item = $litem->getRefItem();

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/Selection/Standard.php

Braces +4 added lines, -8 removed lines

@@ -151,14 +151,12 @@ 
 
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-selection' => $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-selection' => $e->getMessage() );
@@ -391,8 +389,7 @@ 
 				$code = $refItem->getCode();
 				$data[$code]['product.lists.id'] = $listid;
 				$data[$code]['product.id'] = $listItem->getRefId();
-			}
-			else
+			} else
 			{
 				$code = $refItem->getCode() . '_copy';
 				$data[$code]['product.lists.id'] = '';
@@ -523,8 +520,7 @@ 
 
 				$item = $prodItem;
 				$item->setId( null );
-			}
-			else
+			} else
 			{
 				$litem = $listItems[$listid];
 				$item = $litem->getRefItem();

Doc Comments +3 added lines, -3 removed lines

@@ -61,7 +61,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()
 	{
@@ -84,7 +84,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()
 	{
@@ -107,7 +107,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/Image/Standard.php

Braces +5 added lines, -10 removed lines

@@ -155,16 +155,14 @@ 
 			$mediaManager->commit();
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-image' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 
 			$mediaManager->rollback();
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
 			$error = array( 'product-item-image' => $e->getMessage() );
@@ -451,21 +449,18 @@ 
 				if( $mediaId !== '' && isset( $mediaItems[$mediaId] ) )
 				{
 					$item = $mediaItems[$mediaId];
-				}
-				else if( ( $file = $view->value( $files, $num ) ) !== null )
+				} else if( ( $file = $view->value( $files, $num ) ) !== null )
 				{
 					$item = $mediaItem;
 					$item->setId( null );
 
 					$cntl->add( $item, $file );
 					$num++;
-				}
-				else
+				} else
 				{
 					throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'No file uploaded for %1$d. new image', $num+1 ) );
 				}
-			}
-			else
+			} else
 			{
 				$litem = $listItems[$listid];
 				$item = $litem->getRefItem();

Doc Comments +3 added lines, -3 removed lines

@@ -61,7 +61,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()
 	{
@@ -84,7 +84,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()
 	{
@@ -107,7 +107,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()
 	{