_super_attribute_codes positioning wrong in backend

[pquerner]

Hey,

as seen in this image

The configurable product has two configurable options "sw_size" and "sw_packing_unit" and the "sw_packing_unit" should be on the last position as configurable attribute.

However in the backend, it looks like this:

As seen in this image, the "Packing Unit" attribute (= sw_packing_unit) is on the first positon as a configurable attribute, which we dont want.

Is this a bug ? Does the positioning inside the "_super_attribute_codes" array even matter?

FastSimpleImport Version: "b5beca53d40133671548f827e4c4e9f4571cda08"
Magento Version: 1.9.2.1

Thanks!

[avstudnitz]

@pquerner Could you please provide your import array in text form?
I assume, if it's a bug, it's a Mage_ImportExport bug. Feel free to provide a pull request.

[pquerner]

902 => 
    array (size=38)
      'sku' => string 'SW5144017' (length=9)
      '_type' => string 'simple' (length=6)
      'formula' => string 'default: default formula' (length=24)
      'attributereplacement_template' => string 'standard: standard en' (length=21)
      '_attribute_set' => string 'Swarovski Elements' (length=18)
      '_product_websites' => string 'base' (length=4)
      'name' => string 'Swarovski Kristalle {{attl:sw_component_id}} {{attl:sw_size}}, {{attl:sw_stone_color}} {{attl:sw_effects}}, {{attl:sw_packing_unit}} Stück' (length=139)
      'name_short' => string 'Swarovski...' (length=12)
      'description' => string '<p>Verwenden Sie die Swarovski Kristalle in der Gr&ouml;&szlig;e {{attv:sw_size}}.

Die Swarovski-Farbe \"{{attv:sw_stone_color}} {{attv:sw_effects}} \" erstrahlt in den Farben: {{attl:sw_stone_color}} .</p>

<p><em>Strasssteine werden auch B&uuml;gelsteine, Glitzersteine, Crystalsteine, Kristallsteine, Strasssteinchen, Strass-Steine oder Glassteine genannt.</em></p>
' (length=375)
      'short_description' => string 'Swarovski Kristalle {{attl:sw_component_id}}  \"{{attl:sw_stone_color}} {{attl:sw_effects}}\" sind hochwertige Glassteine, die in Farbstreuung und Glanz annähernd einem Diamanten gleichen.' (length=189)
      'price' => int 0
      'cost' => string '71.8' (length=4)
      'visibility' => int 3
      'manufacturer' => string 'swarovski' (length=9)
      'tax_class_id' => int 1
      'base_price_unit' => string 'unit_pcs' (length=8)
      'base_price_base_unit' => string 'unit_pcs' (length=8)
      'base_price_base_amount' => string '1' (length=1)
      'country_of_manufacture' => string 'AT' (length=2)
      'qty' => int 0
      'is_in_stock' => int 1
      'status' => int 1
      'base_price_amount' => string '96' (length=2)
      'url_key' => string 'swarovski-elements_2808_5144017' (length=31)
      'sw_product_name' => string '2808 MM 14,0 CRYSTAL POWROSE' (length=28)
      'sw_id' => string '85970' (length=5)
      'sw_system_nr' => string '5144017' (length=7)
      'sw_system_code' => string 'e3QVC2287kIfgCcnjWRWrm' (length=22)
      'sw_article_nr' => string '2808M14001PROS' (length=14)
      'sw_in_stock' => string '3264' (length=4)
      'sw_min_purchase' => string '96' (length=2)
      'sw_packing_unit' => string '96' (length=2)
      'sw_component_id' => string '2808' (length=4)
      'sw_size' => string 'MM14_0' (length=6)
      'sw_effects' => string 'PROS' (length=4)
      'sw_stone_color' => string '001' (length=3)
      'sw_status' => string 'in program' (length=10)
      'sw_foiling' => string 'EMPTY' (length=5)
  909 => 
    array (size=24)
      'sku' => string 'SWC_2808_001_PROS' (length=17)
      '_type' => string 'configurable' (length=12)
      '_attribute_set' => string 'Swarovski Elements' (length=18)
      '_product_websites' => string 'base' (length=4)
      'price' => int 0
      'manufacturer' => string 'swarovski' (length=9)
      'name' => string 'Swarovski Kristalle {{attl:sw_component_id}} {{attl:sw_size}}, {{attl:sw_stone_color}} {{attl:sw_effects}}, {{attl:sw_packing_unit}} Stück' (length=139)
      'name_short' => string 'Swarovski...' (length=12)
      'cost' => int 0
      'description' => string '<p>Verwenden Sie die Swarovski Kristalle in der Gr&ouml;&szlig;e {{attv:sw_size}}.

Die Swarovski-Farbe \"{{attv:sw_stone_color}} {{attv:sw_effects}} \" erstrahlt in den Farben: {{attl:sw_stone_color}} .</p>

<p><em>Strasssteine werden auch B&uuml;gelsteine, Glitzersteine, Crystalsteine, Kristallsteine, Strasssteinchen, Strass-Steine oder Glassteine genannt.</em></p>
' (length=375)
      'short_description' => string 'Swarovski Kristalle {{attl:sw_component_id}}  \"{{attl:sw_stone_color}} {{attl:sw_effects}}\" sind hochwertige Glassteine, die in Farbstreuung und Glanz annähernd einem Diamanten gleichen.' (length=189)
      'status' => int 1
      'visibility' => int 2
      'tax_class_id' => int 1
      'is_in_stock' => int 1
      '_super_products_sku' => 
        array (size=3)
          0 => string 'SW5144010' (length=9)
          1 => string 'SW5144013' (length=9)
          2 => string 'SW5144017' (length=9)
      'attributereplacement_template' => string 'standard: standard en' (length=21)
      '_super_attribute_code' => 
        array (size=2)
          0 => string 'sw_size' (length=7)
          1 => string 'sw_packing_unit' (length=15)
      'sw_component_id' => string '2808' (length=4)
      'sw_effects' => string 'PROS' (length=4)
      'sw_stone_color' => string '001' (length=3)
      'sw_status' => string 'in program' (length=10)
      'sw_foiling' => string 'EMPTY' (length=5)
      'url_key' => string 'swarovski-elements_2808_' (length=24)

[pquerner]

In the table catalog_product_super_attribute the "position" field equals always 0. Then the last one inserted simply wins.

Position 0 probably rises from here:

src/app/code/community/AvS/FastSimpleImport/Model/Import/Entity/Product/Type/Configurable.php::206
$superAttributes['attributes'][$productId][$attrParams['id']] = array(
                        'product_super_attribute_id' => $productSuperAttrId, 'position' => 0
                    );

//Edit
I can confirm thats the line above.
I manually set it to 5 and watched the database. ==> I got the 5 in there. ;)
So know we need some sort of solution so we can pass in a position on our own without breaking legacy imports. Any idea @avstudnitz ?

[riker09]

He, funny. I stumbled upon the exact same problem. Good to see that there's a PR for that. I applied that patch locally and gave it a quick run, seems to do the trick. Thank you, @pquerner ! Let me know when you are in Braunschweig, Germany, and I'll invite you for a beer or two. ;)