Skip to content

Commit 65e1ba1

Browse files
committed
2.0.0.0-dev88
* Fixed bugs: * Fixed an issue when PayPal Express Checkout Payflow Edition and PayPal Payments Advanced were available for multiple checkout * Fixed an issue when the Bill me later button did not redirect to https when secure url was enabled for frontend * Fixed an issue when the Billing agreement option was available in multishipping checkout, even if there were no signed agreements * Fixed an issue when DoExpressCheckout request instead of DoCapture did not allow to do refund, when using PayPal Express Checkout Payflow Edition * Fixed an issue when eWay was not present on checkout if Base Currency was set to AUD * Fixed an issue with fatal error occurring when placing order via SagePay with 3D Secure enabled * Fixed an issue when the FedEx shipping method had no option to specify unit for weight attribute * Fixed an issue with inability to create credit memo for PalPal Express Checkout/Payments Pro/Payments Pro Hosted Solution (NVP family), if partial refund was initiated on the PayPal side * Fixed an issue when a guest user could not return product to store, if product was paid using PayPal * Fixed an issue when PayPal Payments Pro Hosted Solution Fraud protection did not work properly * Fixed an issue when JavaScript took values from default config for payment methods and used them on the website scope * Fixed an issue with incorrect address in request to shipping carrier (DHL International) in case the address contained diacritic letters * Fixed an issue when it was possible to hack currency in PayPal Website Payments Standard * Fixed an issue when no rows were added to the PayPal Settlement report grid while fetching it from custom server * Fixed an issue when order had the Suspected Fraud status after creating partial invoice on the PayPal side * Fixed an issue when PayPal Payflow Pro did not properly implement CUSTREF and INVNUM * Fixed an issue with PayPal errors handling during IPN postback * Fixed an issue when the Paypal Express Checkout button was not available on product page for several product types * Fixed an issue with PayPal Payflow Pro and Payflow Link broken unit tests * Fixed an issue when PayPal Payments Pro Hosted Solution had the City parameter duplicated in the State parameter for UK * Fixed an issue with remove multiple HTTP 100/101 headers * Fixed an issue when SagePay did not transfer shopping cart information * Fixed an issue when transaction records were absent on Transaction tab for Ogone * Fixed an issue when partial cancel with SagePay Direct was unavailable * Fixed an issue when order did not place using PayPal Payments Pro Hosted Solution * Fixed an issue when order did not place using Authorize.net Direct Post from backend * Fixed an issue when sort order for payment methods did not work * Fixed an issue with multiple schema of language.xml * Fixed an issue with infinite loop in language inheritance * Fixed an issue with residual "scopes" logic in i18n implementation * Fixed an issue when search did not work for the CMS Blocks grid * Fixed an issue when WSDL for one scope was cached and displayed for all scopes * Fixed an issue with unit tests coverage build failure * Fixed an issue when custom options were lost after product import * Fixed an issue when product did not show in backend grid if store contained several store view * Fixed an issue when the Recurring Profile section was not updated after changing product template * Fixed an issue with incorrect discount calculation * Fixed an issue when customer could not register during Checkout if Guest Checkout was disabled * Fixed an issue when shopping cart price rule was not applied after updating items and qty in the shopping cart * Fixed an issue when updated and created dates were not shown for Billing Agreement in the Billing Agreement Grid in the backend * Fixed an issue with broken design on the multiple addresses order review page * Fixed an issue when sort by did not work in frontend for Yes/No attributes when Flat catalog was disabled * Fixed an issue when a new blank CMS page was displayed after saving the CMS page entity * Fixed an issue when product attributes were absent on the Product page after switching to another product template * Fixed a 404 error after saving mass update product attributes form * Fixed an issue when it was impossible to perform search by all tax classes on the Advanced Search page * Fixed an issue when attribute order for configurable product was not preserved after saving product * Fixed an issue with no results for the Product Best Sellers report * Fixed a fatal error when opening tax configuration page in the backend * Fixed an error occurring when opening the Tax Zones and Rates page in the backend * Fixed a 404 error occurring while searching products on the New Review page * Fixed an error when performing search in the Tax rate grid * Payments implementation: * Ported correct behaviour for Fraud Management in PayPal Payflow Pro from M1 to M2 * Implemented ability to use negative line items for PayPal Payflowpro * Language packs: * Implemented ability to use multiple packages for the same language from one vendor * GitHub requests: * [#587] (#587) -- The "install/Magento/basic/*_*/layout/*.xml" pattern cannot be processed in "/mnt/fs01/test/mdt/htdocs/app/design/" path Warning!Invalid argument supplied for foreach() * Unit tests coverage: * Magento\Catalog\Model\Product * Service layer implementation: * Created ConfigurableProduct service * Created CompositeProduct service * Refactored TaxCalculationService * Refactored Google Shopping to use tax service * Exposed TaxRate and TaxRule search functions as WebAPI TaxCalculationService * Refactored QuoteDetails and QuoteDetailsItem to use tax class name * Refactored gift wrapping to use tax/weee services * Performed more tax refactoring for service layer * Improved unit test coverage * Indexer-less implementation of URL Rewrites functionality in new UrlRedirect module: * Implemented URL Rewrites generators for all entities: CMS page, product, category * Implemented URL Rewrites matching in the frontend * Added the following functional tests: * Activate Integration * Add Compared Products * Create Bundle Product * Clear All Compare Products * Create CMS Block * Create CMS Page * Create Custom Variable * Create Integration * Create Grouped Product * Create Search Term * Delete Assigned to Template Product Attribute * Delete CMS Block * Delete CMS Page Rewrite * Delete Compare Products * Delete Custom URL Rewrite * Delete Integration * Delete Product Template * Duplicate Product * Edit Search Term * Update Bundle Product * Update CMS Block * Update CMS Page URL Rewrite * Update Custom Variable * Update Custom URL Rewrite * Update Customer on Frontend * Update Integration * Update Product Template * Update Virtual Product
1 parent c8b7223 commit 65e1ba1

File tree

1,022 files changed

+50576
-9867
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

1,022 files changed

+50576
-9867
lines changed

CHANGELOG.md

Lines changed: 112 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,110 @@
1+
2.0.0.0-dev88
2+
=============
3+
* Fixed bugs:
4+
* Fixed an issue when PayPal Express Checkout Payflow Edition and PayPal Payments Advanced were available for multiple checkout
5+
* Fixed an issue when the Bill me later button did not redirect to https when secure url was enabled for frontend
6+
* Fixed an issue when the Billing agreement option was available in multishipping checkout, even if there were no signed agreements
7+
* Fixed an issue when DoExpressCheckout request instead of DoCapture did not allow to do refund, when using PayPal Express Checkout Payflow Edition
8+
* Fixed an issue when eWay was not present on checkout if Base Currency was set to AUD
9+
* Fixed an issue with fatal error occurring when placing order via SagePay with 3D Secure enabled
10+
* Fixed an issue when the FedEx shipping method had no option to specify unit for weight attribute
11+
* Fixed an issue with inability to create credit memo for PalPal Express Checkout/Payments Pro/Payments Pro Hosted Solution (NVP family), if partial refund was initiated on the PayPal side
12+
* Fixed an issue when a guest user could not return product to store, if product was paid using PayPal
13+
* Fixed an issue when PayPal Payments Pro Hosted Solution Fraud protection did not work properly
14+
* Fixed an issue when JavaScript took values from default config for payment methods and used them on the website scope
15+
* Fixed an issue with incorrect address in request to shipping carrier (DHL International) in case the address contained diacritic letters
16+
* Fixed an issue when it was possible to hack currency in PayPal Website Payments Standard
17+
* Fixed an issue when no rows were added to the PayPal Settlement report grid while fetching it from custom server
18+
* Fixed an issue when order had the Suspected Fraud status after creating partial invoice on the PayPal side
19+
* Fixed an issue when PayPal Payflow Pro did not properly implement CUSTREF and INVNUM
20+
* Fixed an issue with PayPal errors handling during IPN postback
21+
* Fixed an issue when the Paypal Express Checkout button was not available on product page for several product types
22+
* Fixed an issue with PayPal Payflow Pro and Payflow Link broken unit tests
23+
* Fixed an issue when PayPal Payments Pro Hosted Solution had the City parameter duplicated in the State parameter for UK
24+
* Fixed an issue with remove multiple HTTP 100/101 headers
25+
* Fixed an issue when SagePay did not transfer shopping cart information
26+
* Fixed an issue when transaction records were absent on Transaction tab for Ogone
27+
* Fixed an issue when partial cancel with SagePay Direct was unavailable
28+
* Fixed an issue when order did not place using PayPal Payments Pro Hosted Solution
29+
* Fixed an issue when order did not place using Authorize.net Direct Post from backend
30+
* Fixed an issue when sort order for payment methods did not work
31+
* Fixed an issue with multiple schema of language.xml
32+
* Fixed an issue with infinite loop in language inheritance
33+
* Fixed an issue with residual "scopes" logic in i18n implementation
34+
* Fixed an issue when search did not work for the CMS Blocks grid
35+
* Fixed an issue when WSDL for one scope was cached and displayed for all scopes
36+
* Fixed an issue with unit tests coverage build failure
37+
* Fixed an issue when custom options were lost after product import
38+
* Fixed an issue when product did not show in backend grid if store contained several store view
39+
* Fixed an issue when the Recurring Profile section was not updated after changing product template
40+
* Fixed an issue with incorrect discount calculation
41+
* Fixed an issue when customer could not register during Checkout if Guest Checkout was disabled
42+
* Fixed an issue when shopping cart price rule was not applied after updating items and qty in the shopping cart
43+
* Fixed an issue when updated and created dates were not shown for Billing Agreement in the Billing Agreement Grid in the backend
44+
* Fixed an issue with broken design on the multiple addresses order review page
45+
* Fixed an issue when sort by did not work in frontend for Yes/No attributes when Flat catalog was disabled
46+
* Fixed an issue when a new blank CMS page was displayed after saving the CMS page entity
47+
* Fixed an issue when product attributes were absent on the Product page after switching to another product template
48+
* Fixed a 404 error after saving mass update product attributes form
49+
* Fixed an issue when it was impossible to perform search by all tax classes on the Advanced Search page
50+
* Fixed an issue when attribute order for configurable product was not preserved after saving product
51+
* Fixed an issue with no results for the Product Best Sellers report
52+
* Fixed a fatal error when opening tax configuration page in the backend
53+
* Fixed an error occurring when opening the Tax Zones and Rates page in the backend
54+
* Fixed a 404 error occurring while searching products on the New Review page
55+
* Fixed an error when performing search in the Tax rate grid
56+
* Payments implementation:
57+
* Ported correct behaviour for Fraud Management in PayPal Payflow Pro from M1 to M2
58+
* Implemented ability to use negative line items for PayPal Payflowpro
59+
* Language packs:
60+
* Implemented ability to use multiple packages for the same language from one vendor
61+
* GitHub requests:
62+
* [#587] (https://github.com/magento/magento2/issues/587) -- The "install/Magento/basic/*_*/layout/*.xml" pattern cannot be processed in "/mnt/fs01/test/mdt/htdocs/app/design/" path Warning!Invalid argument supplied for foreach()
63+
* Unit tests coverage:
64+
* Magento\Catalog\Model\Product
65+
* Service layer implementation:
66+
* Created ConfigurableProduct service
67+
* Created CompositeProduct service
68+
* Refactored TaxCalculationService
69+
* Refactored Google Shopping to use tax service
70+
* Exposed TaxRate and TaxRule search functions as WebAPI TaxCalculationService
71+
* Refactored QuoteDetails and QuoteDetailsItem to use tax class name
72+
* Refactored gift wrapping to use tax/weee services
73+
* Performed more tax refactoring for service layer
74+
* Improved unit test coverage
75+
* Indexer-less implementation of URL Rewrites functionality in new UrlRedirect module:
76+
* Implemented URL Rewrites generators for all entities: CMS page, product, category
77+
* Implemented URL Rewrites matching in the frontend
78+
* Added the following functional tests:
79+
* Activate Integration
80+
* Add Compared Products
81+
* Create Bundle Product
82+
* Clear All Compare Products
83+
* Create CMS Block
84+
* Create CMS Page
85+
* Create Custom Variable
86+
* Create Integration
87+
* Create Grouped Product
88+
* Create Search Term
89+
* Delete Assigned to Template Product Attribute
90+
* Delete CMS Block
91+
* Delete CMS Page Rewrite
92+
* Delete Compare Products
93+
* Delete Custom URL Rewrite
94+
* Delete Integration
95+
* Delete Product Template
96+
* Duplicate Product
97+
* Edit Search Term
98+
* Update Bundle Product
99+
* Update CMS Block
100+
* Update CMS Page URL Rewrite
101+
* Update Custom Variable
102+
* Update Custom URL Rewrite
103+
* Update Customer on Frontend
104+
* Update Integration
105+
* Update Product Template
106+
* Update Virtual Product
107+
1108
2.0.0.0-dev87
2109
=============
3110
* Service layer updates:
@@ -12,6 +119,7 @@
12119
* [#584] (https://github.com/magento/magento2/issues/584) -- Merge and minify js - Exception
13120
* [#585] (https://github.com/magento/magento2/pull/585) -- Add forgotten return statement
14121
* [#592] (https://github.com/magento/magento2/issues/592) -- Module name pattern
122+
* [#618] (https://github.com/magento/magento2/issues/618) -- Fix of unit tests failure on Travis CI
15123
* Tax calculation updates:
16124
* Separate and display Weee line item totals from Tax
17125
* Fixed bugs:
@@ -30,18 +138,16 @@
30138
* Fixed an issue when configurable product was out of stock in Google Shopping while being in stock in the Magento backend
31139
* Fixed an issue when swipe gesture in menu widget was not supported on mobile
32140
* Fixed an issue when it was impossible to enter alpha-numeric zip code on the stage of estimating shipping and tax rates
33-
* Fixed an issue when it was impossible to edit gift card account
34141
* Fixed an issue when custom price was not applied when editing an order
35-
* Fixed an issue when items were not returned to stock after unsuccessful order was placed
142+
* Fixed an issue when items were not returned to stock after unsuccessful order was placed
36143
* Fixed an issue when error message appeared "Cannot save the credit memo” while creating credit memo
37144
* Fixed an issue when Catalog price rule was not shown for the product if price was less than a discount
38145
* Indexer implementation:
39146
* Implemented a new Stock indexer
40147
* Implemented a new EAV indexer
41-
* Fixed failed L1 plan on phpunit 4.1.0
42-
* Minor updates for integration test framework
43-
* Split action controllers classes into action classes
44-
* Added public MTF repository to the packagist.org
148+
* Minor updates for integration test framework
149+
* Split action controllers classes into action classes
150+
* Added public MTF repository to the packagist.org
45151
* Added the following functional tests:
46152
* Create Admin User
47153
* Create Category

app/code/Magento/AdminNotification/Block/System/Messages.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
* Do not edit or add to this file if you wish to upgrade Magento to newer
1818
* versions in the future. If you wish to customize Magento for your
1919
* needs please refer to http://www.magentocommerce.com for more information.
20-
*
20+
*
2121
* @copyright Copyright (c) 2014 X.commerce, Inc. (http://www.magentocommerce.com)
2222
* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
2323
*/
@@ -129,7 +129,7 @@ protected function _getMessagesUrl()
129129
}
130130

131131
/**
132-
* Initialize Syste,Message dialog widget
132+
* Initialize system message dialog widget
133133
*
134134
* @return string
135135
*/

app/code/Magento/Authorizenet/view/adminhtml/web/js/direct-post.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ directPost.prototype = {
138138
loadOrderIframe : function() {
139139
if (this.orderRequestSent) {
140140
$(this.iframeId).hide();
141-
var data = $('order-' + this.iframeId).contentWindow.document.body.innerHTML;
141+
var data = $('order-' + this.iframeId).contentWindow.document.body.getElementsByTagName('pre')[0].innerHTML;
142142
this.saveAdminOrderSuccess(data);
143143
this.orderRequestSent = false;
144144
}

app/code/Magento/Authorizenet/view/frontend/templates/directpost/form.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ $_orderUrl = $this->helper('Magento\Authorizenet\Helper\Data')->getPlaceOrderFro
9191
<input type="number" title="<?php echo __('Card Verification Number') ?>" data-container="cc-cvv" class="input-text cvv" id="<?php echo $_code ?>_cc_cid" name="payment[cc_cid]" value="" data-validate='{required:true, "validate-cc-cvn":"#<?php echo $_code ?>_cc_type"}' autocomplete="off"/>
9292
<?php $_content = '<img src=\"'.$this->getViewFileUrl('Magento_Checkout::cvv.gif').'\" alt=\"'.__('Card Verification Number Visual Reference').'\" title=\"'.__('Card Verification Number Visual Reference').'\" />'; ?>
9393
<div class="note">
94-
<a href="#" id="directpost-cvv-what-is-this" class="action cvv" title="<?php echo __('What is this?') ?>" data-mage-init='{"tooltip": {"content": "<?php echo $_content ?>"}}'><span><?php echo __('What is this?') ?></span></a>
94+
<a href="#" id="directpost-cvv-what-is-this" class="action cvv" title="<?php echo $this->escapeHtml(__('What is this?'));?>" data-mage-init='{"tooltip": {"content": "<?php echo $_content ?>"}}'><span><?php echo __('What is this?') ?></span></a>
9595
</div>
9696
</div>
9797
</div>

app/code/Magento/Backend/Controller/Adminhtml/System/AbstractConfig.php

Lines changed: 14 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -27,25 +27,32 @@
2727
*/
2828
namespace Magento\Backend\Controller\Adminhtml\System;
2929

30-
use Magento\Framework\App\Action\NotFoundException;
31-
3230
abstract class AbstractConfig extends \Magento\Backend\App\AbstractAction
3331
{
3432
/**
3533
* @var \Magento\Backend\Model\Config\Structure
3634
*/
3735
protected $_configStructure;
3836

37+
/**
38+
* @var ConfigSectionChecker
39+
*/
40+
protected $_sectionChecker;
41+
3942
/**
4043
* @param \Magento\Backend\App\Action\Context $context
4144
* @param \Magento\Backend\Model\Config\Structure $configStructure
45+
* @param ConfigSectionChecker $sectionChecker
4246
*/
4347
public function __construct(
4448
\Magento\Backend\App\Action\Context $context,
45-
\Magento\Backend\Model\Config\Structure $configStructure
49+
\Magento\Backend\Model\Config\Structure $configStructure,
50+
ConfigSectionChecker $sectionChecker
4651
) {
4752
parent::__construct($context);
4853
$this->_configStructure = $configStructure;
54+
$this->_sectionChecker = $sectionChecker;
55+
4956
}
5057

5158
/**
@@ -56,12 +63,8 @@ public function __construct(
5663
*/
5764
public function dispatch(\Magento\Framework\App\RequestInterface $request)
5865
{
59-
$section = null;
6066
if (!$request->getParam('section')) {
61-
$section = $this->_configStructure->getFirstSection();
62-
$request->setParam('section', $section->getId());
63-
} else {
64-
$this->_isSectionAllowed($request->getParam('section'));
67+
$request->setParam('section', $this->_configStructure->getFirstSection()->getId());
6568
}
6669
return parent::dispatch($request);
6770
}
@@ -73,33 +76,9 @@ public function dispatch(\Magento\Framework\App\RequestInterface $request)
7376
*/
7477
protected function _isAllowed()
7578
{
76-
return $this->_authorization->isAllowed('Magento_Adminhtml::config');
77-
}
78-
79-
/**
80-
* Check if specified section allowed in ACL
81-
*
82-
* Will forward to deniedAction(), if not allowed.
83-
*
84-
* @param string $sectionId
85-
* @throws \Exception
86-
* @return bool
87-
* @throws NotFoundException
88-
*/
89-
protected function _isSectionAllowed($sectionId)
90-
{
91-
try {
92-
if (false == $this->_configStructure->getElement($sectionId)->isAllowed()) {
93-
throw new \Exception('');
94-
}
95-
return true;
96-
} catch (\Zend_Acl_Exception $e) {
97-
throw new NotFoundException();
98-
} catch (\Exception $e) {
99-
$this->deniedAction();
100-
$this->_actionFlag->set('', self::FLAG_NO_DISPATCH, true);
101-
return false;
102-
}
79+
$sectionId = $this->_request->getParam('section');
80+
return $this->_authorization->isAllowed('Magento_Adminhtml::config')
81+
|| $this->_configStructure->getElement($sectionId)->isAllowed();
10382
}
10483

10584
/**
Lines changed: 69 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,69 @@
1+
<?php
2+
/**
3+
* Magento
4+
*
5+
* NOTICE OF LICENSE
6+
*
7+
* This source file is subject to the Open Software License (OSL 3.0)
8+
* that is bundled with this package in the file LICENSE.txt.
9+
* It is also available through the world-wide-web at this URL:
10+
* http://opensource.org/licenses/osl-3.0.php
11+
* If you did not receive a copy of the license and are unable to
12+
* obtain it through the world-wide-web, please send an email
13+
* to license@magentocommerce.com so we can send you a copy immediately.
14+
*
15+
* DISCLAIMER
16+
*
17+
* Do not edit or add to this file if you wish to upgrade Magento to newer
18+
* versions in the future. If you wish to customize Magento for your
19+
* needs please refer to http://www.magentocommerce.com for more information.
20+
*
21+
* @copyright Copyright (c) 2014 X.commerce, Inc. (http://www.magentocommerce.com)
22+
* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
23+
*/
24+
25+
namespace Magento\Backend\Controller\Adminhtml\System\Config;
26+
27+
use Magento\Backend\Controller\Adminhtml\System\ConfigSectionChecker;
28+
29+
abstract class AbstractScopeConfig extends \Magento\Backend\Controller\Adminhtml\System\AbstractConfig
30+
{
31+
/**
32+
* @var \Magento\Backend\Model\Config
33+
*/
34+
protected $_backendConfig;
35+
36+
/**
37+
* @param \Magento\Backend\App\Action\Context $context
38+
* @param \Magento\Backend\Model\Config\Structure $configStructure
39+
* @param ConfigSectionChecker $sectionChecker
40+
* @param \Magento\Backend\Model\Config $backendConfig
41+
*/
42+
public function __construct(
43+
\Magento\Backend\App\Action\Context $context,
44+
\Magento\Backend\Model\Config\Structure $configStructure,
45+
ConfigSectionChecker $sectionChecker,
46+
\Magento\Backend\Model\Config $backendConfig
47+
) {
48+
$this->_backendConfig = $backendConfig;
49+
parent::__construct($context, $configStructure, $sectionChecker);
50+
}
51+
52+
/**
53+
* Sets scope for backend config
54+
*
55+
* @param string $sectionId
56+
* @return bool
57+
*/
58+
protected function isSectionAllowed($sectionId)
59+
{
60+
$website = $this->getRequest()->getParam('website');
61+
$store = $this->getRequest()->getParam('store');
62+
if ($store) {
63+
$this->_backendConfig->setStore($store);
64+
} elseif ($website) {
65+
$this->_backendConfig->setWebsite($website);
66+
}
67+
return parent::isSectionAllowed($sectionId);
68+
}
69+
}

app/code/Magento/Backend/Controller/Adminhtml/System/Config/Edit.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
*/
2525
namespace Magento\Backend\Controller\Adminhtml\System\Config;
2626

27-
class Edit extends \Magento\Backend\Controller\Adminhtml\System\AbstractConfig
27+
class Edit extends AbstractScopeConfig
2828
{
2929
/**
3030
* Edit configuration section

app/code/Magento/Backend/Controller/Adminhtml/System/Config/Index.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
*/
2525
namespace Magento\Backend\Controller\Adminhtml\System\Config;
2626

27-
class Index extends \Magento\Backend\Controller\Adminhtml\System\AbstractConfig
27+
class Index extends AbstractScopeConfig
2828
{
2929
/**
3030
* Index action

app/code/Magento/Backend/Controller/Adminhtml/System/Config/Save.php

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,18 +53,20 @@ class Save extends AbstractConfig
5353
/**
5454
* @param \Magento\Backend\App\Action\Context $context
5555
* @param \Magento\Backend\Model\Config\Structure $configStructure
56+
* @param \Magento\Backend\Controller\Adminhtml\System\ConfigSectionChecker $sectionChecker
5657
* @param \Magento\Backend\Model\Config\Factory $configFactory
5758
* @param \Magento\Framework\Cache\FrontendInterface $cache
5859
* @param \Magento\Framework\Stdlib\String $string
5960
*/
6061
public function __construct(
6162
\Magento\Backend\App\Action\Context $context,
6263
\Magento\Backend\Model\Config\Structure $configStructure,
64+
\Magento\Backend\Controller\Adminhtml\System\ConfigSectionChecker $sectionChecker,
6365
\Magento\Backend\Model\Config\Factory $configFactory,
6466
\Magento\Framework\Cache\FrontendInterface $cache,
6567
\Magento\Framework\Stdlib\String $string
6668
) {
67-
parent::__construct($context, $configStructure);
69+
parent::__construct($context, $configStructure, $sectionChecker);
6870
$this->_configFactory = $configFactory;
6971
$this->_cache = $cache;
7072
$this->string = $string;
@@ -160,10 +162,6 @@ protected function _saveAdvanced()
160162
public function execute()
161163
{
162164
try {
163-
if (false == $this->_isSectionAllowed($this->getRequest()->getParam('section'))) {
164-
throw new \Exception(__('This section is not allowed.'));
165-
}
166-
167165
// custom save logic
168166
$this->_saveSection();
169167
$section = $this->getRequest()->getParam('section');

app/code/Magento/Backend/Controller/Adminhtml/System/Config/State.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
*/
2525
namespace Magento\Backend\Controller\Adminhtml\System\Config;
2626

27-
class State extends \Magento\Backend\Controller\Adminhtml\System\AbstractConfig
27+
class State extends AbstractScopeConfig
2828
{
2929
/**
3030
* Save fieldset state through AJAX

0 commit comments

Comments
 (0)