Skip to content

Commit 452b2e3

Browse files
authored
ENGCOM-5930: #878 - Deprecated the column increment_id #951
2 parents 1b41eb0 + bcc5890 commit 452b2e3

File tree

3 files changed

+14
-14
lines changed

3 files changed

+14
-14
lines changed

app/code/Magento/SalesGraphQl/Model/Resolver/Orders.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ public function resolve(
5656
$items[] = [
5757
'id' => $order->getId(),
5858
'increment_id' => $order->getIncrementId(),
59+
'order_number' => $order->getIncrementId(),
5960
'created_at' => $order->getCreatedAt(),
6061
'grand_total' => $order->getGrandTotal(),
6162
'status' => $order->getStatus(),

app/code/Magento/SalesGraphQl/etc/schema.graphqls

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@ type Query {
77

88
type CustomerOrder @doc(description: "Order mapping fields") {
99
id: Int
10-
increment_id: String
10+
increment_id: String @deprecated(reason: "Use the order_number instaed.")
11+
order_number: String! @doc(description: "The order number")
1112
created_at: String
1213
grand_total: Float
1314
status: String

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

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,7 @@ public function testOrdersQuery()
3838
query {
3939
customerOrders {
4040
items {
41-
id
42-
increment_id
43-
created_at
41+
order_number
4442
grand_total
4543
status
4644
}
@@ -54,27 +52,27 @@ public function testOrdersQuery()
5452

5553
$expectedData = [
5654
[
57-
'increment_id' => '100000002',
55+
'order_number' => '100000002',
5856
'status' => 'processing',
5957
'grand_total' => 120.00
6058
],
6159
[
62-
'increment_id' => '100000003',
60+
'order_number' => '100000003',
6361
'status' => 'processing',
6462
'grand_total' => 130.00
6563
],
6664
[
67-
'increment_id' => '100000004',
65+
'order_number' => '100000004',
6866
'status' => 'closed',
6967
'grand_total' => 140.00
7068
],
7169
[
72-
'increment_id' => '100000005',
70+
'order_number' => '100000005',
7371
'status' => 'complete',
7472
'grand_total' => 150.00
7573
],
7674
[
77-
'increment_id' => '100000006',
75+
'order_number' => '100000006',
7876
'status' => 'complete',
7977
'grand_total' => 160.00
8078
]
@@ -84,19 +82,19 @@ public function testOrdersQuery()
8482

8583
foreach ($expectedData as $key => $data) {
8684
$this->assertEquals(
87-
$data['increment_id'],
88-
$actualData[$key]['increment_id'],
89-
"increment_id is different than the expected for order - " . $data['increment_id']
85+
$data['order_number'],
86+
$actualData[$key]['order_number'],
87+
"order_number is different than the expected for order - " . $data['order_number']
9088
);
9189
$this->assertEquals(
9290
$data['grand_total'],
9391
$actualData[$key]['grand_total'],
94-
"grand_total is different than the expected for order - " . $data['increment_id']
92+
"grand_total is different than the expected for order - " . $data['order_number']
9593
);
9694
$this->assertEquals(
9795
$data['status'],
9896
$actualData[$key]['status'],
99-
"status is different than the expected for order - " . $data['increment_id']
97+
"status is different than the expected for order - " . $data['order_number']
10098
);
10199
}
102100
}

0 commit comments

Comments
 (0)