Merge branch '2.4-develop' into 21853

magento · Jan 11, 2021 · be3f84b · be3f84b
2 parents 910eb41 + 63434d2
commit be3f84b
Show file tree

Hide file tree

Showing 123 changed files with 3,891 additions and 553 deletions.
diff --git a/app/code/Magento/Backend/view/adminhtml/templates/widget/grid/extended.phtml b/app/code/Magento/Backend/view/adminhtml/templates/widget/grid/extended.phtml
@@ -14,12 +14,13 @@
  *  getPagerVisibility()
  *  getVarNamePage()
  */
-$numColumns = count($block->getColumns());
 
 /**
  * @var \Magento\Backend\Block\Widget\Grid\Extended $block
  * @var \Magento\Framework\View\Helper\SecureHtmlRenderer $secureRenderer
  */
+$numColumns = count($block->getColumns());
+
 ?>
 <?php if ($block->getCollection()): ?>
     <?php if ($block->canDisplayContainer()): ?>
@@ -285,7 +286,9 @@ $numColumns = count($block->getColumns());
         </table>
 
     </div>
+        <?php if ($block->canDisplayContainer()): ?>
 </div>
+        <?php endif; ?>
     <?php
     /** @var \Magento\Framework\Json\Helper\Data $jsonHelper */
     $jsonHelper = $block->getData('jsonHelper');

diff --git a/app/code/Magento/Bundle/Model/Sales/Order/Pdf/Items/Invoice.php b/app/code/Magento/Bundle/Model/Sales/Order/Pdf/Items/Invoice.php
@@ -8,6 +8,7 @@
 namespace Magento\Bundle\Model\Sales\Order\Pdf\Items;
 
 use Magento\Framework\Data\Collection\AbstractDb;
+use Magento\Framework\DataObject;
 use Magento\Framework\Filesystem;
 use Magento\Framework\Filter\FilterManager;
 use Magento\Framework\Model\Context;
@@ -69,50 +70,52 @@ public function __construct(
     }
 
     /**
-     * Draw item line
+     * Draw bundle product item line
      *
      * @return void
-     * @SuppressWarnings(PHPMD.CyclomaticComplexity)
-     * @SuppressWarnings(PHPMD.NPathComplexity)
-     * @SuppressWarnings(PHPMD.ExcessiveMethodLength)
      */
     public function draw()
     {
-        $order = $this->getOrder();
-        $item = $this->getItem();
-        $pdf = $this->getPdf();
-        $page = $this->getPage();
+        $draw = $this->drawChildrenItems();
+        $draw = $this->drawCustomOptions($draw);
 
+        $page = $this->getPdf()->drawLineBlocks($this->getPage(), $draw, ['table_header' => true]);
+
+        $this->setPage($page);
+    }
+
+    /**
+     * Draw bundle product children items
+     *
+     * @return array
+     */
+    private function drawChildrenItems(): array
+    {
         $this->_setFontRegular();
-        $items = $this->getChildren($item);
 
         $prevOptionId = '';
         $drawItems = [];
-
-        foreach ($items as $childItem) {
-            $line = [];
-
+        $optionId = 0;
+        $lines = [];
+        foreach ($this->getChildren($this->getItem()) as $childItem) {
+            $index = array_key_last($lines) !== null ? array_key_last($lines) + 1 : 0;
             $attributes = $this->getSelectionAttributes($childItem);
             if (is_array($attributes)) {
                 $optionId = $attributes['option_id'];
-            } else {
-                $optionId = 0;
             }
 
             if (!isset($drawItems[$optionId])) {
                 $drawItems[$optionId] = ['lines' => [], 'height' => 15];
             }
 
             if ($childItem->getOrderItem()->getParentItem() && $prevOptionId != $attributes['option_id']) {
-                $line[0] = [
+                $lines[$index][] = [
                     'font' => 'italic',
                     'text' => $this->string->split($attributes['option_label'], 45, true, true),
                     'feed' => 35,
                 ];
 
-                $drawItems[$optionId] = ['lines' => [$line], 'height' => 15];
-
-                $line = [];
+                $index++;
                 $prevOptionId = $attributes['option_id'];
             }
 
@@ -124,35 +127,97 @@ public function draw()
                 $feed = 35;
                 $name = $childItem->getName();
             }
-            $line[] = ['text' => $this->string->split($name, 35, true, true), 'feed' => $feed];
+            $lines[$index][] = ['text' => $this->string->split($name, 35, true, true), 'feed' => $feed];
 
-            // draw SKUs
-            if (!$childItem->getOrderItem()->getParentItem()) {
-                $text = [];
-                foreach ($this->string->split($item->getSku(), 17) as $part) {
-                    $text[] = $part;
-                }
-                $line[] = ['text' => $text, 'feed' => 255];
-            }
+            $lines = $this->drawSkus($childItem, $lines);
 
-            // draw prices
-            if ($this->canShowPriceInfo($childItem)) {
-                $price = $order->formatPriceTxt($childItem->getPrice());
-                $line[] = ['text' => $price, 'feed' => 395, 'font' => 'bold', 'align' => 'right'];
-                $line[] = ['text' => $childItem->getQty() * 1, 'feed' => 435, 'font' => 'bold'];
+            $lines = $this->drawPrices($childItem, $lines);
+        }
+        $drawItems[$optionId]['lines'] = $lines;
 
-                $tax = $order->formatPriceTxt($childItem->getTaxAmount());
-                $line[] = ['text' => $tax, 'feed' => 495, 'font' => 'bold', 'align' => 'right'];
+        return $drawItems;
+    }
 
-                $row_total = $order->formatPriceTxt($childItem->getRowTotal());
-                $line[] = ['text' => $row_total, 'feed' => 565, 'font' => 'bold', 'align' => 'right'];
+    /**
+     * Draw sku parts
+     *
+     * @param DataObject $childItem
+     * @param array $lines
+     * @return array
+     */
+    private function drawSkus(DataObject $childItem, array $lines): array
+    {
+        $index = array_key_last($lines);
+        if (!$childItem->getOrderItem()->getParentItem()) {
+            $text = [];
+            foreach ($this->string->split($this->getItem()->getSku(), 17) as $part) {
+                $text[] = $part;
             }
+            $lines[$index][] = ['text' => $text, 'feed' => 255];
+        }
+
+        return $lines;
+    }
 
-            $drawItems[$optionId]['lines'][] = $line;
+    /**
+     * Draw prices for bundle product children items
+     *
+     * @param DataObject $childItem
+     * @param array $lines
+     * @return array
+     */
+    private function drawPrices(DataObject $childItem, array $lines): array
+    {
+        $index = array_key_last($lines);
+        if ($this->canShowPriceInfo($childItem)) {
+            $lines[$index][] = ['text' => $childItem->getQty() * 1, 'feed' => 435, 'align' => 'right'];
+
+            $tax = $this->getOrder()->formatPriceTxt($childItem->getTaxAmount());
+            $lines[$index][] = ['text' => $tax, 'feed' => 495, 'font' => 'bold', 'align' => 'right'];
+
+            $item = $this->getItem();
+            $this->_item = $childItem;
+            $feedPrice = 380;
+            $feedSubtotal = $feedPrice + 185;
+            foreach ($this->getItemPricesForDisplay() as $priceData) {
+                if (isset($priceData['label'])) {
+                    // draw Price label
+                    $lines[$index][] = ['text' => $priceData['label'], 'feed' => $feedPrice, 'align' => 'right'];
+                    // draw Subtotal label
+                    $lines[$index][] = ['text' => $priceData['label'], 'feed' => $feedSubtotal, 'align' => 'right'];
+                    $index++;
+                }
+                // draw Price
+                $lines[$index][] = [
+                    'text' => $priceData['price'],
+                    'feed' => $feedPrice,
+                    'font' => 'bold',
+                    'align' => 'right',
+                ];
+                // draw Subtotal
+                $lines[$index][] = [
+                    'text' => $priceData['subtotal'],
+                    'feed' => $feedSubtotal,
+                    'font' => 'bold',
+                    'align' => 'right',
+                ];
+                $index++;
+            }
+            $this->_item = $item;
         }
 
-        // custom options
-        $options = $item->getOrderItem()->getProductOptions();
+        return $lines;
+    }
+
+    /**
+     * Draw bundle product custom options
+     *
+     * @param array $draw
+     * @return array
+     */
+    private function drawCustomOptions(array $draw): array
+    {
+        $options = $this->getItem()->getOrderItem()->getProductOptions();
         if ($options && isset($options['options'])) {
             foreach ($options['options'] as $option) {
                 $lines = [];
@@ -180,12 +245,10 @@ public function draw()
                     $lines[][] = ['text' => $text, 'feed' => 40];
                 }
 
-                $drawItems[] = ['lines' => $lines, 'height' => 15];
+                $draw[] = ['lines' => $lines, 'height' => 15];
             }
         }
 
-        $page = $pdf->drawLineBlocks($page, $drawItems, ['table_header' => true]);
-
-        $this->setPage($page);
+        return $draw;
     }
 }
diff --git a/app/code/Magento/Bundle/Test/Unit/Model/Product/TypeTest.php b/app/code/Magento/Bundle/Test/Unit/Model/Product/TypeTest.php
@@ -11,7 +11,6 @@
 use Magento\Bundle\Model\Product\Type;
 use Magento\Bundle\Model\ResourceModel\BundleFactory;
 use Magento\Bundle\Model\ResourceModel\Option\Collection;
-use Magento\CatalogRule\Model\ResourceModel\Product\CollectionProcessor;
 use Magento\Bundle\Model\ResourceModel\Selection\Collection as SelectionCollection;
 use Magento\Bundle\Model\ResourceModel\Selection\CollectionFactory;
 use Magento\Bundle\Model\Selection;
@@ -28,6 +27,7 @@
 use Magento\CatalogInventory\Api\StockStateInterface;
 use Magento\CatalogInventory\Model\StockRegistry;
 use Magento\CatalogInventory\Model\StockState;
+use Magento\CatalogRule\Model\ResourceModel\Product\CollectionProcessor;
 use Magento\Framework\DataObject;
 use Magento\Framework\EntityManager\EntityMetadataInterface;
 use Magento\Framework\EntityManager\MetadataPool;
@@ -1548,6 +1548,10 @@ public function testPrepareForCartAdvancedSpecifyProductOptions()
             ->disableOriginalConstructor()
             ->getMock();
 
+        $buyRequest->method('getOptions')
+            ->willReturn([333 => ['type' => 'image/jpeg']]);
+        $option->method('getId')
+            ->willReturn(333);
         $this->parentClass($group, $option, $buyRequest, $product);
 
         $product->expects($this->any())
@@ -1556,6 +1560,8 @@ public function testPrepareForCartAdvancedSpecifyProductOptions()
         $buyRequest->expects($this->once())
             ->method('getBundleOption')
             ->willReturn([0, '', 'str']);
+        $group->expects($this->once())
+            ->method('validateUserValue');
 
         $result = $this->model->prepareForCartAdvanced($buyRequest, $product);
         $this->assertEquals('Please specify product option(s).', $result);