Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MultiSelect option set migration attributes created as option set #327

Open
jynxeh opened this issue Jul 12, 2022 · 3 comments
Open

MultiSelect option set migration attributes created as option set #327

jynxeh opened this issue Jul 12, 2022 · 3 comments

Comments

@jynxeh
Copy link

jynxeh commented Jul 12, 2022

AttributeManager V 1.2020.10.2

When working with a multi select option set the temporary attribute which is created is not of the multi select type, it is a normal option set.

Steps to reproduce the behavior:

  1. Select an attribute which is of type multi select option set using a global option set
  2. Supply a new name for the attribute (in this example I added _new), check the Convert Attribute Type tick box. Leave the Option set as the same global option set.
  3. Execute the steps
  4. The first step will work, but the following step will fail as it can't find a suitable field.

Example:
image

Expected behaviour is the field should have been created with the same type as the original

I did try to work through the code to see where this might be but having not seen it before I didn't get far. Happy to have a look myself if someone can direct me.

Thanks.

@jynxeh jynxeh added the bug label Jul 12, 2022
@daryllabar
Copy link
Owner

I don't believe I ever added the code to handle Multi-Select Option Sets vs Option Sets. Simple work around is to manually delete the _new field, and recreate it as a Multi-Select, and then re-run the tool, skipping just the create temp field step. You'll probably have to redo it when it comes to recreating it for real as well....

@jynxeh
Copy link
Author

jynxeh commented Jul 12, 2022

Hi @daryllabar - thanks for the quick response, sorry I should have said I've worked around it for now, so no problem, just felt like something worth tracking to be fixed/extended at some point.

@daryllabar
Copy link
Owner

Yep... It's on the To do list... Just not sure when it will get moved to the To Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants