Skip to content

Commit 255e840

Browse files
thiaramusthiaramus
committed
Merge remote-tracking branch 'origin/ACP2E-4311' into PR_2025_11_20_flowers
2 parents 86cf2e0 + 8886771 commit 255e840

File tree

6 files changed

+91
-20
lines changed

6 files changed

+91
-20
lines changed

‎app/code/Magento/Customer/Block/Widget/Dob.php‎

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -410,19 +410,25 @@ public function getTranslatedCalendarConfigJson(): string
410410
$localeData = (new DataBundle())->get($this->localeResolver->getLocale());
411411
$monthsData = $localeData['calendar']['gregorian']['monthNames'];
412412
$daysData = $localeData['calendar']['gregorian']['dayNames'];
413-
413+
$monthsFormat = $monthsData['format'];
414+
$daysFormat = $daysData['format'];
415+
$monthsAbbreviated = $monthsFormat['abbreviated'];
416+
$monthsShort = $monthsAbbreviated ?? $monthsFormat['wide'];
417+
$daysAbbreviated = $daysFormat['abbreviated'];
418+
$daysShort = $daysAbbreviated ?? $daysFormat['wide'];
419+
$daysShortFormat = $daysFormat['short'];
420+
$daysMin = $daysShortFormat ?? $daysShort;
414421
return $this->encoder->encode(
415422
[
416423
'closeText' => __('Done'),
417424
'prevText' => __('Prev'),
418425
'nextText' => __('Next'),
419426
'currentText' => __('Today'),
420-
'monthNames' => array_values(iterator_to_array($monthsData['format']['wide'])),
421-
'monthNamesShort' => array_values(iterator_to_array($monthsData['format']['abbreviated'])),
422-
'dayNames' => array_values(iterator_to_array($daysData['format']['wide'])),
423-
'dayNamesShort' => array_values(iterator_to_array($daysData['format']['abbreviated'])),
424-
'dayNamesMin' =>
425-
array_values(iterator_to_array(($daysData['format']['short']) ?: $daysData['format']['abbreviated'])),
427+
'monthNames' => array_values(iterator_to_array($monthsFormat['wide'])),
428+
'monthNamesShort' => array_values(iterator_to_array($monthsShort)),
429+
'dayNames' => array_values(iterator_to_array($daysFormat['wide'])),
430+
'dayNamesShort' => array_values(iterator_to_array($daysShort)),
431+
'dayNamesMin' => array_values(iterator_to_array($daysMin)),
426432
]
427433
);
428434
}

‎app/code/Magento/Customer/Plugin/ValidateDobOnSave.php‎

Lines changed: 28 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,11 @@
1414
use Magento\Framework\Exception\InputException;
1515
use Magento\Framework\Serialize\Serializer\Json as JsonSerializer;
1616
use Magento\Framework\Stdlib\DateTime;
17+
use Magento\Framework\Locale\ResolverInterface;
1718

19+
/**
20+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
21+
*/
1822
class ValidateDobOnSave
1923
{
2024
/**
@@ -27,16 +31,24 @@ class ValidateDobOnSave
2731
*/
2832
private $json;
2933

34+
/**
35+
* @var ResolverInterface
36+
*/
37+
private $localeResolver;
38+
3039
/**
3140
* @param EavConfig $eavConfig
3241
* @param JsonSerializer $json
42+
* @param ResolverInterface $localeResolver
3343
*/
3444
public function __construct(
3545
EavConfig $eavConfig,
36-
JsonSerializer $json
46+
JsonSerializer $json,
47+
ResolverInterface $localeResolver
3748
) {
3849
$this->eavConfig = $eavConfig;
3950
$this->json = $json;
51+
$this->localeResolver = $localeResolver;
4052
}
4153

4254
/**
@@ -67,8 +79,9 @@ public function aroundSave(
6779
}
6880

6981
if ($dobDate) {
82+
$normalizedDob = $dobDate->format(DateTime::DATE_PHP_FORMAT);
83+
$customer->setDob($normalizedDob);
7084
$attr = $this->eavConfig->getAttribute('customer', 'dob');
71-
7285
$rules = $attr->getData('validate_rules');
7386
if (is_string($rules) && $rules !== '') {
7487
try {
@@ -127,9 +140,21 @@ private function parseDate($value): ?\DateTimeImmutable
127140
$seconds = ($intVal >= 10000000000) ? intdiv($intVal, 1000) : $intVal;
128141
return (new \DateTimeImmutable('@' . $seconds))->setTimezone(new \DateTimeZone('UTC'));
129142
}
143+
$stringValue = (string)$value;
144+
$locale = $this->localeResolver->getLocale();
145+
$formatter = new \IntlDateFormatter(
146+
$locale,
147+
\IntlDateFormatter::SHORT,
148+
\IntlDateFormatter::NONE
149+
);
150+
$formatter->setPattern(DateTime::DATE_INTERNAL_FORMAT);
151+
$timestamp = $formatter->parse($stringValue);
152+
if ($timestamp !== false) {
153+
return new \DateTimeImmutable('@' . $timestamp);
154+
}
130155

131156
try {
132-
return new \DateTimeImmutable((string)$value);
157+
return new \DateTimeImmutable($stringValue);
133158
} catch (\Exception $e) {
134159
return null;
135160
}

‎app/code/Magento/Customer/Test/Unit/Block/Widget/DobTest.php‎

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -447,17 +447,14 @@ protected function getValidationRuleClass($type)
447447
$validationRule->expects($this->any())
448448
->method('getValue')
449449
->willReturn(strtotime(self::MIN_DATE));
450-
}
451-
elseif ($type=="MAX") {
450+
} elseif ($type=="MAX") {
452451
$validationRule->expects($this->any())
453452
->method('getName')
454453
->willReturn(Dob::MAX_DATE_RANGE_KEY);
455454
$validationRule->expects($this->any())
456455
->method('getValue')
457456
->willReturn(strtotime(self::MAX_DATE));
458457
}
459-
460-
461458
return $validationRule;
462459
}
463460

@@ -713,6 +710,25 @@ public static function getTranslatedCalendarConfigJsonDataProvider()
713710
'expectedJson' => '{"closeText":"Done","prevText":"Prev","nextText":"Next","currentText":"Today","monthNames":["Januar","Februar","M\u00e4rz","April","Mai","Juni","Juli","August","September","Oktober","November","Dezember"],"monthNamesShort":["Jan.","Feb.","M\u00e4rz","Apr.","Mai","Juni","Juli","Aug.","Sept.","Okt.","Nov.","Dez."],"dayNames":["Sonntag","Montag","Dienstag","Mittwoch","Donnerstag","Freitag","Samstag"],"dayNamesShort":["So.","Mo.","Di.","Mi.","Do.","Fr.","Sa."],"dayNamesMin":["So.","Mo.","Di.","Mi.","Do.","Fr.","Sa."]}'
714711
// phpcs:enable Generic.Files.LineLength.TooLong
715712
],
713+
[
714+
'locale' => 'ar_SA',
715+
'expectedArray' => [
716+
'closeText' => 'Done',
717+
'prevText' => 'Prev',
718+
'nextText' => 'Next',
719+
'currentText' => 'Today',
720+
'monthNames' => ['يناير', 'فبراير', 'مارس', 'أبريل', 'مايو', 'يونيو', 'يوليو', 'أغسطس', 'سبتمبر',
721+
'أكتوبر', 'نوفمبر', 'ديسمبر'],
722+
'monthNamesShort' => ['يناير', 'فبراير', 'مارس', 'أبريل', 'مايو', 'يونيو', 'يوليو', 'أغسطس',
723+
'سبتمبر', 'أكتوبر', 'نوفمبر', 'ديسمبر'],
724+
'dayNames' => ['الأحد', 'الاثنين', 'الثلاثاء', 'الأربعاء', 'الخميس', 'الجمعة', 'السبت'],
725+
'dayNamesShort' => ['الأحد', 'الاثنين', 'الثلاثاء', 'الأربعاء', 'الخميس', 'الجمعة', 'السبت'],
726+
'dayNamesMin' => ['أحد', 'إثنين', 'ثلاثاء', 'أربعاء', 'خميس', 'جمعة', 'سبت'],
727+
],
728+
// phpcs:disable Generic.Files.LineLength.TooLong
729+
'expectedJson' => '{"closeText":"Done","prevText":"Prev","nextText":"Next","currentText":"Today","monthNames":["\u064a\u0646\u0627\u064a\u0631","\u0641\u0628\u0631\u0627\u064a\u0631","\u0645\u0627\u0631\u0633","\u0623\u0628\u0631\u064a\u0644","\u0645\u0627\u064a\u0648","\u064a\u0648\u0646\u064a\u0648","\u064a\u0648\u0644\u064a\u0648","\u0623\u063a\u0633\u0637\u0633","\u0633\u0628\u062a\u0645\u0628\u0631","\u0623\u0643\u062a\u0648\u0628\u0631","\u0646\u0648\u0641\u0645\u0628\u0631","\u062f\u064a\u0633\u0645\u0628\u0631"],"monthNamesShort":["\u064a\u0646\u0627\u064a\u0631","\u0641\u0628\u0631\u0627\u064a\u0631","\u0645\u0627\u0631\u0633","\u0623\u0628\u0631\u064a\u0644","\u0645\u0627\u064a\u0648","\u064a\u0648\u0646\u064a\u0648","\u064a\u0648\u0644\u064a\u0648","\u0623\u063a\u0633\u0637\u0633","\u0633\u0628\u062a\u0645\u0628\u0631","\u0623\u0643\u062a\u0648\u0628\u0631","\u0646\u0648\u0641\u0645\u0628\u0631","\u062f\u064a\u0633\u0645\u0628\u0631"],"dayNames":["\u0627\u0644\u0623\u062d\u062f","\u0627\u0644\u0627\u062b\u0646\u064a\u0646","\u0627\u0644\u062b\u0644\u0627\u062b\u0627\u0621","\u0627\u0644\u0623\u0631\u0628\u0639\u0627\u0621","\u0627\u0644\u062e\u0645\u064a\u0633","\u0627\u0644\u062c\u0645\u0639\u0629","\u0627\u0644\u0633\u0628\u062a"],"dayNamesShort":["\u0627\u0644\u0623\u062d\u062f","\u0627\u0644\u0627\u062b\u0646\u064a\u0646","\u0627\u0644\u062b\u0644\u0627\u062b\u0627\u0621","\u0627\u0644\u0623\u0631\u0628\u0639\u0627\u0621","\u0627\u0644\u062e\u0645\u064a\u0633","\u0627\u0644\u062c\u0645\u0639\u0629","\u0627\u0644\u0633\u0628\u062a"],"dayNamesMin":["\u0623\u062d\u062f","\u0625\u062b\u0646\u064a\u0646","\u062b\u0644\u0627\u062b\u0627\u0621","\u0623\u0631\u0628\u0639\u0627\u0621","\u062e\u0645\u064a\u0633","\u062c\u0645\u0639\u0629","\u0633\u0628\u062a"]}'
730+
// phpcs:enable Generic.Files.LineLength.TooLong
731+
],
716732
];
717733
}
718734
}

‎app/code/Magento/Customer/Test/Unit/Plugin/ValidateDobOnSaveTest.php‎

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
use Magento\Framework\Serialize\Serializer\Json as JsonSerializer;
1818
use PHPUnit\Framework\MockObject\MockObject;
1919
use PHPUnit\Framework\TestCase;
20+
use Magento\Framework\Locale\ResolverInterface;
2021

2122
/**
2223
* Unit test for validate date of birth plugin
@@ -42,10 +43,13 @@ protected function setUp(): void
4243
$this->eavConfig = $this->createMock(EavConfig::class);
4344
$this->json = $this->createMock(JsonSerializer::class);
4445
$this->repo = $this->createMock(CustomerRepositoryInterface::class);
46+
$localeResolver = $this->createMock(ResolverInterface::class);
47+
$localeResolver->method('getLocale')->willReturn('en_US');
4548

4649
$this->plugin = new ValidateDobOnSave(
4750
$this->eavConfig,
48-
$this->json
51+
$this->json,
52+
$localeResolver
4953
);
5054
}
5155

@@ -322,6 +326,21 @@ public function testDobZeroEpochInvalidThrows(): void
322326
$this->assertFalse($called);
323327
}
324328

329+
public function testDobIsNormalized(): void
330+
{
331+
$dob = '2005-12-01';
332+
$customer = $this->createMock(CustomerInterface::class);
333+
$customer->method('getDob')->willReturn($dob);
334+
$customer->expects($this->once())
335+
->method('setDob')
336+
->with('2005-12-01');
337+
$this->mockAttributeRulesArray(['date_range_min' => '1980-01-01', 'date_range_max' => '2010-12-31']);
338+
$called = false;
339+
$proceed = $this->proceedPlugin($called, $customer);
340+
$this->plugin->aroundSave($this->repo, $proceed, $customer);
341+
$this->assertTrue($called);
342+
}
343+
325344
/**
326345
* Create a proceed closure that marks $called and returns either the given $result or the original $customer.
327346
*

‎lib/internal/Magento/Framework/View/Element/Html/Calendar.php‎

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2014 Adobe
4+
* All Rights Reserved.
55
*/
66
namespace Magento\Framework\View\Element\Html;
77

@@ -74,7 +74,11 @@ protected function _toHtml()
7474
[
7575
'wide' => $this->encoder->encode(array_values(iterator_to_array($daysData['format']['wide']))),
7676
'abbreviated' => $this->encoder->encode(
77-
array_values(iterator_to_array($daysData['format']['abbreviated']))
77+
array_values(
78+
iterator_to_array(
79+
$daysData['format']['abbreviated'] ?? $daysData['format']['wide']
80+
)
81+
)
7882
),
7983
]
8084
);

‎lib/internal/Magento/Framework/View/Test/Unit/Element/Html/CalendarTest.php‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2014 Adobe
4+
* All Rights Reserved.
55
*/
66
declare(strict_types=1);
77

@@ -51,6 +51,7 @@ public static function localesDataProvider()
5151
['lv_LV'],
5252
['sv_SE'],
5353
['de_AT'],
54+
['ar_SA'],
5455
];
5556
}
5657

0 commit comments

Comments
 (0)