MDEE-207: Rename SalesDataExporter module to SalesOrdersDataExporter (#156)

le0n4ik · web-flow · commit b4a3c44a66f0 · 2022-08-08T12:09:46.000-05:00
* MDEE-207: Rename SalesDataExporter module to SalesOrdersDataExporter to eliminate conflicts with the Payments service modules
diff --git a/SalesOrdersDataExporter/Console/Command/Link.php b/SalesOrdersDataExporter/Console/Command/Link.php
@@ -5,7 +5,7 @@
  */
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Console\Command;
+namespace Magento\SalesOrdersDataExporter\Console\Command;
 
 use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface;
 use Magento\DataExporter\Uuid\UuidManager;
diff --git a/SalesOrdersDataExporter/LICENSE.txt b/SalesOrdersDataExporter/LICENSE.txt
diff --git a/SalesOrdersDataExporter/LICENSE_AFL.txt b/SalesOrdersDataExporter/LICENSE_AFL.txt
diff --git a/SalesOrdersDataExporter/Model/Indexer/OrderStatusIdsProvider.php b/SalesOrdersDataExporter/Model/Indexer/OrderStatusIdsProvider.php
@@ -5,7 +5,7 @@
  */
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Indexer;
+namespace Magento\SalesOrdersDataExporter\Model\Indexer;
 
 use Magento\Framework\App\ResourceConnection;
 use Magento\Framework\DB\Select;
diff --git a/SalesOrdersDataExporter/Model/Provider/CreditMemoBaseRowTotal.php b/SalesOrdersDataExporter/Model/Provider/CreditMemoBaseRowTotal.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\Framework\Url\Encoder;
 use Magento\Framework\UrlInterface;
diff --git a/SalesOrdersDataExporter/Model/Provider/CustomAttribute.php b/SalesOrdersDataExporter/Model/Provider/CustomAttribute.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\Framework\Serialize\Serializer\Json;
 
diff --git a/SalesOrdersDataExporter/Model/Provider/DateConverter.php b/SalesOrdersDataExporter/Model/Provider/DateConverter.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\DataExporter\Exception\UnableRetrieveData;
 use Magento\DataExporter\Export\Request\Node;
diff --git a/SalesOrdersDataExporter/Model/Provider/ExternalOrderId.php b/SalesOrdersDataExporter/Model/Provider/ExternalOrderId.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\DataExporter\Export\Request\Node;
 
diff --git a/SalesOrdersDataExporter/Model/Provider/Items/OrderItemAdjustment.php b/SalesOrdersDataExporter/Model/Provider/Items/OrderItemAdjustment.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider\Items;
+namespace Magento\SalesOrdersDataExporter\Model\Provider\Items;
 
 use Magento\Framework\Serialize\Serializer\Json;
 
diff --git a/SalesOrdersDataExporter/Model/Provider/Orders.php b/SalesOrdersDataExporter/Model/Provider/Orders.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\DataExporter\Export\Request\Info;
 use Magento\DataExporter\Export\Request\Node;
diff --git a/SalesOrdersDataExporter/Model/Provider/Payment.php b/SalesOrdersDataExporter/Model/Provider/Payment.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\Framework\Serialize\Serializer\Json;
 
diff --git a/SalesOrdersDataExporter/Model/Provider/Shipping.php b/SalesOrdersDataExporter/Model/Provider/Shipping.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 /**
  * Class for getting shipping information
diff --git a/SalesOrdersDataExporter/Model/Provider/State.php b/SalesOrdersDataExporter/Model/Provider/State.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\Sales\Model\Order;
 
diff --git a/SalesOrdersDataExporter/Model/Provider/TrackingUrl.php b/SalesOrdersDataExporter/Model/Provider/TrackingUrl.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\Framework\Url\Encoder;
 use Magento\Framework\UrlInterface;
diff --git a/SalesOrdersDataExporter/Model/Provider/Uuid.php b/SalesOrdersDataExporter/Model/Provider/Uuid.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Model\Provider;
+namespace Magento\SalesOrdersDataExporter\Model\Provider;
 
 use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface;
 use Magento\DataExporter\Uuid\UuidManager;
diff --git a/SalesOrdersDataExporter/Observer/ReindexOrderFeed.php b/SalesOrdersDataExporter/Observer/ReindexOrderFeed.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Observer;
+namespace Magento\SalesOrdersDataExporter\Observer;
 
 use Magento\Framework\Event\Observer;
 use Magento\Framework\Event\ObserverInterface;
@@ -20,7 +20,7 @@ class ReindexOrderFeed implements ObserverInterface
     /**
      * Review feed indexer id
      */
-    public const ORDER_FEED_INDEXER = 'sales_order_data_exporter';
+    public const ORDER_FEED_INDEXER = 'sales_order_data_exporter_v2';
 
     /**
      * @var IndexerRegistry
diff --git a/SalesOrdersDataExporter/Observer/ReindexOrderFeedOnVoid.php b/SalesOrdersDataExporter/Observer/ReindexOrderFeedOnVoid.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Observer;
+namespace Magento\SalesOrdersDataExporter\Observer;
 
 use Magento\Framework\Event\Observer;
 use Magento\Framework\Event\ObserverInterface;
@@ -20,7 +20,7 @@ class ReindexOrderFeedOnVoid implements ObserverInterface
     /**
      * Sales order feed indexer id
      */
-    public const ORDER_FEED_INDEXER = 'sales_order_data_exporter';
+    public const ORDER_FEED_INDEXER = 'sales_order_data_exporter_v2';
 
     /**
      * @var IndexerRegistry
diff --git a/SalesOrdersDataExporter/Plugin/AssignUuidForCreditMemo.php b/SalesOrdersDataExporter/Plugin/AssignUuidForCreditMemo.php
@@ -5,7 +5,7 @@
  */
 
 declare(strict_types=1);
-namespace Magento\SalesDataExporter\Plugin;
+namespace Magento\SalesOrdersDataExporter\Plugin;
 
 use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface;
 use Magento\DataExporter\Uuid\UuidSaveException;
diff --git a/SalesOrdersDataExporter/Plugin/AssignUuidForOrder.php b/SalesOrdersDataExporter/Plugin/AssignUuidForOrder.php
@@ -5,7 +5,7 @@
  */
 
 declare(strict_types=1);
-namespace Magento\SalesDataExporter\Plugin;
+namespace Magento\SalesOrdersDataExporter\Plugin;
 
 use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface;
 use Magento\DataExporter\Uuid\UuidSaveException;
diff --git a/SalesOrdersDataExporter/Plugin/AssignUuidForShipment.php b/SalesOrdersDataExporter/Plugin/AssignUuidForShipment.php
@@ -5,7 +5,7 @@
  */
 
 declare(strict_types=1);
-namespace Magento\SalesDataExporter\Plugin;
+namespace Magento\SalesOrdersDataExporter\Plugin;
 
 use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface;
 use Magento\DataExporter\Uuid\UuidSaveException;
diff --git a/SalesOrdersDataExporter/README.md b/SalesOrdersDataExporter/README.md
@@ -1,8 +1,8 @@
 # Overview
-SalesDataExporter module allows to collect Orders entities and sales related entities (order items, shipments, invoices, credit memos, payment transactions) for further export to external services.
+SalesOrdersDataExporter module allows to collect Orders entities and sales related entities (order items, shipments, invoices, credit memos, payment transactions) for further export to external services.
 Sales Orders Export works properly only with “Updated On Schedule” indexer mode.
 ### Entities UUID
-Each exported item should have it's own UUID, so only items with UUID assigned will be able to be exported. For example if customer has some orders and then installs SalesDataExporter module - only orders which would be created after installation will be exported.
+Each exported item should have it's own UUID, so only items with UUID assigned will be able to be exported. For example if customer has some orders and then installs SalesOrdersDataExporter module - only orders which would be created after installation will be exported.
 There is also ability to assign UUIDs to already assigned orders and order entities: "commerce-data-export:orders:link" command:
 
 ```shell
@@ -32,5 +32,5 @@ Current time will be used by default
 ### Reindexing
 To export full orders data simply run reindex command:
 ```shell
-bin/magento indexer:reindex sales_order_data_exporter
+bin/magento indexer:reindex sales_order_data_exporter_v2
 ```
diff --git a/SalesOrdersDataExporter/Tests/Integration/AbstractOrderFeedTest.php b/SalesOrdersDataExporter/Tests/Integration/AbstractOrderFeedTest.php
@@ -6,7 +6,7 @@
 
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Tests\Integration;
+namespace Magento\SalesOrdersDataExporter\Tests\Integration;
 
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\DataExporter\Model\FeedInterface;
@@ -34,7 +34,7 @@ abstract class AbstractOrderFeedTest extends TestCase
     /**
      * Order feed indexer
      */
-    private const ORDER_FEED_INDEXER = 'sales_order_data_exporter';
+    private const ORDER_FEED_INDEXER = 'sales_order_data_exporter_v2';
 
     /**
      * @var ResourceConnection
@@ -98,7 +98,7 @@ protected function setUp() : void
         $this->jsonSerializer = Bootstrap::getObjectManager()->create(Json::class);
         $this->orderRepository = Bootstrap::getObjectManager()->create(ProductRepositoryInterface::class);
         $this->storeManager = Bootstrap::getObjectManager()->create(StoreManagerInterface::class);
-        $this->ordersFeed = Bootstrap::getObjectManager()->get(FeedPool::class)->getFeed('salesOrders');
+        $this->ordersFeed = Bootstrap::getObjectManager()->get(FeedPool::class)->getFeed('salesOrdersV2');
         $this->attributeRepository = Bootstrap::getObjectManager()->create(AttributeRepository::class);
         $this->arrayUtils = $objectManager->create(ArrayUtils::class);
         $this->registry = Bootstrap::getObjectManager()->get(Registry::class);
diff --git a/SalesOrdersDataExporter/Tests/Integration/CreateOrderTest.php b/SalesOrdersDataExporter/Tests/Integration/CreateOrderTest.php
@@ -5,7 +5,7 @@
  */
 declare(strict_types=1);
 
-namespace Magento\SalesDataExporter\Tests\Integration;
+namespace Magento\SalesOrdersDataExporter\Tests\Integration;
 
 use Magento\Sales\Api\Data\OrderInterface;
 use Magento\Sales\Api\Data\OrderInterfaceFactory;
@@ -87,7 +87,7 @@ public function testOrderWithTwoProductsInformation(string $orderNumber, array $
      * @param string[] $dataToVerify
      *
      * @dataProvider orderFullWorkflowDataProvider
-     * @magentoDataFixture Magento/SalesDataExporter/_files/order_full_work_flow.php
+     * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_full_work_flow.php
      *
      * @return void
      * @throws \Zend_Db_Statement_Exception
@@ -153,7 +153,7 @@ public function testOrderWithInvoiceAndCustomStatus(string $orderNumber, array $
      * @param string[] $dataToVerify
      *
      * @dataProvider orderWithCreditMemoDataProvider
-     * @magentoDataFixture Magento/SalesDataExporter/_files/order_with_invoice_shipment_creditmemo.php
+     * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_with_invoice_shipment_creditmemo.php
      *
      * @return void
      * @throws \Zend_Db_Statement_Exception
@@ -175,7 +175,7 @@ public function testOrderWithCreditMemo(string $orderNumber, array $dataToVerify
      * @param string[] $dataToVerify
      *
      * @dataProvider orderWithConfigurableProductDataProvider
-     * @magentoDataFixture Magento/SalesDataExporter/_files/order_configurable_product.php
+     * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_configurable_product.php
      *
      * @return void
      * @throws \Zend_Db_Statement_Exception
@@ -197,7 +197,7 @@ public function testOrderWithConfigurableProduct(string $orderNumber, array $dat
      * @param string[] $dataToVerify
      *
      * @dataProvider orderWithTransactionsDataProvider
-     * @magentoDataFixture Magento/SalesDataExporter/_files/transactions_detailed.php
+     * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/transactions_detailed.php
      *
      * @return void
      * @throws \Zend_Db_Statement_Exception
@@ -219,7 +219,7 @@ public function testOrderWithTransactions(string $orderNumber, array $dataToVeri
      * @param string[] $dataToVerify
      *
      * @dataProvider orderWithAdditionalInformationDataProvider
-     * @magentoDataFixture Magento/SalesDataExporter/_files/order_with_additional_information.php
+     * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_with_additional_information.php
      *
      * @return void
      * @throws \Zend_Db_Statement_Exception
diff --git a/SalesOrdersDataExporter/composer.json b/SalesOrdersDataExporter/composer.json
@@ -1,5 +1,5 @@
 {
-    "name": "magento/module-sales-data-exporter",
+    "name": "magento/module-sales-orders-data-exporter",
     "description": "Sales data exporter",
     "config": {
         "sort-packages": true
@@ -14,7 +14,7 @@
             "registration.php"
         ],
         "psr-4": {
-            "Magento\\SalesDataExporter\\": ""
+            "Magento\\SalesOrdersDataExporter\\": ""
         }
     },
     "require": {
diff --git a/SalesOrdersDataExporter/etc/db_schema.xml b/SalesOrdersDataExporter/etc/db_schema.xml
@@ -7,7 +7,7 @@
 -->
 <schema xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:noNamespaceSchemaLocation="urn:magento:framework:Setup/Declaration/Schema/etc/schema.xsd">
-    <table name="sales_data_exporter_orders" resource="default" engine="innodb" comment="Sales Orders Feed Storage">
+    <table name="sales_data_exporter_orders_v2" resource="default" engine="innodb" comment="Sales Orders Feed Storage">
         <column xsi:type="int"
                 name="id"
                 padding="10"
diff --git a/SalesOrdersDataExporter/etc/db_schema_whitelist.json b/SalesOrdersDataExporter/etc/db_schema_whitelist.json
@@ -1,7 +1,7 @@
 {
-    "sales_data_exporter_orders": {
+    "sales_data_exporter_orders_v2": {
         "index": {
-            "SALES_DATA_EXPORTER_ORDERS_MODIFIED_AT": true
+            "SALES_DATA_EXPORTER_ORDERS_V2_MODIFIED_AT": true
         },
         "constraint": {
             "PRIMARY": true
diff --git a/SalesOrdersDataExporter/etc/di.xml b/SalesOrdersDataExporter/etc/di.xml
diff --git a/SalesOrdersDataExporter/etc/et_schema.xml b/SalesOrdersDataExporter/etc/et_schema.xml
diff --git a/SalesOrdersDataExporter/etc/events.xml b/SalesOrdersDataExporter/etc/events.xml
diff --git a/SalesOrdersDataExporter/etc/indexer.xml b/SalesOrdersDataExporter/etc/indexer.xml
diff --git a/SalesOrdersDataExporter/etc/module.xml b/SalesOrdersDataExporter/etc/module.xml
diff --git a/SalesOrdersDataExporter/etc/mview.xml b/SalesOrdersDataExporter/etc/mview.xml
diff --git a/SalesOrdersDataExporter/etc/query.xml b/SalesOrdersDataExporter/etc/query.xml
diff --git a/SalesOrdersDataExporter/registration.php b/SalesOrdersDataExporter/registration.php
diff --git a/_metapackage/composer.json b/_metapackage/composer.json
diff --git a/dev/tests/static/testsuite/Magento/Test/Integrity/_files/blacklist/composer_root_modules_data_export.txt b/dev/tests/static/testsuite/Magento/Test/Integrity/_files/blacklist/composer_root_modules_data_export.txt

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`*/`
`6`	`6`	`declare(strict_types=1);`
`7`	`7`
`8`		`-namespace Magento\SalesDataExporter\Tests\Integration;`
	`8`	`+namespace Magento\SalesOrdersDataExporter\Tests\Integration;`
`9`	`9`
`10`	`10`	`use Magento\Sales\Api\Data\OrderInterface;`
`11`	`11`	`use Magento\Sales\Api\Data\OrderInterfaceFactory;`
`@@ -87,7 +87,7 @@ public function testOrderWithTwoProductsInformation(string $orderNumber, array $`
`87`	`87`	`* @param string[] $dataToVerify`
`88`	`88`	`*`
`89`	`89`	`* @dataProvider orderFullWorkflowDataProvider`
`90`		`- * @magentoDataFixture Magento/SalesDataExporter/_files/order_full_work_flow.php`
	`90`	`+ * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_full_work_flow.php`
`91`	`91`	`*`
`92`	`92`	`* @return void`
`93`	`93`	`* @throws \Zend_Db_Statement_Exception`
`@@ -153,7 +153,7 @@ public function testOrderWithInvoiceAndCustomStatus(string $orderNumber, array $`
`153`	`153`	`* @param string[] $dataToVerify`
`154`	`154`	`*`
`155`	`155`	`* @dataProvider orderWithCreditMemoDataProvider`
`156`		`- * @magentoDataFixture Magento/SalesDataExporter/_files/order_with_invoice_shipment_creditmemo.php`
	`156`	`+ * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_with_invoice_shipment_creditmemo.php`
`157`	`157`	`*`
`158`	`158`	`* @return void`
`159`	`159`	`* @throws \Zend_Db_Statement_Exception`
`@@ -175,7 +175,7 @@ public function testOrderWithCreditMemo(string $orderNumber, array $dataToVerify`
`175`	`175`	`* @param string[] $dataToVerify`
`176`	`176`	`*`
`177`	`177`	`* @dataProvider orderWithConfigurableProductDataProvider`
`178`		`- * @magentoDataFixture Magento/SalesDataExporter/_files/order_configurable_product.php`
	`178`	`+ * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_configurable_product.php`
`179`	`179`	`*`
`180`	`180`	`* @return void`
`181`	`181`	`* @throws \Zend_Db_Statement_Exception`
`@@ -197,7 +197,7 @@ public function testOrderWithConfigurableProduct(string $orderNumber, array $dat`
`197`	`197`	`* @param string[] $dataToVerify`
`198`	`198`	`*`
`199`	`199`	`* @dataProvider orderWithTransactionsDataProvider`
`200`		`- * @magentoDataFixture Magento/SalesDataExporter/_files/transactions_detailed.php`
	`200`	`+ * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/transactions_detailed.php`
`201`	`201`	`*`
`202`	`202`	`* @return void`
`203`	`203`	`* @throws \Zend_Db_Statement_Exception`
`@@ -219,7 +219,7 @@ public function testOrderWithTransactions(string $orderNumber, array $dataToVeri`
`219`	`219`	`* @param string[] $dataToVerify`
`220`	`220`	`*`
`221`	`221`	`* @dataProvider orderWithAdditionalInformationDataProvider`
`222`		`- * @magentoDataFixture Magento/SalesDataExporter/_files/order_with_additional_information.php`
	`222`	`+ * @magentoDataFixture Magento/SalesOrdersDataExporter/_files/order_with_additional_information.php`
`223`	`223`	`*`
`224`	`224`	`* @return void`
`225`	`225`	`* @throws \Zend_Db_Statement_Exception`