Skip to content

Commit 9982931

Browse files
vgoncharenkovgoncharenko
committed
MAGETWO-63626: Remove all usages of serialize/unserilize
1 parent bdd93e2 commit 9982931

File tree

8 files changed

+199
-16
lines changed

8 files changed

+199
-16
lines changed

‎app/code/Magento/Widget/Helper/Conditions.php

Lines changed: 24 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,33 +5,52 @@
55
*/
66
namespace Magento\Widget\Helper;
77

8+
use Magento\Framework\App\ObjectManager;
9+
use Magento\Framework\Serialize\Serializer\Json;
10+
811
/**
9-
* Widget Conditions helper
12+
* Widget Conditions helper.
1013
*/
1114
class Conditions
1215
{
1316
/**
14-
* Encode widget conditions to be used with WYSIWIG
17+
* Instance of serializer interface.
18+
*
19+
* @var Json
20+
*/
21+
private $serializer;
22+
23+
/**
24+
* @param Json $serializer
25+
*/
26+
public function __construct(
27+
Json $serializer = null
28+
) {
29+
$this->serializer = $serializer ?: ObjectManager::getInstance()->get(Json::class);
30+
}
31+
32+
/**
33+
* Encode widget conditions to be used with WYSIWIG.
1534
*
1635
* @param array $value
1736
* @return string
1837
*/
1938
public function encode(array $value)
2039
{
21-
$value = str_replace(['{', '}', '"', '\\'], ['[', ']', '`', '|'], serialize($value));
40+
$value = str_replace(['{', '}', '"', '\\'], ['[', ']', '`', '|'], $this->serializer->serialize($value));
2241
return $value;
2342
}
2443

2544
/**
26-
* Decode previously encoded widget conditions
45+
* Decode previously encoded widget conditions.
2746
*
2847
* @param string $value
2948
* @return array
3049
*/
3150
public function decode($value)
3251
{
3352
$value = str_replace(['[', ']', '`', '|'], ['{', '}', '"', '\\'], $value);
34-
$value = unserialize($value);
53+
$value = $this->serializer->unserialize($value);
3554
return $value;
3655
}
3756
}

‎app/code/Magento/Widget/Model/ResourceModel/Widget.php

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,11 @@
1111
*/
1212
namespace Magento\Widget\Model\ResourceModel;
1313

14+
/**
15+
* Resource model for widget.
16+
*
17+
* @deprecated Data from this table was moved to xml(widget.xml).
18+
*/
1419
class Widget extends \Magento\Framework\Model\ResourceModel\Db\AbstractDb
1520
{
1621
/**
@@ -41,7 +46,7 @@ public function loadPreconfiguredWidget($widgetId)
4146
$widget = $connection->fetchRow($select, $bind);
4247
if (is_array($widget)) {
4348
if ($widget['parameters']) {
44-
$widget['parameters'] = unserialize($widget['parameters']);
49+
$widget['parameters'] = $this->getSerializer()->unserialize($widget['parameters']);
4550
}
4651
return $widget;
4752
}

‎app/code/Magento/Widget/Model/Widget/Instance.php

Lines changed: 29 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
namespace Magento\Widget\Model\Widget;
77

88
use Magento\Framework\App\Filesystem\DirectoryList;
9+
use Magento\Framework\Serialize\Serializer\Json;
10+
use Magento\Framework\App\ObjectManager;
911

1012
/**
1113
* Widget Instance Model
@@ -107,6 +109,27 @@ class Instance extends \Magento\Framework\Model\AbstractModel
107109
*/
108110
protected $conditionsHelper;
109111

112+
/**
113+
* Instance of serializer interface.
114+
*
115+
* @var Json
116+
*/
117+
private $serializer;
118+
119+
/**
120+
* Product type model instance.
121+
*
122+
* @var \Magento\Catalog\Model\Product\Type
123+
*/
124+
private $_productType;
125+
126+
/**
127+
* Widget config reader instance.
128+
*
129+
* @var \Magento\Widget\Model\Config\Reader
130+
*/
131+
private $_reader;
132+
110133
/**
111134
* @param \Magento\Framework\Model\Context $context
112135
* @param \Magento\Framework\Registry $registry
@@ -124,6 +147,7 @@ class Instance extends \Magento\Framework\Model\AbstractModel
124147
* @param \Magento\Framework\Data\Collection\AbstractDb $resourceCollection
125148
* @param array $relatedCacheTypes
126149
* @param array $data
150+
* @param Json $serializer
127151
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
128152
*/
129153
public function __construct(
@@ -142,7 +166,8 @@ public function __construct(
142166
\Magento\Framework\Model\ResourceModel\AbstractResource $resource = null,
143167
\Magento\Framework\Data\Collection\AbstractDb $resourceCollection = null,
144168
array $relatedCacheTypes = [],
145-
array $data = []
169+
array $data = [],
170+
Json $serializer = null
146171
) {
147172
$this->_escaper = $escaper;
148173
$this->_viewFileSystem = $viewFileSystem;
@@ -155,6 +180,7 @@ public function __construct(
155180
$this->conditionsHelper = $conditionsHelper;
156181
$this->_directory = $filesystem->getDirectoryRead(DirectoryList::ROOT);
157182
$this->_namespaceResolver = $namespaceResolver;
183+
$this->serializer = $serializer ?: ObjectManager::getInstance()->get(Json::class);
158184
parent::__construct($context, $registry, $resource, $resourceCollection, $data);
159185
}
160186

@@ -250,7 +276,7 @@ public function beforeSave()
250276
\Magento\Framework\Math\Random::CHARS_LOWERS
251277
);
252278
}
253-
$this->setData('widget_parameters', serialize($parameters));
279+
$this->setData('widget_parameters', $this->serializer->serialize($parameters));
254280
}
255281
$this->setData('page_groups', $tmpPageGroups);
256282
$this->setData('page_group_ids', $pageGroupIds);
@@ -372,7 +398,7 @@ public function getStoreIds()
372398
public function getWidgetParameters()
373399
{
374400
if (is_string($this->getData('widget_parameters'))) {
375-
return unserialize($this->getData('widget_parameters'));
401+
return $this->serializer->unserialize($this->getData('widget_parameters'));
376402
} elseif (null === $this->getData('widget_parameters')) {
377403
return [];
378404
}

‎app/code/Magento/Widget/Setup/UpgradeData.php

Lines changed: 57 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,57 @@
1+
<?php
2+
/**
3+
* Copyright © 2013-2017 Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
7+
namespace Magento\Widget\Setup;
8+
9+
use Magento\Framework\App\ObjectManager;
10+
use Magento\Framework\Setup\UpgradeDataInterface;
11+
use Magento\Framework\Setup\ModuleContextInterface;
12+
use Magento\Framework\Setup\ModuleDataSetupInterface;
13+
use Magento\Framework\DB\FieldDataConverterFactory;
14+
use Magento\Framework\DB\DataConverter\SerializedToJson;
15+
16+
/**
17+
* Upgrade data for widget module.
18+
*/
19+
class UpgradeData implements UpgradeDataInterface
20+
{
21+
/**
22+
* @var FieldDataConverterFactory
23+
*/
24+
private $fieldDataConverterFactory;
25+
26+
/**
27+
* @param FieldDataConverterFactory $fieldDataConverterFactory
28+
*/
29+
public function __construct(
30+
FieldDataConverterFactory $fieldDataConverterFactory
31+
) {
32+
33+
$this->fieldDataConverterFactory = $fieldDataConverterFactory ?: ObjectManager::getInstance()->get(
34+
FieldDataConverterFactory::class
35+
);
36+
}
37+
38+
/**
39+
* {@inheritdoc}
40+
*/
41+
public function upgrade(ModuleDataSetupInterface $setup, ModuleContextInterface $context)
42+
{
43+
$setup->startSetup();
44+
45+
if (version_compare($context->getVersion(), '2.0.1', '<')) {
46+
$fieldDataConverter = $this->fieldDataConverterFactory->create(SerializedToJson::class);
47+
$fieldDataConverter->convert(
48+
$setup->getConnection(),
49+
$setup->getTable('widget_instance'),
50+
'instance_id',
51+
'widget_parameters'
52+
);
53+
}
54+
55+
$setup->endSetup();
56+
}
57+
}

‎app/code/Magento/Widget/Test/Unit/Helper/ConditionsTest.php

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66

77
namespace Magento\Widget\Test\Unit\Helper;
88

9-
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager as ObjectManagerHelper;
10-
119
/**
1210
* Class ConditionsTest
1311
*/
@@ -18,10 +16,26 @@ class ConditionsTest extends \PHPUnit_Framework_TestCase
1816
*/
1917
protected $conditions;
2018

19+
/**
20+
* @var \Magento\Framework\Serialize\Serializer\Json|\PHPUnit_Framework_MockObject_MockObject
21+
*/
22+
private $serializer;
23+
24+
/**
25+
* {@inheritdoc}
26+
*/
2127
protected function setUp()
2228
{
23-
$objectManagerHelper = new ObjectManagerHelper($this);
24-
$this->conditions = $objectManagerHelper->getObject(\Magento\Widget\Helper\Conditions::class);
29+
$this->serializer = $this->getMock(\Magento\Framework\Serialize\Serializer\Json::class);
30+
$this->serializer->method('serialize')->willReturnCallback(function ($value) {
31+
return json_encode($value);
32+
});
33+
$this->serializer->method('unserialize')->willReturnCallback(function ($value) {
34+
return json_decode($value, true);
35+
});
36+
$this->conditions = new \Magento\Widget\Helper\Conditions(
37+
$this->serializer
38+
);
2539
}
2640

2741
public function testEncodeDecode()

‎app/code/Magento/Widget/etc/module.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
*/
77
-->
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
9-
<module name="Magento_Widget" setup_version="2.0.0">
9+
<module name="Magento_Widget" setup_version="2.0.1">
1010
<sequence>
1111
<module name="Magento_Cms"/>
1212
</sequence>

‎dev/tests/integration/testsuite/Magento/Widget/Model/Widget/InstanceTest.php

Lines changed: 33 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,8 @@ public function testGetWidgetSupportedTemplatesByContainer($model)
9999
}
100100

101101
/**
102+
* @covers \Magento\Widget\Model\Widget\Instance::generateLayoutUpdateXml()
103+
* @covers \Magento\Widget\Model\Widget\Instance::getWidgetParameters()
102104
* @param \Magento\Widget\Model\Widget\Instance $model
103105
* @depends testGetWidgetConfigAsArray
104106
*/
@@ -134,7 +136,36 @@ public function testGenerateLayoutUpdateXml(\Magento\Widget\Model\Widget\Instanc
134136
$this->assertContains('<argument name="name" xsi:type="string">types</argument>', $result);
135137
$this->assertContains('<argument name="value" xsi:type="string">type_1,type_2</argument>', $result);
136138
$this->assertContains('<argument name="name" xsi:type="string">conditions_encoded</argument>', $result);
137-
$this->assertContains('s:50:`Magento|CatalogWidget|Model|Rule|Condition|Combine`', $result);
138-
$this->assertContains('s:50:`Magento|CatalogWidget|Model|Rule|Condition|Product`', $result);
139+
$this->assertContains('`Magento||CatalogWidget||Model||Rule||Condition||Combine`', $result);
140+
$this->assertContains('`Magento||CatalogWidget||Model||Rule||Condition||Product`', $result);
141+
}
142+
143+
/**
144+
* @covers \Magento\Widget\Model\Widget\Instance::beforeSave()
145+
* @magentoDataFixture Magento/Widget/_files/new_widget.php
146+
* @dataProvider beforeSaveDataProvider
147+
* @param array $expected
148+
*/
149+
public function testBeforeSave(array $expected)
150+
{
151+
/** @var \Magento\Widget\Model\ResourceModel\Widget\Instance $resourceModel */
152+
$resourceModel = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()
153+
->get(\Magento\Widget\Model\ResourceModel\Widget\Instance::class);
154+
$resourceModel->load($this->_model, 'Magento\Widget\NewSampleWidget', 'instance_type');
155+
156+
$this->assertSame($expected, $this->_model->getWidgetParameters());
157+
}
158+
159+
/**
160+
* @return array
161+
*/
162+
public function beforeSaveDataProvider()
163+
{
164+
return [
165+
# Variation 1
166+
[
167+
['block_id' => '2']
168+
]
169+
];
139170
}
140171
}

‎dev/tests/integration/testsuite/Magento/Widget/_files/new_widget.php

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
<?php
2+
/**
3+
* Copyright © 2013-2017 Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
7+
/** @var \Magento\Widget\Model\ResourceModel\Widget\Instance $resourceModel */
8+
$resourceModel = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()
9+
->get(\Magento\Widget\Model\ResourceModel\Widget\Instance::class);
10+
11+
$model = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()
12+
->get(\Magento\Widget\Model\Widget\Instance::class);
13+
14+
$model->setData(
15+
[
16+
'instance_type' => 'Magento\\Widget\\NewSampleWidget',
17+
'theme_id' => '4',
18+
'title' => 'New Sample widget title',
19+
'store_ids' => [
20+
0 => '0',
21+
],
22+
'widget_parameters' => [
23+
'block_id' => '2',
24+
],
25+
'sort_order' => '0',
26+
'page_groups' => [],
27+
'instance_code' => 'new_sample_widget',
28+
]
29+
);
30+
31+
$resourceModel->save($model);

0 commit comments

Comments
 (0)