Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport] [Resolved : limiter float too generic] #9

Merged
merged 55 commits into from
Jun 6, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
55 commits
Select commit Hold shift + click to select a range
4e3a7fe
Updated font-size variable and standardize #ToDo UI
gelanivishal May 22, 2018
d2f5f02
Moved css from media #TODO
gelanivishal May 22, 2018
371bb07
DocBloc typo fixes
rogyar May 29, 2018
b4171a1
Add missing property, fixed incorrect proprty type, add missing throw…
dmytro-ch Jun 2, 2018
516945b
ENGCOM-1784: [Backport 2.1] Fix minor issues in ui export converter c…
magento-engcom-team Jun 3, 2018
e0544c2
Fixed typos:
dverkade Jun 2, 2018
03d45ee
Fixed return type of wishlist's getImageData in DocBlock
rogyar Jun 3, 2018
02453f6
Refactor javascript code of button split widget
amittiwari024 May 19, 2018
3fae33a
ENGCOM-1798: [BACKPORT 2.1 #15695] Fixed a couple of typos #15716
magento-engcom-team Jun 4, 2018
93cf970
ENGCOM-1799: [Backport 2.1] Fixed return type of wishlist's getImageD…
magento-engcom-team Jun 4, 2018
8c6ed20
fix: support multiple minisearch widget instances
May 17, 2018
56506df
ENGCOM-1802: [Backport] fix: support multiple minisearch widget insta…
magento-engcom-team Jun 4, 2018
5fba6af
ENGCOM-1804: [Backwardport] Refactor javascript code of button split…
magento-engcom-team Jun 4, 2018
3f28206
ENGCOM-1812: [Backport] Moved css from media #TODO #15724
magento-engcom-team Jun 4, 2018
2817ca5
ENGCOM-1813: [Backport] Updated font-size variable and standardize #T…
magento-engcom-team Jun 4, 2018
8f4f052
Error 500 in Module Manager
flancer64 May 15, 2018
13dc017
Fixed according to review
May 21, 2018
bfb02a5
ENGCOM-1813: [Backport] Updated font-size variable and standardize #T…
Jun 4, 2018
1b6eb9c
ENGCOM-1812: [Backport] Moved css from media #TODO #15724
Jun 4, 2018
44a2b9c
ENGCOM-1804: [Backwardport] Refactor javascript code of button split…
Jun 4, 2018
0a1f3c3
ENGCOM-1802: [Backport] fix: support multiple minisearch widget insta…
Jun 4, 2018
0a6caa0
ENGCOM-1799: [Backport 2.1] Fixed return type of wishlist's getImageD…
Jun 4, 2018
f7fd837
ENGCOM-1798: [BACKPORT 2.1 #15695] Fixed a couple of typos #15716
Jun 4, 2018
a55199f
ENGCOM-1828: [Backport] Docblock typo fixes #15573
magento-engcom-team Jun 4, 2018
5020b47
Fix copyright
VladimirZaets Jun 4, 2018
0543650
Merge remote-tracking branch 'engcom/2.1-develop-prs' into 2.1-develo…
VladimirZaets Jun 4, 2018
b01558a
Make necessary space.
chirag-wagento Jun 5, 2018
96b8870
Remove double semicolon from the style sheets.
sanganinamrata Jun 5, 2018
f7893a4
fix: support multiple minisearch widget instances
May 17, 2018
ea51428
change for : fix typo for setCategoryIds
viral-wagento Jun 5, 2018
d89bd3b
ENGCOM-1835: [Backport] Make necessary space. #5 #15805
magento-engcom-team Jun 5, 2018
bd70908
ENGCOM-1812: [Backport] Moved css from media #TODO #15724
Jun 5, 2018
ef4d09f
ENGCOM-1836: [Backport] fix: support multiple minisearch widget insta…
magento-engcom-team Jun 5, 2018
1a53c9d
ENGCOM-1837: [Backport] [Code improvement] #15807
magento-engcom-team Jun 5, 2018
5156366
ENGCOM-1838: [Backport] Remove double semicolon from the style sheets…
magento-engcom-team Jun 5, 2018
15b982b
ENGCOM-1838: [Backport] Remove double semicolon from the style sheets…
Jun 5, 2018
bf9e0e1
ENGCOM-1837: [Backport] [Code improvement] #15807
Jun 5, 2018
5b448fb
ENGCOM-1836: [Backport] fix: support multiple minisearch widget insta…
Jun 5, 2018
cf548b5
ENGCOM-1835: [Backport] Make necessary space. #5 #15805
Jun 5, 2018
16b9f72
Fix typo in Image::open exception message
t-richards May 17, 2018
1a94fd8
change for misleading data-container in product list
viral-wagento Jun 5, 2018
bd413fc
ENGCOM-1841: [Backport] fix typo for setCateroryIds #15814
magento-engcom-team Jun 5, 2018
67781d9
ENGCOM-1842: [BackPort]15319 : misleading data-container in product l…
magento-engcom-team Jun 5, 2018
ee17a20
ENGCOM-1843: [Backport] Fix typo in Image::open exception message #15817
magento-engcom-team Jun 5, 2018
1a8eb96
ENGCOM-1856: [Backport] Error 500 in Module Manager #15756
magento-engcom-team Jun 5, 2018
ba1ff18
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
magento-engcom-team Jun 5, 2018
8f92501
ENGCOM-1856: [Backport] Error 500 in Module Manager #15756
Jun 5, 2018
d0c0fea
ENGCOM-1843: [Backport] Fix typo in Image::open exception message #15817
Jun 5, 2018
27a211e
ENGCOM-1842: [BackPort]15319 : misleading data-container in product l…
Jun 5, 2018
9da6b24
ENGCOM-1841: [Backport] fix typo for setCateroryIds #15814
Jun 5, 2018
52fe2bb
ENGCOM-1828: [Backport] Docblock typo fixes #15573
Jun 5, 2018
caa3a2d
ENGCOM-1784: [Backport 2.1] Fix minor issues in ui export converter c…
Jun 5, 2018
d46050c
ENGCOM-1856: [Backport] Error 500 in Module Manager #15756
Jun 5, 2018
b1f7b25
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
magento-engcom-team Jun 5, 2018
950a668
hitesh-wagento 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
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,11 @@
<?php endif; ?>
</div>

<script>
require(['jquery'], function($){
$('.actions-split')
.on('click.splitDefault', '.action-default', function() {
$(this).siblings('.dropdown-menu').find('.item-default').trigger('click');
});
});
<script type="text/x-magento-init">
{
".actions-split": {
"Magento_Ui/js/grid/controls/button/split": {}
}
}
</script>

Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ $_helper = $this->helper('Magento\Catalog\Helper\Output');
<?php /** @var $_product \Magento\Catalog\Model\Product */ ?>
<?php foreach ($_productCollection as $_product): ?>
<?php /* @escapeNotVerified */ echo($iterator++ == 1) ? '<li class="item product product-item">' : '</li><li class="item product product-item">' ?>
<div class="product-item-info" data-container="product-grid">
<div class="product-item-info" data-container="product-<?= /* @escapeNotVerified */ $viewMode ?>">
<?php
$productImage = $block->getImage($_product, $image);
if ($pos != null) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ public function canDisplay()
}

/**
* Retrieve disply item qty availablity
* Retrieve display item qty availability
*
* @return false
*/
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Search/view/frontend/web/form-mini.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ define([
this.autoComplete = $(this.options.destinationSelector);
this.searchForm = $(this.options.formSelector);
this.submitBtn = this.searchForm.find(this.options.submitBtn)[0];
this.searchLabel = $(this.options.searchLabel);
this.searchLabel = this.searchForm.find(this.options.searchLabel);
this.isExpandable = this.options.isExpandable;

_.bindAll(this, '_onKeyDown', '_onPropertyChange', '_onSubmit');
Expand Down
10 changes: 9 additions & 1 deletion app/code/Magento/Ui/Model/Export/ConvertToCsv.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,10 @@
namespace Magento\Ui\Model\Export;

use Magento\Framework\App\Filesystem\DirectoryList;
use Magento\Framework\Exception\FileSystemException;
use Magento\Framework\Exception\LocalizedException;
use Magento\Framework\Filesystem;
use Magento\Framework\Filesystem\Directory\WriteInterface;
use Magento\Ui\Component\MassAction\Filter;

/**
Expand All @@ -16,7 +18,7 @@
class ConvertToCsv
{
/**
* @var DirectoryList
* @var WriteInterface
*/
protected $directory;

Expand All @@ -30,11 +32,17 @@ class ConvertToCsv
*/
protected $pageSize = null;

/**
* @var Filter
*/
protected $filter;

/**
* @param Filesystem $filesystem
* @param Filter $filter
* @param MetadataProvider $metadataProvider
* @param int $pageSize
* @throws FileSystemException
*/
public function __construct(
Filesystem $filesystem,
Expand Down
12 changes: 11 additions & 1 deletion app/code/Magento/Ui/Model/Export/ConvertToXml.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,10 @@
use Magento\Framework\App\Filesystem\DirectoryList;
use Magento\Framework\Convert\Excel;
use Magento\Framework\Convert\ExcelFactory;
use Magento\Framework\Exception\FileSystemException;
use Magento\Framework\Exception\LocalizedException;
use Magento\Framework\Filesystem;
use Magento\Framework\Filesystem\Directory\WriteInterface;
use Magento\Ui\Component\MassAction\Filter;

/**
Expand All @@ -20,7 +22,7 @@
class ConvertToXml
{
/**
* @var DirectoryList
* @var WriteInterface
*/
protected $directory;

Expand Down Expand Up @@ -49,12 +51,18 @@ class ConvertToXml
*/
protected $fields;

/**
* @var Filter
*/
protected $filter;

/**
* @param Filesystem $filesystem
* @param Filter $filter
* @param MetadataProvider $metadataProvider
* @param ExcelFactory $excelFactory
* @param SearchResultIteratorFactory $iteratorFactory
* @throws FileSystemException
*/
public function __construct(
Filesystem $filesystem,
Expand Down Expand Up @@ -87,6 +95,7 @@ protected function getOptions()
* Returns DB fields list
*
* @return array
* @throws LocalizedException
*/
protected function getFields()
{
Expand All @@ -102,6 +111,7 @@ protected function getFields()
*
* @param DocumentInterface $document
* @return array
* @throws LocalizedException
*/
public function getRowData(DocumentInterface $document)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -109,14 +109,14 @@ define([
* @returns {Group} Chainable.
*/
_setClasses: function () {
var addtional = this.additionalClasses,
var additional = this.additionalClasses,
classes;

if (_.isString(addtional)) {
addtional = this.additionalClasses.split(' ');
if (_.isString(additional)) {
additional = this.additionalClasses.split(' ');
classes = this.additionalClasses = {};

addtional.forEach(function (name) {
additional.forEach(function (name) {
classes[name] = true;
}, this);
}
Expand Down
8 changes: 4 additions & 4 deletions app/code/Magento/Ui/view/base/web/js/form/components/group.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,14 +54,14 @@ define([
* @returns {Group} Chainable.
*/
_setClasses: function () {
var addtional = this.additionalClasses,
var additional = this.additionalClasses,
classes;

if (_.isString(addtional)) {
addtional = this.additionalClasses.split(' ');
if (_.isString(additional)) {
additional = this.additionalClasses.split(' ');
classes = this.additionalClasses = {};

addtional.forEach(function (name) {
additional.forEach(function (name) {
classes[name] = true;
}, this);
}
Expand Down
4 changes: 2 additions & 2 deletions app/code/Magento/Wishlist/CustomerData/Wishlist.php
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php
/**
* Copyright © 2013-2018 Magento, Inc. All rights reserved.
* Copyright © Magento, Inc. All rights reserved.
* See COPYING.txt for license details.
*/
namespace Magento\Wishlist\CustomerData;
Expand Down Expand Up @@ -142,7 +142,7 @@ protected function getItemData(\Magento\Wishlist\Model\Item $wishlistItem)
* Retrieve product image data
*
* @param \Magento\Catalog\Model\Product $product
* @return \Magento\Catalog\Block\Product\Image
* @return array
* @SuppressWarnings(PHPMD.NPathComplexity)
*/
protected function getImageData($product)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -99,8 +99,8 @@
background-color: transparent;
border: none;
color: @link__color;
font-family: @btn__base__font-size;
font-size: 1.5rem; // ToDo UI: Check font-size and standardize
font-family: @font-family__base;
font-size: @base__font-size;

&:hover,
&:active,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -188,10 +188,9 @@
.lib-icon-font-symbol(@icon-list);
}

.limiter {
float: right;

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

& when (@media-common = true) {

// Product Lists
.products {
margin: @indent__l 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -235,9 +235,9 @@
}
}

.limiter {
float: right;
.products.wrapper ~ .toolbar & {
.toolbar {
.products.wrapper ~ & .limiter {
float: right;
display: block;
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,17 +62,16 @@
// Common
// _____________________________________________

& when (@media-common = true) {
body {
.page-wrapper {
.ie9 & {
.lib-css(background-color, @page__background-color);
min-height: 0;
}
}

// ToDo UI: move it directly to .page-wrapper (now it doesn't appear in mediaquery cuz of size of css)
.page-wrapper {
.ie9 & {
.lib-css(background-color, @page__background-color);
min-height: 0;
}
& when (@media-common = true) {
body {
.lib-css(background-color, @page__background-color);
}

//
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
->setVisibility(\Magento\Catalog\Model\Product\Visibility::VISIBILITY_BOTH)
->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED)
->setWebsiteIds([1])
->setCateroryIds([])
->setCategoryIds([])
->setStockData(['qty' => 100, 'is_in_stock' => 1])
->setIsObjectNew(true)
->save();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED
)->setWebsiteIds(
[1]
)->setCateroryIds(
)->setCategoryIds(
[]
)->setStockData(
['qty' => 100, 'is_in_stock' => 1]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
->setVisibility(\Magento\Catalog\Model\Product\Visibility::VISIBILITY_BOTH)
->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED)
->setWebsiteIds([1])
->setCateroryIds([])
->setCategoryIds([])
->setStockData(['use_config_manage_stock' => 1, 'qty' => 100, 'is_qty_decimal' => 0, 'is_in_stock' => 1])
->save();

Expand All @@ -49,7 +49,7 @@
->setVisibility(\Magento\Catalog\Model\Product\Visibility::VISIBILITY_IN_CATALOG)
->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED)
->setWebsiteIds([1])
->setCateroryIds([])
->setCategoryIds([])
->setStockData(['use_config_manage_stock' => 1, 'qty' => 50, 'is_qty_decimal' => 0, 'is_in_stock' => 1])
->save();

Expand All @@ -68,6 +68,6 @@
->setVisibility(\Magento\Catalog\Model\Product\Visibility::VISIBILITY_IN_CATALOG)
->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_DISABLED)
->setWebsiteIds([1])
->setCateroryIds([])
->setCategoryIds([])
->setStockData(['use_config_manage_stock' => 1, 'qty' => 140, 'is_qty_decimal' => 0, 'is_in_stock' => 1])
->save();
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@
)->setVisibility(\Magento\Catalog\Model\Product\Visibility::VISIBILITY_BOTH)
->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED)
->setWebsiteIds([1])
->setCateroryIds([])
->setCategoryIds([])
->setStockData(['qty' => 100, 'is_in_stock' => 1])
->setCanSaveCustomOptions(true)
->setCategoryIds([333])
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
->setVisibility(\Magento\Catalog\Model\Product\Visibility::VISIBILITY_BOTH)
->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED)
->setWebsiteIds([1])
->setCateroryIds([])
->setCategoryIds([])
->setStockData(['qty' => 100, 'is_in_stock' => 1])
->setCanSaveCustomOptions(true)
->setCategoryIds([333])
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED
)->setWebsiteIds(
[1]
)->setCateroryIds(
)->setCategoryIds(
[]
)->setStockData(
['qty' => 100, 'is_in_stock' => 1]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
overflow: hidden;
padding: 5px 6px 3px;
margin-left: 12px;
color: #7a7976;;
color: #7a7976;
}

[class^=" catalog-product-"] .page-actions .action-back:hover,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED
)->setWebsiteIds(
[1]
)->setCateroryIds(
)->setCategoryIds(
[]
)->setStockData(
['qty' => 100, 'is_in_stock' => 1]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,14 @@ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
*/
private function validatePropertiesSorting(PHP_CodeSniffer_File $phpcsFile, $stackPtr, array $properties)
{
// Fix needed for cases when incorrect properties passed for validation due to bug in PHP tokens.
$symbolsForSkip = ['(', 'block'];
$properties = array_filter(
$properties,
function ($var) use ($symbolsForSkip) {
return !in_array($var, $symbolsForSkip);
}
);

$originalProperties = $properties;
sort($properties);
Expand Down
2 changes: 1 addition & 1 deletion lib/internal/Magento/Framework/Image.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public function open()
$this->_adapter->checkDependencies();

if (!file_exists($this->_fileName)) {
throw new \Exception("File '{$this->_fileName}' does not exists.");
throw new \Exception("File '{$this->_fileName}' does not exist.");
}

$this->_adapter->open($this->_fileName);
Expand Down
2 changes: 1 addition & 1 deletion lib/web/extjs/resources/css/ytheme-magento.css
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
background:url(../images/magento/loading_bg.gif) no-repeat #eae2ca;
padding:85px 15px 15px 15px;
font-size:14px;
font-weight:bold;;
font-weight:bold;
color:#611B06;
width:206px;
text-align:center;
Expand Down
5 changes: 4 additions & 1 deletion setup/src/Magento/Setup/Model/Grid/Module.php
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,10 @@ private function getModulesInfo(array $moduleList)
private function addGeneralInfo(array $items)
{
foreach ($items as &$item) {
$item['moduleName'] = $item['moduleName'] ?: $this->packageInfo->getModuleName($item['name']);
$item['moduleName'] = isset($item['moduleName'])
? $item['moduleName']
: $this->packageInfo->getModuleName($item['name']);

$item['enable'] = $this->moduleList->has($item['moduleName']);
$vendorSource = $item['name'] == self::UNKNOWN_PACKAGE_NAME ? $item['moduleName'] : $item['name'];
$item['vendor'] = ucfirst(current(preg_split('%[/_]%', $vendorSource)));
Expand Down