MAGETWO-63858: Certain ASCII Attribute Values gets transformed to HTML code in attributes

Author: Valeriy Nayda

app/code/Magento/Swatches/Block/Adminhtml/Attribute/Edit/Options/AbstractSwatch.php

@@ -78,7 +78,9 @@ public function __construct(
     /**
      * Create store values
      *
-     * @codeCoverageIgnore
+     * Method not intended to escape HTML entities
+     * Escaping will be applied in template files
+     *
      * @param integer $storeId
      * @param integer $optionId
      * @return array
@@ -88,10 +90,8 @@ protected function createStoreValues($storeId, $optionId)
         $value = [];
         $storeValues = $this->getStoreOptionValues($storeId);
         $swatchStoreValue = isset($storeValues['swatch']) ? $storeValues['swatch'] : null;
-        $value['store' . $storeId] = isset($storeValues[$optionId]) ?
-            $this->escapeHtml($storeValues[$optionId]) : '';
-        $value['swatch' . $storeId] = isset($swatchStoreValue[$optionId]) ?
-            $this->escapeHtml($swatchStoreValue[$optionId]) : '';
+        $value['store' . $storeId] = isset($storeValues[$optionId]) ? $storeValues[$optionId] : '';
+        $value['swatch' . $storeId] = isset($swatchStoreValue[$optionId]) ? $swatchStoreValue[$optionId] : '';
 
         return $value;
     }

app/code/Magento/Swatches/Block/Adminhtml/Attribute/Edit/Options/Visual.php

@@ -18,7 +18,9 @@ class Visual extends AbstractSwatch
     /**
      * Create store values
      *
-     * @codeCoverageIgnore
+     * Method not intended to escape HTML entities
+     * Escaping will be applied in template files
+     *
      * @param integer $storeId
      * @param integer $optionId
      * @return array
@@ -37,16 +39,16 @@ protected function createStoreValues($storeId, $optionId)
         }
 
         if (isset($storeValues[$optionId])) {
-            $value['store' . $storeId] = $this->escapeHtml($storeValues[$optionId]);
+            $value['store' . $storeId] = $storeValues[$optionId];
         }
 
         if (isset($swatchStoreValue[$optionId])) {
-            $value['defaultswatch' . $storeId] = $this->escapeHtml($swatchStoreValue[$optionId]);
+            $value['defaultswatch' . $storeId] = $swatchStoreValue[$optionId];
         }
 
         $swatchStoreValue = $this->reformatSwatchLabels($swatchStoreValue);
         if (isset($swatchStoreValue[$optionId])) {
-            $value['swatch' . $storeId] = $this->escapeHtml($swatchStoreValue[$optionId]);
+            $value['swatch' . $storeId] = $swatchStoreValue[$optionId];
         }
 
         return $value;