ACP2E-4243: Issue with Update bundle option price per website via Import

- address issue when child product is not assigned to all websites the bundle product is assigned to resulting in missing website price for that child product

Author: bubasuma

app/code/Magento/BundleImportExport/Model/Export/RowCustomizer.php

@@ -579,6 +579,9 @@ private function getProductOptionCollection(
             $optionCollection = $product->getTypeInstance()
                 ->getOptionsCollection($product)
                 ->setOrder('position', Collection::SORT_ORDER_ASC);
+            // Ensure children products are not filtered by website.
+            // We need to export all children products regardless of the website they are assigned to.
+            $product->getTypeInstance()->setStoreFilter(Store::DEFAULT_STORE_ID, $product);
             $selectionCollection = $product->getTypeInstance()
                 ->getSelectionsCollection(
                     $product->getTypeInstance()->getOptionsIds($product),

dev/tests/integration/testsuite/Magento/BundleImportExport/Model/Export/RowCustomizerTest.php

@@ -9,8 +9,14 @@
 use Magento\Bundle\Test\Fixture\Link as BundleSelectionFixture;
 use Magento\Bundle\Test\Fixture\Product as BundleProductFixture;
 use Magento\Catalog\Api\Data\ProductInterface;
+use Magento\Catalog\Helper\Data;
 use Magento\Catalog\Test\Fixture\Product as ProductFixture;
 use Magento\Store\Model\ScopeInterface;
+use Magento\Store\Model\StoreManagerInterface;
+use Magento\Store\Test\Fixture\Group as GroupFixture;
+use Magento\Store\Test\Fixture\Store as StoreFixture;
+use Magento\Store\Test\Fixture\Website as WebsiteFixture;
+use Magento\TestFramework\Fixture\AppArea;
 use Magento\TestFramework\Fixture\Config;
 use Magento\TestFramework\Fixture\DataFixture;
 use Magento\TestFramework\Fixture\DataFixtureStorageManager;
@@ -36,6 +42,16 @@ class RowCustomizerTest extends \PHPUnit\Framework\TestCase
      * @var \Magento\Catalog\Api\ProductRepositoryInterface
      */
     private $productRepository;
+    
+    /**
+     * @var StoreManagerInterface
+     */
+    private $storeManager;
+    
+    /**
+     * @var int
+     */
+    private $currentStoreId;
 
     /**
      * @inheritdoc
@@ -49,6 +65,14 @@ protected function setUp(): void
         $this->productRepository = $this->objectManager->get(
             \Magento\Catalog\Api\ProductRepositoryInterface::class
         );
+        $this->storeManager = $this->objectManager->get(StoreManagerInterface::class);
+        $this->currentStoreId = $this->storeManager->getStore()->getId();
+    }
+    
+    protected function tearDown(): void
+    {
+        $this->storeManager->setCurrentStore($this->currentStoreId);
+        parent::tearDown();
     }
 
     /**
@@ -136,10 +160,15 @@ function ($input) {
     }
 
     #[
+        // export is performed in global scope
+        AppArea('global'),
         DbIsolation(false),
-        Config(\Magento\Catalog\Helper\Data::XML_PATH_PRICE_SCOPE, 1, ScopeInterface::SCOPE_STORE, 'default'),
+        Config(Data::XML_PATH_PRICE_SCOPE, 1, ScopeInterface::SCOPE_STORE, 'default'),
+        Config(Data::XML_PATH_PRICE_SCOPE, 1, ScopeInterface::SCOPE_STORE, 'bundle_test_store2'),
         DataFixture(ScopeFixture::class, as: 'global_scope'),
-        DataFixture(ScopeFixture::class, ['code' => 'default'], as: 'default_store'),
+        DataFixture(WebsiteFixture::class, as: 'website2'),
+        DataFixture(GroupFixture::class, ['website_id' => '$website2.id$'], 'group2'),
+        DataFixture(StoreFixture::class, ['store_group_id' => '$group2.id$', 'code' => 'bundle_test_store2'], 'store2'),
         DataFixture(ProductFixture::class, as: 'p1'),
         DataFixture(ProductFixture::class, as: 'p2'),
         DataFixture(BundleSelectionFixture::class, ['sku' => '$p1.sku$', 'price' => 10, 'price_type' => 0], 'link1'),
@@ -148,7 +177,7 @@ function ($input) {
         DataFixture(BundleOptionFixture::class, ['product_links' => ['$link2$']], 'opt2'),
         DataFixture(
             BundleProductFixture::class,
-            ['price' => 50,'price_type' => 1, '_options' => ['$opt1$','$opt2$']],
+            ['price' => 50,'price_type' => 1, 'website_ids' => [1, '$website2.id$'], '_options' => ['$opt1$','$opt2$']],
             'bundle',
             'global_scope',
         ),
@@ -161,30 +190,34 @@ public function testExportWhenPriceScopeIsWebsite(): void
         $sku2 = $fixtures->get('p2')->getSku();
         $opt1 = $fixtures->get('opt1')->getTitle();
         $opt2 = $fixtures->get('opt2')->getTitle();
-        $store = $fixtures->get('default_store');
+        $websiteCode = $fixtures->get('website2')->getCode();
+        $store = $fixtures->get('store2');
 
         $data['bundle_values'] = "name=$opt1,type=select,required=1,sku=$sku1,price=10.0000" .
             ",default=0,default_qty=1.0000,price_type=fixed,can_change_qty=0" .
             "|name=$opt2,type=select,required=1,sku=$sku2,price=20.0000" .
             ",default=0,default_qty=1.0000,price_type=percent,can_change_qty=0";
         $this->assertBundleValues($data, $bundleProduct);
 
-        // Update selection prices in default store
-        $bundleProduct = $this->productRepository->get($bundleProduct->getSku(), false, $store->getId());
+        // Update selection prices in second store
+        $bundleProduct = $this->productRepository->get($bundleProduct->getSku(), true);
+        $this->storeManager->setCurrentStore($store->getId());
+        $bundleProduct->setStoreId($store->getId());
         $extension = $bundleProduct->getExtensionAttributes();
         $options = $extension->getBundleProductOptions();
         $options[0]->getProductLinks()[0]->setPrice(40);
         $options[0]->getProductLinks()[0]->setPriceType(1);
         $options[1]->getProductLinks()[0]->setPrice(50);
         $options[1]->getProductLinks()[0]->setPriceType(1);
         $this->productRepository->save($bundleProduct);
+        $this->storeManager->setCurrentStore($this->currentStoreId);
 
         $data['bundle_values'] = "name=$opt1,type=select,required=1,sku=$sku1,price=10.0000" .
             ",default=0,default_qty=1.0000,price_type=fixed,can_change_qty=0" .
-            ",price_website_base=40.000000,price_type_website_base=percent" .
+            ",price_website_$websiteCode=40.000000,price_type_website_$websiteCode=percent" .
             "|name=$opt2,type=select,required=1,sku=$sku2,price=20.0000" .
             ",default=0,default_qty=1.0000,price_type=percent,can_change_qty=0" .
-            ",price_website_base=50.000000,price_type_website_base=percent";
+            ",price_website_$websiteCode=50.000000,price_type_website_$websiteCode=percent";
         $this->assertBundleValues($data, $bundleProduct);
     }