Fix PHP 8.0 deprecations - required parameter follows optional parameter

src/Sylius/Behat/Context/Setup/CustomerContext.php

@@ -94,7 +94,7 @@ public function theStoreHasCustomerWithFirstName($email, $firstName)
      * @Given the store has customer :email with name :fullName since :since
      * @Given the store has customer :email with name :fullName and phone number :phoneNumber since :since
      */
-    public function theStoreHasCustomerWithNameAndRegistrationDate($email, $fullName, $phoneNumber = null, $since)
+    public function theStoreHasCustomerWithNameAndRegistrationDate($email, $fullName, $since, $phoneNumber = null)
     {
         $names = explode(' ', $fullName);
         $customer = $this->createCustomer($email, $names[0], $names[1], new \DateTime($since), $phoneNumber);

src/Sylius/Behat/Context/Setup/OrderContext.php

@@ -641,9 +641,9 @@ public function customersHaveAddedProductsToTheCartForTotalOf($numberOfCustomers
      * @Given then :numberOfCustomers more customers have placed :numberOfOrders orders for total of :total
      */
     public function customersHavePlacedOrdersForTotalOf(
+        string $total,
         int $numberOfCustomers = 1,
-        int $numberOfOrders = 1,
-        string $total
+        int $numberOfOrders = 1
     ): void {
         $this->createOrders($numberOfCustomers, $numberOfOrders, $total);
     }

src/Sylius/Behat/Context/Ui/Admin/ManagingPaymentMethodsContext.php

@@ -72,7 +72,7 @@ public function iWantToModifyAPaymentMethod(PaymentMethodInterface $paymentMetho
      * @When I rename it to :name in :language
      * @When I remove its name from :language translation
      */
-    public function iNameItIn($name = null, $language)
+    public function iNameItIn(string $language, ?string $name = null): void
     {
         /** @var CreatePageInterface|UpdatePageInterface $currentPage */
         $currentPage = $this->currentPageResolver->getCurrentPageWithForm([$this->createPage, $this->updatePage]);

src/Sylius/Behat/Context/Ui/Admin/ManagingProductAssociationTypesContext.php

@@ -92,7 +92,7 @@ public function iDoNotNameIt()
      * @When I rename it to :name in :language
      * @When I remove its name from :language translation
      */
-    public function iRenameItToInLanguage($name = null, $language)
+    public function iRenameItToInLanguage(string $language, ?string $name = null): void
     {
         $this->updatePage->nameItIn($name ?? '', $language);
     }

src/Sylius/Behat/Context/Ui/Admin/ManagingProductOptionsContext.php

@@ -101,7 +101,7 @@ public function iNameItInLanguage($name, $language)
      * @When I rename it to :name in :language
      * @When I remove its name from :language translation
      */
-    public function iRenameItToInLanguage($name = null, $language)
+    public function iRenameItToInLanguage(string $language, ?string $name = null): void
     {
         $this->updatePage->nameItIn($name ?? '', $language);
     }

src/Sylius/Bundle/OrderBundle/Form/Type/CartItemType.php

@@ -25,7 +25,7 @@ class CartItemType extends AbstractResourceType
 
     public function __construct(
         string $dataClass,
-        array $validationGroups = [],
+        array $validationGroups,
         DataMapperInterface $dataMapper
     ) {
         parent::__construct($dataClass, $validationGroups);

src/Sylius/Bundle/OrderBundle/Form/Type/OrderItemType.php

@@ -25,7 +25,7 @@ final class OrderItemType extends AbstractResourceType
 
     public function __construct(
         string $dataClass,
-        array $validationGroups = [],
+        array $validationGroups,
         DataMapperInterface $dataMapper
     ) {
         parent::__construct($dataClass, $validationGroups);

src/Sylius/Bundle/PayumBundle/Form/Type/GatewayConfigType.php

@@ -28,7 +28,7 @@ final class GatewayConfigType extends AbstractResourceType
 
     public function __construct(
         string $dataClass,
-        array $validationGroups = [],
+        array $validationGroups,
         FormTypeRegistryInterface $gatewayConfigurationTypeRegistry
     ) {
         parent::__construct($dataClass, $validationGroups);

src/Sylius/Bundle/PromotionBundle/Form/Type/AbstractConfigurablePromotionElementType.php

@@ -27,7 +27,7 @@ abstract class AbstractConfigurablePromotionElementType extends AbstractResource
     /** @var FormTypeRegistryInterface */
     private $formTypeRegistry;
 
-    public function __construct(string $dataClass, array $validationGroups = [], FormTypeRegistryInterface $formTypeRegistry)
+    public function __construct(string $dataClass, array $validationGroups, FormTypeRegistryInterface $formTypeRegistry)
     {
         parent::__construct($dataClass, $validationGroups);
 

src/Sylius/Bundle/ShippingBundle/Form/Type/AbstractConfigurableShippingMethodElementType.php

@@ -27,7 +27,7 @@ abstract class AbstractConfigurableShippingMethodElementType extends AbstractRes
     /** @var FormTypeRegistryInterface */
     private $formTypeRegistry;
 
-    public function __construct(string $dataClass, array $validationGroups = [], FormTypeRegistryInterface $formTypeRegistry)
+    public function __construct(string $dataClass, array $validationGroups, FormTypeRegistryInterface $formTypeRegistry)
     {
         parent::__construct($dataClass, $validationGroups);