Skip to content

Commit

Permalink
ENGCOM-4284: [Backport] Fixed #17861 Customer Name Prefix shows white…
Browse files Browse the repository at this point in the history
… space when extra separator is addes #21245

 - Merge Pull Request #21245 from mage2pratik/magento2:2.2-develop-PR-port-20896
 - Merged commits:
   1. 77c780d
   2. 5eeecac
   3. 69cf6ce
  • Loading branch information
magento-engcom-team committed Feb 15, 2019
2 parents 994348f + 69cf6ce commit 70684cc
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions app/code/Magento/Customer/Model/Options.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,11 @@
use Magento\Config\Model\Config\Source\Nooptreq as NooptreqSource;
use Magento\Customer\Helper\Address as AddressHelper;
use Magento\Framework\Escaper;
use Magento\Store\Api\Data\StoreInterface;

/**
* Customer Options.
*/
class Options
{
/**
Expand Down Expand Up @@ -38,7 +42,7 @@ public function __construct(
/**
* Retrieve name prefix dropdown options
*
* @param null $store
* @param null|string|bool|int|StoreInterface $store
* @return array|bool
*/
public function getNamePrefixOptions($store = null)
Expand All @@ -52,7 +56,7 @@ public function getNamePrefixOptions($store = null)
/**
* Retrieve name suffix dropdown options
*
* @param null $store
* @param null|string|bool|int|StoreInterface $store
* @return array|bool
*/
public function getNameSuffixOptions($store = null)
Expand All @@ -64,7 +68,9 @@ public function getNameSuffixOptions($store = null)
}

/**
* @param $options
* Unserialize and clear name prefix or suffix options.
*
* @param string $options
* @param bool $isOptional
* @return array|bool
*
Expand All @@ -91,7 +97,7 @@ private function prepareNamePrefixSuffixOptions($options, $isOptional = false)
return false;
}
$result = [];
$options = explode(';', $options);
$options = array_filter(explode(';', $options));
foreach ($options as $value) {
$value = $this->escaper->escapeHtml(trim($value));
$result[$value] = $value;
Expand Down

0 comments on commit 70684cc

Please sign in to comment.