Merge branch '2.4-develop' into 2.4-develop-com-pr4

Author: mykhailomatiola

app/code/Magento/Backend/Block/Media/Uploader.php

@@ -87,7 +87,7 @@ protected function _construct()
 
         $this->setId($this->getId() . '_Uploader');
 
-        $uploadUrl = $this->_urlBuilder->addSessionParam()->getUrl('adminhtml/*/upload');
+        $uploadUrl = $this->_urlBuilder->getUrl('adminhtml/*/upload');
         $this->getConfig()->setUrl($uploadUrl);
         $this->getConfig()->setParams(['form_key' => $this->getFormKey()]);
         $this->getConfig()->setFileField('file');

app/code/Magento/Catalog/Block/Adminhtml/Product/Helper/Form/Gallery/Content.php

@@ -95,7 +95,7 @@ protected function _prepareLayout()
         );
 
         $this->getUploader()->getConfig()->setUrl(
-            $this->_urlBuilder->addSessionParam()->getUrl('catalog/product_gallery/upload')
+            $this->_urlBuilder->getUrl('catalog/product_gallery/upload')
         )->setFileField(
             'image'
         )->setFilters(

app/code/Magento/Catalog/Model/Product/Url.php

@@ -101,15 +101,10 @@ public function getUrlInStore(\Magento\Catalog\Model\Product $product, $params =
      */
     public function getProductUrl($product, $useSid = null)
     {
-        if ($useSid === null) {
-            $useSid = $this->sidResolver->getUseSessionInUrl();
-        }
-
         $params = [];
         if (!$useSid) {
             $params['_nosid'] = true;
         }
-
         return $this->getUrl($product, $params);
     }
 

app/code/Magento/Catalog/Test/Unit/Model/Product/UrlTest.php

@@ -159,12 +159,11 @@ public function testGetUrl(
                 $this->assertEquals($requestPathProduct, $this->model->getUrlInStore($product, $routeParams));
                 break;
             case 'getProductUrl':
-                $this->assertEquals($requestPathProduct, $this->model->getProductUrl($product, true));
+                $this->assertEquals($requestPathProduct, $this->model->getProductUrl($product, null));
                 $this->sidResolver
-                    ->expects($this->once())
+                    ->expects($this->never())
                     ->method('getUseSessionInUrl')
                     ->will($this->returnValue(true));
-                $this->assertEquals($requestPathProduct, $this->model->getProductUrl($product, null));
                 break;
         }
     }
@@ -212,7 +211,7 @@ public function getUrlDataProvider()
                 1,
                 1,
                 [],
-                ['_direct' => '/product/url/path', '_query' => []],
+                ['_direct' => '/product/url/path', '_query' => [], '_nosid' => true],
                 null,
                 null,
             ]

app/code/Magento/Cms/Block/Adminhtml/Wysiwyg/Images/Content/Uploader.php

@@ -35,6 +35,8 @@ public function __construct(
     }
 
     /**
+     * Constructor
+     *
      * @return void
      */
     protected function _construct()
@@ -49,7 +51,7 @@ protected function _construct()
             $files[] = '*.' . $ext;
         }
         $this->getConfig()->setUrl(
-            $this->_urlBuilder->addSessionParam()->getUrl('cms/*/upload', ['type' => $type])
+            $this->_urlBuilder->getUrl('cms/*/upload', ['type' => $type])
         )->setFileField(
             'image'
         )->setFilters(

app/code/Magento/ConfigurableProduct/Ui/DataProvider/Product/Form/Modifier/ConfigurablePanel.php

@@ -447,7 +447,7 @@ protected function getRows()
                                 'smallImage' => '${$.provider}:${$.parentScope}.small_image',
                             ],
                             'uploaderConfig' => [
-                                'url' => $this->urlBuilder->addSessionParam()->getUrl(
+                                'url' => $this->urlBuilder->getUrl(
                                     'catalog/product_gallery/upload'
                                 ),
                             ],

app/code/Magento/Customer/Model/Session.php

@@ -19,6 +19,7 @@
  * @method string getNoReferer()
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  * @SuppressWarnings(PHPMD.CookieAndSessionMisuse)
+ * @SuppressWarnings(PHPMD.TooManyFields)
  * @since 100.0.2
  */
 class Session extends \Magento\Framework\Session\SessionManager
@@ -108,6 +109,11 @@ class Session extends \Magento\Framework\Session\SessionManager
      */
     protected $response;
 
+    /**
+     * @var AccountConfirmation
+     */
+    private $accountConfirmation;
+
     /**
      * Session constructor.
      *
@@ -511,13 +517,6 @@ public function authenticate($loginUrl = null)
             $this->response->setRedirect($loginUrl);
         } else {
             $arguments = $this->_customerUrl->getLoginUrlParams();
-            if ($this->_createUrl()->getUseSession()) {
-                $arguments += [
-                    '_query' => [
-                        $this->sidResolver->getSessionIdQueryParam($this->_session) => $this->_session->getSessionId(),
-                    ]
-                ];
-            }
             $this->response->setRedirect(
                 $this->_createUrl()->getUrl(\Magento\Customer\Model\Url::ROUTE_ACCOUNT_LOGIN, $arguments)
             );
@@ -535,8 +534,6 @@ public function authenticate($loginUrl = null)
      */
     protected function _setAuthUrl($key, $url)
     {
-        $url = $this->_coreUrl->removeRequestParam($url, $this->sidResolver->getSessionIdQueryParam($this));
-        // Add correct session ID to URL if needed
         $url = $this->_createUrl()->getRebuiltUrl($url);
         return $this->storage->setData($key, $url);
     }

app/code/Magento/Customer/Test/Unit/Model/SessionTest.php

@@ -147,10 +147,10 @@ public function testAuthenticate()
         $urlMock->expects($this->once())
             ->method('getRebuiltUrl')
             ->willReturn('');
-        $this->urlFactoryMock->expects($this->exactly(4))
+        $this->urlFactoryMock->expects($this->exactly(3))
             ->method('create')
             ->willReturn($urlMock);
-        $urlMock->expects($this->once())
+        $urlMock->expects($this->never())
             ->method('getUseSession')
             ->willReturn(false);
 

app/code/Magento/Downloadable/Block/Adminhtml/Catalog/Product/Edit/Tab/Downloadable/Links.php

@@ -11,7 +11,7 @@
  * @author      Magento Core Team <core@magentocommerce.com>
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  *
- * @deprecated
+ * @deprecated in favor of new class which adds grid links
  * @see \Magento\Downloadable\Ui\DataProvider\Product\Form\Modifier\Links
  */
 class Links extends \Magento\Backend\Block\Template
@@ -398,7 +398,7 @@ public function getFileFieldName($type)
      */
     public function getUploadUrl($type)
     {
-        return $this->_urlFactory->create()->addSessionParam()->getUrl(
+        return $this->_urlFactory->create()->getUrl(
             'adminhtml/downloadable_file/upload',
             ['type' => $type, '_secure' => true]
         );

app/code/Magento/Downloadable/Block/Adminhtml/Catalog/Product/Edit/Tab/Downloadable/Samples.php

@@ -10,7 +10,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  *
- * @deprecated
+ * @deprecated because of new class which adds grids samples
  * @see \Magento\Downloadable\Ui\DataProvider\Product\Form\Modifier\Samples
  */
 class Samples extends \Magento\Backend\Block\Widget
@@ -250,7 +250,7 @@ public function getUploadButtonHtml()
      */
     public function getConfigJson()
     {
-        $url = $this->_urlFactory->create()->addSessionParam()->getUrl(
+        $url = $this->_urlFactory->create()->getUrl(
             'adminhtml/downloadable_file/upload',
             ['type' => 'samples', '_secure' => true]
         );