Skip to content

Commit 0664ace

Browse files
author
Oleksandr Gorkun
committed
MC-29561: Deprecate SID query parameter-related methods
1 parent 2ae9498 commit 0664ace

File tree

3 files changed

+6
-12
lines changed

3 files changed

+6
-12
lines changed

app/code/Magento/Catalog/Model/Template/Filter.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ public function setUseAbsoluteLinks($flag)
8080
*
8181
* @param bool $flag
8282
* @return $this
83+
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
8384
* @deprecated SID query parameter is not used in URLs anymore.
8485
*/
8586
public function setUseSessionInUrl($flag)

app/code/Magento/Email/Model/Template/Filter.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,7 @@ public function setUseAbsoluteLinks($flag)
264264
*
265265
* @param bool $flag
266266
* @return $this
267+
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
267268
* @deprecated SID query parameter is not used in URLs anymore.
268269
*/
269270
public function setUseSessionInUrl($flag)

app/code/Magento/Store/Model/StoreSwitcher/CleanTargetUrl.php

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414

1515
/**
1616
* Remove SID, from_store, store from target url.
17+
*
18+
* Used in store-switching process in HTML frontend.
1719
*/
1820
class CleanTargetUrl implements StoreSwitcherInterface
1921
{
@@ -22,29 +24,18 @@ class CleanTargetUrl implements StoreSwitcherInterface
2224
*/
2325
private $urlHelper;
2426

25-
/**
26-
* @var \Magento\Framework\Session\Generic
27-
*/
28-
private $session;
29-
30-
/**
31-
* @var \Magento\Framework\Session\SidResolverInterface
32-
*/
33-
private $sidResolver;
34-
3527
/**
3628
* @param UrlHelper $urlHelper
3729
* @param \Magento\Framework\Session\Generic $session
3830
* @param \Magento\Framework\Session\SidResolverInterface $sidResolver
31+
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
3932
*/
4033
public function __construct(
4134
UrlHelper $urlHelper,
4235
\Magento\Framework\Session\Generic $session,
4336
\Magento\Framework\Session\SidResolverInterface $sidResolver
4437
) {
4538
$this->urlHelper = $urlHelper;
46-
$this->session = $session;
47-
$this->sidResolver = $sidResolver;
4839
}
4940

5041
/**
@@ -54,6 +45,7 @@ public function __construct(
5445
* @param StoreInterface $targetStore store where to go to
5546
* @param string $redirectUrl original url requested for redirect after switching
5647
* @return string redirect url
48+
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
5749
*/
5850
public function switch(StoreInterface $fromStore, StoreInterface $targetStore, string $redirectUrl): string
5951
{

0 commit comments

Comments
 (0)