@@ -268,14 +268,12 @@ discard block |
||
| 268 | 268 | $orderItem = $orderCntl->add( $basket->getId(), ['order.type' => 'web'] )->store(); |
| 269 | 269 | |
| 270 | 270 | $context->getSession()->set( 'aimeos/orderid', $orderItem->getId() ); |
| 271 | - } |
|
| 272 | - elseif( $view->param( 'cp_payment', null ) !== null ) |
|
| 271 | + } elseif( $view->param( 'cp_payment', null ) !== null ) |
|
| 273 | 272 | { |
| 274 | 273 | $parts = \Aimeos\MShop\Order\Item\Base\Base::PARTS_ALL; |
| 275 | 274 | $orderItem = $orderCntl->get( $context->getSession()->get( 'aimeos/orderid' ), false ); |
| 276 | 275 | $basket = $basketCntl->load( $orderItem->getBaseId(), $parts, false ); |
| 277 | - } |
|
| 278 | - else |
|
| 276 | + } else |
|
| 279 | 277 | { |
| 280 | 278 | return; |
| 281 | 279 | } |
@@ -285,8 +283,7 @@ discard block |
||
| 285 | 283 | $orderCntl->save( $orderItem->setPaymentStatus( \Aimeos\MShop\Order\Item\Base::PAY_AUTHORIZED ) ); |
| 286 | 284 | $view->standardUrlNext = $this->getUrlConfirm( $view, [], [] ); |
| 287 | 285 | $view->standardMethod = 'POST'; |
| 288 | - } |
|
| 289 | - else // no payment service available |
|
| 286 | + } else // no payment service available |
|
| 290 | 287 | { |
| 291 | 288 | $view = $this->addFormData( $view, $form ); |
| 292 | 289 | } |
@@ -83,8 +83,11 @@ |
||
| 83 | 83 | $this->config( 'client/html/checkout/standard/summary/address', 'common/summary/address-standard' ), |
| 84 | 84 | ['addresses' => $addresses, 'type' => 'delivery'] |
| 85 | 85 | ); ?> |
| 86 | - <?php else : ?> |
|
| 87 | - <?= $enc->html( $this->translate( 'client', 'like billing address' ), $enc::TRUST ); ?> |
|
| 86 | + <?php else { |
|
| 87 | + : ?> |
|
| 88 | + <?= $enc->html( $this->translate( 'client', 'like billing address' ), $enc::TRUST ); |
|
| 89 | +} |
|
| 90 | +?> |
|
| 88 | 91 | <?php endif; ?> |
| 89 | 92 | </div> |
| 90 | 93 | </div> |
@@ -41,14 +41,17 @@ |
||
| 41 | 41 | $this->translate( 'currency', $price->getCurrencyId() ) |
| 42 | 42 | ) ); ?> |
| 43 | 43 | </span> |
| 44 | - <?php else : ?> |
|
| 44 | + <?php else { |
|
| 45 | + : ?> |
|
| 45 | 46 | <span class="price-value"> |
| 46 | 47 | <?= $enc->html( sprintf( |
| 47 | 48 | /// Price format with price value (%1$s) and currency (%2$s) |
| 48 | 49 | $this->translate( 'client/code', 'price:default', null, 0, false ) ?: $this->translate( 'client', '%1$s %2$s' ), |
| 49 | 50 | $this->number( $price->getCosts() > 0 ? $price->getCosts() : 0, $price->getPrecision() ), |
| 50 | 51 | $this->translate( 'currency', $price->getCurrencyId() ) |
| 51 | - ) ); ?> |
|
| 52 | + ) ); |
|
| 53 | +} |
|
| 54 | +?> |
|
| 52 | 55 | </span> |
| 53 | 56 | <?php endif; ?> |
| 54 | 57 | |
@@ -219,9 +219,12 @@ |
||
| 219 | 219 | $context = $this->getContext(); |
| 220 | 220 | $cntl = \Aimeos\Controller\Frontend::create( $context, 'customer' ); |
| 221 | 221 | |
| 222 | - try { |
|
| 222 | + try |
|
| 223 | + { |
|
| 223 | 224 | $id = $cntl->find( $addr->getEmail() )->getId(); |
| 224 | - } catch( \Exception $e ) { |
|
| 225 | + } |
|
| 226 | + catch( \Exception $e ) |
|
| 227 | + { |
|
| 225 | 228 | $id = $new ? $cntl->add( $addr->toArray() )->store()->get()->getId() : null; |
| 226 | 229 | } |
| 227 | 230 | |
@@ -645,15 +645,16 @@ discard block |
||
| 645 | 645 | $disable = $view->config( 'client/html/checkout/standard/address/billing/disable-new', false ); |
| 646 | 646 | $type = \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT; |
| 647 | 647 | |
| 648 | - if( ( $option = $view->param( 'ca_billingoption', 'null' ) ) === 'null' && $disable === false ) // new address |
|
| 648 | + if( ( $option = $view->param( 'ca_billingoption', 'null' ) ) === 'null' && $disable === false ) { |
|
| 649 | + // new address |
|
| 649 | 650 | { |
| 650 | 651 | $params = $view->param( 'ca_billing', [] ); |
| 652 | + } |
|
| 651 | 653 | |
| 652 | 654 | if( ( $view->addressBillingError = $this->checkFields( $params ) ) !== [] ) { |
| 653 | 655 | throw new \Aimeos\Client\Html\Exception( sprintf( 'At least one billing address part is missing or invalid' ) ); |
| 654 | 656 | } |
| 655 | - } |
|
| 656 | - else // existing address |
|
| 657 | + } else // existing address |
|
| 657 | 658 | { |
| 658 | 659 | $params = $view->param( 'ca_billing_' . $option, [] ) + $view->param( 'ca_extra', [] ); |
| 659 | 660 | |
@@ -797,8 +798,7 @@ discard block |
||
| 797 | 798 | $invalid[$key] = $name; |
| 798 | 799 | unset( $params[$key] ); |
| 799 | 800 | } |
| 800 | - } |
|
| 801 | - else |
|
| 801 | + } else |
|
| 802 | 802 | { |
| 803 | 803 | unset( $params[$key] ); |
| 804 | 804 | } |
@@ -524,19 +524,22 @@ discard block |
||
| 524 | 524 | $disable = $view->config( 'client/html/checkout/standard/address/delivery/disable-new', false ); |
| 525 | 525 | $type = \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_DELIVERY; |
| 526 | 526 | |
| 527 | - if( ( $option = $view->param( 'ca_deliveryoption', 'null' ) ) === 'null' && $disable === false ) // new address |
|
| 527 | + if( ( $option = $view->param( 'ca_deliveryoption', 'null' ) ) === 'null' && $disable === false ) { |
|
| 528 | + // new address |
|
| 528 | 529 | { |
| 529 | 530 | $params = $view->param( 'ca_delivery', [] ); |
| 531 | + } |
|
| 530 | 532 | |
| 531 | 533 | if( ( $view->addressDeliveryError = $this->checkFields( $params ) ) !== [] ) { |
| 532 | 534 | throw new \Aimeos\Client\Html\Exception( sprintf( 'At least one delivery address part is missing or invalid' ) ); |
| 533 | 535 | } |
| 534 | 536 | |
| 535 | 537 | $ctrl->addAddress( $type, $params, 0 ); |
| 536 | - } |
|
| 537 | - else if( ( $option = $view->param( 'ca_deliveryoption' ) ) !== 'like' ) // existing address |
|
| 538 | + } else if( ( $option = $view->param( 'ca_deliveryoption' ) ) !== 'like' ) { |
|
| 539 | + // existing address |
|
| 538 | 540 | { |
| 539 | 541 | $params = $view->param( 'ca_delivery_' . $option, [] ); |
| 542 | + } |
|
| 540 | 543 | |
| 541 | 544 | if( !empty( $params ) && ( $view->addressDeliveryError = $this->checkFields( $params ) ) !== [] ) { |
| 542 | 545 | throw new \Aimeos\Client\Html\Exception( sprintf( 'At least one delivery address part is missing or invalid' ) ); |
@@ -549,13 +552,11 @@ discard block |
||
| 549 | 552 | $params = array_replace( $address->toArray(), $params + ['order.base.address.addressid' => $option] ); |
| 550 | 553 | $addr = $ctrl->addAddress( $type, $params, 0 )->get()->getAddress( $type, 0 ); // sanitize address first |
| 551 | 554 | $custCntl->addAddressItem( $address->copyFrom( $addr ), $option )->store(); // update existing address |
| 552 | - } |
|
| 553 | - else |
|
| 555 | + } else |
|
| 554 | 556 | { |
| 555 | 557 | $ctrl->addAddress( $type, $params, 0 ); |
| 556 | 558 | } |
| 557 | - } |
|
| 558 | - else |
|
| 559 | + } else |
|
| 559 | 560 | { |
| 560 | 561 | $ctrl->deleteAddress( $type ); |
| 561 | 562 | } |
@@ -787,8 +788,7 @@ discard block |
||
| 787 | 788 | $invalid[$key] = $name; |
| 788 | 789 | unset( $params[$key] ); |
| 789 | 790 | } |
| 790 | - } |
|
| 791 | - else |
|
| 791 | + } else |
|
| 792 | 792 | { |
| 793 | 793 | unset( $params[$key] ); |
| 794 | 794 | } |
@@ -129,8 +129,11 @@ |
||
| 129 | 129 | <span class="value col-7"> |
| 130 | 130 | <?php if( $interval = $this->get( 'listsIntervalItems', map() )->get( $item->getInterval() ) ) : ?> |
| 131 | 131 | <?= $enc->html( $interval->getName(), $enc::TRUST ); ?> |
| 132 | - <?php else : ?> |
|
| 133 | - <?= $enc->html( $item->getInterval(), $enc::TRUST ); ?> |
|
| 132 | + <?php else { |
|
| 133 | + : ?> |
|
| 134 | + <?= $enc->html( $item->getInterval(), $enc::TRUST ); |
|
| 135 | +} |
|
| 136 | +?> |
|
| 134 | 137 | <?php endif; ?> |
| 135 | 138 | </span> |
| 136 | 139 | </div> |
@@ -57,9 +57,12 @@ discard block |
||
| 57 | 57 | <p class="order-notice"> |
| 58 | 58 | <?= $enc->html( $this->translate( 'client', 'Please enter your payment details' ), $enc::TRUST ); ?> |
| 59 | 59 | </p> |
| 60 | - <?php else : ?> |
|
| 60 | + <?php else { |
|
| 61 | + : ?> |
|
| 61 | 62 | <p class="order-notice"> |
| 62 | - <?= $enc->html( $this->translate( 'client', 'You will now be forwarded to the next step' ), $enc::TRUST ); ?> |
|
| 63 | + <?= $enc->html( $this->translate( 'client', 'You will now be forwarded to the next step' ), $enc::TRUST ); |
|
| 64 | +} |
|
| 65 | +?> |
|
| 63 | 66 | </p> |
| 64 | 67 | <?php endif; ?> |
| 65 | 68 | |
@@ -72,8 +75,11 @@ discard block |
||
| 72 | 75 | <input type="hidden" id="process-<?= $id; ?>" value="<?= $enc->attr( $value ); ?>" |
| 73 | 76 | name="<?= $enc->attr( $this->formparam( [$item->getInternalCode(), $key], $prefix ) ); ?>" |
| 74 | 77 | /> |
| 75 | - <?php else : ?> |
|
| 76 | - <input type="hidden" id="process-<?= $id; ?>" value="<?= $enc->attr( $value ); ?>" |
|
| 78 | + <?php else { |
|
| 79 | + : ?> |
|
| 80 | + <input type="hidden" id="process-<?= $id; |
|
| 81 | +} |
|
| 82 | +?>" value="<?= $enc->attr( $value ); ?>" |
|
| 77 | 83 | name="<?= $enc->attr( $this->formparam( $item->getInternalCode(), $prefix ) ); ?>" |
| 78 | 84 | /> |
| 79 | 85 | <?php endif; ?> |
@@ -161,10 +167,13 @@ discard block |
||
| 161 | 167 | <?= $enc->html( $this->translate( 'client', 'Proceed' ), $enc::TRUST ); ?> |
| 162 | 168 | </a> |
| 163 | 169 | |
| 164 | - <?php else : ?> |
|
| 170 | + <?php else { |
|
| 171 | + : ?> |
|
| 165 | 172 | |
| 166 | 173 | <button class="btn btn-primary btn-lg btn-action"> |
| 167 | - <?= $enc->html( $this->translate( 'client', 'Proceed' ), $enc::TRUST ); ?> |
|
| 174 | + <?= $enc->html( $this->translate( 'client', 'Proceed' ), $enc::TRUST ); |
|
| 175 | +} |
|
| 176 | +?> |
|
| 168 | 177 | </button> |
| 169 | 178 | |
| 170 | 179 | <?php endif; ?> |
@@ -319,13 +319,11 @@ |
||
| 319 | 319 | { |
| 320 | 320 | $item = $item->set( 'checked', true ); |
| 321 | 321 | unset( $attrparams['f_attrid'][$key] ); |
| 322 | - } |
|
| 323 | - elseif( ( $key = array_search( $id, $oneIds ) ) !== false ) |
|
| 322 | + } elseif( ( $key = array_search( $id, $oneIds ) ) !== false ) |
|
| 324 | 323 | { |
| 325 | 324 | $item = $item->set( 'checked', true ); |
| 326 | 325 | unset( $attrparams['f_oneid'][$key] ); |
| 327 | - } |
|
| 328 | - elseif( ( $key = array_search( $id, $optIds ) ) !== false ) |
|
| 326 | + } elseif( ( $key = array_search( $id, $optIds ) ) !== false ) |
|
| 329 | 327 | { |
| 330 | 328 | $item = $item->set( 'checked', true ); |
| 331 | 329 | unset( $attrparams['f_optid'][$key] ); |