Skip to content

Commit 2880aeb

Browse files
oshmyheliukoshmyheliuk
committed
Merge remote-tracking branch 'origin/2.4-develop' into B2B-2036
2 parents 3e99507 + 0af0fba commit 2880aeb

File tree

15 files changed

+224
-46
lines changed

15 files changed

+224
-46
lines changed

‎app/code/Magento/CatalogInventory/Model/StockStateProvider.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ public function checkQuoteItemQty(StockItemInterface $stockItem, $qty, $summaryQ
135135
$result->addData($this->checkQtyIncrements($stockItem, $qty)->getData());
136136

137137
$result->setItemIsQtyDecimal($stockItem->getIsQtyDecimal());
138-
if (!$stockItem->getIsQtyDecimal() && (floor($qty) !== $qty)) {
138+
if (!$stockItem->getIsQtyDecimal() && (floor($qty) !== (float) $qty)) {
139139
$result->setHasError(true)
140140
->setMessage(__('You cannot use decimal quantity for this product.'))
141141
->setErrorCode('qty_decimal')

‎app/code/Magento/Sales/Model/Order/Payment.php‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2272,6 +2272,21 @@ public function setCcLast4($ccLast4)
22722272
return $this->setData(OrderPaymentInterface::CC_LAST_4, $ccLast4);
22732273
}
22742274

2275+
/**
2276+
* @inheritDoc
2277+
*
2278+
* Because cc_last_4 data attribute violates data contract (use underscore (_) between alphanumerical characters),
2279+
* this ad hoc method is for setting cc_last_4 data value in \Magento\Framework\Api\DataObjectHelper::_setDataValues
2280+
*/
2281+
public function setCustomAttribute($attributeCode, $attributeValue)
2282+
{
2283+
if ($attributeCode === OrderPaymentInterface::CC_LAST_4) {
2284+
return parent::setData(OrderPaymentInterface::CC_LAST_4, $attributeValue);
2285+
}
2286+
2287+
return parent::setCustomAttribute($attributeCode, $attributeValue);
2288+
}
2289+
22752290
/**
22762291
* @inheritdoc
22772292
*/

‎dev/tests/api-functional/testsuite/Magento/Webapi/WsdlGenerationFromDataObjectTest.php‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,10 @@ public function testNoAuthorizedServices()
7171
curl_setopt($connection, CURLOPT_RETURNTRANSFER, 1);
7272
$responseContent = curl_exec($connection);
7373
$this->assertEquals(curl_getinfo($connection, CURLINFO_HTTP_CODE), 401);
74-
$this->assertStringContainsString("The consumer isn't authorized to access %resources.", $responseContent);
74+
$this->assertStringContainsString(
75+
"The consumer isn't authorized to access %resources.",
76+
htmlspecialchars_decode($responseContent, ENT_QUOTES)
77+
);
7578
}
7679

7780
public function testInvalidWsdlUrlNoServices()

‎dev/tests/integration/testsuite/Magento/Framework/DB/Adapter/Pdo/MysqlTest.php‎

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,30 @@ protected function tearDown(): void
3434
restore_error_handler();
3535
}
3636

37+
/**
38+
* Check PDO stringify fetches options
39+
*/
40+
public function testStringifyFetchesTrue(): void
41+
{
42+
$tableName = $this->resourceConnection->getTableName('table_with_int_column');
43+
$columnId = 'integer_column';
44+
$adapter = $this->getDbAdapter();
45+
46+
$table = $adapter
47+
->newTable($tableName)
48+
->addColumn($columnId, Table::TYPE_INTEGER);
49+
$adapter->createTable($table);
50+
$adapter->insert($tableName, [$columnId => 100]);
51+
52+
$select = $adapter->select()
53+
->from($tableName)
54+
->columns([$columnId])
55+
->limit(1);
56+
$result = $adapter->fetchOne($select);
57+
$this->assertIsString($result);
58+
$adapter->dropTable($tableName);
59+
}
60+
3761
/**
3862
* Test lost connection re-initializing
3963
*

‎lib/internal/Magento/Framework/Api/DataObjectHelper.php‎

Lines changed: 88 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
use Magento\Framework\Reflection\MethodsMap;
1010

1111
/**
12-
* Data object helper.
12+
* Service class allow populating object from array data
1313
*
1414
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
1515
*/
@@ -94,49 +94,69 @@ public function populateWithArray($dataObject, array $data, $interfaceName)
9494
* @param string $interfaceName
9595
* @return $this
9696
* @SuppressWarnings(PHPMD.CyclomaticComplexity)
97+
* @SuppressWarnings(PHPMD.NPathComplexity)
9798
*/
9899
protected function _setDataValues($dataObject, array $data, $interfaceName)
99100
{
100-
$dataObjectMethods = get_class_methods(get_class($dataObject));
101-
foreach ($data as $key => $value) {
102-
/* First, verify is there any setter for the key on the Service Data Object */
103-
$camelCaseKey = \Magento\Framework\Api\SimpleDataObjectConverter::snakeCaseToUpperCamelCase($key);
104-
$possibleMethods = [
105-
'set' . $camelCaseKey,
106-
'setIs' . $camelCaseKey,
107-
];
108-
if ($key === CustomAttributesDataInterface::CUSTOM_ATTRIBUTES
109-
&& ($dataObject instanceof ExtensibleDataInterface)
110-
&& is_array($data[$key])
111-
&& !empty($data[$key])
112-
) {
113-
foreach ($data[$key] as $customAttribute) {
114-
$dataObject->setCustomAttribute(
115-
$customAttribute[AttributeInterface::ATTRIBUTE_CODE],
116-
$customAttribute[AttributeInterface::VALUE]
117-
);
118-
}
119-
} elseif ($methodNames = array_intersect($possibleMethods, $dataObjectMethods)) {
120-
$methodName = array_values($methodNames)[0];
121-
if (!is_array($value)) {
122-
if ($methodName === 'setExtensionAttributes' && $value === null) {
123-
// Cannot pass a null value to a method with a typed parameter
101+
if (empty($data)) {
102+
return $this;
103+
}
104+
$setMethods = $this->getSetters($dataObject);
105+
if ($dataObject instanceof ExtensibleDataInterface
106+
&& !empty($data[CustomAttributesDataInterface::CUSTOM_ATTRIBUTES])
107+
) {
108+
foreach ($data[CustomAttributesDataInterface::CUSTOM_ATTRIBUTES] as $customAttribute) {
109+
$dataObject->setCustomAttribute(
110+
$customAttribute[AttributeInterface::ATTRIBUTE_CODE],
111+
$customAttribute[AttributeInterface::VALUE]
112+
);
113+
}
114+
unset($data[CustomAttributesDataInterface::CUSTOM_ATTRIBUTES]);
115+
}
116+
if ($dataObject instanceof \Magento\Framework\Model\AbstractModel) {
117+
$simpleData = array_filter($data, static function ($e) {
118+
return is_scalar($e) || is_null($e);
119+
});
120+
if (isset($simpleData['id'])) {
121+
$dataObject->setId($simpleData['id']);
122+
unset($simpleData['id']);
123+
}
124+
$simpleData = array_intersect_key($simpleData, $setMethods);
125+
$dataObject->addData($simpleData);
126+
$data = array_diff_key($data, $simpleData);
127+
if (\count($data) === 0) {
128+
return $this;
129+
}
130+
}
131+
foreach (array_intersect_key($data, $setMethods) as $key => $value) {
132+
$methodName = SimpleDataObjectConverter::snakeCaseToUpperCamelCase($key);
133+
134+
if (!is_array($value)) {
135+
if ($methodName !== 'ExtensionAttributes' || $value !== null) {
136+
if (method_exists($dataObject, 'set' . $methodName)) {
137+
$dataObject->{'set' . $methodName}($value);
124138
} else {
125-
$dataObject->$methodName($value);
139+
$dataObject->{'setIs' . $methodName}($value);
126140
}
127-
} else {
128-
$getterMethodName = 'get' . $camelCaseKey;
129-
$this->setComplexValue($dataObject, $getterMethodName, $methodName, $value, $interfaceName);
130141
}
131-
} elseif ($dataObject instanceof CustomAttributesDataInterface) {
132-
$dataObject->setCustomAttribute($key, $value);
142+
} else {
143+
$getterMethodName = 'get' . $methodName;
144+
$this->setComplexValue($dataObject, $getterMethodName, 'set' . $methodName, $value, $interfaceName);
133145
}
146+
unset($data[$key]);
134147
}
135148

149+
if ($dataObject instanceof CustomAttributesDataInterface) {
150+
foreach ($data as $key => $value) {
151+
$dataObject->setCustomAttribute($key, $value);
152+
}
153+
}
136154
return $this;
137155
}
138156

139157
/**
158+
* Set complex (like object) value using $methodName based on return type of $getterMethodName
159+
*
140160
* @param mixed $dataObject
141161
* @param string $getterMethodName
142162
* @param string $methodName
@@ -179,7 +199,7 @@ protected function setComplexValue(
179199
} elseif (is_subclass_of($returnType, \Magento\Framework\Api\ExtensionAttributesInterface::class)) {
180200
foreach ($value as $extensionAttributeKey => $extensionAttributeValue) {
181201
$extensionAttributeGetterMethodName
182-
= 'get' . \Magento\Framework\Api\SimpleDataObjectConverter::snakeCaseToUpperCamelCase(
202+
= 'get' . SimpleDataObjectConverter::snakeCaseToUpperCamelCase(
183203
$extensionAttributeKey
184204
);
185205
$methodReturnType = $this->methodsMapProcessor->getMethodReturnType(
@@ -260,4 +280,40 @@ public function getCustomAttributeValueByType(array $attributeValues, $type)
260280
}
261281
return $attributeValueArray;
262282
}
283+
284+
/** @var array */
285+
private array $settersCache = [];
286+
287+
/**
288+
* Get list of setters for object
289+
*
290+
* @param object $dataObject
291+
* @return array
292+
*/
293+
private function getSetters(object $dataObject): array
294+
{
295+
$class = get_class($dataObject);
296+
if (!isset($this->settersCache[$class])) {
297+
$dataObjectMethods = get_class_methods($class);
298+
// use regexp to manipulate with method list as it use jit starting with PHP 7.3
299+
$setters = array_filter(
300+
explode(
301+
',',
302+
strtolower(
303+
// (0) remove all not setter
304+
// (1) add _ before upper letter
305+
// (2) remove set_ in start of name
306+
// (3) add name without is_ prefix
307+
preg_replace(
308+
['/(^|,)(?!set)[^,]*/S','/(.)([A-Z])/S', '/(^|,)set_/iS', '/(^|,)is_([^,]+)/is'],
309+
['', '$1_$2', '$1', '$1$2,is_$2'],
310+
implode(',', $dataObjectMethods)
311+
)
312+
)
313+
)
314+
);
315+
$this->settersCache[$class] = array_flip($setters);
316+
}
317+
return $this->settersCache[$class];
318+
}
263319
}

‎lib/internal/Magento/Framework/Api/Test/Unit/DataObjectHelperTest.php‎

Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@
2424
use Magento\Framework\Reflection\MethodsMap;
2525
use Magento\Framework\Reflection\TypeProcessor;
2626
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
27+
use Magento\Sales\Api\Data\OrderPaymentInterface;
28+
use Magento\Sales\Model\Order\Payment;
2729
use PHPUnit\Framework\MockObject\MockObject;
2830
use PHPUnit\Framework\TestCase;
2931

@@ -297,6 +299,66 @@ public function testPopulateWithArrayWithCustomAttributes(): void
297299
);
298300
}
299301

302+
/**
303+
* @return void
304+
*/
305+
public function testPopulateWithArrayWithOrderPaymentAttributes(): void
306+
{
307+
$method = 'companycredit';
308+
$customerPaymentId = null;
309+
$additionalData = null;
310+
$poNumber = 'ReferenceNumber934829dek2';
311+
$cc_type = "Debit";
312+
$cc_number_enc = "393993138";
313+
$cc_last_4 = "3982";
314+
$cc_owner = "John Doe";
315+
$cc_exp_month = "05";
316+
$cc_exp_year = "24";
317+
$cc_number = '1234567890';
318+
$cc_cid = null;
319+
$cc_ss_issue = null;
320+
$cc_ss_start_month = "0";
321+
$cc_ss_start_year = "0";
322+
323+
/** @var OrderPaymentInterface $orderPaymentObject */
324+
$orderPaymentObject = $this->objectManager->getObject(
325+
Payment::class,
326+
['dataObjectHelper' => $this->dataObjectHelper]
327+
);
328+
329+
$data = [
330+
'method' => $method,
331+
'customer_payment_id' => $customerPaymentId,
332+
'additionalData' => $additionalData,
333+
'additionalInformation' => [],
334+
'po_number' => $poNumber,
335+
'cc_type' => $cc_type,
336+
'cc_number_enc' => $cc_number_enc,
337+
'cc_last_4' => $cc_last_4,
338+
'cc_owner' => $cc_owner,
339+
'cc_exp_month' => $cc_exp_month,
340+
'cc_exp_year' => $cc_exp_year,
341+
'cc_number' => $cc_number,
342+
'cc_cid' => $cc_cid,
343+
'cc_ss_issue' => $cc_ss_issue,
344+
'cc_ss_start_month' => $cc_ss_start_month,
345+
'cc_ss_start_year' => $cc_ss_start_year
346+
];
347+
$this->dataObjectHelper->populateWithArray(
348+
$orderPaymentObject,
349+
$data,
350+
OrderPaymentInterface::class
351+
);
352+
$this->assertEquals($method, $orderPaymentObject->getMethod());
353+
$this->assertEquals($cc_exp_month, $orderPaymentObject->getCcExpMonth());
354+
$this->assertEquals($cc_exp_year, $orderPaymentObject->getCcExpYear());
355+
$this->assertEquals($cc_last_4, $orderPaymentObject->getCcLast4());
356+
$this->assertEquals($cc_owner, $orderPaymentObject->getCcOwner());
357+
$this->assertEquals($cc_number_enc, $orderPaymentObject->getCcNumberEnc());
358+
$this->assertEquals($poNumber, $orderPaymentObject->getPoNumber());
359+
$this->assertEquals($cc_type, $orderPaymentObject->getCcType());
360+
}
361+
300362
/**
301363
* @param array $data1
302364
* @param array $data2

‎lib/internal/Magento/Framework/App/Test/Unit/StaticResourceTest.php‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -328,6 +328,9 @@ public function testLaunchWrongPathProductionMode()
328328
$this->responseMock->expects($this->once())
329329
->method('setHttpResponseCode')
330330
->with(404);
331+
$this->driverMock->expects($this->once())
332+
->method('getRealPathSafety')
333+
->willReturnArgument(0);
331334
$this->object->launch();
332335
}
333336

‎lib/internal/Magento/Framework/App/Test/Unit/ViewTest.php‎

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,7 @@ public function testLoadLayoutWhenLayoutAlreadyLoaded(): void
155155
$this->expectException('RuntimeException');
156156
$this->expectExceptionMessage('Layout must be loaded only once.');
157157
$this->_view->setIsLayoutLoaded(true);
158+
// phpcs:ignore Magento2.Legacy.ObsoleteResponse
158159
$this->_view->loadLayout();
159160
}
160161

@@ -166,7 +167,8 @@ public function testLoadLayoutWithDefaultSetup(): void
166167
$this->_layoutProcessor
167168
->method('addHandle')
168169
->withConsecutive(['default']);
169-
$this->_requestMock->expects($this->any())->method('getFullActionName')->willReturn('action_name');
170+
$this->_requestMock->method('getFullActionName')->willReturn('action_name');
171+
// phpcs:ignore Magento2.Legacy.ObsoleteResponse
170172
$this->_view->loadLayout();
171173
}
172174

@@ -175,6 +177,8 @@ public function testLoadLayoutWithDefaultSetup(): void
175177
*/
176178
public function testLoadLayoutWhenBlocksNotGenerated(): void
177179
{
180+
$this->_requestMock->method('getFullActionName')->willReturn('action_name');
181+
// phpcs:ignore Magento2.Legacy.ObsoleteResponse
178182
$this->_view->loadLayout('', false, true);
179183
}
180184

@@ -183,6 +187,8 @@ public function testLoadLayoutWhenBlocksNotGenerated(): void
183187
*/
184188
public function testLoadLayoutWhenXmlNotGenerated(): void
185189
{
190+
$this->_requestMock->method('getFullActionName')->willReturn('action_name');
191+
// phpcs:ignore Magento2.Legacy.ObsoleteResponse
186192
$this->_view->loadLayout('', true, false);
187193
}
188194

@@ -263,8 +269,12 @@ public function testGenerateLayoutBlocksWhenFlagIsSet(): void
263269
*/
264270
public function testRenderLayoutIfActionFlagExist(): void
265271
{
266-
$this->_actionFlagMock->expects($this->once())->method('get')->with('', 'no-renderLayout')->willReturn(true);
272+
$this->_actionFlagMock->expects($this->once())
273+
->method('get')
274+
->with('', 'no-renderLayout')
275+
->willReturn(true);
267276
$this->_eventManagerMock->expects($this->never())->method('dispatch');
277+
// phpcs:ignore Magento2.Legacy.ObsoleteResponse
268278
$this->_view->renderLayout();
269279
}
270280

@@ -279,6 +289,7 @@ public function testRenderLayoutWhenOutputNotEmpty(): void
279289
->willReturn(false);
280290
$this->_layoutMock->expects($this->once())->method('addOutputElement')->with('output');
281291
$this->resultPage->expects($this->once())->method('renderResult')->with($this->response);
292+
// phpcs:ignore Magento2.Legacy.ObsoleteResponse
282293
$this->_view->renderLayout('output');
283294
}
284295

@@ -294,6 +305,7 @@ public function testRenderLayoutWhenOutputEmpty(): void
294305

295306
$this->_layoutMock->expects($this->never())->method('addOutputElement');
296307
$this->resultPage->expects($this->once())->method('renderResult')->with($this->response);
308+
// phpcs:ignore Magento2.Legacy.ObsoleteResponse
297309
$this->_view->renderLayout();
298310
}
299311
}

‎lib/internal/Magento/Framework/Code/Generator/EntityAbstract.php‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,8 +157,7 @@ public function getSourceClassNameWithoutNamespace()
157157
*/
158158
protected function _getFullyQualifiedClassName($className)
159159
{
160-
$className = ltrim($className, '\\');
161-
return $className ? '\\' . $className : '';
160+
return $className ? '\\' . ltrim($className, '\\') : '';
162161
}
163162

164163
/**

0 commit comments

Comments
 (0)