Merge pull request #4626 from magento-trigger/MC-19061_revert

 - Revert ticket MC-17003

Author: irenelagno

app/code/Magento/Sales/Block/Adminhtml/Order/Creditmemo/Create/Adjustments.php

@@ -111,20 +111,4 @@ public function getShippingLabel()
         }
         return $label;
     }
-
-    /**
-     * Get update totals url.
-     *
-     * @return string
-     */
-    public function getUpdateTotalsUrl(): string
-    {
-        return $this->getUrl(
-            'sales/*/updateQty',
-            [
-                'order_id' => $this->getSource()->getOrderId(),
-                'invoice_id' => $this->getRequest()->getParam('invoice_id', null),
-            ]
-        );
-    }
 }

app/code/Magento/Sales/Block/Adminhtml/Order/Creditmemo/Create/Items.php

@@ -56,12 +56,7 @@ protected function _prepareLayout()
         $this->addChild(
             'update_button',
             \Magento\Backend\Block\Widget\Button::class,
-            ['label' => __('Update Qty\'s'), 'class' => 'update-button secondary', 'onclick' => $onclick]
-        );
-        $this->addChild(
-            'update_totals_button',
-            \Magento\Backend\Block\Widget\Button::class,
-            ['label' => __('Update Totals'), 'class' => 'update-totals-button secondary', 'onclick' => $onclick]
+            ['label' => __('Update Qty\'s'), 'class' => 'update-button', 'onclick' => $onclick]
         );
 
         if ($this->getCreditmemo()->canRefund()) {
@@ -181,16 +176,6 @@ public function getUpdateButtonHtml()
         return $this->getChildHtml('update_button');
     }
 
-    /**
-     * Get update totals button html
-     *
-     * @return string
-     */
-    public function getUpdateTotalsButtonHtml(): string
-    {
-        return $this->getChildHtml('update_totals_button');
-    }
-
     /**
      * Get update url
      *

app/code/Magento/Sales/Test/Mftf/Test/AdminCheckingCreditMemoUpdateTotalsTest.xml

@@ -18,6 +18,9 @@
             <testCaseId value="MC-18159"/>
             <useCaseId value="MC-17003"/>
             <group value="sales"/>
+            <skip>
+                <issueId value="MC-17003"/>
+            </skip>
         </annotations>
         <before>
             <!--Create product-->

app/code/Magento/Sales/view/adminhtml/templates/order/creditmemo/create/items.phtml

@@ -100,7 +100,6 @@
                 <span class="title"><?= $block->escapeHtml(__('Refund Totals')) ?></span>
             </div>
             <?= $block->getChildHtml('creditmemo_totals') ?>
-            <div class="totals-actions"><?= $block->getUpdateTotalsButtonHtml() ?></div>
             <div class="order-totals-actions">
                 <div class="field choice admin__field admin__field-option field-append-comments">
                     <input id="notify_customer"
@@ -140,9 +139,8 @@ require(['jquery'], function(jQuery){
 
 //<![CDATA[
 var submitButtons = jQuery('.submit-button');
-var updateButtons = jQuery('.update-button,.update-totals-button');
-var fields = jQuery('.qty-input,.order-subtotal-table input[type="text"]');
-
+var updateButtons = jQuery('.update-button');
+var fields = jQuery('.qty-input');
 function enableButtons(buttons) {
     buttons.removeClass('disabled').prop('disabled', false);
 }

app/code/Magento/Theme/Test/Mftf/Test/AdminWatermarkUploadTest.xml

@@ -16,6 +16,9 @@
             <description value="Watermark images should be able to be uploaded in the admin"/>
             <severity value="MAJOR"/>
             <testCaseId value="MC-5796"/>
+            <skip>
+                <issueId value="MC-18496"/>
+            </skip>
             <group value="Watermark"/>
         </annotations>
         <before>

app/design/adminhtml/Magento/backend/Magento_Sales/web/css/source/module/order/_total.less

@@ -22,10 +22,6 @@
     }
 }
 
-.totals-actions {
-    text-align: right;
-}
-
 .order-totals-actions {
     margin-top: @indent__s;
     .actions {

dev/tests/functional/tests/app/Magento/Sales/Test/Block/Adminhtml/Order/Creditmemo/Totals.php

@@ -27,34 +27,6 @@ class Totals extends \Magento\Sales\Test\Block\Adminhtml\Order\Totals
      */
     protected $capture = '[name="invoice[capture_case]"]';
 
-    /**
-     * Refund Shipping css selector.
-     *
-     * @var string
-     */
-    private $refundShippingSelector = '#shipping_amount';
-
-    /**
-     * Adjustment Refund css selector.
-     *
-     * @var string
-     */
-    private $adjustmentRefundSelector = '#adjustment_positive';
-
-    /**
-     * Adjustment Fee css selector.
-     *
-     * @var string
-     */
-    private $adjustmentFeeSelector = '#adjustment_negative';
-
-    /**
-     * Update Totals button css selector.
-     *
-     * @var string
-     */
-    private $updateTotalsSelector = '.update-totals-button';
-
     /**
      * Submit invoice.
      *
@@ -85,44 +57,4 @@ public function setCaptureOption($option)
     {
         $this->_rootElement->find($this->capture, Locator::SELECTOR_CSS, 'select')->setValue($option);
     }
-
-    /**
-     * Get Refund Shipping input element.
-     *
-     * @return \Magento\Mtf\Client\ElementInterface
-     */
-    public function getRefundShippingElement()
-    {
-        return $this->_rootElement->find($this->refundShippingSelector, Locator::SELECTOR_CSS);
-    }
-
-    /**
-     * Get Adjustment Refund input element.
-     *
-     * @return \Magento\Mtf\Client\ElementInterface
-     */
-    public function getAdjustmentRefundElement()
-    {
-        return $this->_rootElement->find($this->adjustmentRefundSelector, Locator::SELECTOR_CSS);
-    }
-
-    /**
-     * Get Adjustment Fee input element.
-     *
-     * @return \Magento\Mtf\Client\ElementInterface
-     */
-    public function getAdjustmentFeeElement()
-    {
-        return $this->_rootElement->find($this->adjustmentFeeSelector, Locator::SELECTOR_CSS);
-    }
-
-    /**
-     * Click update totals button.
-     *
-     * @return void
-     */
-    public function clickUpdateTotals()
-    {
-        $this->_rootElement->find($this->updateTotalsSelector)->click();
-    }
 }

dev/tests/functional/tests/app/Magento/Sales/Test/TestStep/CreateCreditMemoStep.php

@@ -95,12 +95,8 @@ public function run()
             if ($this->compare($items, $refundData)) {
                 $this->orderCreditMemoNew->getFormBlock()->updateQty();
             }
-            $hasChangeTotals = $this->isTotalsDataChanged($refundData);
-            $this->orderCreditMemoNew->getFormBlock()->fillFormData($refundData);
-            if ($hasChangeTotals) {
-                $this->orderCreditMemoNew->getTotalsBlock()->clickUpdateTotals();
-            }
 
+            $this->orderCreditMemoNew->getFormBlock()->fillFormData($refundData);
             $this->orderCreditMemoNew->getFormBlock()->submit();
         }
 
@@ -120,30 +116,4 @@ protected function getCreditMemoIds()
         $this->salesOrderView->getOrderForm()->openTab('creditmemos');
         return $this->salesOrderView->getOrderForm()->getTab('creditmemos')->getGridBlock()->getIds();
     }
-
-    /**
-     * Is totals data changed.
-     *
-     * @param array $data
-     * @return bool
-     */
-    private function isTotalsDataChanged(array $data): bool
-    {
-        $compareData = [
-            'shipping_amount' =>
-                $this->orderCreditMemoNew->getTotalsBlock()->getRefundShippingElement()->getValue(),
-            'adjustment_positive' =>
-                $this->orderCreditMemoNew->getTotalsBlock()->getAdjustmentRefundElement()->getValue(),
-            'adjustment_negative' =>
-                $this->orderCreditMemoNew->getTotalsBlock()->getAdjustmentFeeElement()->getValue(),
-        ];
-
-        foreach ($compareData as $fieldName => $fieldValue) {
-            if (isset($data['form_data'][$fieldName]) && $fieldValue != $data['form_data'][$fieldName]) {
-                return true;
-            }
-        }
-
-        return false;
-    }
 }