MC-42813: Feature request: Cart qty update should be reverted back if the requested qty is not available

Author: dhorytskyi

app/code/Magento/Checkout/Controller/Cart/UpdateItemQty.php

@@ -21,8 +21,6 @@
 
 /**
  * UpdateItemQty ajax request
- *
- * @package Magento\Checkout\Controller\Cart
  */
 class UpdateItemQty extends Action implements HttpPostActionInterface
 {

app/code/Magento/Checkout/view/frontend/templates/cart/item/default.phtml

@@ -5,7 +5,7 @@
  */
 
 // phpcs:disable Magento2.Templates.ThisInTemplate
-// phpcs:disable Magento2.Files.LineLength.MaxExceeded
+// phpcs:disable Generic.Files.LineLength
 
 /** @var $block \Magento\Checkout\Block\Cart\Item\Renderer */
 
@@ -19,58 +19,58 @@ $canApplyMsrp = $helper->isShowBeforeOrderConfirm($product) && $helper->isMinima
 <tbody class="cart item">
     <tr class="item-info">
         <td data-th="<?= $block->escapeHtml(__('Item')) ?>" class="col item">
-            <?php if ($block->hasProductUrl()) :?>
+            <?php if ($block->hasProductUrl()): ?>
                 <a href="<?= $block->escapeUrl($block->getProductUrl()) ?>"
                    title="<?= $block->escapeHtml($block->getProductName()) ?>"
                    tabindex="-1"
                    class="product-item-photo">
-            <?php else :?>
+            <?php else: ?>
                 <span class="product-item-photo">
-            <?php endif;?>
+            <?php endif; ?>
             <?= $block->getImage($block->getProductForThumbnail(), 'cart_page_product_thumbnail')->toHtml() ?>
-            <?php if ($block->hasProductUrl()) :?>
+            <?php if ($block->hasProductUrl()): ?>
                 </a>
-            <?php else :?>
+            <?php else: ?>
                 </span>
             <?php endif; ?>
             <div class="product-item-details">
                 <strong class="product-item-name">
-                    <?php if ($block->hasProductUrl()) :?>
+                    <?php if ($block->hasProductUrl()): ?>
                         <a href="<?= $block->escapeUrl($block->getProductUrl()) ?>"><?= $block->escapeHtml($block->getProductName()) ?></a>
-                    <?php else :?>
+                    <?php else: ?>
                         <?= $block->escapeHtml($block->getProductName()) ?>
                     <?php endif; ?>
                 </strong>
-                <?php if ($_options = $block->getOptionList()) :?>
+                <?php if ($_options = $block->getOptionList()): ?>
                     <dl class="item-options">
-                        <?php foreach ($_options as $_option) :?>
+                        <?php foreach ($_options as $_option): ?>
                             <?php $_formatedOptionValue = $block->getFormatedOptionValue($_option) ?>
                             <dt><?= $block->escapeHtml($_option['label']) ?></dt>
                             <dd>
-                                <?php if (isset($_formatedOptionValue['full_view'])) :?>
+                                <?php if (isset($_formatedOptionValue['full_view'])): ?>
                                         <?= $block->escapeHtml($_formatedOptionValue['full_view'], ['span', 'a']) ?>
-                                    <?php else :?>
-                                    <?= $block->escapeHtml($_formatedOptionValue['value'], ['span', 'a']) ?>
+                                    <?php else: ?>
+                                        <?= $block->escapeHtml($_formatedOptionValue['value'], ['span', 'a']) ?>
                                 <?php endif; ?>
                             </dd>
                         <?php endforeach; ?>
                     </dl>
-                <?php endif;?>
-                <?php if ($messages = $block->getMessages()) :?>
-                    <?php foreach ($messages as $message) :?>
+                <?php endif; ?>
+                <?php if ($messages = $block->getMessages()): ?>
+                    <?php foreach ($messages as $message): ?>
                         <div class= "cart item message <?= $block->escapeHtmlAttr($message['type']) ?>">
                             <div><?= $block->escapeHtml($message['text']) ?></div>
                         </div>
                     <?php endforeach; ?>
                 <?php endif; ?>
                 <?php $addInfoBlock = $block->getProductAdditionalInformationBlock(); ?>
-                <?php if ($addInfoBlock) :?>
+                <?php if ($addInfoBlock): ?>
                     <?= $addInfoBlock->setItem($_item)->toHtml() ?>
-                <?php endif;?>
+                <?php endif; ?>
             </div>
         </td>
 
-        <?php if ($canApplyMsrp) :?>
+        <?php if ($canApplyMsrp): ?>
             <td class="col msrp" data-th="<?= $block->escapeHtml(__('Price')) ?>">
                 <span class="pricing msrp">
                     <span class="msrp notice"><?= $block->escapeHtml(__('See price before order confirmation.')) ?></span>
@@ -89,7 +89,7 @@ $canApplyMsrp = $helper->isShowBeforeOrderConfirm($product) && $helper->isMinima
                     </a>
                 </span>
             </td>
-        <?php else :?>
+        <?php else: ?>
             <td class="col price" data-th="<?= $block->escapeHtml(__('Price')) ?>">
                 <?= $block->getUnitPriceHtml($_item) ?>
             </td>
@@ -117,9 +117,9 @@ $canApplyMsrp = $helper->isShowBeforeOrderConfirm($product) && $helper->isMinima
         </td>
 
         <td class="col subtotal" data-th="<?= $block->escapeHtml(__('Subtotal')) ?>">
-            <?php if ($canApplyMsrp) :?>
+            <?php if ($canApplyMsrp): ?>
                 <span class="cart msrp subtotal">--</span>
-            <?php else :?>
+            <?php else: ?>
                 <?= $block->getRowTotalHtml($_item) ?>
             <?php endif; ?>
         </td>