magento-configurator icon indicating copy to clipboard operation
magento-configurator copied to clipboard

Sort order and default attribute option

Open mblarsen opened this issue 8 years ago • 4 comments

Added the option to specify a default attribute option for select type (only):

options:
  - S
  - M
  - L|default
  - XL

The old implementation would not take into account the sort order.

Additionally unified: insert, update, delete into one process

mblarsen avatar Jan 16 '16 11:01 mblarsen

Hi @mblarsen,

We have just made a major update to configurator. Would you be able to resolve any conflicts in the code and pass it over?

Many thanks :)

chevli avatar Jan 25 '16 11:01 chevli

@chevli sure I'll look into it

mblarsen avatar Jan 26 '16 01:01 mblarsen

I've merged master-branch now. Great work by the way.

mblarsen avatar Jan 26 '16 01:01 mblarsen

@chevli btw, your code will use the following code to set backend_type which is fine:

 $data['backend_type'] = $attribute->getBackendTypeByInput($data['frontend_input']);

However this value is easily be added to the yaml-file which gives you a load of trouble if you don't know what you are doing.

I didn't: http://magento.stackexchange.com/questions/98279/option-false-for-configurable-associated-products-imported-with-magmi

Perhaps it would be wise to throw an exception for types: dropdown, multiselect if it is not int There might be other backend types that shouldn't be messed with.

mblarsen avatar Jan 26 '16 02:01 mblarsen