[Bug]: Shopify: Unneccesary call to DoMapping function resulting in uncalled Event Publisher

[JeppeBylov]

Describe the issue

When shopify imports orders, the event publisher OnBeforeProcessSalesDocuments is never triggered unless all lines are mapped to items.
This is because the report 30104 "Shpfy Sync Orders from Shopify" calls the "DoMapping" function before calling the CreateSalesDocumentForOrders(OrderHeader);

This is a problem for my client, becuase we have a customization that will handle mapping depending on attributes. The customization subscribes to the OnBeforeProcessSalesDocuments publisher.

ALAppExtensions/Apps/W1/Shopify/app/src/Order handling/Reports/ShpfySyncOrdersfromShopify.Report.al

Lines 65 to 67 in 1845570

	if OrderMapping.DoMapping(OrderHeader) and (OrdersToImport."Import Action" = OrdersToImport."Import Action"::New) then
	if Shop."Auto Create Orders" then
	CreateSalesDocumentForOrders(OrderHeader);

Expected behavior

There should be no call to the DoMapping function, as this is already a part of the process of creating sales documents.

Steps to reproduce

1. Create customization that handles mapping based on Order Line Attributes
2. Sell item with Shopify
3. Import order to BC, which gets the "Not everything can be mapped" error
4. Click "Create Sales Documents"
5. Sales order is now created

Additional context

No response

I will provide a fix for a bug

• I will provide a fix for a bug

[JesperSchulz]

Feel free to create e PR for this.