Skip to content

Commit ef34b72

Browse files
committed
MAGETWO-72041: Information Disclosure - Credit Card details & customer passwords compromise by non-admin user
1 parent f34ad6b commit ef34b72

File tree

2 files changed

+4
-10
lines changed

2 files changed

+4
-10
lines changed

app/code/Magento/Customer/Model/Address/Validator/Country.php

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -56,10 +56,7 @@ private function validateCountry(AbstractAddress $address)
5656
//Checking if such country exists.
5757
$errors[] = __(
5858
'Invalid value of "%value" provided for the %fieldName field.',
59-
[
60-
'fieldName' => 'countryId',
61-
'value' => htmlspecialchars($countryId),
62-
]
59+
['fieldName' => 'countryId', 'value' => htmlspecialchars($countryId)]
6360
);
6461
}
6562

@@ -94,10 +91,7 @@ private function validateRegion(AbstractAddress $address)
9491
//If a region is selected then checking if it exists.
9592
$errors[] = __(
9693
'Invalid value of "%value" provided for the %fieldName field.',
97-
[
98-
'fieldName' => 'regionId',
99-
'value' => htmlspecialchars($regionId),
100-
]
94+
['fieldName' => 'regionId', 'value' => htmlspecialchars($regionId)]
10195
);
10296
}
10397

dev/tests/integration/testsuite/Magento/Customer/Api/AddressRepositoryTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -230,11 +230,11 @@ public function testSaveNewInvalidAddress()
230230
$errors = $exception->getErrors();
231231
$this->assertCount(3, $errors);
232232
$this->assertEquals(
233-
'firstname is a required field.',
233+
'"firstname" is required. Enter and try again.',
234234
$errors[0]->getLogMessage()
235235
);
236236
$this->assertEquals(
237-
'lastname is a required field.',
237+
'"lastname" is required. Enter and try again.',
238238
$errors[1]->getLogMessage()
239239
);
240240
$this->assertEquals(

0 commit comments

Comments
 (0)