🔃 [EngCom] Public Pull Requests - 2.3-develop

Accepted Public Pull Requests:
 - magento/magento2#20050: Add missing throws to PHPDoc (by @pmclain)
 - magento/magento2#20046: Added constants to unit codes to make it easier to reuse it if necessary (by @tiagosampaio)
 - magento/magento2#20022: [Fixed Radio alignment issue on Cart page #20021] (by @hitesh-wagento)
 - magento/magento2#19792: Fixed 19791: Logo vertical misalignment.  (by @suryakant-krish)
 - magento/magento2#20005: Minor Typo fix in AbstractFilter.php (by @ajeetsinghcedcoss)
 - magento/magento2#19981: Fix issue 19887 creating new shipment: getting all trackers. (by @Nazar65)
 - magento/magento2#19978: Set correct email message encoding (by @4lexvav)
 - magento/magento2#19941: issue resolved:Undefined Variable $itemsOrderItemId (by @GovindaSharma)
 - magento/magento2#19799: Fix issue 19796 - Sales Order invoice Update Qty's Button is misaligned (by @speedy008)
 - magento/magento2#19914: Move website_name column into columnSet (by @bradleyfrye)
 - magento/magento2#19926: fixed issue #19925 Close button overlapping in shipping address label whenever any user adding new shipping address in mobile view in checkout (by @cedarvinda)
 - magento/magento2#19929: Refactoring in Magento_SalesRule (by @agorbulin)
 - magento/magento2#19900: Fix negative credit memo #19899 (by @wojtekn)
 - magento/magento2#19752: Fixed Typo error  avaialble -> available (by @gulshanchitranshcedcoss)
 - magento/magento2#19715: [TASK] Updated the implements for the Magento_Cms Adminhtml Controllers (by @lewisvoncken)
 - magento/magento2#19620: Fix issue causing attribute not loading when using getList (by @MiliTree)
 - magento/magento2#19467: Use semantic theme color variables, including global primary/secondary colors (by @Brengineer)
 - magento/magento2#19930: Refactoring magento sales sequence (by @agorbulin)
 - magento/magento2#19646: Area Frontend: Fixed checkbox alignment account information page. (by @suryakant-krish)


Fixed GitHub Issues:
 - magento/magento2#20037: CategoryLinkReposity does not list all possible exceptions (reported by @troendheim) has been fixed in magento/magento2#20050 by @pmclain in 2.3-develop branch
   Related commits:
     1. d08ce91
     2. a1d3299

 - magento/magento2#20021: Radio alignment issue on Cart page (reported by @hitesh-wagento) has been fixed in magento/magento2#20022 by @hitesh-wagento in 2.3-develop branch
   Related commits:
     1. 6089f27

 - magento/magento2#19791: Logo vertical misalignment. (reported by @suryakant-krish) has been fixed in magento/magento2#19792 by @suryakant-krish in 2.3-develop branch
   Related commits:
     1. 485b89f
     2. 3a56397
     3. 8e59e4b

 - magento/magento2#19887: creating new shipment: gettting all trackers. after this commit  2307e16 (reported by @aldf) has been fixed in magento/magento2#19981 by @Nazar65 in 2.3-develop branch
   Related commits:
     1. 8048516
     2. 9cf1e3e

 - magento/magento2#19977: E-Mail subject not showing utf-8 characters (reported by @4lexvav) has been fixed in magento/magento2#19978 by @4lexvav in 2.3-develop branch
   Related commits:
     1. bccc949

 - magento/magento2#19940: Exception undefined variable itemsOrderItemId while creating shipment through MSI (reported by @mohammadzakir) has been fixed in magento/magento2#19941 by @GovindaSharma in 2.3-develop branch
   Related commits:
     1. b54ab2d

 - magento/magento2#19796: Sales Order invoice Update Qty's Button is misaligned (reported by @speedy008) has been fixed in magento/magento2#19799 by @speedy008 in 2.3-develop branch
   Related commits:
     1. e30a080
     2. 8a3fa61

 - magento/magento2#19899: Credit memo for $0 order without refunded shipping produces negative credit memo (reported by @wojtekn) has been fixed in magento/magento2#19900 by @wojtekn in 2.3-develop branch
   Related commits:
     1. e840e41
     2. 4a77f29
     3. 5b4fdca

 - magento/magento2#17759: M2.2.5 : CustomerRepository::getList() does not load custom attribute if the name is "company" (reported by @kanduvisla) has been fixed in magento/magento2#19620 by @MiliTree in 2.3-develop branch
   Related commits:
     1. d600179
     2. b99af38
     3. e29c5dd

 - magento/magento2#19645: Area Frontend: Account information page checkbox alignment issue. (reported by @suryakant-krish) has been fixed in magento/magento2#19646 by @suryakant-krish in 2.3-develop branch
   Related commits:
     1. 0811a71

Author: sidolov

app/code/Magento/Backend/Model/Search/Customer.php

@@ -89,7 +89,7 @@ public function load()
 
         $this->searchCriteriaBuilder->setCurrentPage($this->getStart());
         $this->searchCriteriaBuilder->setPageSize($this->getLimit());
-        $searchFields = ['firstname', 'lastname', 'company'];
+        $searchFields = ['firstname', 'lastname', 'billing_company'];
         $filters = [];
         foreach ($searchFields as $field) {
             $filters[] = $this->filterBuilder

app/code/Magento/Catalog/Api/CategoryLinkRepositoryInterface.php

@@ -32,18 +32,20 @@ public function save(\Magento\Catalog\Api\Data\CategoryProductLinkInterface $pro
      *
      * @throws \Magento\Framework\Exception\CouldNotSaveException
      * @throws \Magento\Framework\Exception\StateException
+     * @throws \Magento\Framework\Exception\InputException
      */
     public function delete(\Magento\Catalog\Api\Data\CategoryProductLinkInterface $productLink);
 
     /**
      * Remove the product assignment from the category by category id and sku
      *
-     * @param string $sku
+     * @param string $categoryId
      * @param string $sku
      * @return bool will returned True if products successfully deleted
      *
      * @throws \Magento\Framework\Exception\CouldNotSaveException
      * @throws \Magento\Framework\Exception\StateException
+     * @throws \Magento\Framework\Exception\InputException
      */
     public function deleteByIds($categoryId, $sku);
 }

app/code/Magento/Catalog/Model/Layer/Filter/AbstractFilter.php

@@ -139,7 +139,7 @@ public function apply(\Magento\Framework\App\RequestInterface $request)
     }
 
     /**
-     * Get fiter items count
+     * Get filter items count
      *
      * @return int
      */

app/code/Magento/Cms/Controller/Adminhtml/Block/Edit.php

@@ -5,8 +5,11 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Block;
 
-use Magento\Framework\App\Action\HttpGetActionInterface as HttpGetActionInterface;
+use Magento\Framework\App\Action\HttpGetActionInterface;
 
+/**
+ * Edit CMS block action.
+ */
 class Edit extends \Magento\Cms\Controller\Adminhtml\Block implements HttpGetActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Block/Index.php

@@ -1,13 +1,15 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Block;
 
-use Magento\Framework\App\Action\HttpGetActionInterface as HttpGetActionInterface;
+use Magento\Framework\App\Action\HttpGetActionInterface;
 
+/**
+ * Index action.
+ */
 class Index extends \Magento\Cms\Controller\Adminhtml\Block implements HttpGetActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Block/MassDelete.php

@@ -6,6 +6,7 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Block;
 
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Framework\Controller\ResultFactory;
 use Magento\Backend\App\Action\Context;
 use Magento\Ui\Component\MassAction\Filter;
@@ -14,7 +15,7 @@
 /**
  * Class MassDelete
  */
-class MassDelete extends \Magento\Backend\App\Action
+class MassDelete extends \Magento\Backend\App\Action implements HttpPostActionInterface
 {
     /**
      * Authorization level of a basic admin session

app/code/Magento/Cms/Controller/Adminhtml/Block/NewAction.php

@@ -1,13 +1,15 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Block;
 
-use Magento\Framework\App\Action\HttpGetActionInterface as HttpGetActionInterface;
+use Magento\Framework\App\Action\HttpGetActionInterface;
 
+/**
+ * Create CMS block action.
+ */
 class NewAction extends \Magento\Cms\Controller\Adminhtml\Block implements HttpGetActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Block/Save.php

@@ -1,12 +1,11 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Block;
 
-use Magento\Framework\App\Action\HttpPostActionInterface as HttpPostActionInterface;
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Backend\App\Action\Context;
 use Magento\Cms\Api\BlockRepositoryInterface;
 use Magento\Cms\Model\Block;
@@ -15,6 +14,9 @@
 use Magento\Framework\Exception\LocalizedException;
 use Magento\Framework\Registry;
 
+/**
+ * Save CMS block action.
+ */
 class Save extends \Magento\Cms\Controller\Adminhtml\Block implements HttpPostActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Page/Delete.php

@@ -1,12 +1,16 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
-class Delete extends \Magento\Backend\App\Action
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
+/**
+ * Delete CMS page action.
+ */
+class Delete extends \Magento\Backend\App\Action implements HttpPostActionInterface
 {
     /**
      * Authorization level of a basic admin session

app/code/Magento/Cms/Controller/Adminhtml/Page/Edit.php

@@ -1,14 +1,16 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
-use Magento\Framework\App\Action\HttpGetActionInterface as HttpGetActionInterface;
+use Magento\Framework\App\Action\HttpGetActionInterface;
 use Magento\Backend\App\Action;
 
+/**
+ * Edit CMS page action.
+ */
 class Edit extends \Magento\Backend\App\Action implements HttpGetActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Page/Index.php

@@ -1,15 +1,17 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
-use Magento\Framework\App\Action\HttpGetActionInterface as HttpGetActionInterface;
+use Magento\Framework\App\Action\HttpGetActionInterface;
 use Magento\Backend\App\Action\Context;
 use Magento\Framework\View\Result\PageFactory;
 
+/**
+ * Index action.
+ */
 class Index extends \Magento\Backend\App\Action implements HttpGetActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Page/MassDelete.php

@@ -5,6 +5,7 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Framework\Controller\ResultFactory;
 use Magento\Backend\App\Action\Context;
 use Magento\Ui\Component\MassAction\Filter;
@@ -13,7 +14,7 @@
 /**
  * Class MassDelete
  */
-class MassDelete extends \Magento\Backend\App\Action
+class MassDelete extends \Magento\Backend\App\Action implements HttpPostActionInterface
 {
     /**
      * Authorization level of a basic admin session

app/code/Magento/Cms/Controller/Adminhtml/Page/MassDisable.php

@@ -5,7 +5,7 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
-use Magento\Framework\App\Action\HttpPostActionInterface as HttpPostActionInterface;
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Framework\Controller\ResultFactory;
 use Magento\Backend\App\Action\Context;
 use Magento\Ui\Component\MassAction\Filter;

app/code/Magento/Cms/Controller/Adminhtml/Page/MassEnable.php

@@ -5,6 +5,7 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Framework\Controller\ResultFactory;
 use Magento\Backend\App\Action\Context;
 use Magento\Ui\Component\MassAction\Filter;
@@ -13,7 +14,7 @@
 /**
  * Class MassEnable
  */
-class MassEnable extends \Magento\Backend\App\Action
+class MassEnable extends \Magento\Backend\App\Action implements HttpPostActionInterface
 {
     /**
      * Authorization level of a basic admin session

app/code/Magento/Cms/Controller/Adminhtml/Page/NewAction.php

@@ -6,7 +6,12 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
-class NewAction extends \Magento\Backend\App\Action
+use Magento\Framework\App\Action\HttpGetActionInterface;
+
+/**
+ * Create CMS page action.
+ */
+class NewAction extends \Magento\Backend\App\Action implements HttpGetActionInterface
 {
     /**
      * Authorization level of a basic admin session

app/code/Magento/Cms/Controller/Adminhtml/Page/Save.php

@@ -1,17 +1,19 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Page;
 
-use Magento\Framework\App\Action\HttpPostActionInterface as HttpPostActionInterface;
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Backend\App\Action;
 use Magento\Cms\Model\Page;
 use Magento\Framework\App\Request\DataPersistorInterface;
 use Magento\Framework\Exception\LocalizedException;
 
+/**
+ * Save CMS page action.
+ */
 class Save extends \Magento\Backend\App\Action implements HttpPostActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Page/Widget/Chooser.php

@@ -1,15 +1,17 @@
 <?php
 /**
- *
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
 namespace Magento\Cms\Controller\Adminhtml\Page\Widget;
 
 use Magento\Framework\App\Action\HttpGetActionInterface;
-use Magento\Framework\App\Action\HttpPostActionInterface as HttpPostActionInterface;
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Backend\App\Action;
 
+/**
+ * Chooser Source action.
+ */
 class Chooser extends Action implements HttpPostActionInterface, HttpGetActionInterface
 {
     /**

app/code/Magento/Cms/Controller/Adminhtml/Wysiwyg/Images/DeleteFolder.php

@@ -6,12 +6,13 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Wysiwyg\Images;
 
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Framework\App\Filesystem\DirectoryList;
 
 /**
  * Delete image folder.
  */
-class DeleteFolder extends \Magento\Cms\Controller\Adminhtml\Wysiwyg\Images
+class DeleteFolder extends \Magento\Cms\Controller\Adminhtml\Wysiwyg\Images implements HttpPostActionInterface
 {
     /**
      * @var \Magento\Framework\Controller\Result\JsonFactory

app/code/Magento/Cms/Controller/Adminhtml/Wysiwyg/Images/Upload.php

@@ -6,12 +6,13 @@
  */
 namespace Magento\Cms\Controller\Adminhtml\Wysiwyg\Images;
 
+use Magento\Framework\App\Action\HttpPostActionInterface;
 use Magento\Framework\App\Filesystem\DirectoryList;
 
 /**
  * Upload image.
  */
-class Upload extends \Magento\Cms\Controller\Adminhtml\Wysiwyg\Images
+class Upload extends \Magento\Cms\Controller\Adminhtml\Wysiwyg\Images implements HttpPostActionInterface
 {
     /**
      * @var \Magento\Framework\Controller\Result\JsonFactory

app/code/Magento/Customer/Model/ResourceModel/CustomerRepository.php

@@ -346,7 +346,7 @@ public function getList(SearchCriteriaInterface $searchCriteria)
             ->joinAttribute('billing_telephone', 'customer_address/telephone', 'default_billing', null, 'left')
             ->joinAttribute('billing_region', 'customer_address/region', 'default_billing', null, 'left')
             ->joinAttribute('billing_country_id', 'customer_address/country_id', 'default_billing', null, 'left')
-            ->joinAttribute('company', 'customer_address/company', 'default_billing', null, 'left');
+            ->joinAttribute('billing_company', 'customer_address/company', 'default_billing', null, 'left');
 
         $this->collectionProcessor->process($searchCriteria, $collection);
 

app/code/Magento/Customer/Test/Unit/Model/ResourceModel/CustomerRepositoryTest.php

@@ -618,7 +618,7 @@ public function testGetList()
             ->willReturnSelf();
         $collection->expects($this->at(7))
             ->method('joinAttribute')
-            ->with('company', 'customer_address/company', 'default_billing', null, 'left')
+            ->with('billing_company', 'customer_address/company', 'default_billing', null, 'left')
             ->willReturnSelf();
         $this->collectionProcessorMock->expects($this->once())
             ->method('process')

app/code/Magento/Directory/Model/Config/Source/WeightUnit.php

@@ -3,6 +3,7 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+
 namespace Magento\Directory\Model\Config\Source;
 
 /**
@@ -14,10 +15,23 @@
 class WeightUnit implements \Magento\Framework\Option\ArrayInterface
 {
     /**
-     * {@inheritdoc}
+     * @var string
+     */
+    const CODE_LBS = 'lbs';
+
+    /**
+     * @var string
+     */
+    const CODE_KGS = 'kgs';
+
+    /**
+     * @inheritdoc
      */
     public function toOptionArray()
     {
-        return [['value' => 'lbs', 'label' => __('lbs')], ['value' => 'kgs', 'label' => __('kgs')]];
+        return [
+            ['value' => self::CODE_LBS, 'label' => __('lbs')],
+            ['value' => self::CODE_KGS, 'label' => __('kgs')]
+        ];
     }
 }

app/code/Magento/Email/Model/Transport.php

@@ -87,7 +87,7 @@ public function __construct(
     public function sendMessage()
     {
         try {
-            $zendMessage = Message::fromString($this->message->getRawMessage());
+            $zendMessage = Message::fromString($this->message->getRawMessage())->setEncoding('utf-8');
             if (2 === $this->isSetReturnPath && $this->returnPathValue) {
                 $zendMessage->setSender($this->returnPathValue);
             } elseif (1 === $this->isSetReturnPath && $zendMessage->getFrom()->count()) {

app/code/Magento/Sales/Model/Order/CreditmemoFactory.php

@@ -200,6 +200,7 @@ protected function initData($creditmemo, $data)
     {
         if (isset($data['shipping_amount'])) {
             $creditmemo->setBaseShippingAmount((double)$data['shipping_amount']);
+            $creditmemo->setBaseShippingInclTax((double)$data['shipping_amount']);
         }
         if (isset($data['adjustment_positive'])) {
             $creditmemo->setAdjustmentPositive($data['adjustment_positive']);
@@ -210,6 +211,8 @@ protected function initData($creditmemo, $data)
     }
 
     /**
+     * Calculate product options.
+     *
      * @param Item $orderItem
      * @param int $parentQty
      * @return int