Save Script name instead of index about config-presets

[marcsyp]

Incidentally, one big improvement in the meantime is to do a check on whether the index of the list exists so that there is no exception and all the other element_id values get set properly anyway -- the user would just have to manually change the script to the one desired. As it stands, the exception of an index higher than the script list item count causes no values to be set, which is rough. Thanks.

[marcsyp]

Working Example:

'def config_preset_dropdown_change(dropdown_value, *components_value):
config_preset = config_presets[dropdown_value]
print(f"[Config-Presets] Changed to: {dropdown_value}")

                        # update component values with user preset
                        current_components = dict(zip(component_map.keys(), components_value))
                        #print("Components before:", current_components)
                        current_components.update(config_preset)

                        # transform necessary components from index to value
                        for component_name, component_value in current_components.items():
                            #print(component_name, component_value)
                            if component_name in index_type_components and type(component_value) == int:
                                    if (component_value < len(component_map[component_name].choices)):        # <--------------
                                        current_components[component_name] = component_map[component_name].choices[component_value]

                        #print("Components after :", current_components)
                        
                        return list(current_components.values())'

[wywywywy]

Sorry how do you mean? You can already specify the script name. It does have to be a number.

{
    "Ultimate SD Upscale": {
        "script_list": "Ultimate SD upscale"
    }
}

[marcsyp]

Yes, exactly -- currently it is stored as an int (index position of the script in the list when the configuration was saved), but if it were stored as the name of the script, as you have demonstrated, the CP logic would need a special IF block to handle the script_list field and search for the name of the script in the list of choices and convert that to the index position, then passing that to the component if an index was found.

[wywywywy]

I meant to say it does NOT have to be a number. Setting it to a name already works (though bear in mind it's case sensitive).

This line handles it already.

current_components.update(config_preset)