Skip to content

Commit

Permalink
🔃 [EngCom] Public Pull Requests - 2.3-develop
Browse files Browse the repository at this point in the history
Accepted Public Pull Requests:
 - #16481: Declare module namespace before template path name(Magento_Sales::order/creditmemo.phtml). (by @Sarvesh-A)
 - #16476: [Forwardport] Declare module namespace before template path name(Magento_Sales::order/info.phtml). (by @gelanivishal)
 - #16462: 2.3 develop (by @rsantellan)
 - #16429: [Forwardport] Fix for #8222 (by @0m3r)
 - #15592: Use the timeout when querying Elasticsearch (by @mpchadwick)


Fixed GitHub Issues:
 - #8222: Estimate Shipping and Tax Form not works due to js error in collapsible.js [proposed fix] (reported by @Dart18) has been fixed in #16429 by @0m3r in 2.3-develop branch
   Related commits:
     1. 59a80fd
  • Loading branch information
magento-engcom-team authored Jul 2, 2018
2 parents 717879c + 6824f97 commit d9fb867
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,8 @@ public function deleteMapping($index, $entityType)
*/
public function query($query)
{
return $this->client->search($query);
$params = array_merge($query, ['client' => ['timeout' => $this->clientOptions['timeout']]]);
return $this->client->search($params);
}

/**
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Sales/Block/Order/Creditmemo.php
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ class Creditmemo extends \Magento\Sales\Block\Order\Creditmemo\Items
/**
* @var string
*/
protected $_template = 'order/creditmemo.phtml';
protected $_template = 'Magento_Sales::order/creditmemo.phtml';

/**
* @var \Magento\Framework\App\Http\Context
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Sales/Block/Order/Info.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ class Info extends \Magento\Framework\View\Element\Template
/**
* @var string
*/
protected $_template = 'order/info.phtml';
protected $_template = 'Magento_Sales::order/info.phtml';

/**
* Core registry
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ define([
}

discountSegments = this.totals()['total_segments'].filter(function (segment) {
return segment.code === 'discount';
return segment.code.indexOf('discount') !== -1;
});

return discountSegments.length ? discountSegments[0].title : null;
Expand Down
2 changes: 1 addition & 1 deletion lib/web/mage/collapsible.js
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ define([
_processState: function () {
var anchor = window.location.hash,
isValid = $.mage.isValidSelector(anchor),
urlPath = window.location.pathname.replace('.', ''),
urlPath = window.location.pathname.replace(/\./g, ''),
state;

this.stateKey = encodeURIComponent(urlPath + this.element.attr('id'));
Expand Down

0 comments on commit d9fb867

Please sign in to comment.