From 80384fc5cbcb715e210422222feca07b8359053b Mon Sep 17 00:00:00 2001 From: alojua Date: Tue, 17 Jul 2018 16:29:54 +0200 Subject: [PATCH 1/8] [Fix] Do not modify current list of countries with require states during setup upgrade. Also mark getCountriesWithRequiredStates() as deprecated because is no longer needed --- .../ResourceModel/Country/Collection.php | 4 + .../Magento/Directory/Setup/InstallData.php | 11 -- .../Magento/Directory/Setup/UpgradeData.php | 144 ++++++++---------- app/code/Magento/Directory/etc/di.xml | 4 - 4 files changed, 69 insertions(+), 94 deletions(-) diff --git a/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php b/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php index eb2b7cef0b707..93a6149e10fcf 100644 --- a/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php +++ b/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php @@ -64,6 +64,8 @@ class Collection extends \Magento\Framework\Model\ResourceModel\Db\Collection\Ab private $allowedCountriesReader; /** + * @deprecated + * * @var string[] * @since 100.1.0 */ @@ -349,6 +351,8 @@ public function setForegroundCountries($foregroundCountries) } /** + * @deprecated use \Magento\Directory\Helper\Data::getCountriesWithStatesRequired() instead + * * Get list of countries with required states * * @return \Magento\Directory\Model\Country[] diff --git a/app/code/Magento/Directory/Setup/InstallData.php b/app/code/Magento/Directory/Setup/InstallData.php index 33c3ae4558e16..12c3fba214804 100644 --- a/app/code/Magento/Directory/Setup/InstallData.php +++ b/app/code/Magento/Directory/Setup/InstallData.php @@ -842,16 +842,5 @@ public function install(ModuleDataSetupInterface $setup, ModuleContextInterface 'value' => 1 ] ); - - $countries = $this->directoryData->getCountryCollection()->getCountriesWithRequiredStates(); - $setup->getConnection()->insert( - $setup->getTable('core_config_data'), - [ - 'scope' => 'default', - 'scope_id' => 0, - 'path' => Data::XML_PATH_STATES_REQUIRED, - 'value' => implode(',', array_keys($countries)) - ] - ); } } diff --git a/app/code/Magento/Directory/Setup/UpgradeData.php b/app/code/Magento/Directory/Setup/UpgradeData.php index 4ee9ea33673d7..f4e9c1206525d 100644 --- a/app/code/Magento/Directory/Setup/UpgradeData.php +++ b/app/code/Magento/Directory/Setup/UpgradeData.php @@ -32,7 +32,7 @@ public function __construct(Data $directoryData) } /** - * Upgrades data for Directry module. + * Upgrades data for Directory module. * * @param ModuleDataSetupInterface $setup * @param ModuleContextInterface $context @@ -41,10 +41,10 @@ public function __construct(Data $directoryData) public function upgrade(ModuleDataSetupInterface $setup, ModuleContextInterface $context) { if (version_compare($context->getVersion(), '2.0.1', '<')) { - $this->addCountryRegions($setup, $this->getDataForCroatia()); + $this->addCountryRegions($setup, 'IN', $this->getDataForCroatia()); } if (version_compare($context->getVersion(), '2.0.2', '<')) { - $this->addCountryRegions($setup, $this->getDataForIndia()); + $this->addCountryRegions($setup, 'HR', $this->getDataForIndia()); } } @@ -56,27 +56,27 @@ public function upgrade(ModuleDataSetupInterface $setup, ModuleContextInterface private function getDataForCroatia() { return [ - ['HR', 'HR-01', 'Zagrebačka županija'], - ['HR', 'HR-02', 'Krapinsko-zagorska županija'], - ['HR', 'HR-03', 'Sisačko-moslavačka županija'], - ['HR', 'HR-04', 'Karlovačka županija'], - ['HR', 'HR-05', 'Varaždinska županija'], - ['HR', 'HR-06', 'Koprivničko-križevačka županija'], - ['HR', 'HR-07', 'Bjelovarsko-bilogorska županija'], - ['HR', 'HR-08', 'Primorsko-goranska županija'], - ['HR', 'HR-09', 'Ličko-senjska županija'], - ['HR', 'HR-10', 'Virovitičko-podravska županija'], - ['HR', 'HR-11', 'Požeško-slavonska županija'], - ['HR', 'HR-12', 'Brodsko-posavska županija'], - ['HR', 'HR-13', 'Zadarska županija'], - ['HR', 'HR-14', 'Osječko-baranjska županija'], - ['HR', 'HR-15', 'Šibensko-kninska županija'], - ['HR', 'HR-16', 'Vukovarsko-srijemska županija'], - ['HR', 'HR-17', 'Splitsko-dalmatinska županija'], - ['HR', 'HR-18', 'Istarska županija'], - ['HR', 'HR-19', 'Dubrovačko-neretvanska županija'], - ['HR', 'HR-20', 'Međimurska županija'], - ['HR', 'HR-21', 'Grad Zagreb'] + 'HR-01' => 'Zagrebačka županija', + 'HR-02' => 'Krapinsko-zagorska županija', + 'HR-03' => 'Sisačko-moslavačka županija', + 'HR-04' => 'Karlovačka županija', + 'HR-05' => 'Varaždinska županija', + 'HR-06' => 'Koprivničko-križevačka županija', + 'HR-07' => 'Bjelovarsko-bilogorska županija', + 'HR-08' => 'Primorsko-goranska županija', + 'HR-09' => 'Ličko-senjska županija', + 'HR-10' => 'Virovitičko-podravska županija', + 'HR-11' => 'Požeško-slavonska županija', + 'HR-12' => 'Brodsko-posavska županija', + 'HR-13' => 'Zadarska županija', + 'HR-14' => 'Osječko-baranjska županija', + 'HR-15' => 'Šibensko-kninska županija', + 'HR-16' => 'Vukovarsko-srijemska županija', + 'HR-17' => 'Splitsko-dalmatinska županija', + 'HR-18' => 'Istarska županija', + 'HR-19' => 'Dubrovačko-neretvanska županija', + 'HR-20' => 'Međimurska županija', + 'HR-21' => 'Grad Zagreb', ]; } @@ -88,42 +88,42 @@ private function getDataForCroatia() private function getDataForIndia() { return [ - ['IN', 'AN', 'Andaman and Nicobar Islands'], - ['IN', 'AP', 'Andhra Pradesh'], - ['IN', 'AR', 'Arunachal Pradesh'], - ['IN', 'AS', 'Assam'], - ['IN', 'BR', 'Bihar'], - ['IN', 'CH', 'Chandigarh'], - ['IN', 'CT', 'Chhattisgarh'], - ['IN', 'DN', 'Dadra and Nagar Haveli'], - ['IN', 'DD', 'Daman and Diu'], - ['IN', 'DL', 'Delhi'], - ['IN', 'GA', 'Goa'], - ['IN', 'GJ', 'Gujarat'], - ['IN', 'HR', 'Haryana'], - ['IN', 'HP', 'Himachal Pradesh'], - ['IN', 'JK', 'Jammu and Kashmir'], - ['IN', 'JH', 'Jharkhand'], - ['IN', 'KA', 'Karnataka'], - ['IN', 'KL', 'Kerala'], - ['IN', 'LD', 'Lakshadweep'], - ['IN', 'MP', 'Madhya Pradesh'], - ['IN', 'MH', 'Maharashtra'], - ['IN', 'MN', 'Manipur'], - ['IN', 'ML', 'Meghalaya'], - ['IN', 'MZ', 'Mizoram'], - ['IN', 'NL', 'Nagaland'], - ['IN', 'OR', 'Odisha'], - ['IN', 'PY', 'Puducherry'], - ['IN', 'PB', 'Punjab'], - ['IN', 'RJ', 'Rajasthan'], - ['IN', 'SK', 'Sikkim'], - ['IN', 'TN', 'Tamil Nadu'], - ['IN', 'TG', 'Telangana'], - ['IN', 'TR', 'Tripura'], - ['IN', 'UP', 'Uttar Pradesh'], - ['IN', 'UT', 'Uttarakhand'], - ['IN', 'WB', 'West Bengal'] + 'AN' => 'Andaman and Nicobar Islands', + 'AP' => 'Andhra Pradesh', + 'AR' => 'Arunachal Pradesh', + 'AS' => 'Assam', + 'BR' => 'Bihar', + 'CH' => 'Chandigarh', + 'CT' => 'Chhattisgarh', + 'DN' => 'Dadra and Nagar Haveli', + 'DD' => 'Daman and Diu', + 'DL' => 'Delhi', + 'GA' => 'Goa', + 'GJ' => 'Gujarat', + 'HR' => 'Haryana', + 'HP' => 'Himachal Pradesh', + 'JK' => 'Jammu and Kashmir', + 'JH' => 'Jharkhand', + 'KA' => 'Karnataka', + 'KL' => 'Kerala', + 'LD' => 'Lakshadweep', + 'MP' => 'Madhya Pradesh', + 'MH' => 'Maharashtra', + 'MN' => 'Manipur', + 'ML' => 'Meghalaya', + 'MZ' => 'Mizoram', + 'NL' => 'Nagaland', + 'OR' => 'Odisha', + 'PY' => 'Puducherry', + 'PB' => 'Punjab', + 'RJ' => 'Rajasthan', + 'SK' => 'Sikkim', + 'TN' => 'Tamil Nadu', + 'TG' => 'Telangana', + 'TR' => 'Tripura', + 'UP' => 'Uttar Pradesh', + 'UT' => 'Uttarakhand', + 'WB' => 'West Bengal', ]; } @@ -131,36 +131,22 @@ private function getDataForIndia() * Add country regions data to appropriate tables. * * @param ModuleDataSetupInterface $setup + * @param string $countryId * @param array $data * @return void */ - private function addCountryRegions(ModuleDataSetupInterface $setup, array $data) + private function addCountryRegions(ModuleDataSetupInterface $setup, string $countryId, array $data) { /** * Fill table directory/country_region * Fill table directory/country_region_name for en_US locale */ - foreach ($data as $row) { - $bind = ['country_id' => $row[0], 'code' => $row[1], 'default_name' => $row[2]]; + foreach ($data as $code => $name) { + $bind = ['country_id' => $countryId, 'code' => $code, 'default_name' => $name]; $setup->getConnection()->insert($setup->getTable('directory_country_region'), $bind); $regionId = $setup->getConnection()->lastInsertId($setup->getTable('directory_country_region')); - $bind = ['locale' => 'en_US', 'region_id' => $regionId, 'name' => $row[2]]; + $bind = ['locale' => 'en_US', 'region_id' => $regionId, 'name' => $name]; $setup->getConnection()->insert($setup->getTable('directory_country_region_name'), $bind); } - /** - * Upgrade core_config_data general/region/state_required field. - */ - $countries = $this->directoryData->getCountryCollection()->getCountriesWithRequiredStates(); - $setup->getConnection()->update( - $setup->getTable('core_config_data'), - [ - 'value' => implode(',', array_keys($countries)) - ], - [ - 'scope="default"', - 'scope_id=0', - 'path=?' => Data::XML_PATH_STATES_REQUIRED - ] - ); } } diff --git a/app/code/Magento/Directory/etc/di.xml b/app/code/Magento/Directory/etc/di.xml index 3bc50e3bc0316..b1b94d6d1edb5 100644 --- a/app/code/Magento/Directory/etc/di.xml +++ b/app/code/Magento/Directory/etc/di.xml @@ -34,10 +34,6 @@ DirectoryHelperDataProxy - - FR - DE - From 80001a1692f43acd5fe648e06896e2323c8b6a4f Mon Sep 17 00:00:00 2001 From: alojua Date: Thu, 6 Sep 2018 16:03:43 +0200 Subject: [PATCH 2/8] [Fix] Set country codes in right places --- app/code/Magento/Directory/Setup/UpgradeData.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/code/Magento/Directory/Setup/UpgradeData.php b/app/code/Magento/Directory/Setup/UpgradeData.php index f4e9c1206525d..9a276d7cc7b80 100644 --- a/app/code/Magento/Directory/Setup/UpgradeData.php +++ b/app/code/Magento/Directory/Setup/UpgradeData.php @@ -41,10 +41,10 @@ public function __construct(Data $directoryData) public function upgrade(ModuleDataSetupInterface $setup, ModuleContextInterface $context) { if (version_compare($context->getVersion(), '2.0.1', '<')) { - $this->addCountryRegions($setup, 'IN', $this->getDataForCroatia()); + $this->addCountryRegions($setup, 'HR', $this->getDataForCroatia()); } if (version_compare($context->getVersion(), '2.0.2', '<')) { - $this->addCountryRegions($setup, 'HR', $this->getDataForIndia()); + $this->addCountryRegions($setup, 'IN', $this->getDataForIndia()); } } From 875c3948257032c34cdc2c910ebbebb41a754c6b Mon Sep 17 00:00:00 2001 From: alojua Date: Thu, 6 Sep 2018 16:11:50 +0200 Subject: [PATCH 3/8] [Fix] Add required countries only for IN and HR while keeping previous functionality --- .../Model/ResourceModel/Country/Collection.php | 2 -- app/code/Magento/Directory/Setup/InstallData.php | 11 +++++++++++ app/code/Magento/Directory/Setup/UpgradeData.php | 15 +++++++++++++++ app/code/Magento/Directory/etc/di.xml | 4 ++++ 4 files changed, 30 insertions(+), 2 deletions(-) diff --git a/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php b/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php index 93a6149e10fcf..da5ad64f8b239 100644 --- a/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php +++ b/app/code/Magento/Directory/Model/ResourceModel/Country/Collection.php @@ -64,8 +64,6 @@ class Collection extends \Magento\Framework\Model\ResourceModel\Db\Collection\Ab private $allowedCountriesReader; /** - * @deprecated - * * @var string[] * @since 100.1.0 */ diff --git a/app/code/Magento/Directory/Setup/InstallData.php b/app/code/Magento/Directory/Setup/InstallData.php index 12c3fba214804..33c3ae4558e16 100644 --- a/app/code/Magento/Directory/Setup/InstallData.php +++ b/app/code/Magento/Directory/Setup/InstallData.php @@ -842,5 +842,16 @@ public function install(ModuleDataSetupInterface $setup, ModuleContextInterface 'value' => 1 ] ); + + $countries = $this->directoryData->getCountryCollection()->getCountriesWithRequiredStates(); + $setup->getConnection()->insert( + $setup->getTable('core_config_data'), + [ + 'scope' => 'default', + 'scope_id' => 0, + 'path' => Data::XML_PATH_STATES_REQUIRED, + 'value' => implode(',', array_keys($countries)) + ] + ); } } diff --git a/app/code/Magento/Directory/Setup/UpgradeData.php b/app/code/Magento/Directory/Setup/UpgradeData.php index 9a276d7cc7b80..d149a83ce8e9d 100644 --- a/app/code/Magento/Directory/Setup/UpgradeData.php +++ b/app/code/Magento/Directory/Setup/UpgradeData.php @@ -148,5 +148,20 @@ private function addCountryRegions(ModuleDataSetupInterface $setup, string $coun $bind = ['locale' => 'en_US', 'region_id' => $regionId, 'name' => $name]; $setup->getConnection()->insert($setup->getTable('directory_country_region_name'), $bind); } + + /** + * Upgrade core_config_data general/region/state_required field. + */ + $setup->getConnection()->update( + $setup->getTable('core_config_data'), + [ + 'value' => new \Zend_Db_Expr("CONCAT(value, '," . $countryId . "')") + ], + [ + 'scope="default"', + 'scope_id=0', + 'path=?' => Data::XML_PATH_STATES_REQUIRED + ] + ); } } diff --git a/app/code/Magento/Directory/etc/di.xml b/app/code/Magento/Directory/etc/di.xml index b1b94d6d1edb5..3bc50e3bc0316 100644 --- a/app/code/Magento/Directory/etc/di.xml +++ b/app/code/Magento/Directory/etc/di.xml @@ -34,6 +34,10 @@ DirectoryHelperDataProxy + + FR + DE + From 9d8b04a0c9700e8a5bfe65911c8151672f202de6 Mon Sep 17 00:00:00 2001 From: Sven Reichel Date: Sun, 16 Sep 2018 21:55:53 +0200 Subject: [PATCH 4/8] Cast products "getStoreId()" to int, closes #18079 --- app/code/Magento/Catalog/Model/Category.php | 13 +++++++++---- app/code/Magento/Catalog/Model/Product.php | 4 ++-- 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/app/code/Magento/Catalog/Model/Category.php b/app/code/Magento/Catalog/Model/Category.php index 57cfa75380851..a12eae06d7ec9 100644 --- a/app/code/Magento/Catalog/Model/Category.php +++ b/app/code/Magento/Catalog/Model/Category.php @@ -71,6 +71,11 @@ class Category extends \Magento\Catalog\Model\AbstractModel implements const CACHE_TAG = 'cat_c'; + /** + * Category Store Id + */ + const STORE_ID = 'store_id'; + /**#@+ * Constants */ @@ -588,12 +593,12 @@ public function getStoreIds() * * If store id is underfined for category return current active store id * - * @return integer + * @return int */ public function getStoreId() { - if ($this->hasData('store_id')) { - return (int)$this->_getData('store_id'); + if ($this->hasData(self::STORE_ID)) { + return (int)$this->_getData(self::STORE_ID); } return (int)$this->_storeManager->getStore()->getId(); } @@ -609,7 +614,7 @@ public function setStoreId($storeId) if (!is_numeric($storeId)) { $storeId = $this->_storeManager->getStore($storeId)->getId(); } - $this->setData('store_id', $storeId); + $this->setData(self::STORE_ID, $storeId); $this->getResource()->setStoreId($storeId); return $this; } diff --git a/app/code/Magento/Catalog/Model/Product.php b/app/code/Magento/Catalog/Model/Product.php index 850fb21777585..69a8ffd1665c7 100644 --- a/app/code/Magento/Catalog/Model/Product.php +++ b/app/code/Magento/Catalog/Model/Product.php @@ -525,9 +525,9 @@ protected function getCustomAttributesCodes() public function getStoreId() { if ($this->hasData(self::STORE_ID)) { - return $this->getData(self::STORE_ID); + return (int)$this->getData(self::STORE_ID); } - return $this->_storeManager->getStore()->getId(); + return (int)$this->_storeManager->getStore()->getId(); } /** From 4e62fe32bf6f27b583eeb8d5d2309c7abebd5535 Mon Sep 17 00:00:00 2001 From: adammada Date: Mon, 24 Sep 2018 09:58:16 +0200 Subject: [PATCH 5/8] fix wysiwyg editor not decoding base64 filenames special chars --- lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js b/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js index f2b3365555f3c..3bae5f8eb48b4 100755 --- a/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js +++ b/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js @@ -463,6 +463,11 @@ define([ var url = this.makeDirectiveUrl('%directive%').replace(/([$^.?*!+:=()\[\]{}|\\])/g, '\\$1'), reg = new RegExp(url.replace('%directive%', '([a-zA-Z0-9,_-]+)')); + var uriReg = /___directive\/(.*)\/key\//g; + content = content.replace(uriReg, function (match) { + return decodeURIComponent(match); + }); + return content.gsub(reg, function (match) { //eslint-disable-line no-extra-bind return Base64.mageDecode(match[1]); }); From 484bbba6407f03c7b2261992c0ec2433d00a2cb4 Mon Sep 17 00:00:00 2001 From: Dmytro Cheshun Date: Tue, 18 Sep 2018 15:44:00 +0300 Subject: [PATCH 6/8] Change sort order for customer group options #18101 --- .../Customer/Model/GroupManagement.php | 26 +++++++++++++++---- 1 file changed, 21 insertions(+), 5 deletions(-) diff --git a/app/code/Magento/Customer/Model/GroupManagement.php b/app/code/Magento/Customer/Model/GroupManagement.php index 47d7d7ad1ac41..73396edec6622 100644 --- a/app/code/Magento/Customer/Model/GroupManagement.php +++ b/app/code/Magento/Customer/Model/GroupManagement.php @@ -8,14 +8,16 @@ namespace Magento\Customer\Model; use Magento\Customer\Api\Data\GroupInterface; +use Magento\Customer\Api\Data\GroupInterfaceFactory; +use Magento\Customer\Api\GroupRepositoryInterface; +use Magento\Framework\Api\FilterBuilder; use Magento\Framework\Api\SearchCriteriaBuilder; +use Magento\Framework\Api\SortOrderBuilder; use Magento\Framework\App\Config\ScopeConfigInterface; -use Magento\Framework\Api\FilterBuilder; +use Magento\Framework\App\ObjectManager; +use Magento\Framework\Data\Collection; use Magento\Framework\Exception\NoSuchEntityException; use Magento\Store\Model\StoreManagerInterface; -use Magento\Customer\Api\GroupRepositoryInterface; -use Magento\Customer\Api\Data\GroupInterfaceFactory; -use Magento\Customer\Model\GroupFactory; /** * @SuppressWarnings(PHPMD.CouplingBetweenObjects) @@ -65,6 +67,11 @@ class GroupManagement implements \Magento\Customer\Api\GroupManagementInterface */ protected $filterBuilder; + /** + * @var SortOrderBuilder + */ + private $sortOrderBuilder; + /** * @param StoreManagerInterface $storeManager * @param ScopeConfigInterface $scopeConfig @@ -73,6 +80,7 @@ class GroupManagement implements \Magento\Customer\Api\GroupManagementInterface * @param GroupInterfaceFactory $groupDataFactory * @param SearchCriteriaBuilder $searchCriteriaBuilder * @param FilterBuilder $filterBuilder + * @param SortOrderBuilder $sortOrderBuilder */ public function __construct( StoreManagerInterface $storeManager, @@ -81,7 +89,8 @@ public function __construct( GroupRepositoryInterface $groupRepository, GroupInterfaceFactory $groupDataFactory, SearchCriteriaBuilder $searchCriteriaBuilder, - FilterBuilder $filterBuilder + FilterBuilder $filterBuilder, + SortOrderBuilder $sortOrderBuilder = null ) { $this->storeManager = $storeManager; $this->scopeConfig = $scopeConfig; @@ -90,6 +99,8 @@ public function __construct( $this->groupDataFactory = $groupDataFactory; $this->searchCriteriaBuilder = $searchCriteriaBuilder; $this->filterBuilder = $filterBuilder; + $this->sortOrderBuilder = $sortOrderBuilder ?: ObjectManager::getInstance() + ->get(SortOrderBuilder::class); } /** @@ -155,9 +166,14 @@ public function getLoggedInGroups() ->setConditionType('neq') ->setValue(self::CUST_GROUP_ALL) ->create(); + $groupNameSortOrder = $this->sortOrderBuilder + ->setField('customer_group_code') + ->setDirection(Collection::SORT_ORDER_ASC) + ->create(); $searchCriteria = $this->searchCriteriaBuilder ->addFilters($notLoggedInFilter) ->addFilters($groupAll) + ->addSortOrder($groupNameSortOrder) ->create(); return $this->groupRepository->getList($searchCriteria)->getItems(); } From dd9b2948a03abdc487e712d4237a9180484cb17b Mon Sep 17 00:00:00 2001 From: Yaroslav Rogoza Date: Wed, 19 Sep 2018 14:42:51 +0200 Subject: [PATCH 7/8] Minor code style fixes --- app/code/Magento/Customer/Model/GroupManagement.php | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/app/code/Magento/Customer/Model/GroupManagement.php b/app/code/Magento/Customer/Model/GroupManagement.php index 73396edec6622..eb5d90f2fd7ea 100644 --- a/app/code/Magento/Customer/Model/GroupManagement.php +++ b/app/code/Magento/Customer/Model/GroupManagement.php @@ -20,6 +20,8 @@ use Magento\Store\Model\StoreManagerInterface; /** + * The class contains methods for getting information about a customer group + * * @SuppressWarnings(PHPMD.CouplingBetweenObjects) */ class GroupManagement implements \Magento\Customer\Api\GroupManagementInterface @@ -104,7 +106,7 @@ public function __construct( } /** - * {@inheritdoc} + * @inheritdoc */ public function isReadonly($groupId) { @@ -118,7 +120,7 @@ public function isReadonly($groupId) } /** - * {@inheritdoc} + * @inheritdoc */ public function getDefaultGroup($storeId = null) { @@ -144,7 +146,7 @@ public function getDefaultGroup($storeId = null) } /** - * {@inheritdoc} + * @inheritdoc */ public function getNotLoggedInGroup() { @@ -152,7 +154,7 @@ public function getNotLoggedInGroup() } /** - * {@inheritdoc} + * @inheritdoc */ public function getLoggedInGroups() { @@ -179,7 +181,7 @@ public function getLoggedInGroups() } /** - * {@inheritdoc} + * @inheritdoc */ public function getAllCustomersGroup() { From bf2efffff0da82f33796c82d643403545466911d Mon Sep 17 00:00:00 2001 From: Stanislav Idolov Date: Thu, 27 Sep 2018 11:20:36 +0300 Subject: [PATCH 8/8] Fixed code style issue --- lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js b/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js index 3bae5f8eb48b4..ed105bd2c18f5 100755 --- a/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js +++ b/lib/web/mage/adminhtml/wysiwyg/tiny_mce/setup.js @@ -461,9 +461,9 @@ define([ decodeDirectives: function (content) { // escape special chars in directives url to use it in regular expression var url = this.makeDirectiveUrl('%directive%').replace(/([$^.?*!+:=()\[\]{}|\\])/g, '\\$1'), - reg = new RegExp(url.replace('%directive%', '([a-zA-Z0-9,_-]+)')); + reg = new RegExp(url.replace('%directive%', '([a-zA-Z0-9,_-]+)')), + uriReg = /___directive\/(.*)\/key\//g; - var uriReg = /___directive\/(.*)\/key\//g; content = content.replace(uriReg, function (match) { return decodeURIComponent(match); });