MC-20638: MyAccount :: Order Details :: Shipping Details by Order Number

Author: danielrenaud

app/code/Magento/BundleGraphQl/composer.json

@@ -10,6 +10,8 @@
         "magento/module-quote": "*",
         "magento/module-quote-graph-ql": "*",
         "magento/module-store": "*",
+        "magento/module-sales": "*",
+        "magento/module-sales-graph-ql": "*",
         "magento/framework": "*"
     },
     "license": [

app/code/Magento/SalesGraphQl/Model/Resolver/CustomerOrders/Carrier.php

@@ -60,7 +60,7 @@ private function findCarrierByMethodCode(string $methodCode): ?string
     {
         $allCarrierMethods = $this->carrierMethods->toOptionArray();
 
-        foreach ($allCarrierMethods as $carrierCode => $carrierMethods) {
+        foreach ($allCarrierMethods as $carrierMethods) {
             $carrierLabel = $carrierMethods['label'];
             $carrierMethodOptions = $carrierMethods['value'];
             if (is_array($carrierMethodOptions)) {

app/code/Magento/SalesGraphQl/Model/Resolver/Invoice/InvoiceItems.php

@@ -16,7 +16,7 @@
 use Magento\Sales\Api\Data\InvoiceItemInterface;
 use Magento\Sales\Api\Data\OrderInterface;
 use Magento\Sales\Api\Data\OrderItemInterface;
-use Magento\SalesGraphQl\Model\Resolver\OrderItem\DataProvider as OrderItemProvider;
+use Magento\SalesGraphQl\Model\OrderItem\DataProvider as OrderItemProvider;
 
 /**
  * Resolver for Invoice Items

app/code/Magento/SalesGraphQl/composer.json

@@ -10,9 +10,7 @@
         "magento/module-catalog": "*",
         "magento/module-tax": "*",
         "magento/module-quote": "*",
-        "magento/module-graph-ql": "*"
-    },
-    "suggest": {
+        "magento/module-graph-ql": "*",
         "magento/module-shipping": "*"
     },
     "license": [

dev/tests/api-functional/testsuite/Magento/GraphQl/Sales/CustomerOrders/OrderShipmentsTest.php

@@ -5,6 +5,8 @@
  */
 declare(strict_types=1);
 
+namespace Magento\GraphQl\Sales\CustomerOrders;
+
 use Magento\Framework\DB\Transaction;
 use Magento\Framework\Registry;
 use Magento\GraphQl\GetCustomerAuthenticationHeader;

dev/tests/api-functional/testsuite/Magento/GraphQl/Sales/Fixtures/CustomerPlaceOrder.php

@@ -81,11 +81,11 @@ public function placeOrderWithBundleProduct(array $customerLogin, array $product
     /**
      * Make GraphQl POST request
      *
-     * @param $query
+     * @param string $query
      * @param array $additionalHeaders
      * @return array
      */
-    private function makeRequest($query, $additionalHeaders = []): array
+    private function makeRequest(string $query, array $additionalHeaders = []): array
     {
         $headers = array_merge([$this->getAuthHeader()], $additionalHeaders);
         return $this->gqlClient->post($query, [], '', $headers);

dev/tests/integration/testsuite/Magento/GraphQl/Sales/_files/customer_order_with_simple_shipment.php

@@ -68,5 +68,3 @@
 $track->setCarrierCode('ups');
 $track->setTrackNumber('1234567890');
 $shipmentTrackRepository->save($track);
-
-

dev/tests/integration/testsuite/Magento/GraphQl/Sales/_files/customer_order_with_ups_shipping.php

@@ -40,7 +40,7 @@
 $track->setCarrierCode('ups');
 $track->setTitle('United Parcel Service');
 $track->setTrackNumber('987654321');
-$shipOrder->execute($order->getEntityId(), $shipmentItems, false, false, null, [$track]);
+$shipOrder->execute($order->getId(), $shipmentItems, false, false, null, [$track]);
 
 //Create second Shipment
 $shipmentItems = [];
@@ -53,4 +53,4 @@
         $shipmentItems[] = $shipmentItem;
     }
 }
-$shipOrder->execute($order->getEntityId(), $shipmentItems);
+$shipOrder->execute($order->getId(), $shipmentItems);