Skip to content

Commit a8eec06

Browse files
committed
Rename stock_availability field to is_available and use property promotion
1 parent 2afe92f commit a8eec06

File tree

4 files changed

+19
-31
lines changed

4 files changed

+19
-31
lines changed

app/code/Magento/QuoteGraphQl/Model/CartItem/ProductStock.php

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,20 +31,14 @@ class ProductStock
3131
*/
3232
private const PRODUCT_TYPE_BUNDLE = "bundle";
3333

34-
/**
35-
* @var StockStatusRepositoryInterface
36-
*/
37-
private $stockStatusRepository;
38-
3934
/**
4035
* ProductStock constructor
4136
*
4237
* @param StockStatusRepositoryInterface $stockStatusRepository
4338
*/
4439
public function __construct(
45-
StockStatusRepositoryInterface $stockStatusRepository
40+
private StockStatusRepositoryInterface $stockStatusRepository
4641
) {
47-
$this->stockStatusRepository = $stockStatusRepository;
4842
}
4943

5044
/**

app/code/Magento/QuoteGraphQl/Model/Resolver/CheckProductStockAvailability.php

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,20 +30,14 @@
3030
*/
3131
class CheckProductStockAvailability implements ResolverInterface
3232
{
33-
/**
34-
* @var ProductStock
35-
*/
36-
private ProductStock $productStock;
37-
3833
/**
3934
* CheckProductStockAvailability constructor
4035
*
4136
* @param ProductStock $productStock
4237
*/
4338
public function __construct(
44-
ProductStock $productStock
39+
private ProductStock $productStock
4540
) {
46-
$this->productStock = $productStock;
4741
}
4842

4943
/**
@@ -57,6 +51,6 @@ public function resolve(Field $field, $context, ResolveInfo $info, array $value
5751
/** @var Item $cartItem */
5852
$cartItem = $value['model'];
5953

60-
return $this->productStock->isProductAvailable($cartItem) ? "available" : "unavailable";
54+
return $this->productStock->isProductAvailable($cartItem) ? true : false;
6155
}
6256
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -360,7 +360,7 @@ interface CartItemInterface @typeResolver(class: "Magento\\QuoteGraphQl\\Model\\
360360
id: String! @deprecated(reason: "Use `uid` instead.")
361361
uid: ID! @doc(description: "The unique ID for a `CartItemInterface` object.")
362362
quantity: Float! @doc(description: "The quantity of this item in the cart.")
363-
stock_availability: String @resolver(class: "\\Magento\\QuoteGraphQl\\Model\\Resolver\\CheckProductStockAvailability") @doc(description: "If qty is more than stock display status as unavailable else available.")
363+
is_available: Boolean! @resolver(class: "\\Magento\\QuoteGraphQl\\Model\\Resolver\\CheckProductStockAvailability") @doc(description: "If qty is more than stock display status as unavailable else available.")
364364
prices: CartItemPrices @resolver(class: "\\Magento\\QuoteGraphQl\\Model\\Resolver\\CartItemPrices") @doc(description: "Contains details about the price of the item, including taxes and discounts.")
365365
product: ProductInterface! @doc(description: "Details about an item in the cart.")
366366
errors: [CartItemError!] @resolver(class: "\\Magento\\QuoteGraphQl\\Model\\Resolver\\CartItemErrors") @doc(description: "An array of errors encountered while loading the cart item")

dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/StockAvailabilityTest.php

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ public function testStockStatusUnavailableSimpleProduct(): void
8686
$response = $this->graphQlMutation($query);
8787
$responseDataObject = new DataObject($response);
8888

89-
self::assertEquals('unavailable', $responseDataObject->getData('cart/items/0/stock_availability'));
89+
self::assertEquals(false, $responseDataObject->getData('cart/items/0/is_available'));
9090
}
9191

9292
#[
@@ -103,8 +103,8 @@ public function testStockStatusUnavailableAddSimpleProduct(): void
103103
$responseDataObject = new DataObject($response);
104104

105105
self::assertEquals(
106-
'unavailable',
107-
$responseDataObject->getData('addProductsToCart/cart/items/0/stock_availability')
106+
false,
107+
$responseDataObject->getData('addProductsToCart/cart/items/0/is_available')
108108
);
109109
}
110110

@@ -145,8 +145,8 @@ public function testStockStatusUnavailableBundleProduct(): void
145145
$responseDataObject = new DataObject($response);
146146

147147
self::assertEquals(
148-
'unavailable',
149-
$responseDataObject->getData('cart/items/0/stock_availability')
148+
false,
149+
$responseDataObject->getData('cart/items/0/is_available')
150150
);
151151
}
152152

@@ -200,8 +200,8 @@ public function testStockStatusUnavailableAddBundleProduct(): void
200200
$responseDataObject = new DataObject($response);
201201

202202
self::assertEquals(
203-
'unavailable',
204-
$responseDataObject->getData('addProductsToCart/cart/items/0/stock_availability')
203+
false,
204+
$responseDataObject->getData('addProductsToCart/cart/items/0/is_available')
205205
);
206206
}
207207

@@ -234,8 +234,8 @@ public function testStockStatusUnavailableConfigurableProduct(): void
234234
$responseDataObject = new DataObject($response);
235235

236236
self::assertEquals(
237-
'unavailable',
238-
$responseDataObject->getData('cart/items/0/stock_availability')
237+
false,
238+
$responseDataObject->getData('cart/items/0/is_available')
239239
);
240240
}
241241

@@ -267,8 +267,8 @@ public function testStockStatusUnavailableAddConfigurableProduct(): void
267267
$responseDataObject = new DataObject($response);
268268

269269
self::assertEquals(
270-
'unavailable',
271-
$responseDataObject->getData('addProductsToCart/cart/items/0/stock_availability')
270+
false,
271+
$responseDataObject->getData('addProductsToCart/cart/items/0/is_available')
272272
);
273273
}
274274

@@ -282,7 +282,7 @@ private function getQuery(string $cartId): string
282282
{
283283
cart(cart_id:"{$cartId}"){
284284
items{
285-
stock_availability
285+
is_available
286286
}
287287
}
288288
}
@@ -304,7 +304,7 @@ private function mutationAddSimpleProduct(string $cartId, string $sku, int $qty
304304
) {
305305
cart {
306306
items {
307-
stock_availability
307+
is_available
308308
}
309309
}
310310
}
@@ -333,7 +333,7 @@ private function mutationAddBundleProduct(
333333
) {
334334
cart {
335335
items {
336-
stock_availability
336+
is_available
337337
product {
338338
sku
339339
}
@@ -363,7 +363,7 @@ private function mutationAddConfigurableProduct(
363363
) {
364364
cart {
365365
items {
366-
stock_availability
366+
is_available
367367
product {
368368
sku
369369
}

0 commit comments

Comments
 (0)