Skip to content

[Forwardport] Correct typo correction js files #16157

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

Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
24fb85b
Merge pull request #2 from hitesh-wagento/2.3-develop-PR-port-15734
hitesh-wagento Jun 5, 2018
cc3cdbd
hitesh-wagento Jun 5, 2018
bccf2b2
Merge pull request #3 from hitesh-wagento/2.3-develop-PR-port-15793
hitesh-wagento Jun 5, 2018
46fd094
hitesh-wagento Jun 5, 2018
b3b31f1
Merge pull request #4 from hitesh-wagento/2.3-develop-PR-port-15792
hitesh-wagento Jun 5, 2018
a99ebd0
hitesh-wagento Jun 5, 2018
8d4aa83
Merge pull request #6 from hitesh-wagento/2.3-develop-PR-port-15811
hitesh-wagento Jun 5, 2018
0f23ed7
Add missing table aliases to fields mapping for Customer Group filter…
Jun 5, 2018
0b4443d
Merge pull request #7 from hitesh-wagento/2.3-develop-PR-port-15826
hitesh-wagento Jun 5, 2018
58600f5
Merge pull request #8 from hitesh-wagento/2.3-develop-PR-port-15878
hitesh-wagento Jun 6, 2018
d6f8368
Merge pull request #10 from hitesh-wagento/2.3-develop-PR-port-15914
hitesh-wagento Jun 7, 2018
d9ee180
Attribute set save refactor
AnshuMishra17 Jun 11, 2018
e77424b
code style : 120 line limit
AnshuMishra17 Jun 11, 2018
d4278e4
magento/magento2#15990: Attribute set save admin controller refactor
ihor-sviziev Jun 12, 2018
bad62ef
magento/magento2#15990: Attribute set save admin controller refactor
ihor-sviziev Jun 12, 2018
d942f2b
Merge pull request #13 from hitesh-wagento/2.3-develop-PR-port-15990
hitesh-wagento Jun 12, 2018
e935949
#15205: Postpone instantiation of session config by using a proxy
Jun 7, 2018
69810b7
#15929: Apply changes to HTTP response also to its subclasses
Jun 8, 2018
a8f8831
#15929: Reset changes to HTTP response classes
Jun 8, 2018
a342e9f
Merge pull request #14 from hitesh-wagento/2.3-develop-PR-port-15929
hitesh-wagento Jun 12, 2018
b2ed432
Merge remote-tracking branch 'upstream/2.3-develop' into 2.3-develop
hitesh-wagento Jun 13, 2018
c6f89a1
fixed word typo
ledian-hymetllari Jun 6, 2018
5be1a66
Merge remote-tracking branch 'upstream/2.3-develop' into 2.3-develop
hitesh-wagento Jun 14, 2018
68fbcbf
Merge pull request #15 from hitesh-wagento/2.3-develop-PR-port-15907
hitesh-wagento Jun 14, 2018
6cbf708
hitesh-wagento Jun 5, 2018
4189521
hitesh-wagento Jun 5, 2018
6ceaf1b
hitesh-wagento Jun 5, 2018
b651610
hitesh-wagento Jun 5, 2018
5afd1f9
Merge pull request #18 from hitesh-wagento/2.3-develop-PR-port-15811
hitesh-wagento Jun 15, 2018
79bfa59
Merge remote-tracking branch 'upstream/2.3-develop' into 2.3-develop
hitesh-wagento Jun 15, 2018
c662b3a
Merge branch '2.3-develop' of github.com:hitesh-wagento/magento2 into…
hitesh-wagento Jun 15, 2018
f83d5b5
[Forwardport Correct typo correction js files]
hitesh-wagento Jun 15, 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
47 changes: 36 additions & 11 deletions app/code/Magento/Catalog/Controller/Adminhtml/Product/Set/Save.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@
*/
namespace Magento\Catalog\Controller\Adminhtml\Product\Set;

/**
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
*/
class Save extends \Magento\Catalog\Controller\Adminhtml\Product\Set
{
/**
Expand All @@ -17,22 +20,49 @@ class Save extends \Magento\Catalog\Controller\Adminhtml\Product\Set
* @var \Magento\Framework\Controller\Result\JsonFactory
*/
protected $resultJsonFactory;


/*
* @var \Magento\Eav\Model\Entity\Attribute\SetFactory
*/
private $attributeSetFactory;

/*
* @var \Magento\Framework\Filter\FilterManager
*/
private $filterManager;

/*
* @var \Magento\Framework\Json\Helper\Data
*/
private $jsonHelper;

/**
* @param \Magento\Backend\App\Action\Context $context
* @param \Magento\Framework\Registry $coreRegistry
* @param \Magento\Framework\View\LayoutFactory $layoutFactory
* @param \Magento\Framework\Controller\Result\JsonFactory $resultJsonFactory
* @param \Magento\Eav\Model\Entity\Attribute\SetFactory $attributeSetFactory
* @param \Magento\Framework\Filter\FilterManager $filterManager
* @param \Magento\Framework\Json\Helper\Data $jsonHelper
*/
public function __construct(
\Magento\Backend\App\Action\Context $context,
\Magento\Framework\Registry $coreRegistry,
\Magento\Framework\View\LayoutFactory $layoutFactory,
\Magento\Framework\Controller\Result\JsonFactory $resultJsonFactory
\Magento\Framework\Controller\Result\JsonFactory $resultJsonFactory,
\Magento\Eav\Model\Entity\Attribute\SetFactory $attributeSetFactory = null,
\Magento\Framework\Filter\FilterManager $filterManager = null,
\Magento\Framework\Json\Helper\Data $jsonHelper = null
) {
parent::__construct($context, $coreRegistry);
$this->layoutFactory = $layoutFactory;
$this->resultJsonFactory = $resultJsonFactory;
$this->attributeSetFactory = $attributeSetFactory ?: \Magento\Framework\App\ObjectManager::getInstance()
->get(\Magento\Eav\Model\Entity\Attribute\SetFactory::class);
$this->filterManager = $filterManager ?: \Magento\Framework\App\ObjectManager::getInstance()
->get(\Magento\Framework\Filter\FilterManager::class);
$this->jsonHelper = $jsonHelper ?: \Magento\Framework\App\ObjectManager::getInstance()
->get(\Magento\Framework\Json\Helper\Data::class);
}

/**
Expand Down Expand Up @@ -65,16 +95,12 @@ public function execute()
$isNewSet = $this->getRequest()->getParam('gotoEdit', false) == '1';

/* @var $model \Magento\Eav\Model\Entity\Attribute\Set */
$model = $this->_objectManager->create(\Magento\Eav\Model\Entity\Attribute\Set::class)
->setEntityTypeId($entityTypeId);

/** @var $filterManager \Magento\Framework\Filter\FilterManager */
$filterManager = $this->_objectManager->get(\Magento\Framework\Filter\FilterManager::class);
$model = $this->attributeSetFactory->create()->setEntityTypeId($entityTypeId);

try {
if ($isNewSet) {
//filter html tags
$name = $filterManager->stripTags($this->getRequest()->getParam('attribute_set_name'));
$name = $this->filterManager->stripTags($this->getRequest()->getParam('attribute_set_name'));
$model->setAttributeSetName(trim($name));
} else {
if ($attributeSetId) {
Expand All @@ -85,11 +111,10 @@ public function execute()
__('This attribute set no longer exists.')
);
}
$data = $this->_objectManager->get(\Magento\Framework\Json\Helper\Data::class)
->jsonDecode($this->getRequest()->getPost('data'));
$data = $this->jsonHelper->jsonDecode($this->getRequest()->getPost('data'));

//filter html tags
$data['attribute_set_name'] = $filterManager->stripTags($data['attribute_set_name']);
$data['attribute_set_name'] = $this->filterManager->stripTags($data['attribute_set_name']);

$model->organizeData($data);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ define([
},

/**
* Prepare storages congfig.
* Prepare storages config.
*
* @returns {Object} Chainable.
*/
Expand Down
12 changes: 6 additions & 6 deletions app/code/Magento/Customer/etc/di.xml
Original file line number Diff line number Diff line change
Expand Up @@ -353,19 +353,19 @@
<virtualType name="Magento\Customer\Model\Api\SearchCriteria\CollectionProcessor\GroupFilterProcessor" type="Magento\Framework\Api\SearchCriteria\CollectionProcessor\FilterProcessor">
<arguments>
<argument name="fieldMapping" xsi:type="array">
<item name="code" xsi:type="string">customer_group_code</item>
<item name="id" xsi:type="string">customer_group_id</item>
<item name="tax_class_name" xsi:type="string">class_name</item>
<item name="code" xsi:type="string">main_table.customer_group_code</item>
<item name="id" xsi:type="string">main_table.customer_group_id</item>
<item name="tax_class_name" xsi:type="string">tax_class_table.class_name</item>
</argument>
</arguments>
</virtualType>
<!-- @api -->
<virtualType name="Magento\Customer\Model\Api\SearchCriteria\CollectionProcessor\GroupSortingProcessor" type="Magento\Framework\Api\SearchCriteria\CollectionProcessor\SortingProcessor">
<arguments>
<argument name="fieldMapping" xsi:type="array">
<item name="code" xsi:type="string">customer_group_code</item>
<item name="id" xsi:type="string">customer_group_id</item>
<item name="tax_class_name" xsi:type="string">class_name</item>
<item name="code" xsi:type="string">main_table.customer_group_code</item>
<item name="id" xsi:type="string">main_table.customer_group_id</item>
<item name="tax_class_name" xsi:type="string">tax_class_table.class_name</item>
</argument>
<argument name="defaultOrders" xsi:type="array">
<item name="id" xsi:type="string">ASC</item>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ define([

/**
* Retrieves from the list file which matches
* search criteria implemented in itertor function.
* search criteria implemented in iterator function.
*
* @param {Function} fn - Function that will be invoked
* for each file in the list.
Expand Down Expand Up @@ -196,7 +196,7 @@ define([
},

/**
* Returns path to the files' preview image.
* Returns path to the file's preview image.
*
* @param {Object} file
* @returns {String}
Expand Down Expand Up @@ -319,7 +319,7 @@ define([

/**
* Handler which is invoked when files are choosed for upload.
* May be used for implementation of aditional validation rules,
* May be used for implementation of additional validation rules,
* e.g. total files and a total size rules.
*
* @param {Event} e - Event object.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
padding: @block-auth__dropdown__padding;
}
}

.authentication-wrapper {
float: right;
margin-top: -1.5*@indent__xl;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -198,6 +198,7 @@
.payment-option-title {
.lib-css(padding-left, @checkout-payment-option-content__padding__xl);
}

.payment-option-content {
.payment-option-inner {
+ .actions-toolbar {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -188,6 +188,7 @@
}
}
}

.row-error {
td {
border-top: none;
Expand Down Expand Up @@ -285,6 +286,7 @@
.lib-css(max-width, @checkout-shipping-address__max-width);
}
}

.table-checkout-shipping-method {
width: auto;
}
Expand Down Expand Up @@ -324,6 +326,7 @@
}
}
}

.table-checkout-shipping-method {
min-width: 500px;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
.actions-toolbar:not(:last-child) {
margin-bottom: @indent__xl;
}

.fieldset {
.nested {
.field:not(.choice) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,20 +35,24 @@
.shipping-item {
margin-left:84px;
}

.shipping-label {
font-weight: @font-weight__bold;
margin-right: @indent__s;
}

.shipping-address {
font-weight: @font-weight__regular;
}

.error-block {
color: @color-red10;

.error-label {
font-weight: @font-weight__bold;
margin-right: @indent__s;
}

.error-description {
font-weight: @font-weight__regular;
}
Expand All @@ -64,6 +68,7 @@
.order-id {
float:left;
}

.shipping-item {
margin-left:100px;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -296,6 +296,7 @@
}
}
}

.opc-wrapper {
.form-login,
.form-shipping-address {
Expand All @@ -307,6 +308,7 @@
padding-bottom: @indent__base;
}
}

.table-checkout-shipping-method {
width: auto;
}
Expand Down Expand Up @@ -346,6 +348,7 @@
}
}
}

.table-checkout-shipping-method {
min-width: 500px;
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,28 +1,28 @@
& when (@media-common = true) {
.box-tocart {
.action.instant-purchase {
&:extend(.abs-button-l all);
&:extend(.abs-button-responsive all);
.box-tocart {
.action.instant-purchase {
&:extend(.abs-button-l all);
&:extend(.abs-button-responsive all);

&:not(:last-child) {
margin-bottom: 15px;
}
&:not(:last-child) {
margin-bottom: 15px;
}

vertical-align: top;
vertical-align: top;
}
}
}
}

//
// Desktop
// _____________________________________________

.media-width(@extremum, @break) when (@extremum = 'min') and (@break = @screen__m) {
.box-tocart {
.action.instant-purchase {
margin-bottom: 0;
margin-right: 1%;
width: 49%;
.box-tocart {
.action.instant-purchase {
margin-bottom: 0;
margin-right: 1%;
width: 49%;
}
}
}
}
Loading