-
Notifications
You must be signed in to change notification settings - Fork 146
Description
This is a breaker:
Line 33 in 64a120d
| ? $attrParams['options'][Mage::helper('fastsimpleimport')->strtolower($rowData[$attrCode])] |
If available, mb_strtolower is used instead of PHP strtolower.
See here:
But ImportExport uses PHP strtolower for "getAttributeOptions":
It leads to a PHP notice and an not imported attribute value - when the option contains "Umlaute".
Notice: Undefined index: überspannungsschutz in /var/www/xyz/app/code/community/AvS/FastSimpleImport/Model/Import/Entity/Product/Type/Simple.php on line 33
Attribute Option (actual value): Überspannungsschutz
$str = 'Überspannungsschutz';
strtolower($str) => Überspannungsschutz
mb_strtolower($str, mb_detect_encoding($str)) => überspannungsschutz
So my question to the original author of these lines
Line 33 in 64a120d
| ? $attrParams['options'][Mage::helper('fastsimpleimport')->strtolower($rowData[$attrCode])] |
and or the helper fuction:
Why was it needed?