Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
50 commits
Select commit Hold shift + click to select a range
e47349a
Fixed Purchased Order Form button should visible properly
gelanivishal May 19, 2018
c5d2d77
Minor styles nesting fix
rogyar May 21, 2018
c261156
resolve Error While Trying To Load Quote Item Collection Using Magent…
neeta-wagento Oct 30, 2017
950e317
Update Collection.php
neeta-wagento Oct 30, 2017
b63d2c5
change for : Resolve Knockout non-unique elements id in console error
viral-wagento Jun 5, 2018
c64228d
ENGCOM-1870: [BackPort] Resolve Knockout non-unique elements id in co…
magento-engcom-team Jun 5, 2018
f8397eb
[Backport] Trim username on customer account login page
dankhrapiyush Jun 6, 2018
1c2f385
ENGCOM-1881: [Backport] [Resolved : limiter float too generic] #15880
magento-engcom-team Jun 6, 2018
43835fc
Remove blank line at eof
dankhrapiyush Jun 6, 2018
14721c7
Update comment in trim-username.js
dankhrapiyush Jun 6, 2018
81d8e52
PHPDoc and type hints
May 28, 2018
55f1c88
ENGCOM-1903: [Backport] [Resolved : Changing @tab-content__border var…
magento-engcom-team Jun 7, 2018
fb3141c
ENGCOM-1918: [Backport] Fixed Purchased Order Form button should visi…
magento-engcom-team Jun 7, 2018
936c7de
MAGETWO-86935: [Backport] Restricted User Role gets "Something went w…
OlgaVasyltsun Jun 8, 2018
0922df2
MAGETWO-64616: [BACKPORT] Layout::renderResult() issue
DianaRusin Jun 8, 2018
2fec057
ENGCOM-1926: [Backport] Fixes in catalog component blocks [2.1-develo…
magento-engcom-team Jun 8, 2018
ddf5130
ENGCOM-1930: [Backport-2.1] Trim username on customer account login p…
magento-engcom-team Jun 11, 2018
ea600d6
Merge remote-tracking branch 'origin/MAGETWO-86935' into 2.1-develop-…
DianaRusin Jun 11, 2018
652e97c
MAGETWO-84366: Logo Image Upload is not working in Internet Explorer …
DianaRusin Jun 11, 2018
e00b0ff
MAGETWO-67162: [Backport][GITHUB] CSS Minifying not compatible with C…
OlgaVasyltsun Jun 11, 2018
f794719
Remove unnecessary parameter from invoking toHtml() method
rogyar Jun 11, 2018
caee053
Fixed a minor styling issue in page footer in order to align the colu…
dmytro-ch Jun 11, 2018
a60cd2e
ENGCOM-1947: Fixed a minor styling issue in page footer in order to a…
magento-engcom-team Jun 11, 2018
26d8664
Merge remote-tracking branch 'origin/2.1-develop' into MAGETWO-67162
OlgaVasyltsun Jun 12, 2018
112fb84
ENGCOM-1948: [Backport] #13899 Solve Canada Zip Code pattern #16031
magento-engcom-team Jun 12, 2018
7c3ce72
fix for dropdown toggle icon
Karlasa Jun 11, 2018
da1b898
ENGCOM-1955: [Backport] fix for dropdown toggle icon in cart #16043
magento-engcom-team Jun 12, 2018
66e0058
Simplify echo statement
ishakhsuvarov Jun 12, 2018
957b835
ENGCOM-1962: [Backport] Resolve Error While Trying To Load Quote Item…
magento-engcom-team Jun 12, 2018
eab571a
Merge remote-tracking branch 'origin/MAGETWO-84366' into 2.1-develop-…
OlgaVasyltsun Jun 12, 2018
b2c4205
Merge remote-tracking branch 'origin/MAGETWO-64616' into 2.1-develop-…
svitja Jun 12, 2018
a018521
Merge remote-tracking branch 'origin/MAGETWO-67162' into 2.1-develop-…
DianaRusin Jun 12, 2018
a8289c9
ENGCOM-1950: [Backport 2.1] Wishlist: Remove unnecessary parameter fr…
magento-engcom-team Jun 12, 2018
b710eab
ENGCOM-1955: [Backport] fix for dropdown toggle icon in cart #16043
Jun 12, 2018
7b68ca2
ENGCOM-1950: [Backport 2.1] Wishlist: Remove unnecessary parameter fr…
Jun 12, 2018
486e92c
ENGCOM-1948: [Backport] #13899 Solve Canada Zip Code pattern #16031
Jun 12, 2018
7d2402c
ENGCOM-1947: Fixed a minor styling issue in page footer in order to a…
Jun 12, 2018
5601ed1
ENGCOM-1930: [Backport-2.1] Trim username on customer account login p…
Jun 12, 2018
6b6b3b4
ENGCOM-1926: [Backport] Fixes in catalog component blocks [2.1-develo…
Jun 12, 2018
7a6435a
ENGCOM-1918: [Backport] Fixed Purchased Order Form button should visi…
Jun 12, 2018
1534ac4
ENGCOM-1903: [Backport] [Resolved : Changing @tab-content__border var…
Jun 12, 2018
1a20085
ENGCOM-1881: [Backport] [Resolved : limiter float too generic] #15880
Jun 12, 2018
d659065
Fixed static failures
Jun 12, 2018
ea93dfb
ENGCOM-1903: [Backport] [Resolved : Changing @tab-content__border var…
Jun 13, 2018
b91b8f2
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
Jun 13, 2018
477bea4
Merge pull request #2690 from magento-tsg/2.1-develop-pr54
Jun 13, 2018
4d687fe
ENGCOM-1962: [Backport] Resolve Error While Trying To Load Quote Item…
VladimirZaets Jun 13, 2018
958359d
ENGCOM-1870: [BackPort] Resolve Knockout non-unique elements id in co…
VladimirZaets Jun 13, 2018
1d129d7
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
Jun 13, 2018
f9768b3
fixed word typo
ledian-hymetllari Jun 6, 2018
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/code/Magento/Catalog/Block/Adminhtml/Category/Tree.php
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,7 @@ public function getBreadcrumbsJavascript($path, $javascriptVarName)
*
* @param Node|array $node
* @param int $level
* @return string
* @return array
* @SuppressWarnings(PHPMD.CyclomaticComplexity)
* @SuppressWarnings(PHPMD.NPathComplexity)
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ function (node, e) {
*
* @param \Magento\Framework\Data\Tree\Node|array $node
* @param int $level
* @return string
* @return array
*/
protected function _getNodeJson($node, $level = 0)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ class Element extends \Magento\Backend\Block\Widget\Form\Renderer\Fieldset\Eleme
/**
* Retrieve data object related with form
*
* @return \Magento\Catalog\Model\Product || \Magento\Catalog\Model\Category
* @return \Magento\Catalog\Model\Product|\Magento\Catalog\Model\Category
*/
public function getDataObject()
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,8 +101,7 @@ protected function _prepareColumns()
'type' => 'options',
'options' => ['1' => __('Yes'), '0' => __('No')],
'align' => 'center'
],
'is_user_defined'
]
);

$this->_eventManager->dispatch('product_attribute_grid_build', ['grid' => $this]);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ public function getSelectorOptions()
*
* @param string $labelPart
* @param int $templateId
* @return \Magento\Catalog\Model\ResourceModel\Product\Attribute\Collection
* @return array
*/
public function getSuggestedAttributes($labelPart, $templateId = null)
{
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Catalog/Block/Adminhtml/Rss/Grid/Link.php
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ public function isRssAllowed()
}

/**
* @return string
* @return array
*/
protected function getLinkParams()
{
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Catalog/Block/Category/Rss/Link.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ public function getLabel()
}

/**
* @return string
* @return array
*/
protected function getLinkParams()
{
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Catalog/Block/Product/AbstractProduct.php
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ public function getAddToCompareUrl()
* Gets minimal sales quantity
*
* @param \Magento\Catalog\Model\Product $product
* @return int|null
* @return float|null
*/
public function getMinimalQty($product)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ public function getPrice($price, $includingTax = null)
* Returns price converted to current currency rate
*
* @param float $price
* @return float
* @return float|string
*/
public function getCurrencyPrice($price)
{
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Catalog/Block/Rss/Product/Special.php
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ protected function _construct()
}

/**
* @return string
* @return array
*/
public function getRssData()
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<!-- ko template: getTemplate() --><!-- /ko -->
<!--/ko-->
<form data-hasrequired="* Required Fields">
<fieldset id="billing-new-address-form" class="fieldset address">
<fieldset data-bind="attr: { id:'billing-new-address-form-'+index, value:index}" class="fieldset address">
<!-- ko foreach: getRegion('additional-fieldsets') -->
<!-- ko template: getTemplate() --><!-- /ko -->
<!--/ko-->
Expand Down
10 changes: 10 additions & 0 deletions app/code/Magento/Customer/view/frontend/templates/form/login.phtml
Original file line number Diff line number Diff line change
Expand Up @@ -50,3 +50,13 @@
</form>
</div>
</div>

<script type="text/x-magento-init">
{
".field.email": {
"Magento_Customer/js/trim-username": {
"formSelector": "form.form-login"
}
}
}
</script>
65 changes: 65 additions & 0 deletions app/code/Magento/Customer/view/frontend/web/js/trim-username.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,65 @@
/**
* Copyright © Magento, Inc. All rights reserved.
* See COPYING.txt for license details.
*/

define([
'jquery'
], function ($) {
'use strict';

$.widget('mage.trimUsername', {
options: {
cache: {},
formSelector: 'form',
emailSelector: 'input[type="email"]'
},

/**
* Widget initialization
* @private
*/
_create: function () {
// We need to look outside the module for backward compatibility, since someone can already use the module.
// @todo Narrow this selector in 2.3 so it doesn't accidentally finds the email field from the
// newsletter email field or any other "email" field.
this.options.cache.email = $(this.options.formSelector).find(this.options.emailSelector);
this._bind();
},

/**
* Event binding, will monitor change, keyup and paste events.
* @private
*/
_bind: function () {
if (this.options.cache.email.length) {
this._on(this.options.cache.email, {
'change': this._trimUsername,
'keyup': this._trimUsername,
'paste': this._trimUsername
});
}
},

/**
* Trim username
* @private
*/
_trimUsername: function () {
var username = this._getUsername().trim();

this.options.cache.email.val(username);
},

/**
* Get username value
* @returns {*}
* @private
*/
_getUsername: function () {
return this.options.cache.email.val();
}
});

return $.mage.trimUsername;
});
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ protected function _construct()
*/
public function getStoreId()
{
return (int)$this->_quote->getStoreId();
return (int)$this->_productCollectionFactory->create()->getStoreId();
}

/**
Expand Down
5 changes: 4 additions & 1 deletion app/code/Magento/Ui/Controller/Adminhtml/Index/Render.php
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,10 @@ public function execute()
$aclResource = $component->getData('acl');

if ($aclResource && !$this->_authorization->isAllowed($aclResource)) {
$this->_redirect('admin/noroute');
if (!$this->_request->isAjax()) {
$this->_redirect('admin/noroute');
}

return;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -149,6 +149,15 @@ public function testExecuteAjaxRequestWithoutPermissions($acl, $isAllowed)
->method('getParam')
->with('namespace')
->willReturn($name);

if ($isAllowed === false) {
$this->requestMock->expects($this->once())
->method('isAjax')
->willReturn(true);
}

$this->responseMock->expects($this->never())
->method('setRedirect');
$this->responseMock->expects($this->any())
->method('appendBody')
->with($renderedData);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -304,7 +304,7 @@ define([

if (allowed.passed) {
target.on('fileuploadsend', function (event, postData) {
postData.data.set('param_name', this.paramName);
postData.data.append('param_name', this.paramName);
$(event.currentTarget).off('fileuploadsend');
}.bind(data));

Expand Down
17 changes: 17 additions & 0 deletions app/code/Magento/Ui/view/base/web/js/grid/controls/button/split.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
/**
* Copyright © Magento, Inc. All rights reserved.
* See COPYING.txt for license details.
*/

define([
'jquery'
], function ($) {
'use strict';

return function (data, element) {

$(element).on('click.splitDefault', '.action-default', function () {
$(this).siblings('.dropdown-menu').find('.item-default').trigger('click');
});
};
});
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ $columns = $block->getColumns();
<?php /* @escapeNotVerified */ echo($iterator++ == 1) ? '<li data-row="product-item" class="product-item" id="item_' . $item->getId() . '">' : '</li><li class="product-item" id="item_' . $item->getId() . '">' ?>
<div class="product-item-info">
<?php foreach ($columns as $column): ?>
<?php $column->setItem($item); echo $column->toHtml($item);?>
<?= $column->setItem($item)->toHtml() ?>
<?php endforeach; ?>
</div>
<?php echo($iterator == count($block->getItems())+1) ? '</li>' : '' ?>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -188,11 +188,10 @@
.lib-icon-font-symbol(@icon-list);
}

.limiter {
float: right;

.products.wrapper ~ .toolbar & {
.toolbar {
.products.wrapper ~ & .limiter {
display: block;
float: right;
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -1401,7 +1401,7 @@
&.active {
> .title {
.lib-icon-font-symbol(
@_icon-font-content: @icon-prev,
@_icon-font-content: @icon-up,
@_icon-font-position: after
);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,7 @@

.media-width(@extremum, @break) when (@extremum = 'min') and (@break = @screen__m) {
.product.data.items {
.lib-data-tabs(
@_tab-content-border-top-status: true
);
.lib-data-tabs();
}
}

Expand Down
11 changes: 11 additions & 0 deletions app/design/frontend/Magento/blank/web/css/source/_variables.less
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,17 @@
@font-family-name__base: 'Open Sans';
@font-family__base: @font-family-name__base, @font-family__sans-serif;

//
// Sections variables
// _____________________________________________

//
// Tabs
// ---------------------------------------------

@tab-content__border-top-status: true;


//
// Sidebar
// ---------------------------------------------
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -235,10 +235,10 @@
}
}

.limiter {
float: right;
.products.wrapper ~ .toolbar & {
.toolbar {
.products.wrapper ~ & .limiter {
display: block;
float: right;
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,10 @@
}
}
}

#po_number {
margin-bottom: 20px;
}
}

.payment-method-title {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -327,6 +327,9 @@
.widget.block {
.lib-css(margin, @indent__base 0);
}
.links .widget.block {
margin: 0;
}
}

.no-display:extend(.abs-no-display all) {
Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
"monolog/monolog": "1.16.0",
"oyejorge/less.php": "~1.7.0",
"pelago/emogrifier": "0.1.1",
"tubalmartin/cssmin": "2.4.8-p4",
"tubalmartin/cssmin": "3.0.0",
"magento/magento-composer-installer": "*",
"braintree/braintree_php": "3.7.0",
"symfony/console": "~2.3 <2.7",
Expand Down
Loading