@@ -294,8 +294,7 @@ |
||
| 294 | 294 | $map[$item->getParentId()][$item->getPosition()] = $item; |
| 295 | 295 | |
| 296 | 296 | $subProducts = []; |
| 297 | - } |
|
| 298 | - else |
|
| 297 | + } else |
|
| 299 | 298 | { // in case it's a sub-product |
| 300 | 299 | $subProducts[$item->getPosition()] = $item; |
| 301 | 300 | } |
@@ -296,8 +296,7 @@ |
||
| 296 | 296 | } ); |
| 297 | 297 | |
| 298 | 298 | $service->setId( null )->setAttributeItems( $attrItems->setId( null ) ); |
| 299 | - } |
|
| 300 | - else |
|
| 299 | + } else |
|
| 301 | 300 | { |
| 302 | 301 | unset( $map[$type][$key] ); |
| 303 | 302 | } |
@@ -101,8 +101,7 @@ discard block |
||
| 101 | 101 | foreach( $columns as $name ) { |
| 102 | 102 | $names .= '"' . $name . '", '; $values .= '?, '; |
| 103 | 103 | } |
| 104 | - } |
|
| 105 | - else |
|
| 104 | + } else |
|
| 106 | 105 | { |
| 107 | 106 | foreach( $columns as $name ) { |
| 108 | 107 | $names .= '"' . $name . '" = ?, '; |
@@ -593,11 +592,9 @@ discard block |
||
| 593 | 592 | |
| 594 | 593 | if( isset( $attributes[$prefix] ) && $attributes[$prefix] instanceof $iface ) { |
| 595 | 594 | return $attributes[$prefix]->getInternalDeps(); |
| 596 | - } |
|
| 597 | - elseif( isset( $attributes[$name] ) && $attributes[$name] instanceof $iface ) { |
|
| 595 | + } elseif( isset( $attributes[$name] ) && $attributes[$name] instanceof $iface ) { |
|
| 598 | 596 | return $attributes[$name]->getInternalDeps(); |
| 599 | - } |
|
| 600 | - else if( isset( $attributes['id'] ) && $attributes['id'] instanceof $iface ) { |
|
| 597 | + } else if( isset( $attributes['id'] ) && $attributes['id'] instanceof $iface ) { |
|
| 601 | 598 | return $attributes['id']->getInternalDeps(); |
| 602 | 599 | } |
| 603 | 600 | |
@@ -823,8 +820,7 @@ discard block |
||
| 823 | 820 | */ |
| 824 | 821 | $path = $this->getConfigKey( 'insert' ); |
| 825 | 822 | $sql = $this->addSqlColumns( array_keys( $columns ), $this->getSqlConfig( $path ) ); |
| 826 | - } |
|
| 827 | - else |
|
| 823 | + } else |
|
| 828 | 824 | { |
| 829 | 825 | /** mshop/common/manager/update/mysql |
| 830 | 826 | * Updates an existing record in the database |
@@ -15,7 +15,7 @@ |
||
| 15 | 15 | { |
| 16 | 16 | $this->info( 'Creating test schema', 'vv' ); |
| 17 | 17 | |
| 18 | - $this->db( 'db-test' )->table( 'mshop_test', function( \Aimeos\Upscheme\Schema\Table $table ) { |
|
| 18 | + $this->db( 'db-test' )->table( 'mshop_test', function( \Aimeos\Upscheme\Schema\Table $table ) { |
|
| 19 | 19 | |
| 20 | 20 | $table->engine = 'InnoDB'; |
| 21 | 21 | |
@@ -301,8 +301,7 @@ |
||
| 301 | 301 | */ |
| 302 | 302 | $path = 'mshop/group/manager/insert'; |
| 303 | 303 | $sql = $this->addSqlColumns( array_keys( $columns ), $this->getSqlConfig( $path ) ); |
| 304 | - } |
|
| 305 | - else |
|
| 304 | + } else |
|
| 306 | 305 | { |
| 307 | 306 | /** mshop/group/manager/update/mysql |
| 308 | 307 | * Updates an existing group record in the database |
@@ -55,9 +55,12 @@ |
||
| 55 | 55 | |
| 56 | 56 | foreach( $testdata['group'] as $entry ) |
| 57 | 57 | { |
| 58 | - try { |
|
| 58 | + try |
|
| 59 | + { |
|
| 59 | 60 | $manager->save( $manager->find( $entry['group.code'] )->fromArray( $entry ) ); |
| 60 | - } catch( \Exception $e ) { |
|
| 61 | + } |
|
| 62 | + catch( \Exception $e ) |
|
| 63 | + { |
|
| 61 | 64 | $manager->save( $manager->create()->fromArray( $entry ), false ); |
| 62 | 65 | } |
| 63 | 66 | } |
@@ -620,9 +620,13 @@ discard block |
||
| 620 | 620 | |
| 621 | 621 | if( empty( $item->getInvoiceNumber() ) && $item->getStatusPayment() >= \Aimeos\MShop\Order\Item\Base::PAY_PENDING ) |
| 622 | 622 | { |
| 623 | - try { |
|
| 623 | + try |
|
| 624 | + { |
|
| 624 | 625 | $item->setInvoiceNumber( $this->createInvoiceNumber( $item ) ); |
| 625 | - } catch( \Exception $e ) { // redo on transaction deadlock |
|
| 626 | + } |
|
| 627 | + catch( \Exception $e ) |
|
| 628 | + { |
|
| 629 | +// redo on transaction deadlock |
|
| 626 | 630 | $item->setInvoiceNumber( $this->createInvoiceNumber( $item ) ); |
| 627 | 631 | } |
| 628 | 632 | } |
@@ -673,8 +677,7 @@ discard block |
||
| 673 | 677 | */ |
| 674 | 678 | $path = 'mshop/order/manager/insert'; |
| 675 | 679 | $sql = $this->addSqlColumns( array_keys( $columns ), $this->getSqlConfig( $path ) ); |
| 676 | - } |
|
| 677 | - else |
|
| 680 | + } else |
|
| 678 | 681 | { |
| 679 | 682 | /** mshop/order/manager/update/mysql |
| 680 | 683 | * Updates an existing order record in the database |
@@ -344,7 +344,8 @@ |
||
| 344 | 344 | { |
| 345 | 345 | $stockManager->decrease( [$item->getProductId() => -1 * $how * $item->getQuantity()], $item->getStockType() ); |
| 346 | 346 | |
| 347 | - switch( $item->getType() ) { |
|
| 347 | + switch( $item->getType() ) |
|
| 348 | + { |
|
| 348 | 349 | case 'default': |
| 349 | 350 | $this->updateStockBundle( $item->getParentProductId(), $item->getStockType() ); break; |
| 350 | 351 | case 'select': |