core-2275 fixed checkout and login

spryker-shop · Jan 30, 2018 · f6e9a09 · f6e9a09
1 parent 971932a
commit f6e9a09
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 8 deletions.
diff --git a/src/SprykerShop/Yves/CheckoutPage/Form/DataProvider/ShipmentFormDataProvider.php b/src/SprykerShop/Yves/CheckoutPage/Form/DataProvider/ShipmentFormDataProvider.php
@@ -101,9 +101,10 @@ protected function createAvailableShipmentChoiceList(QuoteTransfer $quoteTransfe
             if (!isset($shipmentMethods[$shipmentMethodTransfer->getCarrierName()])) {
                 $shipmentMethods[$shipmentMethodTransfer->getCarrierName()] = [];
             }
-            $shipmentMethods[$shipmentMethodTransfer->getCarrierName()][$shipmentMethodTransfer->getIdShipmentMethod()] = $this->getShipmentDescription(
+            $description = $this->getShipmentDescription(
                 $shipmentMethodTransfer
             );
+            $shipmentMethods[$shipmentMethodTransfer->getCarrierName()][$description] = $shipmentMethodTransfer->getIdShipmentMethod();
         }
 
         return $shipmentMethods;

diff --git a/src/SprykerShop/Yves/CheckoutPage/Form/Steps/PaymentForm.php b/src/SprykerShop/Yves/CheckoutPage/Form/Steps/PaymentForm.php
@@ -77,8 +77,7 @@ protected function addPaymentMethodChoices(FormBuilderInterface $builder, array
             self::PAYMENT_SELECTION,
             ChoiceType::class,
             [
-                'choices' => array_flip($paymentMethodChoices),
-                'choices_as_values' => true,
+                'choices' => $paymentMethodChoices,
                 'label' => false,
                 'required' => true,
                 'expanded' => true,
@@ -145,15 +144,15 @@ protected function getPaymentMethodChoices(array $paymentMethodSubForms)
             $subFormName = ucfirst($paymentMethodSubForm->getName());
 
             if (!$paymentMethodSubForm instanceof SubFormProviderNameInterface) {
-                $choices[$paymentMethodSubForm->getPropertyPath()] = $subFormName;
+                $choices[$subFormName] = $paymentMethodSubForm->getPropertyPath();
                 continue;
             }
 
             if (!isset($choices[$paymentMethodSubForm->getProviderName()])) {
                 $choices[$paymentMethodSubForm->getProviderName()] = [];
             }
 
-            $choices[$paymentMethodSubForm->getProviderName()][$paymentMethodSubForm->getPropertyPath()] = $subFormName;
+            $choices[$paymentMethodSubForm->getProviderName()][$subFormName] = $paymentMethodSubForm->getPropertyPath();
         }
 
         return $choices;

diff --git a/src/SprykerShop/Yves/CheckoutPage/Form/Steps/ShipmentForm.php b/src/SprykerShop/Yves/CheckoutPage/Form/Steps/ShipmentForm.php
@@ -60,8 +60,7 @@ public function buildForm(FormBuilderInterface $builder, array $options)
     protected function addShipmentMethods(FormBuilderInterface $builder, array $options)
     {
         $builder->add(self::FIELD_ID_SHIPMENT_METHOD, ChoiceType::class, [
-            'choices' => array_flip($options[self::OPTION_SHIPMENT_METHODS]),
-            'choices_as_values' => true,
+            'choices' => $options[self::OPTION_SHIPMENT_METHODS],
             'expanded' => true,
             'multiple' => false,
             'required' => true,

diff --git a/src/SprykerShop/Yves/CheckoutPage/Theme/default/checkout/payment.twig b/src/SprykerShop/Yves/CheckoutPage/Theme/default/checkout/payment.twig
@@ -21,7 +21,7 @@
                                 {{ form_label(_view.paymentForm.paymentSelection[key]) }}
                                 <div class="__toggler-target">
                                     <div class="row columns">
-                                        {{ checkout.include_subform(_view.paymentForm[_view.paymentForm.paymentSelection[key].vars.value]) }}
+                                    {{ checkout.include_subform(_view.paymentForm[_view.paymentForm.paymentSelection[key].vars.value]) }}
                                     </div>
                                 </div>
                             </li>