Skip to content

Commit d4f4da9

Browse files
committed
Merge pull request #231 from magento-south/BUGS
[South] Sprint #34 Bug fixes
2 parents 07f9f16 + d363ea2 commit d4f4da9

File tree

31 files changed

+774
-71
lines changed

31 files changed

+774
-71
lines changed

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)