Skip to content

Commit aa25ca5

Browse files
committed
MAGETWO-57934: [GitHub] Can't use "configurable" as group name in attribute sets M2.1 #6123
1 parent 5b59d43 commit aa25ca5

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

app/code/Magento/Eav/Model/Entity/Attribute/Group.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,8 @@ public function beforeSave()
127127
);
128128
$isReservedSystemName = in_array(strtolower($attributeGroupCode), $this->reservedSystemNames);
129129
if (empty($attributeGroupCode) || $isReservedSystemName) {
130-
// in the following code sha1 is not used for security purposes
131-
$attributeGroupCode = sha1(strtolower($groupName));
130+
// in the following code md5 is not used for security purposes
131+
$attributeGroupCode = md5(strtolower($groupName));
132132
}
133133
$this->setAttributeGroupCode($attributeGroupCode);
134134
}

app/code/Magento/Eav/Test/Unit/Model/Entity/Attribute/GroupTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,9 @@ public function attributeGroupCodeDataProvider()
6868
{
6969
return [
7070
['General Group', 'general-group'],
71-
['configurable', sha1('configurable')],
72-
['configurAble', sha1('configurable')],
73-
['///', sha1('///')],
71+
['configurable', md5('configurable')],
72+
['configurAble', md5('configurable')],
73+
['///', md5('///')],
7474
];
7575
}
7676
}

0 commit comments

Comments
 (0)