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
  • Loading branch information
sidolov authored Feb 15, 2019
2 parents b572676 + 70684cc commit 4c98d48
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 4c98d48

Please sign in to comment.