Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
25 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
f2976b2
[Correct typo mistake]
hitesh-wagento Jun 14, 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 @@ -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