Merge branch '2.4-develop' into mftf-cache-flush-command-improvements

# Conflicts: # app/code/Magento/LoginAsCustomer/Test/Mftf/Test/AdminLoginAsCustomerMultishippingLoggingTest.xml
magento · Dec 14, 2020 · 551dd3f · 551dd3f
2 parents dc02c0a + ad29452
commit 551dd3f
Show file tree

Hide file tree

Showing 182 changed files with 9,140 additions and 3,341 deletions.
diff --git a/app/code/Magento/Backend/Test/Mftf/Test/AdminExpireCustomerSessionTest.xml b/app/code/Magento/Backend/Test/Mftf/Test/AdminExpireCustomerSessionTest.xml
@@ -22,6 +22,8 @@
         <after>
             <!-- 6. Restore default configuration settings. -->
             <magentoCLI command="config:set {{DefaultWebCookieLifetimeConfigData.path}} {{DefaultWebCookieLifetimeConfigData.value}}" stepKey="setDefaultCookieLifetime"/>
+            <!-- Customer Log Out -->
+            <actionGroup ref="StorefrontCustomerLogoutActionGroup" stepKey="customerLogout"/>
             <!-- Delete data -->
             <deleteData createDataKey="createCustomer" stepKey="deleteCustomer"/>
             <actionGroup ref="AdminLogoutActionGroup" stepKey="logout"/>

diff --git a/app/code/Magento/Catalog/Block/Adminhtml/Product/Helper/Form/Gallery/Content.php b/app/code/Magento/Catalog/Block/Adminhtml/Product/Helper/Form/Gallery/Content.php
@@ -209,15 +209,22 @@ public function getImagesJson()
      */
     private function sortImagesByPosition($images)
     {
-        if (is_array($images)) {
+        $nullPositions = [];
+        foreach ($images as $index => $image) {
+            if ($image['position'] === null) {
+                $nullPositions[] = $image;
+                unset($images[$index]);
+            }
+        }
+        if (is_array($images) && !empty($images)) {
             usort(
                 $images,
                 function ($imageA, $imageB) {
                     return ($imageA['position'] < $imageB['position']) ? -1 : 1;
                 }
             );
         }
-        return $images;
+        return array_merge($images, $nullPositions);
     }
 
     /**

diff --git a/app/code/Magento/Catalog/Block/Product/View/Options/Type/Date.php b/app/code/Magento/Catalog/Block/Product/View/Options/Type/Date.php
@@ -5,6 +5,11 @@
  */
 namespace Magento\Catalog\Block\Product\View\Options\Type;
 
+use DateTimeZone;
+use Magento\Framework\App\ObjectManager;
+use Magento\Framework\Data\Form\FilterFactory;
+use Magento\Framework\Stdlib\DateTime;
+
 /**
  * Product options text type block
  *
@@ -27,22 +32,30 @@ class Date extends \Magento\Catalog\Block\Product\View\Options\AbstractOptions
      */
     protected $_catalogProductOptionTypeDate;
 
+    /**
+     * @var FilterFactory
+     */
+    private $filterFactory;
+
     /**
      * @param \Magento\Framework\View\Element\Template\Context $context
      * @param \Magento\Framework\Pricing\Helper\Data $pricingHelper
      * @param \Magento\Catalog\Helper\Data $catalogData
      * @param \Magento\Catalog\Model\Product\Option\Type\Date $catalogProductOptionTypeDate
      * @param array $data
+     * @param FilterFactory|null $filterFactory
      */
     public function __construct(
         \Magento\Framework\View\Element\Template\Context $context,
         \Magento\Framework\Pricing\Helper\Data $pricingHelper,
         \Magento\Catalog\Helper\Data $catalogData,
         \Magento\Catalog\Model\Product\Option\Type\Date $catalogProductOptionTypeDate,
-        array $data = []
+        array $data = [],
+        ?FilterFactory $filterFactory = null
     ) {
         $this->_catalogProductOptionTypeDate = $catalogProductOptionTypeDate;
         parent::__construct($context, $pricingHelper, $catalogData, $data);
+        $this->filterFactory = $filterFactory ?? ObjectManager::getInstance()->get(FilterFactory::class);
     }
 
     /**
@@ -77,14 +90,24 @@ public function getDateHtml()
     public function getCalendarDateHtml()
     {
         $option = $this->getOption();
-        $value = $this->getProduct()->getPreconfiguredValues()->getData('options/' . $option->getId() . '/date');
+        $values = $this->getProduct()->getPreconfiguredValues()->getData('options/' . $option->getId());
 
         $yearStart = $this->_catalogProductOptionTypeDate->getYearStart();
         $yearEnd = $this->_catalogProductOptionTypeDate->getYearEnd();
 
-        $dateFormat = $this->_localeDate->getDateFormat(\IntlDateFormatter::SHORT);
+        $dateFormat = $this->_localeDate->getDateFormatWithLongYear();
         /** Escape RTL characters which are present in some locales and corrupt formatting */
         $escapedDateFormat = preg_replace('/[^MmDdYy\/\.\-]/', '', $dateFormat);
+        $value = null;
+        if (is_array($values)) {
+            $date = $this->getInternalDateString($values);
+            if ($date !== null) {
+                $dateFilter = $this->filterFactory->create('date', ['format' => $escapedDateFormat]);
+                $value = $dateFilter->outputFilter($date);
+            } elseif (isset($values['date'])) {
+                $value = $values['date'];
+            }
+        }
         $calendar = $this->getLayout()->createBlock(
             \Magento\Framework\View\Element\Html\Date::class
         )->setId(
@@ -158,8 +181,8 @@ public function getTimeHtml()
      * Return drop-down html with range of values
      *
      * @param string $name Id/name of html select element
-     * @param int $from  Start position
-     * @param int $to    End position
+     * @param int $from Start position
+     * @param int $to End position
      * @param int|null $value Value selected
      * @return string Formatted Html
      */
@@ -209,9 +232,8 @@ protected function _getHtmlSelect($name, $value = null)
 
         $select->setExtraParams($extraParams);
         if ($value === null) {
-            $value = $this->getProduct()->getPreconfiguredValues()->getData(
-                'options/' . $option->getId() . '/' . $name
-            );
+            $values = $this->getProduct()->getPreconfiguredValues()->getData('options/' . $option->getId());
+            $value = is_array($values) ? $this->parseDate($values, $name) : null;
         }
         if ($value !== null) {
             $select->setValue($value);
@@ -233,4 +255,56 @@ protected function _getValueWithLeadingZeros($value)
         }
         return $value < 10 ? '0' . $value : $value;
     }
+
+    /**
+     * Get internal date format of provided value
+     *
+     * @param array $value
+     * @return string|null
+     */
+    private function getInternalDateString(array $value): ?string
+    {
+        $result = null;
+        if (!empty($value['date']) && !empty($value['date_internal'])) {
+            $dateTimeZone = new DateTimeZone($this->_localeDate->getConfigTimezone());
+            $dateTimeObject = date_create_from_format(
+                DateTime::DATETIME_PHP_FORMAT,
+                $value['date_internal'],
+                $dateTimeZone
+            );
+            if ($dateTimeObject !== false) {
+                $result = $dateTimeObject->format(DateTime::DATE_PHP_FORMAT);
+            }
+        } elseif (!empty($value['day']) && !empty($value['month']) && !empty($value['year'])) {
+            $dateTimeObject = $this->_localeDate->date();
+            $dateTimeObject->setDate((int) $value['year'], (int) $value['month'], (int) $value['day']);
+            $result = $dateTimeObject->format(DateTime::DATE_PHP_FORMAT);
+        }
+        return $result;
+    }
+
+    /**
+     * Parse option value and return the requested part
+     *
+     * @param array $value
+     * @param string $part [year, month, day, hour, minute, day_part]
+     * @return string|null
+     */
+    private function parseDate(array $value, string $part): ?string
+    {
+        $result = null;
+        if (!empty($value['date']) && !empty($value['date_internal'])) {
+            $formatDate = explode(' ', $value['date_internal']);
+            $date = explode('-', $formatDate[0]);
+            $value['year'] = $date[0];
+            $value['month'] = $date[1];
+            $value['day'] = $date[2];
+        }
+
+        if (isset($value[$part])) {
+            $result = (string) $value[$part];
+        }
+
+        return $result;
+    }
 }
diff --git a/app/code/Magento/Catalog/Model/Product/Gallery/CreateHandler.php b/app/code/Magento/Catalog/Model/Product/Gallery/CreateHandler.php
@@ -267,29 +267,50 @@ protected function processNewAndExistingImages($product, array &$images)
     {
         foreach ($images as &$image) {
             if (empty($image['removed'])) {
+                $isNew = empty($image['value_id']);
                 $data = $this->processNewImage($product, $image);
 
-                if (!$product->isObjectNew()) {
-                    $this->resourceModel->deleteGalleryValueInStore(
-                        $image['value_id'],
-                        $product->getData($this->metadata->getLinkField()),
-                        $product->getStoreId()
-                    );
-                }
                 // Add per store labels, position, disabled
                 $data['value_id'] = $image['value_id'];
                 $data['label'] = isset($image['label']) ? $image['label'] : '';
-                $data['position'] = isset($image['position']) ? (int)$image['position'] : 0;
+                $data['position'] = isset($image['position']) && $image['position'] !== ''
+                    ? (int)$image['position']
+                    : null;
                 $data['disabled'] = isset($image['disabled']) ? (int)$image['disabled'] : 0;
                 $data['store_id'] = (int)$product->getStoreId();
 
                 $data[$this->metadata->getLinkField()] = (int)$product->getData($this->metadata->getLinkField());
 
-                $this->resourceModel->insertGalleryValueInStore($data);
+                $this->saveGalleryStoreValue($product, $data);
+                if ($isNew && $data['store_id'] !== Store::DEFAULT_STORE_ID) {
+                    $dataForDefaultScope = $data;
+                    $dataForDefaultScope['store_id'] = Store::DEFAULT_STORE_ID;
+                    $dataForDefaultScope['disabled'] = 0;
+                    $dataForDefaultScope['label'] = null;
+                    $this->saveGalleryStoreValue($product, $dataForDefaultScope);
+                }
             }
         }
     }
 
+    /**
+     * Save media gallery store value
+     *
+     * @param Product $product
+     * @param array $data
+     */
+    private function saveGalleryStoreValue(Product $product, array $data): void
+    {
+        if (!$product->isObjectNew()) {
+            $this->resourceModel->deleteGalleryValueInStore(
+                $data['value_id'],
+                $data[$this->metadata->getLinkField()],
+                $data['store_id']
+            );
+        }
+        $this->resourceModel->insertGalleryValueInStore($data);
+    }
+
     /**
      * Processes image as new.
      *

diff --git a/app/code/Magento/Catalog/Model/Product/Gallery/ReadHandler.php b/app/code/Magento/Catalog/Model/Product/Gallery/ReadHandler.php
@@ -64,9 +64,9 @@ public function execute($entity, $arguments = [])
 
         $this->addMediaDataToProduct(
             $entity,
-            $mediaEntries
+            $this->sortMediaEntriesByPosition($mediaEntries)
         );
-        
+
         return $entity;
     }
 
@@ -108,7 +108,7 @@ public function getAttribute()
      * Find default value
      *
      * @param string $key
-     * @param string[] &$image
+     * @param string[] $image
      * @return string
      * @deprecated 101.0.1
      * @since 101.0.0
@@ -121,4 +121,30 @@ protected function findDefaultValue($key, &$image)
 
         return '';
     }
+
+    /**
+     * Sort media entries by position
+     *
+     * @param array $mediaEntries
+     * @return array
+     */
+    private function sortMediaEntriesByPosition(array $mediaEntries): array
+    {
+        $mediaEntriesWithNullPositions = [];
+        foreach ($mediaEntries as $index => $mediaEntry) {
+            if ($mediaEntry['position'] === null) {
+                $mediaEntriesWithNullPositions[] = $mediaEntry;
+                unset($mediaEntries[$index]);
+            }
+        }
+        if (!empty($mediaEntries)) {
+            usort(
+                $mediaEntries,
+                function ($entryA, $entryB) {
+                    return ($entryA['position'] < $entryB['position']) ? -1 : 1;
+                }
+            );
+        }
+        return array_merge($mediaEntries, $mediaEntriesWithNullPositions);
+    }
 }
diff --git a/app/code/Magento/Catalog/Model/Product/Option/Type/File.php b/app/code/Magento/Catalog/Model/Product/Option/Type/File.php
@@ -6,10 +6,11 @@
 
 namespace Magento\Catalog\Model\Product\Option\Type;
 
+use Magento\Catalog\Model\Product\Exception as ProductException;
+use Magento\Catalog\Helper\Product as ProductHelper;
 use Magento\Framework\App\Filesystem\DirectoryList;
 use Magento\Framework\Filesystem;
 use Magento\Framework\Exception\LocalizedException;
-use Magento\Catalog\Model\Product\Exception as ProductException;
 use Magento\Framework\Serialize\Serializer\Json;
 use Magento\Framework\App\ObjectManager;
 
@@ -91,6 +92,11 @@ class File extends \Magento\Catalog\Model\Product\Option\Type\DefaultType
      */
     private $filesystem;
 
+    /**
+     * @var ProductHelper
+     */
+    private $productHelper;
+
     /**
      * @param \Magento\Checkout\Model\Session $checkoutSession
      * @param \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig
@@ -103,6 +109,7 @@ class File extends \Magento\Catalog\Model\Product\Option\Type\DefaultType
      * @param array $data
      * @param Filesystem $filesystem
      * @param Json|null $serializer
+     * @param ProductHelper|null $productHelper
      * @SuppressWarnings(PHPMD.ExcessiveParameterList)
      */
     public function __construct(
@@ -116,7 +123,8 @@ public function __construct(
         \Magento\Framework\Escaper $escaper,
         array $data = [],
         Filesystem $filesystem = null,
-        Json $serializer = null
+        Json $serializer = null,
+        ProductHelper $productHelper = null
     ) {
         $this->_itemOptionFactory = $itemOptionFactory;
         $this->_urlBuilder = $urlBuilder;
@@ -129,6 +137,7 @@ public function __construct(
         $this->validatorInfo = $validatorInfo;
         $this->validatorFile = $validatorFile;
         $this->serializer = $serializer ?: ObjectManager::getInstance()->get(Json::class);
+        $this->productHelper = $productHelper ?: ObjectManager::getInstance()->get(ProductHelper::class);
         parent::__construct($checkoutSession, $scopeConfig, $data);
     }
 
@@ -223,12 +232,21 @@ public function validateUserValue($values)
         $this->setIsValid(true);
         $option = $this->getOption();
 
+        if (isset($values['files_prefix'])) {
+            $processingParams = ['files_prefix' => $values['files_prefix']];
+            $processingParams = array_merge($this->_getProcessingParams()->getData(), $processingParams);
+            $this->productHelper->addParamsToBuyRequest($this->getRequest(), $processingParams);
+        }
+
         /*
          * Check whether we receive uploaded file or restore file by: reorder/edit configuration or
          * previous configuration with no newly uploaded file
          */
         $fileInfo = null;
-        if (isset($values[$option->getId()]) && is_array($values[$option->getId()])) {
+        if (isset($values[$option->getId()])) {
+            if (is_string($values[$option->getId()])) {
+                $values[$option->getId()] = explode(',', $values[$option->getId()]);
+            }
             // Legacy style, file info comes in array with option id index
             $fileInfo = $values[$option->getId()];
         } else {

diff --git a/...og/Test/Mftf/ActionGroup/AdminCheckNameToggleOnProductsMassAttributeUpdateActionGroup.xml b/...og/Test/Mftf/ActionGroup/AdminCheckNameToggleOnProductsMassAttributeUpdateActionGroup.xml
@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+ /**
+  * Copyright © Magento, Inc. All rights reserved.
+  * See COPYING.txt for license details.
+  */
+-->
+
+<actionGroups xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+              xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/actionGroupSchema.xsd">
+    <actionGroup name="AdminCheckNameToggleOnProductsMassAttributesUpdateActionGroup">
+        <annotations>
+            <description>Click the "Change" checkbox for the "Name" field on the Products Masss Attributes Update page.</description>
+        </annotations>
+
+        <click selector="{{AdminEditProductAttributesSection.ChangeAttributeNameToggle}}" stepKey="toggleToChangeName"/>
+
+    </actionGroup>
+</actionGroups>