Closed piotrekkaminski closed 6 years ago
From @orlangur on October 30, 2017 12:6
must be global and is not super in row(s)
Such message has no sense as every global select is considered to be super by design:
// save super attributes for simplier and quicker search in future
if ('select' == $attrParams['type'] && 1 == $attrParams['is_global']) {
$this->_superAttributes[$attrParams['code']] = $attrParams;
}
However, check for is_global
could be implemented as separate with separate error message.
From @magento-engcom-team on November 17, 2017 10:30
@brosenberger, thank you for your report. We've created internal ticket(s) MAGETWO-84059 to track progress on the issue.
@piotrekkaminski - I've reviewed this.
TESTED ON [Magento Commerce 2.2.2] and [2.3-develop]
If the import detects that a configurable_variations attribute code is not super the following checks are needed:
At the moment we only have a check to see if the attribute is in the list of super attributes:
if (!$this->_isAttributeSuper($superAttrCode)) { FAIL - DISPLAY ERROR }
Is the intended result:
to try pinpoint exactly which one(s) of the above the attribute has failed on? That will require extra coding. I'm not sure all the information is available at that moment in time.
to simply expand the message to alert the user of all possible reasons why.
Thanks, Tadhg
I'd prefer detailed message when available, when not a list of possible reasons. The easier we can make it for people to avoid looking at CSV trying to guess what's wrong, the better.
@piotrekkaminski - Agreed. It's on my radar. Thanks. T
Quick update from me:
If the import detects that a configurable_variations attribute code is not super one the following error messages will be displayed:
Attribute code does not exist or is not in chosen attribute set. Attribute code needs to have an Input Type of "Dropdown", "Visual Swatch" or Text Swatch" Attribute code needs to have a Scope of "Global".
This covers all reasons why is not super. I will submit a pull request early next week for this.
Thanks. T.
Thank you @tadhgbowe for your fix for this issue. PR https://github.com/magento-engcom/import-export-improvements/pull/115 has added more detailed information to error messages here.
From @brosenberger on October 30, 2017 10:53
Preconditions
Steps to reproduce
Expected result
Actual result
Copied from original issue: magento/magento2#11871