Skip to content

Commit 6da226c

Browse files
committed
Merge remote-tracking branch 'origin/2.4.6-develop' into AC-7681-v3
2 parents 244566e + 03c2d71 commit 6da226c

File tree

41 files changed

+327
-197
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+327
-197
lines changed

app/code/Magento/Catalog/Test/Unit/Model/CategoryTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ public function testMoveWhenCannotFindParentCategory(): void
214214
{
215215
$this->expectException('Magento\Framework\Exception\LocalizedException');
216216
$this->expectExceptionMessage('Sorry, but we can\'t find the new parent category you selected.');
217-
$this->markTestIncomplete('MAGETWO-31165');
217+
$this->markTestSkipped('MAGETWO-31165');
218218
$parentCategory = $this->createPartialMock(
219219
Category::class,
220220
['getId', 'setStoreId', 'load']
@@ -260,7 +260,7 @@ public function testMoveWhenParentCategoryIsSameAsChildCategory(): void
260260
$this->expectExceptionMessage(
261261
'We can\'t move the category because the parent category name matches the child category name.'
262262
);
263-
$this->markTestIncomplete('MAGETWO-31165');
263+
$this->markTestSkipped('MAGETWO-31165');
264264
$parentCategory = $this->createPartialMock(
265265
Category::class,
266266
['getId', 'setStoreId', 'load']

app/code/Magento/CatalogGraphQl/Model/Resolver/Products/Query/Search.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,8 @@ private function buildSearchCriteria(array $args, ResolveInfo $info): SearchCrit
186186
{
187187
$productFields = (array)$info->getFieldSelection(1);
188188
$includeAggregations = isset($productFields['filters']) || isset($productFields['aggregations']);
189-
$processedArgs = $this->argsSelection->process((string) $info->fieldName, $args);
189+
$fieldName = $info->fieldName ?? "";
190+
$processedArgs = $this->argsSelection->process((string) $fieldName, $args);
190191
$searchCriteria = $this->searchCriteriaBuilder->build($processedArgs, $includeAggregations);
191192

192193
return $searchCriteria;

app/code/Magento/Fedex/Test/Unit/Model/Source/GenericTest.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,8 @@ protected function setUp(): void
5555
* @return void
5656
* @dataProvider toOptionArrayDataProvider
5757
*/
58-
public function testToOptionArray($code, $methods, $result): void
58+
public function testToOptionArray($methods, $result): void
5959
{
60-
$this->model->code = $code;
6160
$this->shippingFedexMock->expects($this->once())
6261
->method('getCode')
6362
->willReturn($methods);
@@ -74,7 +73,6 @@ public function toOptionArrayDataProvider(): array
7473
{
7574
return [
7675
[
77-
'method',
7876
[
7977
'FEDEX_GROUND' => __('Ground'),
8078
'FIRST_OVERNIGHT' => __('First Overnight')
@@ -85,7 +83,6 @@ public function toOptionArrayDataProvider(): array
8583
]
8684
],
8785
[
88-
'',
8986
false,
9087
[]
9188
]

app/code/Magento/GraphQl/composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"magento/module-webapi": "*",
1010
"magento/module-new-relic-reporting": "*",
1111
"magento/module-authorization": "*",
12-
"webonyx/graphql-php": "^14.11"
12+
"webonyx/graphql-php": "^15.0"
1313
},
1414
"suggest": {
1515
"magento/module-graph-ql-cache": "*"

app/code/Magento/GroupedProduct/Test/Unit/Model/ProductTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,7 @@ protected function setUp(): void
349349
*/
350350
public function testGetProductLinks(): void
351351
{
352-
$this->markTestIncomplete('Skipped due to https://jira.corp.x.com/browse/MAGETWO-36926');
352+
$this->markTestSkipped('Skipped due to https://jira.corp.x.com/browse/MAGETWO-36926');
353353
$linkTypes = ['related' => 1, 'upsell' => 4, 'crosssell' => 5, 'associated' => 3];
354354
$this->linkTypeProviderMock->expects($this->once())->method('getLinkTypes')->willReturn($linkTypes);
355355

app/code/Magento/ImportExport/Test/Unit/Helper/ReportTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ protected function setUp(): void
153153
*/
154154
public function testGetExecutionTime()
155155
{
156-
$this->markTestIncomplete('Invalid mocks used for DateTime object. Investigate later.');
156+
$this->markTestSkipped('Invalid mocks used for DateTime object. Investigate later.');
157157

158158
$startDate = '2000-01-01 01:01:01';
159159
$endDate = '2000-01-01 02:03:04';

app/code/Magento/ImportExport/Test/Unit/Model/Import/Source/ZipTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ protected function setUp(): void
4242
*/
4343
public function testConstructorFileDestinationMatch($fileName, $expectedfileName): void
4444
{
45-
$this->markTestIncomplete('The implementation of constructor has changed. Rewrite test to cover changes.');
45+
$this->markTestSkipped('The implementation of constructor has changed. Rewrite test to cover changes.');
4646

4747
$this->directory->method('getRelativePath')
4848
->withConsecutive([$fileName], [$expectedfileName]);

app/code/Magento/Sales/Helper/Admin.php

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,13 @@ public function escapeHtmlWithLinks($data, $allowedTags = null)
166166

167167
$internalErrors = libxml_use_internal_errors(true);
168168

169-
$data = mb_convert_encoding($data, 'HTML-ENTITIES', 'UTF-8');
169+
$convmap = [0x80, 0x10FFFF, 0, 0x1FFFFF];
170+
$data = mb_encode_numericentity(
171+
$data,
172+
$convmap,
173+
'UTF-8'
174+
);
175+
170176
$domDocument->loadHTML(
171177
'<html><body id="' . $wrapperElementId . '">' . $data . '</body></html>'
172178
);
@@ -192,7 +198,17 @@ public function escapeHtmlWithLinks($data, $allowedTags = null)
192198
}
193199
}
194200

195-
$result = mb_convert_encoding($domDocument->saveHTML(), 'UTF-8', 'HTML-ENTITIES');
201+
$result = mb_decode_numericentity(
202+
// phpcs:ignore Magento2.Functions.DiscouragedFunction
203+
html_entity_decode(
204+
$domDocument->saveHTML(),
205+
ENT_QUOTES|ENT_SUBSTITUTE,
206+
'UTF-8'
207+
),
208+
$convmap,
209+
'UTF-8'
210+
);
211+
196212
preg_match('/<body id="' . $wrapperElementId . '">(.+)<\/body><\/html>$/si', $result, $matches);
197213
$data = !empty($matches) ? $matches[1] : '';
198214
}

composer.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@
4040
"composer/composer": "^2.0, !=2.2.16",
4141
"elasticsearch/elasticsearch": "^7.17||^8.5",
4242
"ezyang/htmlpurifier": "^4.14",
43-
"guzzlehttp/guzzle": "^7.4",
43+
"guzzlehttp/guzzle": "^7.5",
4444
"laminas/laminas-captcha": "^2.12",
4545
"laminas/laminas-code": "^4.5",
4646
"laminas/laminas-db": "^2.15",
@@ -85,7 +85,7 @@
8585
"tedivm/jshrink": "^1.4",
8686
"tubalmartin/cssmin": "^4.1",
8787
"web-token/jwt-framework": "^3.1",
88-
"webonyx/graphql-php": "^14.11",
88+
"webonyx/graphql-php": "^15.0",
8989
"wikimedia/less.php": "^3.2"
9090
},
9191
"require-dev": {
@@ -98,7 +98,7 @@
9898
"magento/magento2-functional-testing-framework": "^4.0",
9999
"pdepend/pdepend": "^2.10",
100100
"phpmd/phpmd": "^2.12",
101-
"phpstan/phpstan": "^1.7",
101+
"phpstan/phpstan": "1.9.2",
102102
"phpunit/phpunit": "^9.5",
103103
"sebastian/phpcpd": "^6.0",
104104
"symfony/finder": "^5.4"

0 commit comments

Comments
 (0)