Skip to content

Commit 9deab4e

Browse files
committed
Helper removed
1 parent d7cf39e commit 9deab4e

File tree

4 files changed

+84
-62
lines changed

4 files changed

+84
-62
lines changed

Helper/Data.php

Lines changed: 0 additions & 48 deletions
This file was deleted.

Observer/RedirectWithoutStoreCode.php

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,31 +15,30 @@ class RedirectWithoutStoreCode implements \Magento\Framework\Event\ObserverInter
1515
protected $storeManager;
1616

1717
/**
18-
* @var \Noon\HideDefaultStoreCode\Helper\Data
18+
* @var \Noon\HideDefaultStoreCode\Service\Config
1919
*/
20-
protected $helper;
20+
protected $config;
2121

2222
/**
2323
* @var \Magento\Framework\UrlInterface
2424
*/
2525
protected $url;
2626

2727
/**
28-
*
2928
* @param \Magento\Framework\App\ActionFlag $actionFlag
3029
* @param \Magento\Store\Model\StoreManagerInterface $storeManager
31-
* @param \Noon\HideDefaultStoreCode\Helper\Data $helper
30+
* @param \Noon\HideDefaultStoreCode\Service\Config $config
3231
* @param \Magento\Framework\UrlInterface $url
3332
*/
3433
public function __construct(
3534
\Magento\Framework\App\ActionFlag $actionFlag,
3635
\Magento\Store\Model\StoreManagerInterface $storeManager,
37-
\Noon\HideDefaultStoreCode\Helper\Data $helper,
36+
\Noon\HideDefaultStoreCode\Service\Config $config,
3837
\Magento\Framework\UrlInterface $url
3938
) {
4039
$this->actionFlag = $actionFlag;
4140
$this->storeManager = $storeManager;
42-
$this->helper = $helper;
41+
$this->config = $config;
4342
$this->url = $url;
4443
}
4544

@@ -58,9 +57,9 @@ public function execute(\Magento\Framework\Event\Observer $observer)
5857
$pos = strpos($url, $this->storeManager->getStore()->getBaseUrl() . $defaultStore->getCode());
5958

6059
if (
61-
$this->helper->isHideDefaultStoreCode() &&
60+
$this->config->isHideDefaultStoreCode() &&
6261
$pos !== false &&
63-
$code = $this->helper->getRedirectCode()
62+
$code = $this->config->getRedirectCode()
6463
) {
6564
$controller = $observer->getData('controller_action');
6665
$url = str_replace('/' . $defaultStore->getCode() . '/', '/', $url);

Plugin/Model/HideDefaultStoreCode.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ class HideDefaultStoreCode
66
{
77
/**
88
*
9-
* @var \Noon\HideDefaultStoreCode\Helper\Data
9+
* @var \Noon\HideDefaultStoreCode\Service\Config
1010
*/
11-
protected $helper;
11+
protected $config;
1212

1313
/**
1414
*
@@ -18,14 +18,14 @@ class HideDefaultStoreCode
1818

1919
/**
2020
*
21-
* @param \Noon\HideDefaultStoreCode\Helper\Data $helper
21+
* @param \Noon\HideDefaultStoreCode\Service\Config $config
2222
* @param \Magento\Store\Model\StoreManagerInterface $storeManager
2323
*/
2424
public function __construct(
25-
\Noon\HideDefaultStoreCode\Helper\Data $helper,
25+
\Noon\HideDefaultStoreCode\Service\Config $config,
2626
\Magento\Store\Model\StoreManagerInterface $storeManager
2727
) {
28-
$this->helper = $helper;
28+
$this->config = $config;
2929
$this->storeManager = $storeManager;
3030
}
3131

@@ -39,7 +39,7 @@ public function afterGetBaseUrl(\Magento\Store\Model\Store $subject, $url)
3939
{
4040
$websiteId = $this->storeManager->getStore()->getWebsiteId();
4141
$defaultStore = $this->storeManager->getWebsite($websiteId)->getDefaultStore();
42-
if ($this->helper->isHideDefaultStoreCode() && !is_null($defaultStore)) {
42+
if ($this->config->isHideDefaultStoreCode() && !is_null($defaultStore)) {
4343
$url = str_replace('/' . $defaultStore->getCode() . '/', '/', $url);
4444
}
4545
return $url;

Service/Config.php

Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,71 @@
1+
<?php
2+
3+
namespace Noon\HideDefaultStoreCode\Service;
4+
5+
class Config
6+
{
7+
const XML_PATH_HIDE_DEFAULT_STORE_CODE = 'web/url/hide_default_store_code';
8+
const XML_PATH_REDIRECT_TO_URL_WITHOUT_STORE_CODE = 'web/url/redirect_to_url_without_store_code';
9+
10+
const NO_REDIRECT = 0;
11+
const PERMANENT_REDIRECT = 301;
12+
const TEMPORARY_REDIRECT = 302;
13+
14+
/**
15+
* @var \Magento\Framework\App\Config\ScopeConfigInterface
16+
*/
17+
private $scopeConfig;
18+
19+
/**
20+
* Config constructor.
21+
*
22+
* @param \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig
23+
*/
24+
public function __construct(
25+
\Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig
26+
) {
27+
$this->scopeConfig = $scopeConfig;
28+
}
29+
30+
/**
31+
* @param $field
32+
* @param null $storeId
33+
* @return mixed
34+
*/
35+
private function getConfigValue($field, $storeId = null)
36+
{
37+
return $this->scopeConfig->getValue(
38+
$field,
39+
\Magento\Store\Model\ScopeInterface::SCOPE_STORE,
40+
$storeId
41+
);
42+
}
43+
44+
/**
45+
* @param null $storeId
46+
* @return mixed
47+
*/
48+
public function isHideDefaultStoreCode($storeId = null)
49+
{
50+
return $this->getConfigValue(self::XML_PATH_HIDE_DEFAULT_STORE_CODE, $storeId);
51+
}
52+
53+
/**
54+
* @param null $storeId
55+
* @return int
56+
*/
57+
public function getRedirectCode($storeId = null)
58+
{
59+
$code = $this->getConfigValue(self::XML_PATH_REDIRECT_TO_URL_WITHOUT_STORE_CODE, $storeId);
60+
61+
if ($code == 301) {
62+
return self::PERMANENT_REDIRECT;
63+
}
64+
65+
if ($code == 1) {
66+
return self::TEMPORARY_REDIRECT;
67+
}
68+
69+
return self::NO_REDIRECT;
70+
}
71+
}

0 commit comments

Comments
 (0)