Skip to content

Commit 9b1e1bb

Browse files
author
Yuri Kovsher
committed
Merge remote-tracking branch 'tango-ce/result_factory' into MAGETWO-36105
2 parents 82c9ede + 0754a60 commit 9b1e1bb

File tree

61 files changed

+3575
-646
lines changed

Some content is hidden

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

61 files changed

+3575
-646
lines changed

app/code/Magento/Backend/App/Action/Context.php

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
*/
66
namespace Magento\Backend\App\Action;
77

8+
use Magento\Framework\Controller\ResultFactory;
9+
810
/**
911
* Backend Controller context
1012
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
@@ -62,6 +64,7 @@ class Context extends \Magento\Framework\App\Action\Context
6264
* @param \Magento\Framework\App\ViewInterface $view
6365
* @param \Magento\Framework\Message\ManagerInterface $messageManager
6466
* @param \Magento\Backend\Model\View\Result\RedirectFactory $resultRedirectFactory
67+
* @param \Magento\Framework\Controller\ResultFactory $resultFactory
6568
* @param \Magento\Backend\Model\Session $session
6669
* @param \Magento\Framework\AuthorizationInterface $authorization
6770
* @param \Magento\Backend\Model\Auth $auth
@@ -83,6 +86,7 @@ public function __construct(
8386
\Magento\Framework\App\ViewInterface $view,
8487
\Magento\Framework\Message\ManagerInterface $messageManager,
8588
\Magento\Backend\Model\View\Result\RedirectFactory $resultRedirectFactory,
89+
ResultFactory $resultFactory,
8690
\Magento\Backend\Model\Session $session,
8791
\Magento\Framework\AuthorizationInterface $authorization,
8892
\Magento\Backend\Model\Auth $auth,
@@ -102,7 +106,8 @@ public function __construct(
102106
$actionFlag,
103107
$view,
104108
$messageManager,
105-
$resultRedirectFactory
109+
$resultRedirectFactory,
110+
$resultFactory
106111
);
107112

108113
$this->_session = $session;

app/code/Magento/Catalog/Block/Product/View/Options/Type/Select.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public function getValuesHtml()
4646
if ($_option->getType() == \Magento\Catalog\Model\Product\Option::OPTION_TYPE_DROP_DOWN) {
4747
$select->setName('options[' . $_option->getid() . ']')->addOption('', __('-- Please Select --'));
4848
} else {
49-
$select->setName('options[' . $_option->getid() . '][]');
49+
$select->setName('options[' . $_option->getid() . ']');
5050
$select->setClass('multiselect admin__control-multiselect' . $require . ' product-custom-option');
5151
}
5252
foreach ($_option->getValues() as $_value) {
@@ -141,7 +141,7 @@ public function getValuesHtml()
141141
' name="options[' .
142142
$_option->getId() .
143143
']' .
144-
$arraySign .
144+
(!empty($arraySign) ? '[' . $htmlValue . ']' : '') .
145145
'" id="options_' .
146146
$_option->getId() .
147147
'_' .

app/code/Magento/Catalog/view/adminhtml/web/catalog/product/composite/configure.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ ProductConfigure.prototype = {
195195
return false;
196196
}
197197

198-
this._initWindowElements();
198+
this.initialize();
199199
this.current.listType = listType;
200200
this.current.itemId = itemId;
201201
this.confirmedCurrentId = this._getConfirmedBlockId(listType, itemId);

app/code/Magento/Cms/Api/BlockRepositoryInterface.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
/**
1111
* CMS block CRUD interface.
12+
* @api
1213
*/
1314
interface BlockRepositoryInterface
1415
{

app/code/Magento/Cms/Api/Data/BlockInterface.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
/**
99
* CMS block interface.
10+
* @api
1011
*/
1112
interface BlockInterface
1213
{

app/code/Magento/Cms/Api/Data/BlockSearchResultsInterface.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
/**
1111
* Interface for cms block search results.
12+
* @api
1213
*/
1314
interface BlockSearchResultsInterface extends SearchResultsInterface
1415
{

app/code/Magento/Cms/Api/Data/PageInterface.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
/**
99
* CMS page interface.
10+
* @api
1011
*/
1112
interface PageInterface
1213
{

app/code/Magento/Cms/Api/Data/PageSearchResultsInterface.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
/**
1111
* Interface for cms page search results.
12+
* @api
1213
*/
1314
interface PageSearchResultsInterface extends SearchResultsInterface
1415
{

app/code/Magento/Cms/Api/PageRepositoryInterface.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
/**
1111
* CMS page CRUD interface.
12+
* @api
1213
*/
1314
interface PageRepositoryInterface
1415
{

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -420,10 +420,12 @@ public function initiatePasswordReset($email, $template, $websiteId = null)
420420
)
421421
);
422422
}
423+
return true;
423424
} catch (MailException $e) {
424425
// If we are not able to send a reset password email, this should be ignored
425426
$this->logger->critical($e);
426427
}
428+
return false;
427429
}
428430

429431
/**

0 commit comments

Comments
 (0)